The --volume-service-name option is not used and should be removed

Bug #1570593 reported by Matt Riedemann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-novaclient
Fix Released
Low
Kristi Nikolla

Bug Description

This option and the code using it is basically dead now:

https://github.com/openstack/python-novaclient/blob/a967e40185d4a8682194604c2225638e1ec88ed1/novaclient/shell.py#L524-L534

Since we removed the volume proxy CLI and APIs here:

https://github.com/openstack/python-novaclient/commit/a42570268915f42405ed0b0a67c25686b5db22ce

So we should clean up that code.

Changed in python-novaclient:
assignee: nobody → Kristi Nikolla (knikolla)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-novaclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/306473

Changed in python-novaclient:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-novaclient (master)

Reviewed: https://review.openstack.org/306473
Committed: https://git.openstack.org/cgit/openstack/python-novaclient/commit/?id=74c3da93f5bfb019ae310ef811ea3d8137fcff44
Submitter: Jenkins
Branch: master

commit 74c3da93f5bfb019ae310ef811ea3d8137fcff44
Author: Kristi Nikolla <email address hidden>
Date: Fri Apr 15 10:35:59 2016 -0400

    Deprecated the `--volume-service-name` option

    The `--volume-service-name` name option needs to be deprecated
    since the volume proxy API and CLI have been deprecated.

    Change-Id: I8af9b62e56c5ab7bffa79b2700b3c9213aaad65d
    Closes-Bug: 1570593

Changed in python-novaclient:
status: In Progress → Fix Released
Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/python-novaclient 5.0.0

This issue was fixed in the openstack/python-novaclient 5.0.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to python-novaclient (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/465502

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to python-novaclient (master)

Reviewed: https://review.openstack.org/465502
Committed: https://git.openstack.org/cgit/openstack/python-novaclient/commit/?id=3a762401245de8c2941ea41d6168c17ddd30aacd
Submitter: Jenkins
Branch: master

commit 3a762401245de8c2941ea41d6168c17ddd30aacd
Author: Stephen Finucane <email address hidden>
Date: Wed May 17 11:57:16 2017 +0100

    Remove various deprecated options

    These are all deprecated with at least one release and can be removed
    now.

    Change-Id: I5d11eda2a6b35de98f0484492f597a87df882013
    Related-Bug: 1570593

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.