commit 9272613b40c2d103f79e897541b5d3121959ede1
author Charles Chan <rascov@gmail.com> Fri Feb 16 17:20:54 2018 -0800
committer Charles Chan <charles@opennetworking.org> Wed Feb 21 22:26:54 2018 +0000
tree d3dd0d8f265fbeb08c6a78837560058fb266517e
parent 465c7be771e36fc0c38b045ccf90e4a7550e72b0

Skip unconfigured devices rather than aborting the entire process

Change-Id: Iae0c165dc4b15798a53b9aa269f0304f2d610fdf

apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java

diff --git a/apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java b/apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java
index 882bbe7..5baa6a9 100644
--- a/apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java
+++ b/apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java
@@ -338,9 +338,8 @@
                     try {
                         isEdge = config.isEdgeDevice(targetSw.id());
                     } catch (DeviceConfigNotFoundException e) {
-                        log.warn(e.getMessage() + "aborting populateSubnet");
-                        populationStatus = Status.ABORTED;
-                        return;
+                        log.warn(e.getMessage() + "aborting populateSubnet on targetSw {}", targetSw.id());
+                        continue;
                     }
                     if (dstSw.equals(targetSw.id()) || !isEdge ||
                             (cpts.size() == 2 &&