A (physical port + vlan) or (physical port + no tag) can be bound to a
logical switch. When one unbinds (physical port + no tag) from a logical
switch, the emulator inadvertantly deletes flows for (physical port + vlan)
too. This commit fixes it.

VMware-BZ: #1598643
Reported-by: Mike Qing <[email protected]>
Signed-off-by: Gurucharan Shetty <[email protected]>
---
 AUTHORS       |    1 +
 vtep/ovs-vtep |    4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/AUTHORS b/AUTHORS
index 96bdc4f..e7c68da 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -353,6 +353,7 @@ Michael Shigorin        [email protected]
 Mihir Gangar            [email protected]
 Mike Bursell            [email protected]
 Mike Kruze              [email protected]
+Mike Qing               [email protected]
 Min Chen                [email protected]
 Mikael Doverhag         [email protected]
 Mrinmoy Das             [email protected]
diff --git a/vtep/ovs-vtep b/vtep/ovs-vtep
index 97397b0..31ff159 100755
--- a/vtep/ovs-vtep
+++ b/vtep/ovs-vtep
@@ -545,8 +545,8 @@ def del_binding(binding, ls):
                   % (ps_name, port_no, vlan_))
         ovs_ofctl("del-flows %s in_port=%s" % (ps_name, patch_no))
     else:
-        ovs_ofctl("del-flows %s in_port=%s" % (ps_name, port_no))
-        ovs_ofctl("del-flows %s in_port=%s" % (ps_name, patch_no))
+        ovs_ofctl("--strict del-flows %s in_port=%s" % (ps_name, port_no))
+        ovs_ofctl("--strict del-flows %s in_port=%s" % (ps_name, patch_no))
 
     ls.del_lbinding(lbinding)
 
-- 
1.7.9.5

_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to