)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"6d3ebaac6a661976cf56a17a392b6354e5e56046","unresolved":false,"context_lines":[{"line_number":10,"context_line":"is either replaced with a LOG+raise or removed completely when not"},{"line_number":11,"context_line":"needed (again, HACKING.rst suggests the try clause should be avoided"},{"line_number":12,"context_line":"too letting the original exception to flow)."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ief779bada1aa53837d62e3f58a7d3418eef494ba"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAASn%2F%2F%2B7s%3D","line":13,"updated":"2013-08-27 07:57:24.000000000","message":"I think you should write \"blueprint replace-self-fail\" in this commit message.","commit_id":"9bab2edd299aed5ef5f7c74ed1841c281e93bce9"}],"tempest/api/compute/test_authorization.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"6d3ebaac6a661976cf56a17a392b6354e5e56046","unresolved":false,"context_lines":[{"line_number":16,"context_line":"#    under the License."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from tempest import clients"},{"line_number":19,"context_line":"from tempest import exceptions"},{"line_number":20,"context_line":"from tempest.api import compute"},{"line_number":21,"context_line":"from tempest.api.compute import base"},{"line_number":22,"context_line":"from tempest.common.utils.data_utils import parse_image_id"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAASn%2F%2F%2FNw%3D","line":19,"updated":"2013-08-27 07:57:24.000000000","message":"This change causes pep8 error.","commit_id":"9bab2edd299aed5ef5f7c74ed1841c281e93bce9"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"6d3ebaac6a661976cf56a17a392b6354e5e56046","unresolved":false,"context_lines":[{"line_number":207,"context_line":"            self.alt_keypairs_client.base_url \u003d self.saved_base_url"},{"line_number":208,"context_line":"            if (resp[\u0027status\u0027] is not None):"},{"line_number":209,"context_line":"                resp, _ \u003d self.alt_keypairs_client.delete_keypair(k_name)"},{"line_number":210,"context_line":"                LOG.error(\"Create keypair request should not happen \""},{"line_number":211,"context_line":"                          \"if the tenant id does not match the current user\")"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    @attr(type\u003d\u0027gate\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAASn%2F%2F%2FMM%3D","line":210,"updated":"2013-08-27 07:57:24.000000000","message":"Is a raise not necessary here?","commit_id":"9bab2edd299aed5ef5f7c74ed1841c281e93bce9"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"6d3ebaac6a661976cf56a17a392b6354e5e56046","unresolved":false,"context_lines":[{"line_number":258,"context_line":"            self.alt_security_client.base_url \u003d self.saved_base_url"},{"line_number":259,"context_line":"            if resp[\u0027status\u0027] is not None:"},{"line_number":260,"context_line":"                self.alt_security_client.delete_security_group(resp[\u0027id\u0027])"},{"line_number":261,"context_line":"                LOG.error(\"Create Security Group request should not happen if\""},{"line_number":262,"context_line":"                          \"the tenant id does not match the current user\")"},{"line_number":263,"context_line":""},{"line_number":264,"context_line":"    @attr(type\u003d\u0027gate\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAASn%2F%2F%2B%2BE%3D","line":261,"updated":"2013-08-27 07:57:24.000000000","message":"ditto","commit_id":"9bab2edd299aed5ef5f7c74ed1841c281e93bce9"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"6d3ebaac6a661976cf56a17a392b6354e5e56046","unresolved":false,"context_lines":[{"line_number":300,"context_line":"            self.alt_security_client.base_url \u003d self.saved_base_url"},{"line_number":301,"context_line":"            if resp[\u0027status\u0027] is not None:"},{"line_number":302,"context_line":"                self.alt_security_client.delete_security_group_rule(resp[\u0027id\u0027])"},{"line_number":303,"context_line":"                LOG.error(\"Create security group rule request should not \""},{"line_number":304,"context_line":"                          \"happen if the tenant id does not match the\""},{"line_number":305,"context_line":"                          \" current user\")"},{"line_number":306,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAASn%2F%2F%2B%2BA%3D","line":303,"updated":"2013-08-27 07:57:24.000000000","message":"and here.","commit_id":"9bab2edd299aed5ef5f7c74ed1841c281e93bce9"}],"tempest/api/identity/admin/v3/test_endpoints.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"6d3ebaac6a661976cf56a17a392b6354e5e56046","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        resp, fetched_endpoints \u003d self.client.list_endpoints()"},{"line_number":85,"context_line":"        fetched_endpoints_id \u003d [e[\u0027id\u0027] for e in fetched_endpoints]"},{"line_number":86,"context_line":"        self.assertIn(endpoint[\u0027id\u0027], fetched_endpoints_id)"},{"line_number":87,"context_line":"        # Deleting the endpoint created in this method"},{"line_number":88,"context_line":"        resp, body \u003d self.client.delete_endpoint(endpoint[\u0027id\u0027])"},{"line_number":89,"context_line":"        self.assertEqual(resp[\u0027status\u0027], \u0027204\u0027)"},{"line_number":90,"context_line":"        self.assertEqual(body, \u0027\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAASn%2F%2F%2FHo%3D","line":87,"updated":"2013-08-27 07:57:24.000000000","message":"In this commit, I think if an error occurs in the above, the endpoint is not deleted.","commit_id":"9bab2edd299aed5ef5f7c74ed1841c281e93bce9"}],"tempest/api/object_storage/test_object_services.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"ceb609d329ea65b5273182539f8287f0f6701944","unresolved":false,"context_lines":[{"line_number":216,"context_line":"            src_container_name, object_name, dst_container_name,"},{"line_number":217,"context_line":"            object_name)"},{"line_number":218,"context_line":"        self.assertEqual(resp[\u0027status\u0027], \u0027201\u0027)"},{"line_number":219,"context_line":"        # check if object is present in destination container"},{"line_number":220,"context_line":"        resp, body \u003d self.object_client.get_object(dst_container_name,"},{"line_number":221,"context_line":"                                                   object_name)"},{"line_number":222,"context_line":"        self.assertEqual(body, data)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAASn%2F%2F6aw%3D","line":219,"updated":"2013-08-28 02:58:47.000000000","message":"very minor nit - I\u0027d prefer that we keep some vertical whitespace around even around (before) comments. I think it makes the code easier to read","commit_id":"92f7719f6513b37ab72e8d6bb5b1b2830b102c15"}],"tempest/scenario/manager.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"6d3ebaac6a661976cf56a17a392b6354e5e56046","unresolved":false,"context_lines":[{"line_number":29,"context_line":"import neutronclient.v2_0.client"},{"line_number":30,"context_line":"import novaclient.client"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"from tempest import exceptions"},{"line_number":33,"context_line":"from tempest.api.network import common as net_common"},{"line_number":34,"context_line":"from tempest.common import isolated_creds"},{"line_number":35,"context_line":"from tempest.common import ssh"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAASn%2F%2F%2FF4%3D","line":32,"updated":"2013-08-27 07:57:24.000000000","message":"This change causes a pep8 error.","commit_id":"9bab2edd299aed5ef5f7c74ed1841c281e93bce9"}]}