l2 pop: check for more than 1 first active port on a node
With high concurrency more than 1 port may be activated on an
OVS agent at the same time (like VM port + a DVR port),
so the patch mitigates the condition by checking for 1 or 2
first active ports.
Given that the condition also contains "or self.agent_restarted(context)"
which makes it True first 180 sec (by default) after agent restart,
I believe the downside of changing 1 to 2 should be negligible.
Please see bug for more details on the issue.
Closes-Bug: #1789846
Change-Id: Ieab0186cbe05185d47bbf5a31141563cf923f66f
(cherry picked from commit b32db30874a7729c4e9209cfc18e106a7e9fc697)
Reviewed: https:/ /review. openstack. org/600151 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=1ba5e6964b5 eadd660f2f333e7 74ce174c4e6f06
Committed: https:/
Submitter: Zuul
Branch: stable/pike
commit 1ba5e6964b5eadd 660f2f333e774ce 174c4e6f06
Author: Oleg Bondarev <email address hidden>
Date: Thu Aug 30 12:25:07 2018 +0400
l2 pop: check for more than 1 first active port on a node
With high concurrency more than 1 port may be activated on an
OVS agent at the same time (like VM port + a DVR port),
so the patch mitigates the condition by checking for 1 or 2
first active ports.
Given that the condition also contains "or self.agent_ restarted( context) "
which makes it True first 180 sec (by default) after agent restart,
I believe the downside of changing 1 to 2 should be negligible.
Please see bug for more details on the issue.
Closes-Bug: #1789846 5d47bbf5a311415 63cf923f66f c4e9209cfc18e10 6a7e9fc697)
Change-Id: Ieab0186cbe0518
(cherry picked from commit b32db30874a7729