Cinder Client ignores auth plugin in favor of keystone auth

Bug #1380729 reported by Robert Myers
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-cinderclient
Fix Released
Medium
Robert Myers

Bug Description

A recent change to the cinder client to use the adapter from keystone client. This changed a line that ignored using the session if 'auth_plugin' is defined. Now if you try to use a custom auth_plugin it always uses the default keystone v2 or v3 implementations. The following check should be re-instated:

    # Don't use sessions if third party plugin is used
    if session and not auth_plugin:

Robert Myers (rmyers)
Changed in cinder:
assignee: nobody → Robert Myers (rmyers)
Robert Myers (rmyers)
Changed in cinder:
status: New → In Progress
Revision history for this message
Xing Yang (xing-yang) wrote :
Mike Perez (thingee)
affects: cinder → python-cinderclient
Changed in python-cinderclient:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-cinderclient (master)

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

commit 7783767f23494b549b74cc2843631c8aed456866
Author: Robert Myers <email address hidden>
Date: Mon Oct 13 13:03:15 2014 -0500

    Don't use sessions if third party plugin is used

    If an auth_plugin is used do not try to load the session
    from the default keystone providers.

    Change-Id: If2a87e23cabde006833de70e5c7aa066d95dbf50
    Closes-Bug: #1380729

Changed in python-cinderclient:
status: In Progress → Fix Committed
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.