openstack
/
nova
Code Issues Proposed changes

Merge "Pass the actual target in os-agents policy"

This commit is contained in:
Zuul 2020-03-31 20:03:22 +00:00 committed by Gerrit Code Review
parent 232486132b 4af5f17205
commit 15974f476f
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
nova/api/openstack/compute/agents.py
View File

@ -52,7 +52,7 @@ class AgentController(wsgi.Controller):
def index(self, req):
"""Return a list of all agent builds. Filter by hypervisor."""
context = req.environ['nova.context']
context.can(agents_policies.BASE_POLICY_NAME % 'list')
context.can(agents_policies.BASE_POLICY_NAME % 'list', target={})
hypervisor = None
agents = []
if 'hypervisor' in req.GET:
@ -75,7 +75,7 @@ class AgentController(wsgi.Controller):
def update(self, req, id, body):
"""Update an existing agent build."""
context = req.environ['nova.context']
context.can(agents_policies.BASE_POLICY_NAME % 'update')
context.can(agents_policies.BASE_POLICY_NAME % 'update', target={})
# TODO(oomichi): This parameter name "para" is different from the ones
# of the other APIs. Most other names are resource names like "server"
@ -118,7 +118,7 @@ class AgentController(wsgi.Controller):
def delete(self, req, id):
"""Deletes an existing agent build."""
context = req.environ['nova.context']
context.can(agents_policies.BASE_POLICY_NAME % 'delete')
context.can(agents_policies.BASE_POLICY_NAME % 'delete', target={})
try:
utils.validate_integer(id, 'id')
@ -140,7 +140,7 @@ class AgentController(wsgi.Controller):
def create(self, req, body):
"""Creates a new agent build."""
context = req.environ['nova.context']
context.can(agents_policies.BASE_POLICY_NAME % 'create')
context.can(agents_policies.BASE_POLICY_NAME % 'create', target={})
agent = body['agent']
hypervisor = agent['hypervisor']