]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Iterate over same port_id if more than one exists
authorJohn Schwarz <jschwarz@redhat.com>
Thu, 18 Sep 2014 08:24:53 +0000 (11:24 +0300)
committerJohn Schwarz <jschwarz@redhat.com>
Thu, 2 Oct 2014 08:51:21 +0000 (11:51 +0300)
commite90d6d6c833db51964037afd1c4aa1f521ee380a
tree27bd8d31548cf62de703fa7bdeb944042b5ae6cc
parent205162f58050fcb94db53cd51b674d2093dfe700
Iterate over same port_id if more than one exists

In certain cases where multiple ports can have the same
external_ids:iface_id property, the ovs agent will arbitrarily choose
one and ignore the rest. In case the chosen port isn't on the
integration bridge the ovs agent is managing, an error is returned to
the calling function. This is faulty since one of the other ports may
belong to the correct bridge and it should be chosen instead.

This is interesting for future L3 HA integration tests, where 2
different instances of l3 agents are needed to run on the same machine.
In this case, both agents will register ports which have the same
iface_id property, but obviously only one of the ports is relevant for
each agent.

Closes-bug: #1370914
Related-bug: #1374947
Change-Id: I05d70417e0f78ae51a9ce377fc93a3f12046b313
neutron/agent/linux/ovs_lib.py
neutron/tests/unit/agent/linux/test_ovs_lib.py