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).
Reviewed: https:/ /review. openstack. org/349221 /git.openstack. org/cgit/ openstack/ python- cinderclient/ commit/ ?id=f66b0d57d12 3941c46a6c27721 a513927a7c6490
Committed: https:/
Submitter: Jenkins
Branch: master
commit f66b0d57d123941 c46a6c27721a513 927a7c6490
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: Ibfcccedee5baf4 3f5b5c517d37e3f 046c8743078
Closes-Bug: 1608166