Cinder

Dell PowerVault PVMEISCSIDriver driver cannot manage volumes

Bug #1922255 reported by Richard
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
In Progress
Low
Chris M

Bug Description

Information:
OpenStack deployed with kolla-ansible, version is Victoria, but using latest master of Cinder images from dockerhub to get access to the PowerVault driver as Wallaby is not released yet.

From the documentation for the driver:
https://opendev.org/openstack/cinder/src/branch/master/doc/source/configuration/block-storage/drivers/dell-emc-powervault-me.rst
It says that managing (importing) a volume is possible.
("Manage and unmanage a volume." is under "Supported Operations".)

Yet when trying to use `cinder --os-volume-api-version 3.8 manageable-list host01@pool-b#B`:
ERROR: The server has either erred or is incapable of performing the requested operation. (HTTP 500) (Request-ID: req-12e76049-eebf-44a1-8771-4f93608530ba)`

An extract of cinder-volume.log contains:
2021-04-01 13:49:07.508 31 ERROR cinder.volume.manager [req-6c2f7e58-ccc4-4ea0-8e54-a4861c919409 3c16147710844ff78a6f9f0be18cff06 ed0b2146e7f34f0daf57945044b01781 - - -] Listing manageable volumes fa$2021-04-01 13:49:07.508 31 ERROR cinder.volume.manager Traceback (most recent call last):
2021-04-01 13:49:07.508 31 ERROR cinder.volume.manager File "/var/lib/kolla/venv/lib/python3.8/site-packages/cinder/volume/manager.py", line 3177, in get_manageable_volumes
2021-04-01 13:49:07.508 31 ERROR cinder.volume.manager driver_entries = self.driver.get_manageable_volumes(
2021-04-01 13:49:07.508 31 ERROR cinder.volume.manager File "/var/lib/kolla/venv/lib/python3.8/site-packages/cinder/volume/driver.py", line 2255, in get_manageable_volumes
2021-04-01 13:49:07.508 31 ERROR cinder.volume.manager raise NotImplementedError(msg)
2021-04-01 13:49:07.508 31 ERROR cinder.volume.manager NotImplementedError: Get manageable volumes not implemented.

We can see this function is not implemented, so the documentation should not say that managing/importing is supported.

The ability to import an existing volume that Cinder doesn't know about seems like an important feature.
Is the driver created by Dell, by OpenStack? Having this feature added for Wallaby would be fantastic.

Also, there is an error in the example back-end entries.
https://opendev.org/openstack/cinder/src/branch/master/doc/source/configuration/block-storage/drivers/dell-emc-powervault-me.rst

There are three instances of `pvme_backend_name` on the page, but this is incorrect, it should be replaced with `pvme_pool_name`.

Sofia Enriquez (lsofia-enriquez)
Changed in cinder:
importance: Undecided → Low
Chris M (pots)
Changed in cinder:
assignee: nobody → Chris M (pots)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
status: New → In Progress
Revision history for this message
Chris M (pots) wrote :

The issue with the examples in the documentation is addressed by change https://review.opendev.org/c/openstack/cinder/+/809965.

The proposed fix for missing functionality (https://review.opendev.org/c/openstack/cinder/+/808837) also depends on another change: https://review.opendev.org/c/openstack/cinder/+/809968.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.opendev.org/c/openstack/cinder/+/809965
Committed: https://opendev.org/openstack/cinder/commit/8f68fd47d8ea8bba84622106f2afbf8b054069b8
Submitter: "Zuul (22348)"
Branch: master

commit 8f68fd47d8ea8bba84622106f2afbf8b054069b8
Author: Chris M <email address hidden>
Date: Mon Sep 20 04:24:51 2021 +0000

    Dell PowerVault driver: Fix documentation typos

    As reported in Bug #1922255, the PowerVault driver documentation used
    the wrong term to refer to the backend pool name.

    Partial-Bug: #1922255

    Change-Id: I147a3fd79d17f4092bcdd48d715ac71c072b1c9e

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

Fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/cinder/+/811155

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/cinder/+/811156

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.opendev.org/c/openstack/cinder/+/808837
Committed: https://opendev.org/openstack/cinder/commit/bff1d26f6fc7c492ee9cfcde0f09868c34a5cd20
Submitter: "Zuul (22348)"
Branch: master

commit bff1d26f6fc7c492ee9cfcde0f09868c34a5cd20
Author: Chris M <email address hidden>
Date: Mon Sep 13 20:46:26 2021 +0000

    Dell PowerVault: Fix "cinder manageable-list"

    The Dell PowerVault ME driver claims support for importing volumes and
    although it does support "cinder manage", some related functions were
    missing, so it did not support "cinder manageable-list" or the related
    snapshot functions.

    Partial-Bug: #1922255
    Depends-on: https://review.opendev.org/c/openstack/cinder/+/809968

    Change-Id: I73958099b32e44e7e4875d0eba0e2c0096a12252

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

Reviewed: https://review.opendev.org/c/openstack/cinder/+/811156
Committed: https://opendev.org/openstack/cinder/commit/de55fab4854c85e518b9c9da497cb8cc94d4619e
Submitter: "Zuul (22348)"
Branch: stable/xena

commit de55fab4854c85e518b9c9da497cb8cc94d4619e
Author: Chris M <email address hidden>
Date: Mon Sep 20 04:24:51 2021 +0000

    Dell PowerVault driver: Fix documentation typos

    As reported in Bug #1922255, the PowerVault driver documentation used
    the wrong term to refer to the backend pool name.

    Partial-Bug: #1922255

    Change-Id: I147a3fd79d17f4092bcdd48d715ac71c072b1c9e
    (cherry picked from commit 8f68fd47d8ea8bba84622106f2afbf8b054069b8)

tags: added: in-stable-xena
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.opendev.org/c/openstack/cinder/+/811155
Committed: https://opendev.org/openstack/cinder/commit/5bebe93ef8c96db703d38349e39010a92535fd63
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 5bebe93ef8c96db703d38349e39010a92535fd63
Author: Chris M <email address hidden>
Date: Mon Sep 13 20:46:26 2021 +0000

    Dell PowerVault: Fix "cinder manageable-list"

    The Dell PowerVault ME driver claims support for importing volumes and
    although it does support "cinder manage", some related functions were
    missing, so it did not support "cinder manageable-list" or the related
    snapshot functions.

    Partial-Bug: #1922255
    Depends-on: https://review.opendev.org/c/openstack/cinder/+/809968

    Change-Id: I73958099b32e44e7e4875d0eba0e2c0096a12252
    (cherry picked from commit bff1d26f6fc7c492ee9cfcde0f09868c34a5cd20)

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

Other bug subscribers

Remote bug watches

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