Add http_codes decorator where missing

This commit adds the decorator http_codes to a number of the API put
methods so that all now use it. This ensures any exceptions generated by
the methods return the expected status code and the response body is
JSON encoded.

Change-Id: Id24ef3687afe21e36adae225efdef7150b5d5318
Partial-bug: 1665015
This commit is contained in:
git-harry 2017-03-09 09:16:51 +00:00
parent c18917cdc4
commit cbb246111b
4 changed files with 6 additions and 0 deletions

View File

@ -57,6 +57,7 @@ class CellById(base.Resource):
cell = utils.get_resource_with_vars(request_args, cell_obj)
return cell, 200, None
@base.http_codes
def put(self, context, id, request_data):
"""Update existing cell."""
cell_obj = dbapi.cells_update(context, id, request_data)

View File

@ -68,6 +68,7 @@ class HostById(base.Resource):
return host, 200, None
@base.http_codes
def put(self, context, id, request_data):
"""Update existing host data, or create if it does not exist."""
host_obj = dbapi.hosts_update(context, id, request_data)

View File

@ -55,6 +55,7 @@ class NetworkById(base.Resource):
device['variables'] = jsonutils.to_primitive(obj.variables)
return device, 200, None
@base.http_codes
def put(self, context, id, request_data):
"""Update existing network values."""
net_obj = dbapi.networks_update(context, id, request_data)
@ -124,6 +125,7 @@ class NetworkDeviceById(base.Resource):
return device, 200, None
@base.http_codes
def put(self, context, id, request_data):
"""Update existing device values."""
net_obj = dbapi.network_devices_update(context, id, request_data)
@ -203,6 +205,7 @@ class NetworkInterfaceById(base.Resource):
interface['variables'] = jsonutils.to_primitive(obj.variables)
return interface, 200, None
@base.http_codes
def put(self, context, id, request_data):
"""Update existing network interface values."""
net_obj = dbapi.network_interfaces_update(context, id, request_data)

View File

@ -73,6 +73,7 @@ class RegionsById(base.Resource):
region = utils.get_resource_with_vars(request_args, region_obj)
return region, 200, None
@base.http_codes
def put(self, context, id, request_data):
"""Update existing region."""
region_obj = dbapi.regions_update(context, id, request_data)