yahoo-eng-team team mailing list archive

[OpenStack Infra <1730933@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.openstack.org/522245
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=05a73c0f3a9f8edf9024f9870279bc6fb7bba2e7
Submitter: Zuul
Branch:    master

commit 05a73c0f3a9f8edf9024f9870279bc6fb7bba2e7
Author: Silvan Kaiser <silvan@xxxxxxxxxxx>
Date:   Wed Nov 22 14:35:11 2017 +0100

    Added mount fstype based validation of Quobyte mounts
    
    The validation of Quobyte mounts is extended to validate based
    on a mounts file system type being set to "fuse.quobyte".
    This includes adding a new StaleVolumeMount exception type to
    the Nova exceptions.
    This also closes a bug concerning multi-registry configurations
    for Quobyte volumes due to no longer using the is_mounted()
    method that failed in that case.
    Finally this adds exception handling for the unmount call that
    is issued on trying to mount an already mounted volume.
    
    Closes-Bug: #1730933
    Closes-Bug: #1737131
    
    Change-Id: Ia5a23ce1123a68608ee2ec6f2ac5dca02da67c59


** Changed in: nova
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1730933

Title:
  Quobyte mount validation needs update

Status in Cinder:
  Fix Released
Status in OpenStack Compute (nova):
  Fix Released

Bug description:
  The Quobyte driver currently validates mounts, amongst other checks,
  by verifying the mounts device field has a 'quobyte@' prefix [1].
  Setting this will be replaced in the Quobyte client by setting a fuse
  subtype. The driver should be able to cope with both variants in order
  to stay compatible to all Quobyte releases.

  [1]
  https://github.com/openstack/cinder/blob/fb27334719fb612d2d5386b7d9de374d4a415d81/cinder/volume/drivers/quobyte.py#L476

To manage notifications about this bug go to:
https://bugs.launchpad.net/cinder/+bug/1730933/+subscriptions