get_registered_limits() does not use the Enforcer cache
Bug #1964848 reported by
melanie witt
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
oslo.limit |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
This was missed when the get_registered_
https:/
Since caching had been added in _get_limit() and get_registered_
description: | updated |
Changed in oslo.limit: | |
status: | New → In Progress |
To post a comment you must log in.
Reviewed: https:/ /review. opendev. org/c/openstack /oslo.limit/ +/828813 /opendev. org/openstack/ oslo.limit/ commit/ c67c9cf0df47947 834b00bc2cbbd45 24521bc011
Committed: https:/
Submitter: "Zuul (22348)"
Branch: master
commit c67c9cf0df47947 834b00bc2cbbd45 24521bc011
Author: melanie witt <email address hidden>
Date: Fri Feb 11 01:34:41 2022 +0000
Move Enforcer caching closer to limit retrieval
In change I22234e0bb6b3a1 cecb29a6b99a3af cd02ffdbf5f added a registered_ limits( ) interface but missed making it work with the limits( ) could not make use of
get_
Enforcer cache because caching was implemented in a higher layer,
_get_limit() and get_registered_
_get_limit().
This moves the caching logic to where limits are retrieved from limits( ) will use caching and any
keystone, so that get_registered_
interfaces that might be added in the future will be able to benefit
from using the cache.
Closes-Bug: #1964848
Change-Id: I28cdd4d3f4927b 94acea3309b3699 9850cf2ee2a