assignment fails for ldap get_user

Bug #1200769 reported by Sahdev Zala
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
High
Brant Knudson

Bug Description

Seems like a bug associated with a recent implementation of assignment.

  File "/opt/stack/keystone/tests/_ldap_livetest.py", line 119, in test_ldap_dereferencing
    user_ref = self.identity_api.get_user('alt_fake1')
  File "/opt/stack/keystone/keystone/identity/backends/ldap.py", line 85, in get_user
    return self.assignment._set_default_domain(ref)
AttributeError: 'Identity' object has no attribute 'assignment'

Dolph Mathews (dolph)
Changed in keystone:
status: New → Triaged
importance: Undecided → High
Sahdev Zala (spzala)
Changed in keystone:
assignee: nobody → Sahdev Zala (spzala)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

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

Changed in keystone:
status: Triaged → In Progress
Changed in keystone:
assignee: Sahdev Zala (spzala) → Brant Knudson (blk-u)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/37962
Committed: http://github.com/openstack/keystone/commit/639d62cdbacbe3c3da5183137d900d82bea5389d
Submitter: Jenkins
Branch: master

commit 639d62cdbacbe3c3da5183137d900d82bea5389d
Author: Brant Knudson <email address hidden>
Date: Fri Jul 19 15:10:51 2013 -0500

    Use assignment_api rather than assignment

    When normal dependency injection is used the variables
    are named like "xxx_api".
    This change makes it so that the identity backend and
    drivers use "assignment_api" rather than "assignment"
    so it will be easier to switch to normal dependency
    injection.

    Part of fix for bug 1200769

    Change-Id: I7805b338c48d57ca1922bb622a3f474f2341f4ac

Changed in keystone:
status: In Progress → Fix Committed
Brant Knudson (blk-u)
Changed in keystone:
status: Fix Committed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/37821
Committed: http://github.com/openstack/keystone/commit/3268a4b96449647faa76e5521660516fbc9e8a84
Submitter: Jenkins
Branch: master

commit 3268a4b96449647faa76e5521660516fbc9e8a84
Author: Brant Knudson <email address hidden>
Date: Fri Jul 19 17:11:43 2013 -0500

    Use dependency injection for assignment and identity

    The assignment and identity backends don't use normal
    dependency injection because of a circular reference.

    Support for circular references was added, so these
    backends can now use normal dependency injection.

    Fixes: bug 1200769
    Fixes: bug 1204605

    Change-Id: I8716a65b984c26fcaaf2595463db09e1f02cb475

Changed in keystone:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in keystone:
milestone: none → havana-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: havana-3 → 2013.2
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.