[FT] Error in "test_virtual_port_host_update"

[Rodolfo Alonso]

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/

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/841298

[Lajos Katona (lajos-katona)]

It happened a few times:
https://opensearch.logs.openstack.org/_dashboards/app/discover/?security_tenant=global#/?_g=(filters:!(),refreshInterval:(pause:!t,value:0),time:(from:now-7d,to:now))&_a=(columns:!(build_name),filters:!(),index:'94869730-aea8-11ec-9e6a-83741af3fdcd',interval:auto,query:(language:kuery,query:'message:%22AssertionError:%20Expected%20!'update_virtual_port_host!'%20to%20be%20called%20once%22'),sort:!())

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/neutron/+/841298
Committed: https://opendev.org/openstack/neutron/commit/494c477b21ebbaa441b55d63755af72cc24244af
Submitter: "Zuul (22348)"
Branch: master

commit 494c477b21ebbaa441b55d63755af72cc24244af
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Sat May 7 02:15:16 2022 +0000

    [OVN][FT] Wait until virtual parents are written

    In "test_virtual_port_host_update", wait until VIP virtual parents
    have been updated in the SB "Port_Binding" register. Then, the
    test should check if "update_virtual_port_host" has been called or
    not.

    Closes-Bug: #1971672
    Change-Id: Ifa04bb59f4b9acd308299cfa44f2316394d14505

[yatin (yatinkarel)]

Seeing again https://cb16041cfb3c54cedd2e-24bc61d83ed5aece64ab40b405cf025c.ssl.cf5.rackcdn.com/797121/16/check/neutron-functional-with-uwsgi/12ca1dc/testr_results.html

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/848156

[OpenStack Infra (hudson-openstack)]

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 <email address hidden>
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

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/neutron 21.0.0.0rc1 release candidate.