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 <mq...@vmware.com>
Signed-off-by: Gurucharan Shetty <g...@ovn.org>
---
 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        m...@osdn.org.ua
 Mihir Gangar            gang...@vmware.com
 Mike Bursell            mike.burs...@citrix.com
 Mike Kruze              mkr...@nicira.com
+Mike Qing               mq...@vmware.com
 Min Chen                ustcer.tonyc...@gmail.com
 Mikael Doverhag         mdover...@nicira.com
 Mrinmoy Das             mr...@ixiacom.com
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
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to