]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
All API controllers inherit from wsgi.Controller
authorling-yun <zengyunling@huawei.com>
Tue, 3 Dec 2013 08:19:17 +0000 (16:19 +0800)
committerling-yun <zengyunling@huawei.com>
Wed, 4 Dec 2013 06:53:08 +0000 (14:53 +0800)
Most of the Cinder Public API Controllers inherit from wsgi.Controller,
but still some Cinder Public APIs controllers directly inherit from
object. So it's necessary to unify all the Cinder Public API Controllers
inherit from wsgi.Controller.
1. For code cleanup.
2. When we try to do body valid check in api's put/post method later, we
can use the helper function is_valid_body instead of using try...catch
block.

Change-Id: I88812224e0b182939cfccce63982cb8f95792891
Closes-Bug: #1257198

cinder/api/contrib/hosts.py
cinder/api/contrib/services.py
cinder/api/v1/limits.py
cinder/api/v1/snapshot_metadata.py
cinder/api/v1/volume_metadata.py
cinder/api/v2/limits.py
cinder/api/v2/snapshot_metadata.py
cinder/api/v2/volume_metadata.py

index cf11395330e0bee0e832c2078c27f4e730ca7cb5..69e8070e79c5f6b1d19172fdc0cc142fefb22380 100644 (file)
@@ -138,7 +138,7 @@ def check_host(fn):
     return wrapped
 
 
-class HostController(object):
+class HostController(wsgi.Controller):
     """The Hosts API controller for the OpenStack API."""
     def __init__(self):
         self.api = volume_api.HostAPI()
index a72ae30475eb672494bfd5d3b0ed249119e1724e..d242e5209980e7357f6bced073793b9f2a9a1a50 100644 (file)
@@ -65,7 +65,7 @@ class ServicesUpdateTemplate(xmlutil.TemplateBuilder):
         return xmlutil.MasterTemplate(root, 1)
 
 
-class ServiceController(object):
+class ServiceController(wsgi.Controller):
     @wsgi.serializers(xml=ServicesIndexTemplate)
     def index(self, req):
         """Return a list of all running services.
index aeac101ed1302bbfa50c22bae671a9f47638524b..de70243f98521beb429bc77504951e968fa79caa 100644 (file)
@@ -75,7 +75,7 @@ class LimitsTemplate(xmlutil.TemplateBuilder):
         return xmlutil.MasterTemplate(root, 1, nsmap=limits_nsmap)
 
 
-class LimitsController(object):
+class LimitsController(wsgi.Controller):
     """Controller for accessing limits in the OpenStack API."""
 
     @wsgi.serializers(xml=LimitsTemplate)
index c009fbba14cbed31fbb5273b3623e0f6c46677d5..87997aec4132b187f1b70395c3146366a71b64ea 100644 (file)
@@ -24,7 +24,7 @@ from cinder import volume
 from webob import exc
 
 
-class Controller(object):
+class Controller(wsgi.Controller):
     """The volume metadata API controller for the OpenStack API."""
 
     def __init__(self):
index 3e196d1147b5c26d27a64cbb8e4090cd9be62476..141bbba731ff3666f8067c68f3d5479abcc89d36 100644 (file)
@@ -24,7 +24,7 @@ from cinder import volume
 from webob import exc
 
 
-class Controller(object):
+class Controller(wsgi.Controller):
     """The volume metadata API controller for the OpenStack API."""
 
     def __init__(self):
index face1038dbc61ffd1aeb5a838397b5299a3c6fd5..d8f17d424ac6cec05a89e6e452bed19354382b14 100644 (file)
@@ -74,7 +74,7 @@ class LimitsTemplate(xmlutil.TemplateBuilder):
         return xmlutil.MasterTemplate(root, 1, nsmap=limits_nsmap)
 
 
-class LimitsController(object):
+class LimitsController(wsgi.Controller):
     """Controller for accessing limits in the OpenStack API."""
 
     @wsgi.serializers(xml=LimitsTemplate)
index c009fbba14cbed31fbb5273b3623e0f6c46677d5..87997aec4132b187f1b70395c3146366a71b64ea 100644 (file)
@@ -24,7 +24,7 @@ from cinder import volume
 from webob import exc
 
 
-class Controller(object):
+class Controller(wsgi.Controller):
     """The volume metadata API controller for the OpenStack API."""
 
     def __init__(self):
index b9449da0c81bb61b24a6a2afbb8cc0e1ca9822a7..cf8ef548b7f85c32ea2bddf5a54dc77cdb381516 100644 (file)
@@ -23,7 +23,7 @@ from cinder import exception
 from cinder import volume
 
 
-class Controller(object):
+class Controller(wsgi.Controller):
     """The volume metadata API controller for the OpenStack API."""
 
     def __init__(self):