diff options
| author | Gurucharan Shetty <guru@ovn.org> | 2016-03-01 15:14:14 -0800 |
|---|---|---|
| committer | Gurucharan Shetty <guru@ovn.org> | 2016-03-17 09:37:42 -0700 |
| commit | 6d1c67619440ecf30f99a805ef1076b8560d39e2 (patch) | |
| tree | 1c93b183371570e848784749b7b66bef835a8055 /vtep | |
| parent | 87530bc1c1c1046c9bcff454e96df15fbb31d1cf (diff) | |
| download | openvswitch-6d1c67619440ecf30f99a805ef1076b8560d39e2.tar.gz | |
ovs-vtep: Delete flows with untagged vlan carefully.
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: #1609938
Reported-by: Mike Qing <mqing@vmware.com>
Signed-off-by: Gurucharan Shetty <guru@ovn.org>
Acked-by: Ryan Moats <rmoats@us.ibm.com>
Diffstat (limited to 'vtep')
| -rwxr-xr-x | vtep/ovs-vtep | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/vtep/ovs-vtep b/vtep/ovs-vtep index 97397b0ca..31ff15928 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) |