[OPNFLWPLUG-400] Set-Vlan action on OF1.3 should remove 2 flows (simlar to addFlow) Created: 13/Apr/15  Updated: 27/Sep/21  Resolved: 09/Jun/15

Status: Resolved
Project: OpenFlowPlugin
Component/s: General
Affects Version/s: None
Fix Version/s: None

Type: Bug
Reporter: Muralidhar Assignee: Martin Bobak
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All


External issue ID: 2987

 Description   

<snip>

---------- Forwarded message ----------
From: Michal Rehak -X (mirehak - Pantheon Technologies SRO at Cisco) <mirehak@cisco.com>
Date: Mon, Apr 13, 2015 at 4:57 AM
Subject: RE: [openflowplugin-dev] Set VLAN with Opendaylight for OF1.0/1.3
To: Muralidhar Kakulapati <murali@nuviso.com>, Abhijit Kumbhare <abhijitkoss@gmail.com>
Cc: "controller-dev@lists.opendaylight.org" <controller-dev@lists.opendaylight.org>, "openflowplugin-dev@lists.opendaylight.org" <openflowplugin-dev@lists.opendaylight.org>

Hi Murali,
you are right - in way to device (addFlow) there might be 2 flows pushed having only 1 source flow. But when removing flows there is always 1:1 ration used. Please file this as bug.

Once this should get solved by cleaning md-sal models covering the flow. Currently there are sometimes 2 ways on how to specify a match or action. But till then we should fix this by removing 2 flows too.

Regards,
Michal
</snip>

Please refer to the following mail thread:

https://lists.opendaylight.org/pipermail/controller-dev/2015-April/008854.html



 Comments   
Comment by Claudio David Gasparini [ 24/Apr/15 ]

Fix on https://git.opendaylight.org/gerrit/#/c/18992/

Comment by Claudio David Gasparini [ 24/Apr/15 ]

Correct one, fix on https://git.opendaylight.org/gerrit/#/c/19011/

Comment by Claudio David Gasparini [ 07/May/15 ]

https://git.opendaylight.org/gerrit/#/c/19138/

Comment by Abhijit Kumbhare [ 03/Jun/15 ]

Is this planned to be merged Michal? I see the patch https://git.opendaylight.org/gerrit/#/c/19138/ in a waiting for review stage.

Comment by Martin Bobak [ 08/Jun/15 ]

rebased to stable/lithium https://git.opendaylight.org/gerrit/22089

Comment by Michal Rehak [ 09/Jun/15 ]

merged

Generated at Wed Feb 07 20:32:22 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.