Remove extra policy check

This is a follow-up change to the ensure shares APIs [1]. An
additional policy check was added, but it was not needed, as it
was already being performed by the decorator, so we remove it.

[1] https://review.opendev.org/c/openstack/manila/+/924279

Change-Id: Iab25d5a6a3906dfdb1cb3473188edcc5fafc80d4
(cherry picked from commit 697c9bf3d34b4eb291158c99ae859da10a0c2c59)
This commit is contained in:
Carlos Eduardo 2024-09-18 11:08:55 -03:00
parent 5b58f34534
commit 7fd6a54e42

View File

@ -23,7 +23,6 @@ from manila.api.views import services as services_views
from manila import db
from manila import exception
from manila.i18n import _
from manila import policy
from manila.services import api as service_api
@ -173,7 +172,6 @@ class ServiceController(ServiceMixin, wsgi.Controller):
"""Starts ensure shares for a given manila-share binary."""
context = req.environ['manila.context']
policy.check_policy(context, 'service', 'ensure_shares')
host = body.get('host', None)
if not host:
raise webob.exc.HTTPBadRequest('Missing host parameter.')