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)
(cherry picked from commit 1ba5e6964b5eadd660f2f333e774ce174c4e6f06)
Reviewed: https:/ /review. openstack. org/649417 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=e8033b74c1c af6af6428cd814b ac56cda2a8e600
Committed: https:/
Submitter: Zuul
Branch: stable/ocata
commit e8033b74c1caf6a f6428cd814bac56 cda2a8e600
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) 660f2f333e774ce 174c4e6f06)
Change-Id: Ieab0186cbe0518
(cherry picked from commit b32db30874a7729
(cherry picked from commit 1ba5e6964b5eadd