Renaming ovsdb-managed-node-augmentation to ovsdb-bridge-augmentation. 76/16276/1
authorSharad Mishra <sharad.d.mishra@intel.com>
Tue, 10 Mar 2015 07:57:16 +0000 (00:57 -0700)
committerSharad Mishra <sharad.d.mishra@intel.com>
Tue, 10 Mar 2015 07:59:31 +0000 (00:59 -0700)
ovsdb yang model was using ovsdb-managed-node for bridges. This was causing confusion and was renamed to ovsdb-bridge.

Change-Id: I73477cd62f4c361738fa5cb10cfbff7e1ebe1cd7
Signed-off-by: Sharad Mishra <sharad.d.mishra@intel.com>
southbound/southbound-api/src/main/yang/ovsdb.yang
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/OvsdbConnectionManager.java
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/OvsdbManagedNodeDataChangeListener.java
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbBridgeUpdateCommand.java

index f46b89ea317ea3fdfcf8da468f182da60ab022f5..ac27f72709aec7bbb15c4e75f90f5807c2415f8a 100644 (file)
@@ -106,7 +106,7 @@ module ovsdb {
 
     augment "/topo:network-topology/topo:topology/topo:node" {
         description "Augmentation for bridge nodes managed by ovsdb";
-        ext:augment-identifier "ovsdb-managed-node-augmentation";
+        ext:augment-identifier "ovsdb-bridge-augmentation";
         uses ovsdb-bridge-attributes;
     }
 
index 95ca90d9e34de59c49b09051cfc9a6fd4c411fc5..7946d03404e09851d3afa2395b1ecbd5c0a6cac0 100644 (file)
@@ -23,7 +23,7 @@ import org.opendaylight.ovsdb.southbound.transactions.md.OvsdbNodeRemoveCommand;
 import org.opendaylight.ovsdb.southbound.transactions.md.TransactionInvoker;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.overlay.rev150105.IpPortLocator;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbBridgeAttributes;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbManagedNodeAugmentation;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbBridgeAugmentation;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbNodeAugmentation;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbNodeRef;
 import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.Node;
@@ -130,7 +130,7 @@ public class OvsdbConnectionManager implements OvsdbConnectionListener, AutoClos
     public OvsdbConnectionInstance getConnectionInstance(Node node) {
         Preconditions.checkNotNull(node);
         OvsdbNodeAugmentation ovsdbNode = node.getAugmentation(OvsdbNodeAugmentation.class);
-        OvsdbManagedNodeAugmentation ovsdbManagedNode = node.getAugmentation(OvsdbManagedNodeAugmentation.class);
+        OvsdbBridgeAugmentation ovsdbManagedNode = node.getAugmentation(OvsdbBridgeAugmentation.class);
         if(ovsdbNode != null) {
             return getConnectionInstance(ovsdbNode);
         } else if (ovsdbManagedNode != null) {
index 57dddfd31757a15c577617f77c0a9ce036578b53..52944d2423b5308a2a616151a39ad05439e2fd9a 100644 (file)
@@ -15,7 +15,7 @@ import org.opendaylight.controller.md.sal.common.api.data.AsyncDataBroker.DataCh
 import org.opendaylight.controller.md.sal.common.api.data.AsyncDataChangeEvent;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.ovsdb.lib.OvsdbClient;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbManagedNodeAugmentation;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbBridgeAugmentation;
 import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.NetworkTopology;
 import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.Topology;
 import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.TopologyKey;
@@ -37,11 +37,11 @@ public class OvsdbManagedNodeDataChangeListener implements DataChangeListener, A
         LOG.info("Registering OvsdbManagedNodeDataChangeListener");
         this.cm = cm;
         this.db = db;
-        InstanceIdentifier<OvsdbManagedNodeAugmentation> path = InstanceIdentifier
+        InstanceIdentifier<OvsdbBridgeAugmentation> path = InstanceIdentifier
                 .create(NetworkTopology.class)
                 .child(Topology.class, new TopologyKey(SouthboundConstants.OVSDB_TOPOLOGY_ID))
                 .child(Node.class)
-                .augmentation(OvsdbManagedNodeAugmentation.class);
+                .augmentation(OvsdbBridgeAugmentation.class);
         registration = db.registerDataChangeListener(LogicalDatastoreType.CONFIGURATION, path, this, DataChangeScope.ONE);
 
     }
@@ -52,15 +52,15 @@ public class OvsdbManagedNodeDataChangeListener implements DataChangeListener, A
        LOG.info("Received change to ovsdbManagedNode: {}", changes);
        for( Entry<InstanceIdentifier<?>, DataObject> created : changes.getCreatedData().entrySet()) {
            // TODO validate we have the correct kind of InstanceIdentifier
-           if(created.getValue() instanceof OvsdbManagedNodeAugmentation) {
+           if(created.getValue() instanceof OvsdbBridgeAugmentation) {
                LOG.debug("Received request to create {}",created.getValue());
-               OvsdbClient client = cm.getClient((OvsdbManagedNodeAugmentation)created.getValue());
+               OvsdbClient client = cm.getClient((OvsdbBridgeAugmentation)created.getValue());
                if(client != null) {
                    LOG.debug("Found client for {}", created.getValue());
                } else {
                    LOG.debug("Did not find client for {}",created.getValue());
                }
-               // TODO - translate the OvsdbManagedNodeAugmentation to libary/ transacts to create the node
+               // TODO - translate the OvsdbBridgeAugmentation to libary/ transacts to create the node
            }
        }
        // TODO handle case of updates to ovsdb managed nodes as needed
index ba9f729ecbaf0487cd817d5018c43cc722f6d35c..209529c8a24f2ee6643198d082c26e8f4a30507e 100644 (file)
@@ -16,8 +16,8 @@ import org.opendaylight.ovsdb.southbound.SouthboundMapper;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev130715.Uuid;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbBridgeName;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbBridgeRef;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbManagedNodeAugmentation;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbManagedNodeAugmentationBuilder;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbBridgeAugmentation;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbBridgeAugmentationBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbNodeAugmentation;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbNodeAugmentationBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.rev150105.OvsdbNodeRef;
@@ -57,11 +57,11 @@ public class OvsdbBridgeUpdateCommand extends AbstractTransactionCommand {
                 NodeId manageNodeId = SouthboundMapper.createManagedNodeId(getKey(), new OvsdbBridgeName(bridge.getName()));
                 managedNodeBuilder.setNodeId(manageNodeId);
 
-                OvsdbManagedNodeAugmentationBuilder ovsdbManagedNodeBuilder = new OvsdbManagedNodeAugmentationBuilder();
+                OvsdbBridgeAugmentationBuilder ovsdbManagedNodeBuilder = new OvsdbBridgeAugmentationBuilder();
                 ovsdbManagedNodeBuilder.setBridgeName(new OvsdbBridgeName(bridge.getName()));
                 ovsdbManagedNodeBuilder.setBridgeUuid(new Uuid(bridge.getUuid().toString()));
                 ovsdbManagedNodeBuilder.setManagedBy(new OvsdbNodeRef(nodePath));
-                managedNodeBuilder.addAugmentation(OvsdbManagedNodeAugmentation.class, ovsdbManagedNodeBuilder.build());
+                managedNodeBuilder.addAugmentation(OvsdbBridgeAugmentation.class, ovsdbManagedNodeBuilder.build());
 
                 InstanceIdentifier<Node> managedNodePath = SouthboundMapper.createInstanceIdentifier(manageNodeId);