)]}'
{"tools/abandon_old_reviews.sh":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"8d4595b3be0959d010b89ce8fd648446afdf69fe","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    abandon_review $review $blocked_msg"},{"line_number":63,"context_line":"done"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"# then purge all the reviews that are \u003e 4w with no changes and Jenkins has -1ed"},{"line_number":66,"context_line":"failing_reviews\u003d$(ssh review.openstack.org \"gerrit query  --current-patch-set --format json $PROJECTS status:open age:4w NOT label:Verified\u003e\u003d1,jenkins\" | jq .currentPatchSet.revision | grep -v null | sed \u0027s/\"//g\u0027)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"failing_msg\u003d$(cat \u003c\u003cEOF"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9a41bdd9_a602f03c","line":65,"updated":"2015-07-17 16:45:12.000000000","message":"I suspect we want to skip WIP reviews for this.","commit_id":"7811570e5cc9a160cc9824ec391412ae25b7ef70"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"31443ca67f3ecc76a2ac9571aae3c2590fbab064","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    abandon_review $review $blocked_msg"},{"line_number":63,"context_line":"done"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"# then purge all the reviews that are \u003e 4w with no changes and Jenkins has -1ed"},{"line_number":66,"context_line":"failing_reviews\u003d$(ssh review.openstack.org \"gerrit query  --current-patch-set --format json $PROJECTS status:open age:4w NOT label:Verified\u003e\u003d1,jenkins\" | jq .currentPatchSet.revision | grep -v null | sed \u0027s/\"//g\u0027)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"failing_msg\u003d$(cat \u003c\u003cEOF"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3a50d1a3_8087af60","line":65,"in_reply_to":"3a50d1a3_55096a2b","updated":"2015-07-20 14:57:50.000000000","message":"OK, I figured out I can add \"-label:Workflow\u003d-1\" to the gerrit query to exclude anything explicitly marked as such.\n\nI think that does end up excluding a lot of patches that look like they really have been abandoned by this point. Looking for \"-message:WIP\" doesn\u0027t catch everything either.\n\nSince the maintainer can just click the restore button, and this points to that in the comment that gets posted, wouldn\u0027t that be enough?\n\nAs far as I know, no comments or history would be lost be doing so. And that would make it explicit from the patch owner that they have not just forgotten about the patch and are still working on it.\n\nBTW, the Fuel project has been having a good discussion around some of this lately here: http://lists.openstack.org/pipermail/openstack-dev/2015-July/068840.html","commit_id":"7811570e5cc9a160cc9824ec391412ae25b7ef70"},{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"44d5e0ce045f05bc373d87deed78f979f3e8be8e","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    abandon_review $review $blocked_msg"},{"line_number":63,"context_line":"done"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"# then purge all the reviews that are \u003e 4w with no changes and Jenkins has -1ed"},{"line_number":66,"context_line":"failing_reviews\u003d$(ssh review.openstack.org \"gerrit query  --current-patch-set --format json $PROJECTS status:open age:4w NOT label:Verified\u003e\u003d1,jenkins\" | jq .currentPatchSet.revision | grep -v null | sed \u0027s/\"//g\u0027)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"failing_msg\u003d$(cat \u003c\u003cEOF"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3a50d1a3_f46867bc","line":65,"in_reply_to":"3a50d1a3_8087af60","updated":"2015-07-27 16:15:28.000000000","message":"I agree that I want WIP to not be touched. There was also discussion on this in Nova:\n\nhttp://lists.openstack.org/pipermail/openstack-dev/2014-November/050888.html","commit_id":"7811570e5cc9a160cc9824ec391412ae25b7ef70"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"b723f61b6fe4af25f15f14362e0ee3bc2b7aa5cf","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    abandon_review $review $blocked_msg"},{"line_number":63,"context_line":"done"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"# then purge all the reviews that are \u003e 4w with no changes and Jenkins has -1ed"},{"line_number":66,"context_line":"failing_reviews\u003d$(ssh review.openstack.org \"gerrit query  --current-patch-set --format json $PROJECTS status:open age:4w NOT label:Verified\u003e\u003d1,jenkins\" | jq .currentPatchSet.revision | grep -v null | sed \u0027s/\"//g\u0027)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"failing_msg\u003d$(cat \u003c\u003cEOF"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3a50d1a3_54291b36","line":65,"in_reply_to":"3a50d1a3_f46867bc","updated":"2015-07-27 16:21:14.000000000","message":"Done","commit_id":"7811570e5cc9a160cc9824ec391412ae25b7ef70"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"96b468b4e35e681d09e148c11426f2d737cd4399","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    abandon_review $review $blocked_msg"},{"line_number":63,"context_line":"done"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"# then purge all the reviews that are \u003e 4w with no changes and Jenkins has -1ed"},{"line_number":66,"context_line":"failing_reviews\u003d$(ssh review.openstack.org \"gerrit query  --current-patch-set --format json $PROJECTS status:open age:4w NOT label:Verified\u003e\u003d1,jenkins\" | jq .currentPatchSet.revision | grep -v null | sed \u0027s/\"//g\u0027)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"failing_msg\u003d$(cat \u003c\u003cEOF"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3a50d1a3_55096a2b","line":65,"in_reply_to":"9a41bdd9_318ff824","updated":"2015-07-20 13:58:39.000000000","message":"It depends.  https://review.openstack.org/#/c/147186/  is sitting around waiting for Nova bug fixes.\n\nI don\u0027t really want it abandoned because I want it to be known that the work is out there and coming when it can.  People can still review it or base further patchsets on it.  This is one of the uses of the WIP indicator, I think.  But it\u0027s also kind of an unusual case, so, I don\u0027t know really.","commit_id":"7811570e5cc9a160cc9824ec391412ae25b7ef70"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"22388f70471a7410413c8b0feed7712035548c03","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    abandon_review $review $blocked_msg"},{"line_number":63,"context_line":"done"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"# then purge all the reviews that are \u003e 4w with no changes and Jenkins has -1ed"},{"line_number":66,"context_line":"failing_reviews\u003d$(ssh review.openstack.org \"gerrit query  --current-patch-set --format json $PROJECTS status:open age:4w NOT label:Verified\u003e\u003d1,jenkins\" | jq .currentPatchSet.revision | grep -v null | sed \u0027s/\"//g\u0027)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"failing_msg\u003d$(cat \u003c\u003cEOF"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9a41bdd9_318ff824","line":65,"in_reply_to":"9a41bdd9_a602f03c","updated":"2015-07-17 16:56:36.000000000","message":"If they haven\u0027t been touched for over 4 weeks are they still really \"in progress\"?\n\nI suppose it should be easy enough to look for a Workflow -1 and skip those. They still probably need to get cleared out at some point.","commit_id":"7811570e5cc9a160cc9824ec391412ae25b7ef70"}]}