From 06e605648223a2e3182313688a5f08bdbb43b38e Mon Sep 17 00:00:00 2001 From: jichenjc Date: Thu, 14 May 2015 21:56:24 +0800 Subject: [PATCH] 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 --- nova/api/openstack/compute/plugins/v3/cells.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/nova/api/openstack/compute/plugins/v3/cells.py b/nova/api/openstack/compute/plugins/v3/cells.py index 363cf2a3ae7e..a910a147fe6c 100644 --- a/nova/api/openstack/compute/plugins/v3/cells.py +++ b/nova/api/openstack/compute/plugins/v3/cells.py @@ -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)