commit 94fa13ed46f5a6fa2f535f3284a94c361054a4c0
author Victor Silva <victor@waltznetworks.com> Fri Jul 22 16:08:33 2016 -0300
committer Victor Silva <victor@waltznetworks.com> Wed Aug 24 15:15:16 2016 -0300
tree 7a81d768fcabbcb6054c93a75f7d8fb8ce889cc7
parent 8ba356cd7f483ca54fb74fc9751d52adb713c22f

[ONOS-5166] GroupModBuilder: no longer ignore weights

We provide an interface for assigning weights to buckets
in a group and thus we should not silently ignore it.
Properly set bucket weight to bucket.weight().

Change-Id: I7dce89e793140ee365a016f4bf26f0ccf206b8dd

providers/openflow/group/src/main/java/org/onosproject/provider/of/group/impl/GroupModBuilder.java

diff --git a/providers/openflow/group/src/main/java/org/onosproject/provider/of/group/impl/GroupModBuilder.java b/providers/openflow/group/src/main/java/org/onosproject/provider/of/group/impl/GroupModBuilder.java
index d473e66..464c2cc 100644
--- a/providers/openflow/group/src/main/java/org/onosproject/provider/of/group/impl/GroupModBuilder.java
+++ b/providers/openflow/group/src/main/java/org/onosproject/provider/of/group/impl/GroupModBuilder.java
@@ -152,7 +152,7 @@
             OFBucket.Builder bucketBuilder = factory.buildBucket();
             bucketBuilder.setActions(actions);
             if (type == GroupDescription.Type.SELECT) {
-                bucketBuilder.setWeight(1);
+                bucketBuilder.setWeight(bucket.weight());
             }
 
             if (type == GroupDescription.Type.FAILOVER && bucket.watchPort() != null) {
@@ -192,7 +192,7 @@
             OFBucket.Builder bucketBuilder = factory.buildBucket();
             bucketBuilder.setActions(actions);
             if (type == GroupDescription.Type.SELECT) {
-                bucketBuilder.setWeight(1);
+                bucketBuilder.setWeight(bucket.weight());
             }
             if (type == GroupDescription.Type.FAILOVER && bucket.watchPort() != null) {
                 bucketBuilder.setWatchPort(OFPort.of((int) bucket.watchPort().toLong()));