commit | b1f02f2e07eba6cce77fbac27aeaa2d421f6a2d7 | [log] [tgz] |
---|---|---|
author | Charles Chan <rascov@gmail.com> | Fri Aug 31 20:19:33 2018 -0700 |
committer | Charles Chan <charles@opennetworking.org> | Wed Sep 05 17:34:13 2018 +0000 |
tree | 02f92173da28fdaaabd781e8b2330306a5a3e4b9 | |
parent | 9d7db8a6cb005fe5a4b8d62b4bce5ab69a0bdfe9 [diff] |
Do not change vlan_meta flow if we are only editing the buckets There may be some other buckets in the hash group relying on the vlan_meta flow Change-Id: Ie208328df0e8b43c5dd836f0d9c43785bc74c908
diff --git a/pipelines/fabric/src/main/java/org/onosproject/pipelines/fabric/pipeliner/FabricNextPipeliner.java b/pipelines/fabric/src/main/java/org/onosproject/pipelines/fabric/pipeliner/FabricNextPipeliner.java index e5207eb..479b1d4 100644 --- a/pipelines/fabric/src/main/java/org/onosproject/pipelines/fabric/pipeliner/FabricNextPipeliner.java +++ b/pipelines/fabric/src/main/java/org/onosproject/pipelines/fabric/pipeliner/FabricNextPipeliner.java
@@ -74,7 +74,10 @@ public PipelinerTranslationResult next(NextObjective nextObjective) { PipelinerTranslationResult.Builder resultBuilder = PipelinerTranslationResult.builder(); - processNextVlanMeta(nextObjective, resultBuilder); + if (nextObjective.op() != Objective.Operation.ADD_TO_EXISTING && + nextObjective.op() != Objective.Operation.REMOVE_FROM_EXISTING) { + processNextVlanMeta(nextObjective, resultBuilder); + } switch (nextObjective.type()) { case SIMPLE: