Missing qos_specs_id for volume type show

[Liyingjun]

qos_specs_id is not returned for volume type show [1], we should add this to make it convenience to get qos specs associated to the volume type.

[1]: http://developer.openstack.org/api-ref-blockstorage-v2.html

[John Griffith (john-griffith)]

Probably would be a good thing to look at adding, but I'd argue it's an enhancement/improvement as opposed to a bug. Would you agree?

[OpenStack Infra (hudson-openstack)]

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

[Liyingjun (liyingjun)]

Yes, it's an improvement. Do i need to file a bp for this? it just a few lines change.

[John Griffith (john-griffith)]

@Liyingjun
Oh cool! You already proposed a change for it, that's great!! No, in my opinion this doesn't need a bp. The only thing I'm likely going to comment on in the review is backporting it to V1. We typically don't do that with features/enhancements particularly since we've deprecated V1.

Thanks a lot for submitting this!!

[Huang Zhiteng (zhiteng-huang)]

We should take care of this bug first: https://bugs.launchpad.net/cinder/+bug/1351971 before expose even more behind-the-scene details, which deployer would most want to hide, to non-admin users.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Mike Perez (<email address hidden>) on branch: master
Review: https://review.openstack.org/169153
Reason: Over a month with no update.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/234414
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=b2e92b85b5ea2fc4058d6c0f6e9e3fddc2a9036e
Submitter: Jenkins
Branch: master

commit b2e92b85b5ea2fc4058d6c0f6e9e3fddc2a9036e
Author: Nate Potter <email address hidden>
Date: Tue Oct 13 19:53:23 2015 +0000

    Add qos_specs_id to volume type show

    Adding qos_specs_id to the output of volume
    type show to make getting the qos specs for
    a volume type more convenient. However, this
    variable will only be visible when type show
    is called with the admin role.

    APIImpact
    Change-Id: Id8a2333267c9a51b36c6adfbd142dcc4fbe4f090
    Closes-Bug: #1438488

[Thierry Carrez (ttx)]

This issue was fixed in the openstack/cinder 8.0.0.0b1 development milestone.