)]}'
{"tools/abandon_old_reviews.sh":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"ebdf455d22f9ace8fb6aa9c858aa87a1af5cc87f","unresolved":false,"context_lines":[{"line_number":19,"context_line":"#"},{"line_number":20,"context_line":"#   Host review.openstack.org"},{"line_number":21,"context_line":"#   User \u003cyourgerritusername\u003e"},{"line_number":22,"context_line":"#   Port 29418"},{"line_number":23,"context_line":"#"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"# Note: due to gerrit bug somewhere, this double posts messages. :("}],"source_content_type":"text/x-sh","patch_set":1,"id":"3a961159_d9b61d3e","line":22,"updated":"2015-01-13 11:31:16.000000000","message":"ssh has -p argument, since the hostname is already hard coded, the port should should be handled in the same way.","commit_id":"20a3eed7bd197565cb82cfca9eb3f895f5284d1b"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"7238ac4b547ceea002ab8ddd68bd8db14a3e5176","unresolved":false,"context_lines":[{"line_number":19,"context_line":"#"},{"line_number":20,"context_line":"#   Host review.openstack.org"},{"line_number":21,"context_line":"#   User \u003cyourgerritusername\u003e"},{"line_number":22,"context_line":"#   Port 29418"},{"line_number":23,"context_line":"#"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"# Note: due to gerrit bug somewhere, this double posts messages. :("}],"source_content_type":"text/x-sh","patch_set":1,"id":"1a930d6b_9cdb2c7b","line":22,"in_reply_to":"3a961159_d9b61d3e","updated":"2015-01-27 01:49:49.000000000","message":"I think it\u0027s fine to put it there.","commit_id":"20a3eed7bd197565cb82cfca9eb3f895f5284d1b"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"ebdf455d22f9ace8fb6aa9c858aa87a1af5cc87f","unresolved":false,"context_lines":[{"line_number":31,"context_line":"function abandon_review {"},{"line_number":32,"context_line":"    local gitid\u003d$1"},{"line_number":33,"context_line":"    shift"},{"line_number":34,"context_line":"    local msg\u003d$@"},{"line_number":35,"context_line":"    echo \"Abandoning $gitid\""},{"line_number":36,"context_line":"    echo ssh review.openstack.org gerrit review $gitid --abandon --message \\\"$msg\\\""},{"line_number":37,"context_line":"    if [[ \"$DRYRUN\" -ne 1 ]]; then"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3a961159_591ead60","line":34,"updated":"2015-01-13 11:31:16.000000000","message":"Use $* or \"$@\" if you need to preserve the white spaces.\n\nUsing $2 and quoiting at the caller side is better.","commit_id":"20a3eed7bd197565cb82cfca9eb3f895f5284d1b"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"ebdf455d22f9ace8fb6aa9c858aa87a1af5cc87f","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    echo \"Abandoning $gitid\""},{"line_number":36,"context_line":"    echo ssh review.openstack.org gerrit review $gitid --abandon --message \\\"$msg\\\""},{"line_number":37,"context_line":"    if [[ \"$DRYRUN\" -ne 1 ]]; then"},{"line_number":38,"context_line":"        ssh review.openstack.org gerrit review $gitid --abandon --message \\\"$msg\\\""},{"line_number":39,"context_line":"    fi"},{"line_number":40,"context_line":"}"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"3a961159_796fe9c4","line":38,"updated":"2015-01-13 11:31:16.000000000","message":"The $msg is not quoted on the calling side: \\\"\"$msg\"\\\".\n\nThe \u0027\"\u0027  needs to be escaped from the message.","commit_id":"20a3eed7bd197565cb82cfca9eb3f895f5284d1b"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"ebdf455d22f9ace8fb6aa9c858aa87a1af5cc87f","unresolved":false,"context_lines":[{"line_number":42,"context_line":"DRYRUN\u003d1"},{"line_number":43,"context_line":"PROJECT\u003d\"openstack/tempest\""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"blocked_reviews\u003d$(ssh review.openstack.org \"gerrit query --current-patch-set --format json project:$PROJECT status:open age:4w label:Code-Review\u003c\u003d-2\" | jq .currentPatchSet.revision | grep -v null | sed \u0027s/\"//g\u0027)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"blocked_msg\u003d$(cat \u003c\u003cEOF"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"3a961159_d9e45d3e","line":45,"range":{"start_line":45,"start_character":18,"end_line":45,"end_character":42},"updated":"2015-01-13 11:31:16.000000000","message":"Instead of repeating the ssh -p 29418 review.openstack.org create helper function \u0027gerrit_cmd\u0027 which applies the usual ssh parameters (maybe even more what we have now).\nUse \"$@\" for the extra args. (ssh $SSH_OPTS \"$@\")","commit_id":"20a3eed7bd197565cb82cfca9eb3f895f5284d1b"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"ebdf455d22f9ace8fb6aa9c858aa87a1af5cc87f","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":"for review in $failing_reviews; do"},{"line_number":84,"context_line":"    echo \"Failing review $review\""},{"line_number":85,"context_line":"    abandon_review $review $failing_msg"},{"line_number":86,"context_line":"done"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3a961159_d9529ddd","line":85,"updated":"2015-01-13 11:31:16.000000000","message":"The string argument needs to be quoted in order to preserve white spaces.","commit_id":"20a3eed7bd197565cb82cfca9eb3f895f5284d1b"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"7238ac4b547ceea002ab8ddd68bd8db14a3e5176","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":"for review in $failing_reviews; do"},{"line_number":84,"context_line":"    echo \"Failing review $review\""},{"line_number":85,"context_line":"    abandon_review $review $failing_msg"},{"line_number":86,"context_line":"done"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1a930d6b_3c41580b","line":85,"in_reply_to":"3a961159_d9529ddd","updated":"2015-01-27 01:49:49.000000000","message":"It actually doesn\u0027t, it\u0027s been working fine as is.","commit_id":"20a3eed7bd197565cb82cfca9eb3f895f5284d1b"}]}