[ovs-dev] [PATCH] ovs-vtep: Delete flows with untagged vlan carefully.

[Gurucharan Shetty]

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 <mqing at vmware.com>
Signed-off-by: Gurucharan Shetty <guru at 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        mike at osdn.org.ua
 Mihir Gangar            gangarm at vmware.com
 Mike Bursell            mike.bursell at citrix.com
 Mike Kruze              mkruze at nicira.com
+Mike Qing               mqing at vmware.com
 Min Chen                ustcer.tonychan at gmail.com
 Mikael Doverhag         mdoverhag at nicira.com
 Mrinmoy Das             mrdas at 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