openstack
/
nova
Code Issues Proposed changes

Remove 400 as expected error 

400 is not an expected error for create server and update_all
metadata so , remove it from API layer.
TrivialFix

Closes-Bug: 1718335

Change-Id: I655c908df5c989f9d3654db0ed0c28618c141fb7
This commit is contained in:
jichenjc 2017-09-16 11:07:56 +08:00
parent 2af92486b2
commit 550e6dcd70
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
nova/api/openstack/compute/server_metadata.py
View File

@ -56,7 +56,7 @@ class ServerMetadataController(wsgi.Controller):
context.can(sm_policies.POLICY_ROOT % 'index')
return {'metadata': self._get_metadata(context, server_id)}
@extensions.expected_errors((400, 403, 404, 409))
@extensions.expected_errors((403, 404, 409))
# NOTE(gmann): Returns 200 for backwards compatibility but should be 201
# as this operation complete the creation of metadata.
@validation.schema(server_metadata.create)
@ -88,7 +88,7 @@ class ServerMetadataController(wsgi.Controller):
return {'meta': meta_item}
@extensions.expected_errors((400, 403, 404, 409))
@extensions.expected_errors((403, 404, 409))
@validation.schema(server_metadata.update_all)
def update_all(self, req, server_id, body):
context = req.environ['nova.context']