yahoo-eng-team team mailing list archive

[Henry Nash <henryn@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

The assignment manager has a comment implying that the code in the
init() method is working around a circular dependency between identity
and assignment.  While there is a dependency between assignment and
identity in terms of supporting the backward compatibility of the config
options for which driver is used, I don't believe there is a "circular"
dependency.  The comment should be corrected and be in the init() method
itself.

** Affects: keystone
     Importance: Low
     Assignee: Henry Nash (henry-nash)
         Status: New

** Description changed:

  The assignment manager has a comment implying that the code in the
  init() method is working around a circular dependency between identity
  and assignment.  While there is a dependency between assignment and
  identity in terms of supporting the backward compatibility of the config
  options for which driver is used, I don't believe there is a "circular"
- dependency.  The comment should be correct and be in the init() method
+ dependency.  The comment should be corrected and be in the init() method
  itself.

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Keystone.
https://bugs.launchpad.net/bugs/1407342

Title:
  Incorrect comment about circular dependency in assignment manager

Status in OpenStack Identity (Keystone):
  New

Bug description:
  The assignment manager has a comment implying that the code in the
  init() method is working around a circular dependency between identity
  and assignment.  While there is a dependency between assignment and
  identity in terms of supporting the backward compatibility of the
  config options for which driver is used, I don't believe there is a
  "circular" dependency.  The comment should be corrected and be in the
  init() method itself.

To manage notifications about this bug go to:
https://bugs.launchpad.net/keystone/+bug/1407342/+subscriptions