Comment 2 for bug 1608166

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

Reviewed: https://review.openstack.org/349221
Committed: https://git.openstack.org/cgit/openstack/python-cinderclient/commit/?id=f66b0d57d123941c46a6c27721a513927a7c6490
Submitter: Jenkins
Branch: master

commit f66b0d57d123941c46a6c27721a513927a7c6490
Author: Steve Martinelli <email address hidden>
Date: Sat Jul 30 20:54:23 2016 -0700

    print endpoints using new keystoneauth catalog object

    cinderclient was assuming an identity v2 styled service catalog
    would always be returned (when using `cinder endpoints`).

    keystoneclient would return either a v2 or v3 styled catalog,
    whereas keystoneauth abstracts out the differences and handles
    them internally. the result is that there is no need to look
    for specific ['serviceCatalog'] or ['catalog'] keys in the
    dictionary returned from keystoneauth.

    it should be noted that perhaps cinderclient should deprecate
    the ability to list endpoints since that is mostly an identity
    and admin level operation, and likely an artifact from early
    openstack days, it should now be handled by openstackclient.
    further, it's not clear whether the command is meant to list all
    endpoints or just the endpoints in the user's token (which it does
    now).

    Change-Id: Ibfcccedee5baf43f5b5c517d37e3f046c8743078
    Closes-Bug: 1608166