yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #89230
[Bug 1971672] Re: [FT] Error in "test_virtual_port_host_update"
Reviewed: https://review.opendev.org/c/openstack/neutron/+/848156
Committed: https://opendev.org/openstack/neutron/commit/753fbff11bb7500db4293cb79fbf3d7cb0b2b121
Submitter: "Zuul (22348)"
Branch: master
commit 753fbff11bb7500db4293cb79fbf3d7cb0b2b121
Author: Rodolfo Alonso Hernandez <ralonsoh@xxxxxxxxxx>
Date: Wed Jun 29 18:12:04 2022 +0000
[FT] Ensure "update_virtual_port_host" has been called
An OVS DB event once matched, is stored in a cache and processed in
``ovsdbapp.event.RowEventHandler.notify_loop``. The method
``update_virtual_port_host`` is called in the ``run`` method of the
event. Sometimes in the FT execution, the test checks if the
method has been called when the event ``run`` method has not been
executed yet.
This patch adds a short active wait to ensure the event is actually
processed and the method called.
This bug can be easily reproduced if in the
``PortBindingUpdateVirtualPortsEvent.run`` method, a 2 seconds delay
is added:
import time; time.sleep(2)
Closes-Bug: #1971672
Change-Id: I2e389e3cce70505ce01f03c09b36fa9d55be964a
** 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/1971672
Title:
[FT] Error in "test_virtual_port_host_update"
Status in neutron:
Fix Released
Bug description:
Error executing
neutron.tests.functional.plugins.ml2.drivers.ovn.mech_driver.ovsdb.test_ovsdb_monitor.TestNBDbMonitorOverSsl.test_virtual_port_host_update
Log:
https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_207/840448/2/check/neutron-
functional-with-uwsgi/2074ad0/testr_results.html
Snippet: https://paste.opendev.org/show/bVvDh4svozwjBpLokYre/
To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1971672/+subscriptions
References