]> git.feebdaed.xyz Git - 0xmirror/openstack.git/commitdiff
Update git submodules
authorBalazs Gibizer <gibi@redhat.com>
Thu, 18 Dec 2025 17:23:53 +0000 (18:23 +0100)
committerGerrit Code Review <review@openstack.org>
Thu, 18 Dec 2025 17:23:53 +0000 (17:23 +0000)
* Update nova from branch 'master'
  to 081130d329f2c7386ddbada8c0e5ac47188f5c12
  - Do not mock threading.Event.wait

    There were libvirt disk attach / detach unit test cases that mocked
    threading.Event.wait to speed up the test execution as the libvirt event
    based device detach has a 20 seconds timeout.

    However this mock is very wide. It effects all the threading.Event usage
    during the test case. As we are switched to the native threaded backed
    of oslo.service we got an LoopingCall implementation that relies on
    threading.Event to signal when the call is finished. In these test cases
    the wide mock broke this logic and the LoopingCall signalled "done" while
    the actual call in the background was still running making that call
    leaking out from the test case and therefore under the fixtures the test
    case tore down after the test case finished. This caused that later the
    call hit non mocked code paths like importing the real libvirt lib
    instead of using our libvirt fixture causing late failures.

    The fix is to remove the wide mock and instead change the detach timeout
    in these test cases.

    Later follow ups will check put protection in place to avoid
    re-introducing this wide mock.

    Closes-Bug: #2136815
    Change-Id: I3f40d9dad6ef87e6fa4db0ea4065a77421e8c271
Signed-off-by: Balazs Gibizer <gibi@redhat.com>
nova

diff --git a/nova b/nova
index 3e5a2dbad73ff61d27db8750480e2c0b52133676..081130d329f2c7386ddbada8c0e5ac47188f5c12 160000 (submodule)
--- a/nova
+++ b/nova
@@ -1 +1 @@
-Subproject commit 3e5a2dbad73ff61d27db8750480e2c0b52133676
+Subproject commit 081130d329f2c7386ddbada8c0e5ac47188f5c12