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

Bug #1913607 reported by Corey Bryant
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
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.

Revision history for this message
Corey Bryant (corey.bryant) wrote :
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 :

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
no longer affects: cloud-archive/queens
no longer affects: cinder (Ubuntu Bionic)
Revision history for this message
Corey Bryant (corey.bryant) wrote :

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 :

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 : 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 :

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 :

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 :

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.

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-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 :

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 : Update Released

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 :

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 : Please test proposed package

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
Changed in cloud-archive:
status: Fix Released → Fix Committed
Revision history for this message
James Page (james-page) wrote :

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 :

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 :

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

---------------
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).

 -- 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 : Update Released

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
Corey Bryant (corey.bryant) wrote :

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 :

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
Corey Bryant (corey.bryant) wrote :

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 :

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
Corey Bryant (corey.bryant) wrote :

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 : Fix included in openstack/cinder 18.0.0.0rc1

This issue was fixed in the openstack/cinder 18.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.