Merge "makes renderer ignore controller-config"
authorCédric Ollivier <ollivier.cedric@gmail.com>
Fri, 1 Sep 2017 07:27:29 +0000 (07:27 +0000)
committerGerrit Code Review <gerrit@opendaylight.org>
Fri, 1 Sep 2017 07:27:29 +0000 (07:27 +0000)
renderer/src/main/java/org/opendaylight/transportpce/renderer/RendererNotificationsImpl.java

index 15fd24aee82f0e7fcde94ef67e124fdc2ed08f65..8ddd081d44c15a75b89fa3e3835fee7504fbc54f 100644 (file)
@@ -171,6 +171,11 @@ public class RendererNotificationsImpl implements DataTreeChangeListener<Node> {
                 nodeId = rootNode.getDataAfter().getKey().getNodeId().getValue();
             }
             if (nnode != null) {
+                if (nodeId.equals("controller-config")) {
+                    // We shouldn't process controller-config as an OpenROAM device
+                    LOG.info("{} ignored: org-openroadm-device advertised but not a real ROADM device", nodeId);
+                    return;
+                }
                 if (rootNode.getModificationType() == ModificationType.WRITE) {
                     LOG.info("Node added " + nodeId);
 
@@ -212,4 +217,4 @@ public class RendererNotificationsImpl implements DataTreeChangeListener<Node> {
             }
         }
     }
-}
\ No newline at end of file
+}