← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1571342] [NEW] Misleading variable naming is used in ldap driver

 

Public bug reported:

In keystone/common/ldap/core.py, the get_all() method of EnabledEmuMixIn
is using "tenant" as the return object. This is misleading as the core
method is not using as project only but also other identity or resources
(such as user and group). We should change to name to be more generic to
avoid confusion.

https://github.com/openstack/keystone/blob/master/keystone/common/ldap/core.py#L1921-L1932

** Affects: keystone
     Importance: Undecided
     Assignee: Kahou Lei (kahou82)
         Status: In Progress

-- 
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/1571342

Title:
  Misleading variable naming is used in ldap driver

Status in OpenStack Identity (keystone):
  In Progress

Bug description:
  In keystone/common/ldap/core.py, the get_all() method of
  EnabledEmuMixIn is using "tenant" as the return object. This is
  misleading as the core method is not using as project only but also
  other identity or resources (such as user and group). We should change
  to name to be more generic to avoid confusion.

  https://github.com/openstack/keystone/blob/master/keystone/common/ldap/core.py#L1921-L1932

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


Follow ups