assignment fails for ldap get_user

Bug #1200769 reported by Sahdev Zala on 2013-07-12
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
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) on 2013-07-12
Changed in keystone:
status: New → Triaged
importance: Undecided → High
Sahdev Zala (spzala) on 2013-07-17
Changed in keystone:
assignee: nobody → Sahdev Zala (spzala)

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)

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) on 2013-07-23
Changed in keystone:
status: Fix Committed → In Progress

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) on 2013-09-05
Changed in keystone:
milestone: none → havana-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-10-17
Changed in keystone:
milestone: havana-3 → 2013.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers