Adding openflow protocol set support in onos for device
Change-Id: Id796e54b536eda19d511a1978657abd6e149f117
diff --git a/protocols/ovsdb/api/src/main/java/org/onosproject/ovsdb/controller/OvsdbBridge.java b/protocols/ovsdb/api/src/main/java/org/onosproject/ovsdb/controller/OvsdbBridge.java
index 34c2d43..6bddef3 100644
--- a/protocols/ovsdb/api/src/main/java/org/onosproject/ovsdb/controller/OvsdbBridge.java
+++ b/protocols/ovsdb/api/src/main/java/org/onosproject/ovsdb/controller/OvsdbBridge.java
@@ -20,16 +20,16 @@
import org.onosproject.net.behaviour.BridgeDescription;
import org.onosproject.net.behaviour.BridgeDescription.FailMode;
import org.onosproject.net.behaviour.ControllerInfo;
-
+import org.onosproject.net.behaviour.ControlProtocolVersion;
import static com.google.common.base.MoreObjects.toStringHelper;
import static org.onosproject.ovsdb.controller.OvsdbConstant.DATAPATH_ID;
import static org.onosproject.ovsdb.controller.OvsdbConstant.DISABLE_INBAND;
-
import java.util.List;
import java.util.Map;
import java.util.Objects;
import java.util.Optional;
+
import static com.google.common.base.Preconditions.checkNotNull;
/**
@@ -47,6 +47,9 @@
/* Adds more properties */
private final Optional<String> datapathType;
+ /* Add control protocol version property */
+ private final Optional<List<ControlProtocolVersion>> controlProtocols;
+
/* other optional configs */
private final Map<String, String> otherConfigs;
@@ -57,16 +60,19 @@
* @param failMode openflow controller fail mode policy
* @param controllers list of openflow controllers
* @param datapathType ovs datapath_type
+ * @param controlProtocols list of control protocols
* @param otherConfigs other configs
*/
private OvsdbBridge(String name, Optional<FailMode> failMode,
List<ControllerInfo> controllers,
Optional<String> datapathType,
+ Optional<List<ControlProtocolVersion>> controlProtocols,
Map<String, String> otherConfigs) {
this.name = checkNotNull(name);
this.failMode = failMode;
this.controllers = controllers;
this.datapathType = datapathType;
+ this.controlProtocols = controlProtocols;
this.otherConfigs = otherConfigs;
}
@@ -107,6 +113,14 @@
}
/**
+ * Returns Control protocol versions of the bridge.
+ * @return List of Control protocols
+ */
+ public Optional<List<ControlProtocolVersion>> controlProtocols() {
+ return controlProtocols;
+ }
+
+ /**
* Returns other configurations of the bridge.
*
* @return map of configurations
@@ -148,6 +162,7 @@
.add("failMode", failMode)
.add("controllers", controllers)
.add("datapathType", datapathType)
+ .add("controlProtocols", controlProtocols)
.add("otherConfigs", otherConfigs)
.toString();
}
@@ -180,6 +195,7 @@
private List<ControllerInfo> controllers = Lists.newArrayList();
private Optional<String> datapathType = Optional.empty();
private Map<String, String> otherConfigs = Maps.newHashMap();
+ private Optional<List<ControlProtocolVersion>> controlProtocols = Optional.empty();
private Builder() {
}
@@ -200,6 +216,9 @@
if (bridgeDesc.datapathType().isPresent()) {
this.datapathType = bridgeDesc.datapathType();
}
+ if (bridgeDesc.controlProtocols().isPresent()) {
+ this.controlProtocols = bridgeDesc.controlProtocols();
+ }
this.name = bridgeDesc.name();
this.failMode = bridgeDesc.failMode();
this.controllers = Lists.newArrayList(bridgeDesc.controllers());
@@ -211,7 +230,7 @@
* @return ovsdb bridge
*/
public OvsdbBridge build() {
- return new OvsdbBridge(name, failMode, controllers, datapathType, otherConfigs);
+ return new OvsdbBridge(name, failMode, controllers, datapathType, controlProtocols, otherConfigs);
}
/**
@@ -292,6 +311,16 @@
}
/**
+ * Returns OVSDB bridge builder with given control protocol Versions.
+ * @param controlProtocols list of control protocols
+ * @return ovsdb bridge builder
+ */
+ public Builder controlProtocols(List<ControlProtocolVersion> controlProtocols) {
+ this.controlProtocols = Optional.ofNullable(controlProtocols);
+ return this;
+ }
+
+ /**
* Returns OVSDB bridge builder with a given disable in-band config.
*
* @return ovsdb bridge builder
diff --git a/protocols/ovsdb/api/src/main/java/org/onosproject/ovsdb/controller/OvsdbConstant.java b/protocols/ovsdb/api/src/main/java/org/onosproject/ovsdb/controller/OvsdbConstant.java
index 54c5fcb..f74186e 100644
--- a/protocols/ovsdb/api/src/main/java/org/onosproject/ovsdb/controller/OvsdbConstant.java
+++ b/protocols/ovsdb/api/src/main/java/org/onosproject/ovsdb/controller/OvsdbConstant.java
@@ -45,6 +45,7 @@
// other configs
public static final String DATAPATH_ID = "datapath-id";
public static final String DISABLE_INBAND = "disable-in-band";
+ public static final String PROTOCOLS = "protocols";
/** Port table. */
public static final String PORT = "Port";
diff --git a/protocols/ovsdb/api/src/main/java/org/onosproject/ovsdb/controller/driver/DefaultOvsdbClient.java b/protocols/ovsdb/api/src/main/java/org/onosproject/ovsdb/controller/driver/DefaultOvsdbClient.java
index b36032b..6056be2 100644
--- a/protocols/ovsdb/api/src/main/java/org/onosproject/ovsdb/controller/driver/DefaultOvsdbClient.java
+++ b/protocols/ovsdb/api/src/main/java/org/onosproject/ovsdb/controller/driver/DefaultOvsdbClient.java
@@ -29,6 +29,7 @@
import org.onlab.packet.IpAddress;
import org.onosproject.net.DeviceId;
import org.onosproject.net.PortNumber;
+import org.onosproject.net.behaviour.ControlProtocolVersion;
import org.onosproject.net.behaviour.ControllerInfo;
import org.onosproject.net.behaviour.MirroringName;
import org.onosproject.net.behaviour.MirroringStatistics;
@@ -485,6 +486,12 @@
bridge.setDatapathType(datapathType);
}
+ if (ovsdbBridge.controlProtocols().isPresent()) {
+ bridge.setProtocols(ovsdbBridge.controlProtocols().get().stream()
+ .map(ControlProtocolVersion::toString)
+ .collect(Collectors.toCollection(HashSet::new)));
+ }
+
String bridgeUuid = getBridgeUuid(ovsdbBridge.name());
if (bridgeUuid == null) {
bridge.setName(ovsdbBridge.name());