openstack
/
keystone
Code Issues Proposed changes

Fix cache invalidation for application credential 

The cache invalidation doesn't work for application credential.

This patch fixed it.

Change-Id: I730df6f4abe1ec63d93e999535de7afad05e76a7
Closes-bug: #1746868
This commit is contained in:
wangxiyuan 2018-02-02 14:37:37 +08:00
parent 10e8e69e02
commit 63fde3eefb
2 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
keystone/application_credential/core.py
View File

@ -183,7 +183,8 @@ class Manager(manager.Manager):
:raises keystone.exception.ApplicationCredentialNotFound: If the
application credential doesn't exist.
"""
self.get_application_credential.invalidate()
self.get_application_credential.invalidate(self,
application_credential_id)
self.driver.delete_application_credential(application_credential_id)
notifications.Audit.deleted(
self._APP_CRED, application_credential_id, initiator)

3
keystone/tests/unit/application_credential/test_backends.py
View File

@ -19,7 +19,6 @@ from keystone.common import driver_hints
from keystone.common import provider_api
import keystone.conf
from keystone import exception
from keystone.tests.unit import utils as test_utils
CONF = keystone.conf.CONF
@ -151,8 +150,6 @@ class ApplicationCredentialTests(object):
hints)
return [ac['id'] for ac in resp]
@test_utils.wip('Waiting to fix the cache invalidation problem from '
'fixing bug 1746868.')
def test_delete_application_credential(self):
app_cred = self._new_app_cred_data(self.user_foo['id'],
project_id=self.tenant_bar['id'])