auth plugin ignored

Bug #1333961 reported by Cory Stone
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-ceilometerclient
Fix Released
Undecided
gordon chung
python-cinderclient
Fix Released
Undecided
Haneef Ali

Bug Description

Since the keystone discovery integration, the --os-auth-plugin flag is ignored.

In 1.0.9:
$ cinder --debug --os-auth-system rackspace list

REQ: curl -i http://10.127.0.152:5000/v2.0/tokens -X POST -H "Content-Type: application/json" -H "Accept: application/json" -H "User-Agent: python-cinderclient" -d '{"auth": {"RAX-KSKEY:apiKeyCredentials": {"username": "demo", "apiKey": "password"}}}'

In master:
$ cinder --debug --os-auth-system rackspace list

DEBUG:keystoneclient.session:REQ: curl -i -X POST http://10.127.0.152:5000/v2.0//tokens -H "Content-Type: application/json" -H "Accept: application/json" -H "User-Agent: python-keystoneclient" -d '{"auth": {"tenantName": "demo", "passwordCredentials": {"username": "demo", "password": "password"}}}'

Revision history for this message
Cory Stone (corystone) wrote :

This example is using the simple rackspace auth plugin.

https://github.com/rackerlabs/rackspace-auth-openstack

Revision history for this message
Haneef Ali (haneef) wrote :

Keystone has auth plugin support, but for that to work, the plugin should be a subclass of

keystoneclient.auth.identity.base.BaseIdentityPlugin

Basically if Rackspace plugin implements that interface then it can be used with session. Till then I will bypass sessions if auth system is other than "keystone".

Please le me know whether this is acceptable.

Changed in python-cinderclient:
assignee: nobody → Haneef Ali (haneef)
Revision history for this message
Cory Stone (corystone) wrote :

It would be good for cinderclient to support the keystoneclient auth plugins as well, but the existing auth_plugin/auth_system should continue to work.

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/102586

Changed in python-cinderclient:
status: New → In Progress
Changed in python-cinderclient:
assignee: Haneef Ali (haneef) → Adam Young (ayoung)
Changed in python-cinderclient:
assignee: Adam Young (ayoung) → Haneef Ali (haneef)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-cinderclient (master)

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

commit b4906c855fe86af6ac8f469ae6e76adddffe6586
Author: Haneef Ali <email address hidden>
Date: Wed Jun 25 10:04:57 2014 -0700

    Fix version discovery and auth_plugins

    V3Client support added version discovery and session supports. Most
    of the external auth system doesn't support this. This fix bypasses
    version discovery if the idenity service doesn't support that. Session
    is used only if no external auth plugin is used

    Change-Id: Ia84a2ad45940d35c5ef740727f5f7477c72ea9d4
    Closes-Bug: #1333961

Changed in python-cinderclient:
status: In Progress → Fix Committed
Changed in python-ceilometerclient:
assignee: nobody → gordon chung (chungg)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-ceilometerclient (master)

Reviewed: https://review.openstack.org/118931
Committed: https://git.openstack.org/cgit/openstack/python-ceilometerclient/commit/?id=bc71ba32c9e69c2a57f612d752fe1128fccc7d40
Submitter: Jenkins
Branch: master

commit bc71ba32c9e69c2a57f612d752fe1128fccc7d40
Author: gordon chung <email address hidden>
Date: Wed Sep 3 20:22:33 2014 -0400

    keystone discovery fallback support

    version discovery and sessions aren't supported by all external
    auth systems. this patch adds a fallback when discovery does not
    work.

    Closes-Bug: #1333961
    Change-Id: Ib253c736e37f8dbc50881f9fc691bfdb4ab87d71

Changed in python-ceilometerclient:
status: In Progress → Fix Committed
Changed in python-ceilometerclient:
milestone: none → 1.3.0
status: Fix Committed → Fix Released
Changed in python-cinderclient:
milestone: none → 1.3.1
status: Fix Committed → Fix Released
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.