]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Add missing match in OVS agent tunnel mac leaning
authorFrancois Eleouet <f.eleouet@gmail.com>
Fri, 30 Aug 2013 20:30:04 +0000 (22:30 +0200)
committerFrancois Eleouet <f.eleouet@gmail.com>
Fri, 30 Aug 2013 20:37:38 +0000 (22:37 +0200)
Learned flows introduced in commit a369f9e39691c01a4e4f7f8668cb37fc17ba03b3
to learn remote mac addresses on tunnels currently only match local VLAN,
but not destination mac address, as a consequence, connectivity may be
erratic as soon as more than two agents are used

Change-Id: I467e47d0dd2c1e5aca12a4e240e348116d530d4f
Closes-Bug: #1219074

neutron/plugins/openvswitch/agent/ovs_neutron_agent.py
neutron/tests/unit/openvswitch/test_ovs_tunnel.py

index 6cdca563c23b31422f0c9c034bc421aa95ebaa64..5216e297163584ba94bb68db965a20c42b19f4c9 100644 (file)
@@ -610,6 +610,7 @@ class OVSNeutronAgent(sg_rpc.SecurityGroupAgentRpcCallbackMixin):
                         "priority=1,"
                         "hard_timeout=300,"
                         "NXM_OF_VLAN_TCI[0..11],"
+                        "NXM_OF_ETH_DST[]=NXM_OF_ETH_SRC[],"
                         "load:0->NXM_OF_VLAN_TCI[],"
                         "load:NXM_NX_TUN_ID[]->NXM_NX_TUN_ID[],"
                         "output:NXM_OF_IN_PORT[]" %
index c9f0f5791b2f58743bf53de4d0870db7cd7978e3..571b2361e676cf70de57bdb12c845ec7a6d29f1c 100644 (file)
@@ -139,6 +139,7 @@ class TunnelTest(base.BaseTestCase):
                         "priority=1,"
                         "hard_timeout=300,"
                         "NXM_OF_VLAN_TCI[0..11],"
+                        "NXM_OF_ETH_DST[]=NXM_OF_ETH_SRC[],"
                         "load:0->NXM_OF_VLAN_TCI[],"
                         "load:NXM_NX_TUN_ID[]->NXM_NX_TUN_ID[],"
                         "output:NXM_OF_IN_PORT[]" %