| commit | 9f676b64af82dd4db4da8a9b2958c4bd1b5258ac | [log] [tgz] |
|---|---|---|
| author | Charles Chan <rascov@gmail.com> | Thu Oct 29 14:58:10 2015 -0700 |
| committer | Gerrit Code Review <gerrit@onlab.us> | Fri Oct 30 15:47:08 2015 +0000 |
| tree | 0774710788adda87395aa360b1a979a7b8866dce | |
| parent | 144bab0c356031a8bb3759c72e8288b62d47da14 [diff] [blame] |
Several fixes for Segment Routing
1. In getSubnetNextObjectiveId, do not create a broadcast group if it does not exist
The creation is done when device is up and when config is added
2. Check if broadcast group exists before creating one
3. Put subnet in a Set instead of a List to avoid duplicated elements
Change-Id: Ifafd8e783a9d98aa62b5e299f560897458b90bb0
diff --git a/apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/RoutingRulePopulator.java b/apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/RoutingRulePopulator.java index 6e4de12..0d7d4c9 100644 --- a/apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/RoutingRulePopulator.java +++ b/apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/RoutingRulePopulator.java
@@ -141,7 +141,7 @@ * @return true if all rules are set successfully, false otherwise */ public boolean populateIpRuleForSubnet(DeviceId deviceId, - List<Ip4Prefix> subnets, + Set<Ip4Prefix> subnets, DeviceId destSw, Set<DeviceId> nextHops) {