yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #49548
[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