Reviewed: https://review.openstack.org/554940 Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=2a1319ab7a21c2492627c38c5549d20eada899ff Submitter: Zuul Branch: master
commit 2a1319ab7a21c2492627c38c5549d20eada899ff Author: Sławek Kapłoński <[email protected]> Date: Wed Mar 21 15:28:51 2018 +0100 Don't set administratively disabled ports as ACTIVE There was a race condition during port update to disable it. In case when neutron-server receives port update call to set admin_state_down on port, it sends PORT_UPDATE notification to agents. Then both L2 and DHCP agents start doing their job with port. L2 agent asks neutron-server about device details and during processing this call server sets port's status to DOWN if its admin_state_up = False. Problem is that sometimes just after that, DHCP agent will send to neutron server notification that provisioning for this port is finished. As there is no any other provisioning block in db (because it is just port update) neutron-server is setting port's status to ACTIVE. This patch fixes this issue by allowing to transition to ACTIVE only ports which are administratively enabled. Change-Id: If506e0ff68fc49748f19618470c85901339a419b Closes-Bug: #1757089 ** Changed in: neutron Status: In Progress => Fix Released -- You received this bug notification because you are a member of Yahoo! Engineering Team, which is subscribed to neutron. https://bugs.launchpad.net/bugs/1757089 Title: Fullstack test_port_shut_down(Linux bridge agent) fails quite often Status in neutron: Fix Released Bug description: Example of failure logs: http://logs.openstack.org/43/540943/7/check/neutron-fullstack/5746360/logs/testr_results.html.gz but I saw it happen at least 3-4 times during last 2 days. To manage notifications about this bug go to: https://bugs.launchpad.net/neutron/+bug/1757089/+subscriptions -- Mailing list: https://launchpad.net/~yahoo-eng-team Post to : [email protected] Unsubscribe : https://launchpad.net/~yahoo-eng-team More help : https://help.launchpad.net/ListHelp
