yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #35601
[Bug 1477451] [NEW] Assumption that db drivers can ignore hints is false
Public bug reported:
For hints, the documentation says that if the driver implementation
doesn't filter by hints, they are handled at the controller level. But
this assumption is false when using 'list users' and 'list groups' API.
This was found while writing a Cassandra backend driver for Keystone.
Similar could be done by commenting out lines from
keystone/identity/backends/sql.py driver file which filters by hints.
I am suspecting that the problem is at these two lines:
https://github.com/openstack/keystone/blob/7a28fdb6385ec31e3d46fe63b22028e599ea66b3/keystone/identity/core.py#L429
I see two alternatives: either fix this, or change the documentation
saying that hints filtering at higher (controller) level might not be
enforced.
** Affects: keystone
Importance: Undecided
Status: New
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Keystone.
https://bugs.launchpad.net/bugs/1477451
Title:
Assumption that db drivers can ignore hints is false
Status in Keystone:
New
Bug description:
For hints, the documentation says that if the driver implementation
doesn't filter by hints, they are handled at the controller level. But
this assumption is false when using 'list users' and 'list groups'
API.
This was found while writing a Cassandra backend driver for Keystone.
Similar could be done by commenting out lines from
keystone/identity/backends/sql.py driver file which filters by hints.
I am suspecting that the problem is at these two lines:
https://github.com/openstack/keystone/blob/7a28fdb6385ec31e3d46fe63b22028e599ea66b3/keystone/identity/core.py#L429
I see two alternatives: either fix this, or change the documentation
saying that hints filtering at higher (controller) level might not be
enforced.
To manage notifications about this bug go to:
https://bugs.launchpad.net/keystone/+bug/1477451/+subscriptions
Follow ups