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:
parent
c18917cdc4
commit
cbb246111b
@ -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)
|
||||
|
@ -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)
|
||||
|
@ -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)
|
||||
|
@ -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)
|
||||
|
Loading…
Reference in New Issue
Block a user