Committed by
Gerrit Code Review
Use OfdpaSetVlanVid extension instead of setVlanVid in group buckets
GroupBucketEntryBuilder now shares the same treatment builder as FlowEntryBuilder. Therefore, we also need to use the extension when building group buckets. https://gerrit.onosproject.org/#/c/8485/ Change-Id: Ic85dd10d1bfd8c0ca26f5811369f0587dc286761
Showing
1 changed file
with
5 additions
and
2 deletions
... | @@ -13,6 +13,7 @@ import org.onlab.packet.MplsLabel; | ... | @@ -13,6 +13,7 @@ import org.onlab.packet.MplsLabel; |
13 | import org.onlab.packet.VlanId; | 13 | import org.onlab.packet.VlanId; |
14 | import org.onosproject.core.ApplicationId; | 14 | import org.onosproject.core.ApplicationId; |
15 | import org.onosproject.core.DefaultGroupId; | 15 | import org.onosproject.core.DefaultGroupId; |
16 | +import org.onosproject.driver.extensions.OfdpaSetVlanVid; | ||
16 | import org.onosproject.net.DeviceId; | 17 | import org.onosproject.net.DeviceId; |
17 | import org.onosproject.net.PortNumber; | 18 | import org.onosproject.net.PortNumber; |
18 | import org.onosproject.net.behaviour.NextGroup; | 19 | import org.onosproject.net.behaviour.NextGroup; |
... | @@ -306,7 +307,8 @@ public class Ofdpa2GroupHandler { | ... | @@ -306,7 +307,8 @@ public class Ofdpa2GroupHandler { |
306 | break; | 307 | break; |
307 | case VLAN_ID: | 308 | case VLAN_ID: |
308 | vlanid = ((L2ModificationInstruction.ModVlanIdInstruction) l2ins).vlanId(); | 309 | vlanid = ((L2ModificationInstruction.ModVlanIdInstruction) l2ins).vlanId(); |
309 | - outerTtb.setVlanId(vlanid); | 310 | + OfdpaSetVlanVid ofdpaSetVlanVid = new OfdpaSetVlanVid(vlanid); |
311 | + outerTtb.extension(ofdpaSetVlanVid, deviceId); | ||
310 | setVlan = true; | 312 | setVlan = true; |
311 | break; | 313 | break; |
312 | case VLAN_POP: | 314 | case VLAN_POP: |
... | @@ -339,7 +341,8 @@ public class Ofdpa2GroupHandler { | ... | @@ -339,7 +341,8 @@ public class Ofdpa2GroupHandler { |
339 | } | 341 | } |
340 | // if vlan is not set, use the vlan in metadata for outerTtb | 342 | // if vlan is not set, use the vlan in metadata for outerTtb |
341 | if (vlanid != null && !setVlan) { | 343 | if (vlanid != null && !setVlan) { |
342 | - outerTtb.setVlanId(vlanid); | 344 | + OfdpaSetVlanVid ofdpaSetVlanVid = new OfdpaSetVlanVid(vlanid); |
345 | + outerTtb.extension(ofdpaSetVlanVid, deviceId); | ||
343 | } | 346 | } |
344 | } | 347 | } |
345 | 348 | ... | ... |
-
Please register or login to post a comment