openstack
/
nova
Code Issues Proposed changes

Remove cell policy check 

we already got policy checks for create,update,delete
and sync_instances of cell, there is no need to check the
general policy of cell and it might lead to inconsistency
problem.

The v2 code is not touched because v2.1 is current using.

UpgradeImpact

Partially implements bp nova-api-policy-final-part

Change-Id: I05ea8b0f3b093b7dbfbe037c21e224acc3c605ad
This commit is contained in:
jichenjc 2015-05-14 21:56:24 +08:00
parent 7134f1e309
commit 06e6056482
1 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
nova/api/openstack/compute/plugins/v3/cells.py
View File

@ -177,7 +177,6 @@ class CellsController(wsgi.Controller):
"""Delete a child or parent cell entry. 'id' is a cell name."""
context = req.environ['nova.context']
authorize(context)
authorize(context, action="delete")
try:
@ -242,7 +241,6 @@ class CellsController(wsgi.Controller):
"""Create a child cell entry."""
context = req.environ['nova.context']
authorize(context)
authorize(context, action="create")
cell = body['cell']
@ -260,7 +258,6 @@ class CellsController(wsgi.Controller):
"""Update a child cell entry. 'id' is the cell name to update."""
context = req.environ['nova.context']
authorize(context)
authorize(context, action="update")
cell = body['cell']
@ -295,7 +292,6 @@ class CellsController(wsgi.Controller):
"""Tell all cells to sync instance info."""
context = req.environ['nova.context']
authorize(context)
authorize(context, action="sync_instances")
project_id = body.pop('project_id', None)