]> git.feebdaed.xyz Git - 0xmirror/openstack.git/commit
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)
commit583706f65996c85b2f5c31ba31b17a8ce4f5e6b0
treee598b6cd2c7b9ab4423c7159a02191be829742fe
parentfe37df8fedf23dff5706fc9b97e7cb0c48c3c0db
Update git submodules

* 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