yahoo-eng-team team mailing list archive

[Ron De Rose <ronald.de.rose@xxxxxxxxx>]

Public bug reported:

Remove dependencies where backend code references code in the core.  The
reasoning being that the core should know about the backends, but the
backends should not know anything about the core (separation of
concerns).  And part of the risk here is a potential for circular
dependencies.

For example, identity backends imports identity:
https://github.com/openstack/keystone/blob/master/keystone/identity/backends/sql.py#L24

Backend code could then reference code in the core, as well as other
modules inside identity; thus, creating a circular dependency.

** Affects: keystone
     Importance: Undecided
     Assignee: Ron De Rose (ronald-de-rose)
         Status: In Progress

** Changed in: keystone
     Assignee: (unassigned) => Ron De Rose (ronald-de-rose)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Identity (keystone).
https://bugs.launchpad.net/bugs/1563101

Title:
  Remove backend dependency on core

Status in OpenStack Identity (keystone):
  In Progress

Bug description:
  Remove dependencies where backend code references code in the core.
  The reasoning being that the core should know about the backends, but
  the backends should not know anything about the core (separation of
  concerns).  And part of the risk here is a potential for circular
  dependencies.

  For example, identity backends imports identity:
  https://github.com/openstack/keystone/blob/master/keystone/identity/backends/sql.py#L24

  Backend code could then reference code in the core, as well as other
  modules inside identity; thus, creating a circular dependency.

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