Keystone Discovery Warnings Displayed

Bug #1448244 reported by Mike Perez
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-cinderclient
Fix Released
Medium
Jamie Lennox

Bug Description

Discovery is now in Cinder client https://review.openstack.org/#/c/145613/ , but when the Keystone catalog contains a url that has a version in it, it displays warnings:

WARNING:keystoneclient.auth.identity.base:Failed to contact the endpoint at http://172.30.128.4:8776/v1/4af36bb30e4e43ffa837a3fd71c22c3f for discovery. Fallback to using that endpoint as the base url.
WARNING:keystoneclient.auth.identity.base:Failed to contact the endpoint at http://172.30.128.4:8776/v1/4af36bb30e4e43ffa837a3fd71c22c3f for discovery. Fallback to using that endpoint as the base url.
WARNING:keystoneclient.auth.identity.base:Failed to contact the endpoint at http://172.30.128.4:8776/v1/4af36bb30e4e43ffa837a3fd71c22c3f for discovery. Fallback to using that endpoint as the base url.

See more examples here:

http://paste.openstack.org/raw/155906/

This is when Cinder is doing its initial endpoint discovery.

Revision history for this message
Mike Perez (thingee) wrote :

Adding Keystone client after talking to Morgan about it.

http://eavesdrop.openstack.org/irclogs/%23openstack-cinder/%23openstack-cinder.2015-04-24.log

Maybe it could be suppressed?

Revision history for this message
Jamie Lennox (jamielennox) wrote :

Mike: what you are looking for is: https://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/discover.py#L310

It's in preperation for a catalog that contains ONLY the root url - in the mean time we need to hack off anything that is excess. We do this with keystone already where we need to remove the /v2 from the urls in the catalog.

https://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/_discover.py#L313

in the main cinderclient file you want to add something like:

from keystoneclient import discover

discover.add_catalog_discover_hack('volume', '/v[12]/\w+/?$', '/')

my regex is probably off.

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

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

Changed in python-cinderclient:
assignee: nobody → Mike Perez (thingee)
status: Triaged → In Progress
Changed in python-cinderclient:
assignee: Mike Perez (thingee) → Jamie Lennox (jamielennox)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-cinderclient (master)

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

commit 4ea65b440994f183a109b0671208dec37088a43e
Author: Mike Perez <email address hidden>
Date: Tue Apr 28 15:01:49 2015 -0700

    Add version removal rule to stop discovery warning

    Cinder recently added version discovery leveraging Keystone Client. For
    cases where the service catalog still contains version numbers, this can
    result in Keystone attempting to do discovery at the base url with the
    version number and giving warnings. This will set a version removal rule
    so Keystone can find the correct base url.

    Change-Id: I71432468fea8bf1e50f180ab7f6dd69ee9aaa7e6
    Closes-Bug: #1448244

Changed in python-cinderclient:
status: In Progress → Fix Committed
Mike Perez (thingee)
no longer affects: python-keystoneclient
Changed in python-cinderclient:
status: Fix Committed → Fix Released
milestone: 1.2.0 → none
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.