)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"08d047234254b2e0bd183ea00ec28fe9d616d2bf","unresolved":false,"context_lines":[{"line_number":13,"context_line":"   exception.NoValidHost,"},{"line_number":14,"context_line":"   exception.InvalidLocalStorage,"},{"line_number":15,"context_line":"   exception.InvalidSharedStorage,"},{"line_number":16,"context_line":"   exception.MigrationPreCheckError"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I40ccb0f268716da20ae4b3de9d295a41a2dba18d"},{"line_number":19,"context_line":"Closes-Bug: #1255001"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAATn%2F9VhU%3D","line":16,"updated":"2013-12-12 14:48:24.000000000","message":"Your patch is no longer related to these other exceptions","commit_id":"fa383dba64784190ab455d28189b39e5ad0c9694"},{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"7615c2cc98a0f1de36fe443b3f2a956ffa38c60a","unresolved":false,"context_lines":[{"line_number":13,"context_line":"   exception.NoValidHost,"},{"line_number":14,"context_line":"   exception.InvalidLocalStorage,"},{"line_number":15,"context_line":"   exception.InvalidSharedStorage,"},{"line_number":16,"context_line":"   exception.MigrationPreCheckError"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I40ccb0f268716da20ae4b3de9d295a41a2dba18d"},{"line_number":19,"context_line":"Closes-Bug: #1255001"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAATn%2F9Rsk%3D","line":16,"in_reply_to":"AAAATn%2F9VhU%3D","updated":"2013-12-13 06:42:48.000000000","message":"I\u0027m sorry.\nThe other patch has merged for these exceptions. \nI will modify this.","commit_id":"fa383dba64784190ab455d28189b39e5ad0c9694"}],"nova/api/openstack/compute/contrib/admin_actions.py":[{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"08d047234254b2e0bd183ea00ec28fe9d616d2bf","unresolved":false,"context_lines":[{"line_number":329,"context_line":"            raise exc.HTTPNotFound(explanation\u003de.format_message())"},{"line_number":330,"context_line":"        except exception.InstanceInvalidState as state_error:"},{"line_number":331,"context_line":"            common.raise_http_conflict_for_instance_invalid_state(state_error,"},{"line_number":332,"context_line":"                    \u0027os-migrateLive\u0027)"},{"line_number":333,"context_line":"        return webob.Response(status_int\u003d202)"},{"line_number":334,"context_line":""},{"line_number":335,"context_line":"    @wsgi.action(\u0027os-resetState\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAATn%2F9Vgc%3D","line":332,"updated":"2013-12-12 14:48:24.000000000","message":"On second thought, this patch is changing more than you describe.  It\u0027s changing the response for all other exceptions to 500 instead of 400, presumably.  I agree that is probably more correct, which is why I approved this before.  Now I wonder if that should be split into another patch, or even left out entirely.  We risk introducing a regression here for something where 400 really is the right response, but we\u0027re no longer catching it.  I\u0027m tempted to just leave this part alone for the v2 API.","commit_id":"fa383dba64784190ab455d28189b39e5ad0c9694"},{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"7615c2cc98a0f1de36fe443b3f2a956ffa38c60a","unresolved":false,"context_lines":[{"line_number":329,"context_line":"            raise exc.HTTPNotFound(explanation\u003de.format_message())"},{"line_number":330,"context_line":"        except exception.InstanceInvalidState as state_error:"},{"line_number":331,"context_line":"            common.raise_http_conflict_for_instance_invalid_state(state_error,"},{"line_number":332,"context_line":"                    \u0027os-migrateLive\u0027)"},{"line_number":333,"context_line":"        return webob.Response(status_int\u003d202)"},{"line_number":334,"context_line":""},{"line_number":335,"context_line":"    @wsgi.action(\u0027os-resetState\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAATn%2F9Rrc%3D","line":332,"in_reply_to":"AAAATn%2F9Vgc%3D","updated":"2013-12-13 06:42:48.000000000","message":"Thanks. \nI just find out the response for all other exceptions to 500 instead of 400. So i think it should be consistent with v3.\nYou mean it should be in another patch?","commit_id":"fa383dba64784190ab455d28189b39e5ad0c9694"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"78b9a794379e975b0d308c1474c76257b9172286","unresolved":false,"context_lines":[{"line_number":337,"context_line":"            LOG.exception(msg)"},{"line_number":338,"context_line":"            # Return messages from scheduler"},{"line_number":339,"context_line":"            raise exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"        return webob.Response(status_int\u003d202)"},{"line_number":342,"context_line":""},{"line_number":343,"context_line":"    @wsgi.action(\u0027os-resetState\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAATn%2F9IYk%3D","side":"PARENT","line":340,"updated":"2013-12-16 14:33:49.000000000","message":"We shoudn\u0027t touch any extra thing.","commit_id":"0b54716424d7d8eaf66a55edee77df9cdf21320a"}]}