)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"8076645ba47588ed651f713f667530e61e228986","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"And fix server\u0027s action confirm_resize return wrong status"},{"line_number":13,"context_line":"code. That because HTTPNoContent overwrite the status code"},{"line_number":14,"context_line":"of decorator \u0027wsgi.response(202)\u0027"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Partially implements blueprint v3-api-unittests"},{"line_number":17,"context_line":"Close-bug: #1233561"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAATH%2F%2F9%2Fo%3D","line":14,"updated":"2013-10-07 10:28:45.000000000","message":"Feels wrong that this bug fix is included in here.  \n\nIts not obvious from the title, and feels like something that might be wanted as a seperate backport.   Can that be split out into a separate change ?","commit_id":"cd558095403a58d0ddeee56d9ffabbb779a3027d"},{"author":{"_account_id":7040,"name":"GuoHui LIu","email":"liuguohui@gmail.com","username":"guohliu"},"change_message_id":"03dcccc7931469800721fc564ef8030bc94f3bc4","unresolved":false,"context_lines":[{"line_number":14,"context_line":"the status code of confirm_resize."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Partially implements blueprint v3-api-unittests"},{"line_number":17,"context_line":"Close-bug: #1233561"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I50a3046866af0a1efdb5c41ded40f973b02843e8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATH%2F%2F6HE%3D","line":17,"updated":"2013-10-08 07:43:33.000000000","message":"nit: Closes-bug, I guess that\u0027s why the patch hasn\u0027t been linked with the LP bug.","commit_id":"5e1c23ca5416d5534b6ec03f94e6b1419548f86c"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"660670876c242df4fe2d9ac9dea1f516f07b64aa","unresolved":false,"context_lines":[{"line_number":14,"context_line":"the status code of confirm_resize."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Partially implements blueprint v3-api-unittests"},{"line_number":17,"context_line":"Close-bug: #1233561"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I50a3046866af0a1efdb5c41ded40f973b02843e8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATH%2F%2F6GE%3D","line":17,"in_reply_to":"AAAATH%2F%2F6HE%3D","updated":"2013-10-08 07:50:13.000000000","message":"oops, thanks","commit_id":"5e1c23ca5416d5534b6ec03f94e6b1419548f86c"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"9730c521979028df7b02d09abd37c8988a621f9b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The status code of confirm_resize should be 202, but it"},{"line_number":10,"context_line":"returns 204 now. That because it returns HTTPNoContent"},{"line_number":11,"context_line":"that overwrite the status code of decorator \u0027wsgi.response(202)\u0027"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"And add API samples for confirm_resize that used to test"},{"line_number":14,"context_line":"the status code of confirm_resize."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAATH%2F%2F6Bw%3D","line":11,"updated":"2013-10-08 08:18:10.000000000","message":"overwrite \u003d\u003e overwrites","commit_id":"f1d5dc7a66f5ac9ccc0d3217d50c26226b8b63e4"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"1355cd9df4af3533b75bcc5c60cf84e2f53b4fa6","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The status code of confirm_resize should be 202, but it"},{"line_number":10,"context_line":"returns 204 now. That because it returns HTTPNoContent"},{"line_number":11,"context_line":"that overwrite the status code of decorator \u0027wsgi.response(202)\u0027"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"And add API samples for confirm_resize that used to test"},{"line_number":14,"context_line":"the status code of confirm_resize."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAATH%2F%2F5%2B0%3D","line":11,"in_reply_to":"AAAATH%2F%2F6Bw%3D","updated":"2013-10-08 08:31:05.000000000","message":"thanks","commit_id":"f1d5dc7a66f5ac9ccc0d3217d50c26226b8b63e4"}],"nova/api/openstack/compute/plugins/v3/servers.py":[{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"8076645ba47588ed651f713f667530e61e228986","unresolved":false,"context_lines":[{"line_number":951,"context_line":"        except exception.InstanceInvalidState as state_error:"},{"line_number":952,"context_line":"            common.raise_http_conflict_for_instance_invalid_state(state_error,"},{"line_number":953,"context_line":"                    \u0027confirm_resize\u0027)"},{"line_number":954,"context_line":"        return exc.HTTPNoContent()"},{"line_number":955,"context_line":""},{"line_number":956,"context_line":"    @wsgi.response(202)"},{"line_number":957,"context_line":"    @wsgi.serializers(xml\u003dFullServerTemplate)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATH%2F%2F9%2Bg%3D","side":"PARENT","line":954,"updated":"2013-10-07 10:28:45.000000000","message":"As per comments on the commit message - I think this change would be better as a separate commit","commit_id":"9501fa954b4d724b665207c2a0188e92f691fdfb"}],"nova/tests/integrated/v3/test_servers.py":[{"author":{"_account_id":7040,"name":"GuoHui LIu","email":"liuguohui@gmail.com","username":"guohliu"},"change_message_id":"1f8c9996f4abfade07ccd470f1f94ba13c6e2ab4","unresolved":false,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def test_server_confirm_resize(self):"},{"line_number":134,"context_line":"        uuid \u003d self.test_server_resize()"},{"line_number":135,"context_line":"        self._test_server_action(uuid, \"confirm_resize\", code\u003d204)"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    def test_server_create_image(self):"},{"line_number":138,"context_line":"        uuid \u003d self._post_server()"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2FVu8%3D","line":135,"updated":"2013-09-29 16:17:11.000000000","message":"202, right?","commit_id":"3db76c9868e7897a5b5811967dcd08ab7878b06f"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"1b98178577e54fa33c9dcedb4e92a3cc182273d5","unresolved":false,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def test_server_confirm_resize(self):"},{"line_number":134,"context_line":"        uuid \u003d self.test_server_resize()"},{"line_number":135,"context_line":"        self._test_server_action(uuid, \"confirm_resize\", code\u003d204)"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    def test_server_create_image(self):"},{"line_number":138,"context_line":"        uuid \u003d self._post_server()"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2FRHI%3D","line":135,"in_reply_to":"AAAAS3%2F%2FVu8%3D","updated":"2013-10-01 08:47:31.000000000","message":"Thanks, there is bug for confirm_resize, i will fix it first","commit_id":"3db76c9868e7897a5b5811967dcd08ab7878b06f"},{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"c4ad8c384376b57c7ef6189626d9ccfbac5ce048","unresolved":false,"context_lines":[{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    def test_server_resize(self):"},{"line_number":98,"context_line":"        self.flags(allow_resize_to_same_host\u003dTrue)"},{"line_number":99,"context_line":"        uuid \u003d self._post_server()"},{"line_number":100,"context_line":"        self._test_server_action(uuid, \"resize\","},{"line_number":101,"context_line":"                                 {\"id\": 2,"},{"line_number":102,"context_line":"                                  \"host\": self._get_host()})"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATH%2F%2FanI%3D","line":99,"updated":"2013-10-16 14:00:08.000000000","message":"I don\u0027t love uuid as a variable name since it shadows a builtin module name, but since you\u0027re not using the uuid module it\u0027s not a real problem.","commit_id":"01a44568cc60bb5a6dd7b55d69b20bba57d1b94b"}]}