Merge "Log policy filters in one line"
This commit is contained in:
commit
4bf5b9d5da
@ -15,6 +15,7 @@
|
||||
|
||||
import copy
|
||||
|
||||
from oslo_log import log as logging
|
||||
from oslo_policy import policy as oslo_policy
|
||||
from oslo_utils import excutils
|
||||
from pecan import hooks
|
||||
@ -29,6 +30,8 @@ from neutron.pecan_wsgi.controllers import quota
|
||||
from neutron.pecan_wsgi.hooks import utils
|
||||
from neutron import policy
|
||||
|
||||
LOG = logging.getLogger(__name__)
|
||||
|
||||
|
||||
def _custom_getter(resource, resource_id):
|
||||
"""Helper function to retrieve resources not served by any plugin."""
|
||||
@ -250,4 +253,7 @@ class PolicyHook(hooks.PecanHook):
|
||||
# This should be migrated to project_id later.
|
||||
if attr_name == 'tenant_id':
|
||||
attributes_to_exclude.append('project_id')
|
||||
if attributes_to_exclude:
|
||||
LOG.debug("Attributes excluded by policy engine: %s",
|
||||
attributes_to_exclude)
|
||||
return attributes_to_exclude
|
||||
|
@ -368,9 +368,6 @@ def check(context, action, target, plugin=None, might_not_exist=False,
|
||||
target,
|
||||
credentials,
|
||||
pluralized=pluralized)
|
||||
# logging applied rules in case of failure
|
||||
if not result:
|
||||
log_rule_list(match_rule)
|
||||
return result
|
||||
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user