)]}'
{"tools/owners.py":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"243ee1f910eb508d6a11ef1dda93ba17bbffaf44","unresolved":false,"context_lines":[{"line_number":186,"context_line":"    return decoded"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"def query_gerrit(method, params\u003d{}):"},{"line_number":190,"context_line":"    \"\"\"Query the Gerrit REST API\"\"\""},{"line_number":191,"context_line":""},{"line_number":192,"context_line":"    # The base URL to Gerrit, for accessing both GitWeb and the REST"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f4e5783_e3fd816e","side":"PARENT","line":189,"updated":"2017-10-13 22:21:14.000000000","message":"We can drop and hard-code the method here, though that does mean tweaking calls to this function elsewhere in the script so can certainly be left as follow-on cleanup.","commit_id":"22eb7cc3ac834ca28a4f9347de1b237f0f53291f"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"0e0ea88c54d36c98ec8b5dfa72ff7c59603f3b1d","unresolved":false,"context_lines":[{"line_number":186,"context_line":"    return decoded"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"def query_gerrit(method, params\u003d{}):"},{"line_number":190,"context_line":"    \"\"\"Query the Gerrit REST API\"\"\""},{"line_number":191,"context_line":""},{"line_number":192,"context_line":"    # The base URL to Gerrit, for accessing both GitWeb and the REST"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f4e5783_a911a754","side":"PARENT","line":189,"in_reply_to":"5f4e5783_e3fd816e","updated":"2017-10-14 01:27:07.000000000","message":"Yeah I was going for the minimal change possible.","commit_id":"22eb7cc3ac834ca28a4f9347de1b237f0f53291f"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"52a54305c1fa589832ec84d34a84e3df2f25d77d","unresolved":false,"context_lines":[{"line_number":202,"context_line":""},{"line_number":203,"context_line":"    url \u003d \u0027http://git.openstack.org/cgit/\u0027 + project + \u0027plain\u0027 + obj"},{"line_number":204,"context_line":"    raw \u003d requester(url, params\u003dparams,"},{"line_number":205,"context_line":"                    headers\u003d{\u0027Accept\u0027: \u0027application/json\u0027})"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    raw.encoding \u003d \u0027utf-8\u0027  # Workaround for cgit encoding"},{"line_number":208,"context_line":"    return yaml.safe_load(raw.text)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f4e5783_b7cd8267","line":205,"updated":"2017-10-14 07:51:00.000000000","message":"And I\u0027ll drop this \u0027cause you know we\u0027re not going to get json when requesting a plain yaml file ;P","commit_id":"714c726a3211dc6e39e433bec26700bd98f4d910"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"243ee1f910eb508d6a11ef1dda93ba17bbffaf44","unresolved":false,"context_lines":[{"line_number":204,"context_line":"    raw \u003d requester(url, params\u003dparams,"},{"line_number":205,"context_line":"                    headers\u003d{\u0027Accept\u0027: \u0027application/json\u0027})"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    raw.encoding \u003d \u0027utf-8\u0027  # Workaround for cgit encoding"},{"line_number":208,"context_line":"    return yaml.safe_load(raw.text)"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5f4e5783_c302856f","line":207,"updated":"2017-10-13 22:21:14.000000000","message":"May not be needed? The script would crash with obvious decoding exceptions if we didn\u0027t do this for Gitweb. It\u0027s worth checking whether cgit provides a content type requests can grok.","commit_id":"714c726a3211dc6e39e433bec26700bd98f4d910"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"52a54305c1fa589832ec84d34a84e3df2f25d77d","unresolved":false,"context_lines":[{"line_number":204,"context_line":"    raw \u003d requester(url, params\u003dparams,"},{"line_number":205,"context_line":"                    headers\u003d{\u0027Accept\u0027: \u0027application/json\u0027})"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    raw.encoding \u003d \u0027utf-8\u0027  # Workaround for cgit encoding"},{"line_number":208,"context_line":"    return yaml.safe_load(raw.text)"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5f4e5783_37bab2c0","line":207,"in_reply_to":"5f4e5783_891a6b6c","updated":"2017-10-14 07:51:00.000000000","message":"Yeah cgit send back:\n\n  Content-Type: text/plain; charset\u003dUTF-8\n\nSo I\u0027ll remove this.","commit_id":"714c726a3211dc6e39e433bec26700bd98f4d910"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"0e0ea88c54d36c98ec8b5dfa72ff7c59603f3b1d","unresolved":false,"context_lines":[{"line_number":204,"context_line":"    raw \u003d requester(url, params\u003dparams,"},{"line_number":205,"context_line":"                    headers\u003d{\u0027Accept\u0027: \u0027application/json\u0027})"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    raw.encoding \u003d \u0027utf-8\u0027  # Workaround for cgit encoding"},{"line_number":208,"context_line":"    return yaml.safe_load(raw.text)"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5f4e5783_891a6b6c","line":207,"in_reply_to":"5f4e5783_c302856f","updated":"2017-10-14 01:27:07.000000000","message":"Agreed","commit_id":"714c726a3211dc6e39e433bec26700bd98f4d910"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"711dba0a3bef56733751f8332e2d721eaf6f7202","unresolved":false,"context_lines":[{"line_number":204,"context_line":"    raw \u003d requester(url, params\u003dparams,"},{"line_number":205,"context_line":"                    headers\u003d{\u0027Accept\u0027: \u0027application/json\u0027})"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    raw.encoding \u003d \u0027utf-8\u0027  # Workaround for cgit encoding"},{"line_number":208,"context_line":"    return yaml.safe_load(raw.text)"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3f4b6375_ba0abc1c","line":207,"updated":"2017-10-26 13:47:52.000000000","message":"Benign, but you mentioned in a comment on patchset #1 that you were going to remove this line. Can always be left to a following cleanup patch though.","commit_id":"4d2ff114c60d6f236810841761273b8bb159ba78"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"6dcc4d88c86e82799d5538446e0ab2141a464461","unresolved":false,"context_lines":[{"line_number":204,"context_line":"    raw \u003d requester(url, params\u003dparams,"},{"line_number":205,"context_line":"                    headers\u003d{\u0027Accept\u0027: \u0027application/json\u0027})"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    raw.encoding \u003d \u0027utf-8\u0027  # Workaround for cgit encoding"},{"line_number":208,"context_line":"    return yaml.safe_load(raw.text)"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3f4b6375_3b0b642a","line":207,"in_reply_to":"3f4b6375_ba0abc1c","updated":"2017-10-26 23:19:02.000000000","message":"D\u0027oh!  I\u0027ll create the follow-up patch.","commit_id":"4d2ff114c60d6f236810841761273b8bb159ba78"}]}