Cinder

test_mount_quobyte_should_reraise_already_mounted_error - No such file or directory: '/etc/mtab'

Bug #1913607 reported by Corey Bryant on 2021-01-28

This bug affects 1 person

	Status	Importance	Assigned to
Cinder	In Progress	Undecided	Corey Bryant
Ubuntu Cloud Archive	Fix Committed	Medium	Unassigned
Stein	Fix Released	Medium	Unassigned
Train	Fix Released	Medium	Unassigned
Ussuri	Fix Released	Medium	Unassigned
Victoria	Fix Committed	Medium	Unassigned
cinder (Ubuntu)	Fix Released	Medium	Unassigned
Focal	Fix Released	Medium	Unassigned
Groovy	Fix Released	Medium	Unassigned
Hirsute	Fix Released	Medium	Unassigned

Bug Description

[Impact]
I'm hitting the following error on ussuri and prior. Worth noting, this is a package build for ubuntu.

======================================================================
FAIL: cinder.tests.unit.volume.drivers.test_quobyte.QuobyteDriverTestCase.test_mount_quobyte_should_reraise_already_mounted_error
cinder.tests.unit.volume.drivers.test_quobyte.QuobyteDriverTestCase.test_mount_quobyte_should_reraise_already_mounted_error
----------------------------------------------------------------------
_StringException: pythonlogging:'': {{{
2021-01-27 18:46:39,044 INFO [keystonemiddleware.auth_token] Starting Keystone auth_token middleware
2021-01-27 18:46:39,047 WARNING [keystonemiddleware._common.config] The option "auth_url" is not known to keystonemiddleware
2021-01-27 18:46:39,047 WARNING [keystonemiddleware.auth_token] AuthToken middleware is set with keystone_authtoken.service_token_roles_required set to False. This is backwards compatible but deprecated behaviour. Please set this to True.
2021-01-27 18:46:39,050 WARNING [keystonemiddleware.auth_token] Configuring www_authenticate_uri to point to the public identity endpoint is required; clients may not be able to authenticate against an admin endpoint
2021-01-27 18:46:39,069 INFO [cinder.volume.drivers.quobyte] The NAS file operations will be run as non privileged user in secure mode. Please ensure your libvirtd settings have been configured accordingly (see section 'OpenStack' in the Quobyte Manual.
2021-01-27 18:46:39,070 INFO [cinder.volume.drivers.quobyte] Mounting volume: quobyte://quobyte-host/openstack-volumes ...
2021-01-27 18:46:39,071 INFO [cinder.volume.drivers.quobyte] Mounting volume: quobyte://quobyte-host/openstack-volumes succeeded
}}}

Traceback (most recent call last):
  File "/<<PKGBUILDDIR>>/cinder/tests/unit/volume/drivers/test_quobyte.py", line 463, in test_mount_quobyte_should_reraise_already_mounted_error
    ensure=False)
  File "/usr/lib/python3/dist-packages/testtools/testcase.py", line 485, in assertRaises
    self.assertThat(our_callable, matcher)
  File "/usr/lib/python3/dist-packages/testtools/testcase.py", line 496, in assertThat
    mismatch_error = self._matchHelper(matchee, matcher, message, verbose)
  File "/usr/lib/python3/dist-packages/testtools/testcase.py", line 547, in _matchHelper
    mismatch = matcher.match(matchee)
  File "/usr/lib/python3/dist-packages/testtools/matchers/_exception.py", line 108, in match
    mismatch = self.exception_matcher.match(exc_info)
  File "/usr/lib/python3/dist-packages/testtools/matchers/_higherorder.py", line 62, in match
    mismatch = matcher.match(matchee)
  File "/usr/lib/python3/dist-packages/testtools/testcase.py", line 475, in match
    reraise(*matchee)
  File "/usr/lib/python3/dist-packages/testtools/_compat3x.py", line 16, in reraise
    raise exc_obj.with_traceback(exc_tb)
  File "/usr/lib/python3/dist-packages/testtools/matchers/_exception.py", line 101, in match
    result = matchee()
  File "/usr/lib/python3/dist-packages/testtools/testcase.py", line 1049, in __call__
    return self._callable_object(*self._args, **self._kwargs)
  File "/<<PKGBUILDDIR>>/cinder/volume/drivers/quobyte.py", line 699, in _mount_quobyte
    self._validate_volume(mount_path)
  File "/<<PKGBUILDDIR>>/cinder/volume/drivers/quobyte.py", line 705, in _validate_volume
    partitions = psutil.disk_partitions(all=True)
  File "/usr/lib/python3/dist-packages/psutil/__init__.py", line 2095, in disk_partitions
    return _psplatform.disk_partitions(all)
  File "/usr/lib/python3/dist-packages/psutil/_pslinux.py", line 1168, in disk_partitions
    partitions = cext.disk_partitions(mtab_path)
FileNotFoundError: [Errno 2] No such file or directory: '/etc/mtab'

This is similiar to: https://bugs.launchpad.net/cinder/+bug/1675861

[Test Case]
The ubuntu package for cinder runs unit tests as part of the build process. Tests are expected to be successful with this fix.

[Regression Potential]
This change only touches unit test code. Any regressions would be limited to failing tests at package build time.

See original description

Tags:

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-01-28:

Fix proposed: https://review.opendev.org/c/openstack/cinder/+/772902

Changed in cinder:
assignee:	nobody → Corey Bryant (corey.bryant)
Changed in cinder (Ubuntu):
status:	New → Invalid
Changed in cinder (Ubuntu Bionic):
status:	New → Triaged
Changed in cinder (Ubuntu Focal):
status:	New → Triaged
status:	Triaged → Incomplete
Changed in cinder (Ubuntu Bionic):
importance:	Undecided → Medium
Changed in cinder (Ubuntu Focal):
importance:	Undecided → Medium
status:	Incomplete → Triaged
Changed in cinder (Ubuntu Bionic):
status:	Triaged → Invalid

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-01-28:

I'm currently only hitting this on focal, bionic-ussuri, bionic-train, and bionic-stein. I'm not hitting it on bionic base, groovy, and hirsute. It's probably intermittent so I think I'll backport to all releases back to bionic.

Changed in cinder (Ubuntu Bionic):
status:	Invalid → Triaged
Changed in cinder (Ubuntu):
status:	Invalid → Triaged
importance:	Undecided → Medium
Changed in cinder (Ubuntu Groovy):
importance:	Undecided → Medium
status:	New → Triaged

Corey Bryant (corey.bryant) on 2021-01-28

no longer affects:	cloud-archive/queens
no longer affects:	cinder (Ubuntu Bionic)

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-01-28:

For ubuntu and the cloud-archive I've uploaded package for all affected releases. The ubuntu uploads are awiting SRU team approval.

description:	updated
Changed in cinder:
status:	New → In Progress

Revision history for this message

Launchpad Janitor (janitor) wrote on 2021-01-29:

This bug was fixed in the package cinder - 2:17.0.1+git2021012507.d26092348-0ubuntu2

---------------
cinder (2:17.0.1+git2021012507.d26092348-0ubuntu2) hirsute; urgency=medium

* d/p/add-mock-psutil-in-quobyte-tests.patch: Add a mock of psutil
disk_partitions to fix failing unit test (LP: #1913607).

-- Corey Bryant <email address hidden> Thu, 28 Jan 2021 12:03:20 -0500

Changed in cinder (Ubuntu Hirsute):
status:	Triaged → Fix Released

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-02-10: Please test proposed package

Hello Corey, or anyone else affected,

Accepted cinder into train-proposed. The package will build now and be available in the Ubuntu Cloud Archive in a few hours, and then in the -proposed repository.

Please help us by testing this new package. To enable the -proposed repository:

sudo add-apt-repository cloud-archive:train-proposed
sudo apt-get update

Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-train-needed to verification-train-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-train-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags:

added: verification-train-needed

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-02-10:

Hello Corey, or anyone else affected,

Accepted cinder into stein-proposed. The package will build now and be available in the Ubuntu Cloud Archive in a few hours, and then in the -proposed repository.

Please help us by testing this new package. To enable the -proposed repository:

sudo add-apt-repository cloud-archive:stein-proposed
sudo apt-get update

Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-stein-needed to verification-stein-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-stein-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags:

added: verification-stein-needed

Revision history for this message

Brian Murray (brian-murray) wrote on 2021-02-16:

Hello Corey, or anyone else affected,

Accepted cinder into groovy-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/cinder/2:17.0.1-0ubuntu2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-groovy to verification-done-groovy. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-groovy. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in cinder (Ubuntu Groovy):
status:	Triaged → Fix Committed
tags:	added: verification-needed verification-needed-groovy

Revision history for this message

Brian Murray (brian-murray) wrote on 2021-02-16:

Hello Corey, or anyone else affected,

Accepted cinder into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/cinder/2:16.2.1-0ubuntu2 in a few hours, and then in the -proposed repository.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in cinder (Ubuntu Focal):
status:	Triaged → Fix Committed
tags:	added: verification-needed-focal

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-02-16:

Hello Corey, or anyone else affected,

Accepted cinder into victoria-proposed. The package will build now and be available in the Ubuntu Cloud Archive in a few hours, and then in the -proposed repository.

Please help us by testing this new package. To enable the -proposed repository:

sudo add-apt-repository cloud-archive:victoria-proposed
sudo apt-get update

Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-victoria-needed to verification-victoria-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-victoria-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags:

added: verification-victoria-needed

Revision history for this message

Chris MacNaughton (chris.macnaughton) wrote on 2021-02-17: Update Released

#10

The verification of the Stable Release Update for cinder has completed successfully and the package has now been released to -updates. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message

Chris MacNaughton (chris.macnaughton) wrote on 2021-02-17:

#11

This bug was fixed in the package cinder - 2:17.0.1-0ubuntu2~cloud0
---------------

cinder (2:17.0.1-0ubuntu2~cloud0) focal-victoria; urgency=medium
.
   * New update for the Ubuntu Cloud Archive.
.
cinder (2:17.0.1-0ubuntu2) groovy; urgency=medium
.
   * d/p/rbd-retry-delete.patch: Fix RBD mirroring race by retrying volume delete
     if VolumeIsBusy in _copy_image_to_volume (LP: #1900775).
   * d/p/add-mock-psutil-in-quobyte-tests.patch: Add a mock of psutil
     disk_partitions to fix failing unit test (LP: #1913607).

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-02-17: Please test proposed package

#12

Hello Corey, or anyone else affected,

Accepted cinder into ussuri-proposed. The package will build now and be available in the Ubuntu Cloud Archive in a few hours, and then in the -proposed repository.

Please help us by testing this new package. To enable the -proposed repository:

sudo add-apt-repository cloud-archive:ussuri-proposed
sudo apt-get update

Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-ussuri-needed to verification-ussuri-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-ussuri-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags:

added: verification-ussuri-needed

Corey Bryant (corey.bryant) on 2021-02-17

Changed in cloud-archive:
status:	Fix Released → Fix Committed

Revision history for this message

James Page (james-page) wrote on 2021-03-02:

#13

Packages all built and unit tested successfully for UCA and Ubuntu so marking verification as done for all targets.

tags:

added: verification-done verification-done-focal verification-done-groovy verification-stein-done verification-train-done verification-ussuri-done verification-victoria-done
removed: verification-needed verification-needed-focal verification-needed-groovy verification-stein-needed verification-train-needed verification-ussuri-needed verification-victoria-needed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2021-03-02:

#14

This bug was fixed in the package cinder - 2:17.0.1-0ubuntu2

---------------
cinder (2:17.0.1-0ubuntu2) groovy; urgency=medium

  * d/p/rbd-retry-delete.patch: Fix RBD mirroring race by retrying volume delete
    if VolumeIsBusy in _copy_image_to_volume (LP: #1900775).
  * d/p/add-mock-psutil-in-quobyte-tests.patch: Add a mock of psutil
    disk_partitions to fix failing unit test (LP: #1913607).

-- Corey Bryant <email address hidden> Tue, 26 Jan 2021 15:12:48 -0500

Changed in cinder (Ubuntu Groovy):
status:	Fix Committed → Fix Released

Revision history for this message

Launchpad Janitor (janitor) wrote on 2021-03-02:

#15

This bug was fixed in the package cinder - 2:16.2.1-0ubuntu2

---------------
cinder (2:16.2.1-0ubuntu2) focal; urgency=medium

-- Corey Bryant <email address hidden> Tue, 26 Jan 2021 15:28:53 -0500

Changed in cinder (Ubuntu Focal):
status:	Fix Committed → Fix Released

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-03-03: Update Released

#16

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-03-03:

#17

This bug was fixed in the package cinder - 2:14.3.1-0ubuntu1~cloud1
---------------

cinder (2:14.3.1-0ubuntu1~cloud1) bionic-stein; urgency=medium
.
   * d/p/rbd-retry-delete.patch: Fix RBD mirroring race by retrying volume delete
     if VolumeIsBusy in _copy_image_to_volume (LP: #1900775).
   * d/p/add-mock-psutil-in-quobyte-tests.patch: Add a mock of psutil
     disk_partitions to fix failing unit test (LP: #1913607).

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-03-03:

#18

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-03-03:

#19

This bug was fixed in the package cinder - 2:16.2.1-0ubuntu2~cloud0
---------------

cinder (2:16.2.1-0ubuntu2~cloud0) bionic-ussuri; urgency=medium
.
   * New update for the Ubuntu Cloud Archive.
.
cinder (2:16.2.1-0ubuntu2) focal; urgency=medium
.
   * d/p/rbd-retry-delete.patch: Fix RBD mirroring race by retrying volume delete
     if VolumeIsBusy in _copy_image_to_volume (LP: #1900775).
   * d/p/add-mock-psutil-in-quobyte-tests.patch: Add a mock of psutil
     disk_partitions to fix failing unit test (LP: #1913607).

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-03-03:

#20

Revision history for this message

Corey Bryant (corey.bryant) wrote on 2021-03-03:

#21

This bug was fixed in the package cinder - 2:15.4.1-0ubuntu1~cloud1
---------------

cinder (2:15.4.1-0ubuntu1~cloud1) bionic-train; urgency=medium
.
   * d/p/rbd-retry-delete.patch: Fix RBD mirroring race by retrying volume delete
     if VolumeIsBusy in _copy_image_to_volume (LP: #1900775).
   * d/p/add-mock-psutil-in-quobyte-tests.patch: Add a mock of psutil
     disk_partitions to fix failing unit test (LP: #1913607).

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2021-03-26: Fix included in openstack/cinder 18.0.0.0rc1

#22

This issue was fixed in the openstack/cinder 18.0.0.0rc1 release candidate.

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

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