openstack
/
keystone
Code Issues Proposed changes

Use keystone.common.provider_api for revoke APIs 

This change converts the usage of self.<provider_api> to
keystone.common.providers_api.ProviderAPIs.<provider_api> in manager
and controller logic. This is the correct way to reference
providers from other managers and controllers now that dependency
injection has been eliminated.

Change-Id: I262536f96c5a1944308252ee2834082b66b1a8cd
This commit is contained in:
Lance Bragstad 2017-12-27 17:31:30 +00:00
parent 1e21c52f3b
commit 2f9b444ab0
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
keystone/revoke/controllers.py
View File

@ -13,10 +13,14 @@
from oslo_utils import timeutils
from keystone.common import controller
from keystone.common import provider_api
from keystone import exception
from keystone.i18n import _
PROVIDERS = provider_api.ProviderAPIs
class RevokeController(controller.V3Controller):
@controller.protected()
def list_revoke_events(self, request):
@ -35,7 +39,7 @@ class RevokeController(controller.V3Controller):
# and should be deprecated in the near future. Fix this before adding
# resource_options or any lazy-loaded relationships to the revocation
# events themselves.
events = self.revoke_api.list_events(last_fetch=last_fetch)
events = PROVIDERS.revoke_api.list_events(last_fetch=last_fetch)
# Build the links by hand as the standard controller calls require ids
response = {'events': [event.to_dict() for event in events],
'links': {