rbd_utils unit tests not running in gate

Bug #1928007 reported by melanie witt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Undecided
melanie witt
Victoria
Fix Released
Undecided
Unassigned
Wallaby
Fix Released
Undecided
Unassigned

Bug Description

Awhile back, change:

https://review.opendev.org/c/openstack/nova/+/746904

moved rbd_utils out from the libvirt driver and into a central location under nova/storage. This move missed adding a __init__.py file under nova/tests/unit/storage, so unit test discovery wasn't picking up the rbd_utils tests and couldn't run them.

There was also a small bug introduced by change:

https://review.opendev.org/c/openstack/nova/+/574301

in RbdTestCase.setUp() that passed nonexistent self.images_rbd_pool to self.flags. It should be self.rbd_pool.

Tags: testing
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Changed in nova:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.opendev.org/c/openstack/nova/+/790511
Committed: https://opendev.org/openstack/nova/commit/8b647f1b3f56879be221b3925570790a1e0e77f8
Submitter: "Zuul (22348)"
Branch: master

commit 8b647f1b3f56879be221b3925570790a1e0e77f8
Author: melanie witt <email address hidden>
Date: Mon May 10 17:31:25 2021 +0000

    rbd: Get rbd_utils unit tests running again

    Awhile back, change I25baf5edd25d9e551686b7ed317a63fd778be533 moved
    rbd_utils out from the libvirt driver and into a central location under
    nova/storage. This move missed adding a __init__.py file under
    nova/tests/unit/storage, so unit test discovery wasn't picking up the
    rbd_utils tests and couldn't run them.

    This adds a __init__.py file under nova/tests/unit/storage to get the
    tests running again.

    This also fixes a small bug introduced by change
    I3032bbe6bd2d6acc9ba0f0cac4d00ed4b4464ceb in RbdTestCase.setUp() that
    passed nonexistent self.images_rbd_pool to self.flags. It should be
    self.rbd_pool.

    Closes-Bug: #1928007

    Change-Id: Ic03a5336abdced883f62f395690c0feac12075c8

Changed in nova:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/wallaby)

Fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/nova/+/790836

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/wallaby)

Reviewed: https://review.opendev.org/c/openstack/nova/+/790836
Committed: https://opendev.org/openstack/nova/commit/8f018d754d5c55e432cd51df99278382b527283e
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 8f018d754d5c55e432cd51df99278382b527283e
Author: melanie witt <email address hidden>
Date: Mon May 10 17:31:25 2021 +0000

    rbd: Get rbd_utils unit tests running again

    Awhile back, change I25baf5edd25d9e551686b7ed317a63fd778be533 moved
    rbd_utils out from the libvirt driver and into a central location under
    nova/storage. This move missed adding a __init__.py file under
    nova/tests/unit/storage, so unit test discovery wasn't picking up the
    rbd_utils tests and couldn't run them.

    This adds a __init__.py file under nova/tests/unit/storage to get the
    tests running again.

    This also fixes a small bug introduced by change
    I3032bbe6bd2d6acc9ba0f0cac4d00ed4b4464ceb in RbdTestCase.setUp() that
    passed nonexistent self.images_rbd_pool to self.flags. It should be
    self.rbd_pool.

    Closes-Bug: #1928007

    Change-Id: Ic03a5336abdced883f62f395690c0feac12075c8
    (cherry picked from commit 8b647f1b3f56879be221b3925570790a1e0e77f8)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/victoria)

Fix proposed to branch: stable/victoria
Review: https://review.opendev.org/c/openstack/nova/+/794624

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/victoria)

Reviewed: https://review.opendev.org/c/openstack/nova/+/794624
Committed: https://opendev.org/openstack/nova/commit/04057099991579bccfb2accb6b0952ba589fe092
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 04057099991579bccfb2accb6b0952ba589fe092
Author: melanie witt <email address hidden>
Date: Mon May 10 17:31:25 2021 +0000

    rbd: Get rbd_utils unit tests running again

    Awhile back, change I25baf5edd25d9e551686b7ed317a63fd778be533 moved
    rbd_utils out from the libvirt driver and into a central location under
    nova/storage. This move missed adding a __init__.py file under
    nova/tests/unit/storage, so unit test discovery wasn't picking up the
    rbd_utils tests and couldn't run them.

    This adds a __init__.py file under nova/tests/unit/storage to get the
    tests running again.

    This also fixes a small bug introduced by change
    I3032bbe6bd2d6acc9ba0f0cac4d00ed4b4464ceb in RbdTestCase.setUp() that
    passed nonexistent self.images_rbd_pool to self.flags. It should be
    self.rbd_pool.

    Closes-Bug: #1928007

    Change-Id: Ic03a5336abdced883f62f395690c0feac12075c8
    (cherry picked from commit 8b647f1b3f56879be221b3925570790a1e0e77f8)
    (cherry picked from commit 8f018d754d5c55e432cd51df99278382b527283e)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 23.0.2

This issue was fixed in the openstack/nova 23.0.2 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 22.2.2

This issue was fixed in the openstack/nova 22.2.2 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 24.0.0.0rc1

This issue was fixed in the openstack/nova 24.0.0.0rc1 release candidate.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.