)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"b42813cc20c10514a574c7a5cf5223e89caf808f","unresolved":false,"context_lines":[{"line_number":9,"context_line":"When keystoneclient selects the v2 endpoint, the quota-show"},{"line_number":10,"context_line":"command fails with"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"          raise AttributeError(_(\"Unknown Attribute: %s\") % name)"},{"line_number":13,"context_line":"on"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"    project \u003d keystone.projects.get(id, subtree_as_ids\u003dsubtree_as_ids)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3a7e1126_f53c63e2","line":12,"range":{"start_line":12,"start_character":10,"end_line":12,"end_character":15},"updated":"2015-12-22 03:09:23.000000000","message":"It is easier to locate if you add a reference link to code and line number.","commit_id":"0a355ad6a1409704e40e9a2c4acdc3f57f91ee0d"},{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"b42813cc20c10514a574c7a5cf5223e89caf808f","unresolved":false,"context_lines":[{"line_number":15,"context_line":"    project \u003d keystone.projects.get(id, subtree_as_ids\u003dsubtree_as_ids)"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-Bug: 1517043"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I89443b6de5b1d70cdcff31fe01719a8cc9878fa5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3a7e1126_7529539a","line":18,"updated":"2015-12-22 03:09:23.000000000","message":"I think you may remove the blank space here.","commit_id":"0a355ad6a1409704e40e9a2c4acdc3f57f91ee0d"}],"cinder/api/contrib/quotas.py":[{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"b42813cc20c10514a574c7a5cf5223e89caf808f","unresolved":false,"context_lines":[{"line_number":192,"context_line":"                                     session\u003dclient_session)"},{"line_number":193,"context_line":"        except exceptions.NotFound:"},{"line_number":194,"context_line":"            # This can happen when there is no V2 endpoint"},{"line_number":195,"context_line":"            return None"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"        try:"},{"line_number":198,"context_line":"            project \u003d keystone.projects.get(id, subtree_as_ids\u003dsubtree_as_ids)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a7e1126_1877e6a0","line":195,"range":{"start_line":195,"start_character":19,"end_line":195,"end_character":23},"updated":"2015-12-22 03:09:23.000000000","message":"I think raise an exception with message like \"You should be using keystone V3\" helps users more.\n\nAt least there should be an warning logged, instead of swallow the exception.","commit_id":"0a355ad6a1409704e40e9a2c4acdc3f57f91ee0d"}],"cinder/tests/unit/api/contrib/test_quotas.py":[{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"b42813cc20c10514a574c7a5cf5223e89caf808f","unresolved":false,"context_lines":[{"line_number":136,"context_line":"        context \u003d self.req.environ[\u0027cinder.context\u0027]"},{"line_number":137,"context_line":"        self.controller._get_project(context, context.project_id)"},{"line_number":138,"context_line":"        ksclient_class.assert_called_once_with(auth_url\u003dself.auth_url,"},{"line_number":139,"context_line":"                                               version\u003d(3,),"},{"line_number":140,"context_line":"                                               session\u003dksclient_session())"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"    @mock.patch(\u0027keystoneclient.client.Client\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a7e1126_5861eed1","line":139,"range":{"start_line":139,"start_character":47,"end_line":139,"end_character":54},"updated":"2015-12-22 03:09:23.000000000","message":"The test case modified here hides the bug, rather than reproduce the bug and verify it is fixed.","commit_id":"0a355ad6a1409704e40e9a2c4acdc3f57f91ee0d"}]}
