Dell PowerVault PVMEISCSIDriver driver cannot manage volumes

[Richard]

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

[OpenStack Infra (hudson-openstack)]

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

[Chris M (pots)]

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.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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)

[OpenStack Infra (hudson-openstack)]

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)