)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"35b4a31d53d0168e15cc1f733577e2a3b9848cb7","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Yusuke Hayashi \u003chayashi-yusuke@jp.fujitsu.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-08-09 13:30:31 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix incorrect exception message from get_volume_api_from_url"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Incorrect exception message was outputted"},{"line_number":10,"context_line":"from get_volume_api_from_url(url) in cinderclient.client."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a4dcd0f_4543026d","line":7,"updated":"2015-08-09 16:48:51.000000000","message":"https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure says \"The first line should be limited to 50 characters ...\"","commit_id":"4cb9458430d60d7185c0d7dbfad9f7cfdfd89ded"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"35b4a31d53d0168e15cc1f733577e2a3b9848cb7","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix incorrect exception message from get_volume_api_from_url"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Incorrect exception message was outputted"},{"line_number":10,"context_line":"from get_volume_api_from_url(url) in cinderclient.client."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Here is example:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a4dcd0f_85490a91","line":9,"updated":"2015-08-09 16:48:51.000000000","message":"s/outputted/output/\n\nEnglish irregular verbs are a pain :-(","commit_id":"4cb9458430d60d7185c0d7dbfad9f7cfdfd89ded"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"35b4a31d53d0168e15cc1f733577e2a3b9848cb7","unresolved":false,"context_lines":[{"line_number":15,"context_line":"\"http://fakeurl/v9/tenants\""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"\u003cException message\u003e"},{"line_number":18,"context_line":"cinderclient.exceptions.UnsupportedVersion: "},{"line_number":19,"context_line":"Invalid client version \u0027v2\u0027. must be one of: v1, v2"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"\u003d\u003e Although input does not include \u0027v2\u0027, "}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a4dcd0f_a5676624","line":18,"updated":"2015-08-09 16:48:51.000000000","message":"remove trailing whitespace","commit_id":"4cb9458430d60d7185c0d7dbfad9f7cfdfd89ded"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"35b4a31d53d0168e15cc1f733577e2a3b9848cb7","unresolved":false,"context_lines":[{"line_number":18,"context_line":"cinderclient.exceptions.UnsupportedVersion: "},{"line_number":19,"context_line":"Invalid client version \u0027v2\u0027. must be one of: v1, v2"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"\u003d\u003e Although input does not include \u0027v2\u0027, "},{"line_number":22,"context_line":"   exception message reports that invalid version v2 is included."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Change-Id: I73ff1581b1ce609b3c64a8a6d88d41a2cc2ed56f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a4dcd0f_65715ee0","line":21,"updated":"2015-08-09 16:48:51.000000000","message":"remove trailing whitespace","commit_id":"4cb9458430d60d7185c0d7dbfad9f7cfdfd89ded"}],"cinderclient/client.py":[{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"35b4a31d53d0168e15cc1f733577e2a3b9848cb7","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        if version in components:"},{"line_number":73,"context_line":"            return version[1:]"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    msg \u003d \"Invalid url: \u0027%s\u0027. It must include one of: %s\" % ("},{"line_number":76,"context_line":"        (url, \u0027, \u0027.join(_VALID_VERSIONS)))"},{"line_number":77,"context_line":"    raise exceptions.UnsupportedVersion(msg)"},{"line_number":78,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1a4dcd0f_453a22e9","line":75,"updated":"2015-08-09 16:48:51.000000000","message":"Please add a period to the end of msg.","commit_id":"4cb9458430d60d7185c0d7dbfad9f7cfdfd89ded"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"551f3103fa85806517e43e319118b29b2006fb44","unresolved":false,"context_lines":[{"line_number":113,"context_line":"        try:"},{"line_number":114,"context_line":"            version \u003d get_volume_api_from_url(self.get_endpoint())"},{"line_number":115,"context_line":"        except exceptions.UnsupportedVersion as e:"},{"line_number":116,"context_line":"            msg \u003d \"Service catalog returned invalid url.\\n%s\" % e.message"},{"line_number":117,"context_line":"            raise exceptions.UnsupportedVersion(msg)"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"        return version"}],"source_content_type":"text/x-python","patch_set":5,"id":"da20952f_b691019d","line":116,"updated":"2015-09-08 13:40:42.000000000","message":"Exception text should have _() translation and formatting the exception message should use six.text_type() to ensure there is no Unicode conversion error when formatting.","commit_id":"7ef2f147564fd0ad9f0871427a3902ed4b430c24"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"551f3103fa85806517e43e319118b29b2006fb44","unresolved":false,"context_lines":[{"line_number":324,"context_line":"            version \u003d get_volume_api_from_url(self.management_url)"},{"line_number":325,"context_line":"        except exceptions.UnsupportedVersion as e:"},{"line_number":326,"context_line":"            if self.management_url \u003d\u003d self.bypass_url:"},{"line_number":327,"context_line":"                msg \u003d ((\"Invalid url was specified in --bypass-url or \""},{"line_number":328,"context_line":"                        \"environment variable CINDERCLIENT_BYPASS_URL.\\n\""},{"line_number":329,"context_line":"                        \"%s\") % e.message)"},{"line_number":330,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":5,"id":"da20952f_f6f539ed","line":327,"updated":"2015-09-08 13:40:42.000000000","message":"Same.","commit_id":"7ef2f147564fd0ad9f0871427a3902ed4b430c24"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"551f3103fa85806517e43e319118b29b2006fb44","unresolved":false,"context_lines":[{"line_number":328,"context_line":"                        \"environment variable CINDERCLIENT_BYPASS_URL.\\n\""},{"line_number":329,"context_line":"                        \"%s\") % e.message)"},{"line_number":330,"context_line":"            else:"},{"line_number":331,"context_line":"                msg \u003d \"Service catalog returned invalid url.\\n%s\" % e.message"},{"line_number":332,"context_line":""},{"line_number":333,"context_line":"            raise exceptions.UnsupportedVersion(msg)"},{"line_number":334,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"da20952f_96db9d7d","line":331,"updated":"2015-09-08 13:40:42.000000000","message":"Same.","commit_id":"7ef2f147564fd0ad9f0871427a3902ed4b430c24"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"856fc8f053546ab24f3a380f7fa3d93011b809d6","unresolved":false,"context_lines":[{"line_number":73,"context_line":"        if version in components:"},{"line_number":74,"context_line":"            return version[1:]"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    msg \u003d \"Invalid url: \u0027%s\u0027. It must include one of: %s.\" % ("},{"line_number":77,"context_line":"        (url, \u0027, \u0027.join(_VALID_VERSIONS)))"},{"line_number":78,"context_line":"    raise exceptions.UnsupportedVersion(msg)"},{"line_number":79,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"da20952f_9c1e530c","line":76,"updated":"2015-09-08 17:08:47.000000000","message":"Sorry, I didn\u0027t point out here is another case where _() needs to be around the text.\n\nGranted, this is how it was prior to your change, but since you are updating it, it would be good to correct the translation marker.","commit_id":"8a45751b0b36896fa2606f302bd67e21a3d9af26"}]}