)]}'
{"tools/add-reviewers.py":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"233b080a397387590fb02b94bce65c31d86a7d81","unresolved":false,"context_lines":[{"line_number":30,"context_line":"def add_reviewers(debug, username, liaisons, review):"},{"line_number":31,"context_line":"    gerrit \u003d ["},{"line_number":32,"context_line":"        \u0027ssh\u0027,"},{"line_number":33,"context_line":"        \u0027-p 29418\u0027,"},{"line_number":34,"context_line":"        \u0027{}@review.openstack.org\u0027.format(username),"},{"line_number":35,"context_line":"        \u0027gerrit\u0027,"},{"line_number":36,"context_line":"        \u0027set-reviewers\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_5b93345b","line":33,"updated":"2015-06-24 20:15:48.000000000","message":"technically -p and 29481 should be separated!","commit_id":"32dca4ceca7a2ab2d3826c2f8f96f2b7f9d21c01"},{"author":{"_account_id":1112,"name":"Everett Toews","email":"everett.toews@rackspace.com","username":"everett-toews"},"change_message_id":"057fd9ea15a308d4adbe1f88928f3437d792776d","unresolved":false,"context_lines":[{"line_number":30,"context_line":"def add_reviewers(debug, username, liaisons, review):"},{"line_number":31,"context_line":"    gerrit \u003d ["},{"line_number":32,"context_line":"        \u0027ssh\u0027,"},{"line_number":33,"context_line":"        \u0027-p 29418\u0027,"},{"line_number":34,"context_line":"        \u0027{}@review.openstack.org\u0027.format(username),"},{"line_number":35,"context_line":"        \u0027gerrit\u0027,"},{"line_number":36,"context_line":"        \u0027set-reviewers\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_a22aec2a","line":33,"in_reply_to":"fa32b979_5b93345b","updated":"2015-07-06 17:06:35.000000000","message":"Done","commit_id":"32dca4ceca7a2ab2d3826c2f8f96f2b7f9d21c01"},{"author":{"_account_id":12321,"name":"Ryan Brown","email":"rybrown@redhat.com","username":"ryansb"},"change_message_id":"2fb509ae1722dd644adb6d96da6aa6e551072274","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    gerrit.append(\u0027{}\u0027.format(review))"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    if debug: logger.debug(\u0027 \u0027.join(gerrit))"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    subprocess.call(gerrit)"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_62117653","line":44,"updated":"2015-06-25 18:05:00.000000000","message":"I\u0027d like to register a minor grumble about single-line if statement here.","commit_id":"32dca4ceca7a2ab2d3826c2f8f96f2b7f9d21c01"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"233b080a397387590fb02b94bce65c31d86a7d81","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    gerrit.append(\u0027{}\u0027.format(review))"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    if debug: logger.debug(\u0027 \u0027.join(gerrit))"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    subprocess.call(gerrit)"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_3ba670ae","line":44,"updated":"2015-06-24 20:15:48.000000000","message":"rather than having if debug here, on line 48 set the log level either to DEBUG or INFO or WARN or else according to cmdline args.","commit_id":"32dca4ceca7a2ab2d3826c2f8f96f2b7f9d21c01"},{"author":{"_account_id":1112,"name":"Everett Toews","email":"everett.toews@rackspace.com","username":"everett-toews"},"change_message_id":"057fd9ea15a308d4adbe1f88928f3437d792776d","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    gerrit.append(\u0027{}\u0027.format(review))"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    if debug: logger.debug(\u0027 \u0027.join(gerrit))"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    subprocess.call(gerrit)"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_022f18a2","line":44,"in_reply_to":"fa32b979_3ba670ae","updated":"2015-07-06 17:06:35.000000000","message":"Done","commit_id":"32dca4ceca7a2ab2d3826c2f8f96f2b7f9d21c01"},{"author":{"_account_id":1112,"name":"Everett Toews","email":"everett.toews@rackspace.com","username":"everett-toews"},"change_message_id":"057fd9ea15a308d4adbe1f88928f3437d792776d","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    gerrit.append(\u0027{}\u0027.format(review))"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    if debug: logger.debug(\u0027 \u0027.join(gerrit))"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    subprocess.call(gerrit)"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_e21c7479","line":44,"in_reply_to":"fa32b979_62117653","updated":"2015-07-06 17:06:35.000000000","message":"Grumble registered.","commit_id":"32dca4ceca7a2ab2d3826c2f8f96f2b7f9d21c01"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"233b080a397387590fb02b94bce65c31d86a7d81","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    debug, username, review \u003d parse_args()"},{"line_number":54,"context_line":"    liaisons \u003d get_liaisons()"},{"line_number":55,"context_line":"    add_reviewers(debug, username, liaisons, review)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    print(\"Added {} reviewers to {}\".format(len(liaisons), review))"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_fbc24833","line":55,"updated":"2015-06-24 20:15:48.000000000","message":"I wonder whether it would be easier to create a API-WG-CPL gerrit group and automatically add it to every patch","commit_id":"32dca4ceca7a2ab2d3826c2f8f96f2b7f9d21c01"},{"author":{"_account_id":1112,"name":"Everett Toews","email":"everett.toews@rackspace.com","username":"everett-toews"},"change_message_id":"057fd9ea15a308d4adbe1f88928f3437d792776d","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    debug, username, review \u003d parse_args()"},{"line_number":54,"context_line":"    liaisons \u003d get_liaisons()"},{"line_number":55,"context_line":"    add_reviewers(debug, username, liaisons, review)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    print(\"Added {} reviewers to {}\".format(len(liaisons), review))"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba3cc151_fd02d5b3","line":55,"in_reply_to":"fa32b979_fbc24833","updated":"2015-07-06 17:06:35.000000000","message":"We\u0027ve gone back and forth on this in the meetings and on #openstack-api. Maintaining gerrit groups requires gerrit admin. That\u0027s likely too much of a barrier to deal with.\n\nOTOH, if we could find a way to easily manage a gerrit group, it could remove the need for something like this script and simplify things. \"easily manage a gerrit group\" requires more investigation.","commit_id":"32dca4ceca7a2ab2d3826c2f8f96f2b7f9d21c01"}]}