Topology manager - implementation of NodeChangeListener 02/18202/2
authorJozef Gloncak <jgloncak@cisco.com>
Mon, 13 Apr 2015 14:32:37 +0000 (16:32 +0200)
committerJozef Gloncak <jgloncak@cisco.com>
Mon, 13 Apr 2015 14:37:35 +0000 (16:37 +0200)
If new FlowCapableNode is added/removed to/from inventory then new node
is added/removed to/from topology.

Change-Id: Ie674e32258b67ffc1fb50849a85805bfcd9c6dbb
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
applications/topology-manager/src/main/java/org/opendaylight/openflowplugin/applications/topology/manager/FlowCapableTopologyProvider.java
applications/topology-manager/src/main/java/org/opendaylight/openflowplugin/applications/topology/manager/LinkChangeListenerImpl.java
applications/topology-manager/src/main/java/org/opendaylight/openflowplugin/applications/topology/manager/NodeChangeListenerImpl.java

index 9a2b7b82f0dd77e6c15d65ad94a9c172d547db99..7b7071cb910863795d60d26f91d72e78ecbdb0e9 100644 (file)
@@ -53,8 +53,8 @@ public class FlowCapableTopologyProvider extends AbstractBindingAwareProvider im
         final OperationProcessor processor = new OperationProcessor(dataBroker);
         final FlowCapableTopologyExporter listener = new FlowCapableTopologyExporter(processor, path);
         this.listenerRegistration = notificationService.registerNotificationListener(listener);
-        linkChangeListener = new LinkChangeListenerImpl(dataBroker);
-        nodeChangeListener = new NodeChangeListenerImpl(dataBroker);
+        linkChangeListener = new LinkChangeListenerImpl(dataBroker, processor);
+        nodeChangeListener = new NodeChangeListenerImpl(dataBroker, processor);
 
         final ReadWriteTransaction tx = dataBroker.newReadWriteTransaction();
         tx.put(LogicalDatastoreType.OPERATIONAL, path, new TopologyBuilder().setKey(key).build(), true);
index f6be043bef5a9c20112e8a73dd8cdc7b94c2a4e1..ef0f4d9fdb0eb262d961fc2592a0a20f628ccea8 100644 (file)
@@ -11,7 +11,6 @@ import java.util.Map;
 import java.util.Set;
 import org.opendaylight.controller.md.sal.common.api.data.AsyncDataChangeEvent;
 import org.opendaylight.yangtools.yang.binding.DataObject;
-
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.md.sal.binding.api.DataChangeListener;
 import org.opendaylight.controller.md.sal.common.api.data.AsyncDataBroker;
@@ -25,8 +24,9 @@ import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 
 public class LinkChangeListenerImpl implements DataChangeListener, AutoCloseable {
     private final ListenerRegistration<DataChangeListener> dataChangeListenerRegistration;
+    private OperationProcessor operationProcessor;
 
-    public LinkChangeListenerImpl(final DataBroker dataBroker) {
+    public LinkChangeListenerImpl(final DataBroker dataBroker, final OperationProcessor operationProcessor) {
         dataChangeListenerRegistration = dataBroker.registerDataChangeListener(
                 LogicalDatastoreType.OPERATIONAL,
                 InstanceIdentifier.builder(Nodes.class)
@@ -35,6 +35,7 @@ public class LinkChangeListenerImpl implements DataChangeListener, AutoCloseable
                         .augmentation(FlowCapableNodeConnector.class)
                         .build(),
                 this, AsyncDataBroker.DataChangeScope.BASE);
+        this.operationProcessor = operationProcessor;
     }
 
     @Override
index fbf29b37f9b14c6db6ab648ba18bcc21f16cf66d..5fdd6e507ca756ac3f74490185eeceaca6c7f84e 100644 (file)
@@ -7,8 +7,19 @@
  */
 package org.opendaylight.openflowplugin.applications.topology.manager;
 
-import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.FlowCapableNode;
 
+import org.opendaylight.controller.md.sal.binding.api.ReadWriteTransaction;
+import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.NodeBuilder;
+import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.NodeId;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.NodeKey;
+import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.TopologyId;
+import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.TopologyKey;
+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.NetworkTopology;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import java.util.Map.Entry;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.FlowCapableNode;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.Node;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.Nodes;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
@@ -22,53 +33,109 @@ import org.opendaylight.yangtools.yang.binding.DataObject;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 import org.opendaylight.controller.md.sal.common.api.data.AsyncDataBroker;
 
-    public class NodeChangeListenerImpl implements DataChangeListener, AutoCloseable {
-        private final ListenerRegistration<DataChangeListener> dataChangeListenerRegistration;
-
-        public NodeChangeListenerImpl(final DataBroker dataBroker) {
-            dataChangeListenerRegistration = dataBroker.registerDataChangeListener(
-                    LogicalDatastoreType.OPERATIONAL,
-                    InstanceIdentifier.builder(Nodes.class)
-                            .child(Node.class)
-                            .augmentation(FlowCapableNode.class)
-                            .build(),
-                    this, AsyncDataBroker.DataChangeScope.BASE);
-        }
+public class NodeChangeListenerImpl implements DataChangeListener, AutoCloseable {
+    private final static Logger LOG = LoggerFactory.getLogger(NodeChangeListenerImpl.class);
 
-        @Override
-        public void onDataChanged(AsyncDataChangeEvent<InstanceIdentifier<?>, DataObject> change) {
-            processAddedNode(change.getCreatedData());
-            processUpdatedNode(change.getUpdatedData());
-            processRemovedNode(change.getRemovedPaths());
-        }
+    private final static String topologyId = "topology id";
 
-        /**
-         * @param removedPaths
-         */
-        private void processRemovedNode(Set<InstanceIdentifier<?>> removedPaths) {
-            // TODO Auto-generated method stub
+    /**
+     * instance identifier to Node in network topology model (yangtools)
+     */
+    private static final InstanceIdentifier<Topology> II_TO_TOPOLOGY =
+            InstanceIdentifier
+            .builder(NetworkTopology.class)
+            .child(Topology.class, new TopologyKey(new TopologyId(topologyId)))
+            .build();
 
-        }
+    private final ListenerRegistration<DataChangeListener> dataChangeListenerRegistration;
+    private OperationProcessor operationProcessor;
+
+    public NodeChangeListenerImpl(final DataBroker dataBroker, final OperationProcessor operationProcessor) {
+        //TODO: listener on FlowCapableNode. what if node id in Node.class is changed (it won't be caught by this listener)
+        dataChangeListenerRegistration = dataBroker.registerDataChangeListener(LogicalDatastoreType.OPERATIONAL,
+                InstanceIdentifier.builder(Nodes.class).child(Node.class).augmentation(FlowCapableNode.class).build(),
+                this, AsyncDataBroker.DataChangeScope.BASE);
+        this.operationProcessor = operationProcessor;
+    }
+
+    @Override
+    public void onDataChanged(AsyncDataChangeEvent<InstanceIdentifier<?>, DataObject> change) {
+        processAddedNode(change.getCreatedData());
+        processUpdatedNode(change.getUpdatedData());
+        processRemovedNode(change.getRemovedPaths());
+    }
 
-        /**
-         * @param updatedData
-         */
-        private void processUpdatedNode(Map<InstanceIdentifier<?>, DataObject> updatedData) {
-            // TODO Auto-generated method stub
+    /**
+     * @param removedPaths
+     */
+    private void processRemovedNode(Set<InstanceIdentifier<?>> removedNodes) {
+        for (final InstanceIdentifier<?> removedNode : removedNodes) {
+            operationProcessor.enqueueOperation(new TopologyOperation() {
 
+                @Override
+                public void applyOperation(ReadWriteTransaction transaction) {
+                    transaction.delete(LogicalDatastoreType.OPERATIONAL, removedNode);
+                }
+            });
         }
+    }
 
-        /**
-         * @param createdData
-         */
-        private void processAddedNode(Map<InstanceIdentifier<?>, DataObject> createdData) {
-            // TODO Auto-generated method stub
+    /**
+     * @param updatedData
+     */
+    private void processUpdatedNode(Map<InstanceIdentifier<?>, DataObject> updatedData) {
+        //TODO: only node id is used from incomming data object.
+        //if it is changed what should happen? Listener is on FlocCapableNode so change
+        //of node id (only data which are used) isn't caught.
+    }
 
+    /**
+     * @param createdData
+     */
+    private void processAddedNode(Map<InstanceIdentifier<?>, DataObject> addedDatas) {
+        for (Entry<InstanceIdentifier<?>, DataObject> addedData : addedDatas.entrySet()) {
+            if (addedData.getValue() instanceof FlowCapableNode) {
+                createNewNodeInTopology(addedData.getKey(), (FlowCapableNode) (addedData.getValue()));
+            } else {
+                LOG.debug("Expected data of type FlowCapableNode but {} was obtainedl", addedData.getClass().getName());
+            }
         }
 
-        @Override
-        public void close() throws Exception {
-            dataChangeListenerRegistration.close();
+    }
+
+    /**
+     * @param iiToNodeInInventory
+     * @param addedData
+     */
+    private void createNewNodeInTopology(InstanceIdentifier<?> iiToNodeInInventory, final FlowCapableNode addedData) {
+        final NodeBuilder topologyNodeBuilder = new NodeBuilder();
+        final NodeKey inventoryNodeKey = iiToNodeInInventory.firstKeyOf(Node.class, NodeKey.class);
+        if (inventoryNodeKey != null) {
+            NodeId nodeIdInTopology = new NodeId(inventoryNodeKey.getId().getValue());
+            org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.NodeKey nodeKeyInTopology = new org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.NodeKey(nodeIdInTopology);
+            topologyNodeBuilder.setNodeId(nodeIdInTopology);
+
+            final InstanceIdentifier<org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.Node> iiToTopologyNode = II_TO_TOPOLOGY
+            .builder()
+            .child(org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.Node.class, nodeKeyInTopology)
+            .build();
+
+            operationProcessor.enqueueOperation(new TopologyOperation() {
+
+                @Override
+                public void applyOperation(ReadWriteTransaction transaction) {
+                    transaction.put(LogicalDatastoreType.OPERATIONAL, iiToTopologyNode, topologyNodeBuilder.build());
+                }
+            });
+        } else {
+            LOG.debug("Inventory node key is null. Data can't be written to topology");
         }
 
-    }
\ No newline at end of file
+    }
+
+    @Override
+    public void close() throws Exception {
+        dataChangeListenerRegistration.close();
+    }
+
+}
\ No newline at end of file