)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"84a168ac56c217d6b3eef315dc1d61b1e190df31","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Use -c when retrieving resource provider uuid"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Using head and cut within the $() can lead to situations where messages"},{"line_number":12,"context_line":"other than what we expect are cut before we\u0027re able to see their full"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Id0635f512011c96e0683a26298c7382c711ac3a3"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"3f79a3b5_24131eb3","line":12,"range":{"start_line":12,"start_character":34,"end_line":12,"end_character":69},"updated":"2018-11-27 21:07:04.000000000","message":"...go on...","commit_id":"bf7c33612be665024b88ad5af82656f9cb61a43b"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"b24ce358c606e9d4a0ea6080d234b5afdd320eb1","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Use -c when retrieving resource provider uuid"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Using head and cut within the $() can lead to situations where messages"},{"line_number":12,"context_line":"other than what we expect are cut before we\u0027re able to see their full"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Id0635f512011c96e0683a26298c7382c711ac3a3"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"3f79a3b5_c4b82a79","line":12,"range":{"start_line":12,"start_character":34,"end_line":12,"end_character":69},"in_reply_to":"3f79a3b5_24131eb3","updated":"2018-11-27 21:12:43.000000000","message":"ooops. yeah. full output. You can see it in some of the logs I referenced on the swift fixes","commit_id":"bf7c33612be665024b88ad5af82656f9cb61a43b"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"84a168ac56c217d6b3eef315dc1d61b1e190df31","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Using head and cut within the $() can lead to situations where messages"},{"line_number":12,"context_line":"other than what we expect are cut before we\u0027re able to see their full"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Id0635f512011c96e0683a26298c7382c711ac3a3"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"3f79a3b5_c4c10aff","line":13,"updated":"2018-11-27 21:07:04.000000000","message":"So what was the issue here - something was failing and you couldn\u0027t tell what b/c this was swallowing the output?","commit_id":"bf7c33612be665024b88ad5af82656f9cb61a43b"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"b24ce358c606e9d4a0ea6080d234b5afdd320eb1","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Using head and cut within the $() can lead to situations where messages"},{"line_number":12,"context_line":"other than what we expect are cut before we\u0027re able to see their full"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Id0635f512011c96e0683a26298c7382c711ac3a3"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"3f79a3b5_64a7d652","line":13,"in_reply_to":"3f79a3b5_c4c10aff","updated":"2018-11-27 21:12:43.000000000","message":"Yeah, when I do a cleanup on this, I\u0027ll make the commit message more verbose.","commit_id":"bf7c33612be665024b88ad5af82656f9cb61a43b"}],"projects/60_nova/resources.sh":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"84a168ac56c217d6b3eef315dc1d61b1e190df31","unresolved":false,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    key\u003d\"$1\""},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    # Get the uuid of the first resource provider, cut separately"},{"line_number":57,"context_line":"    # to avoid swallowing unexpected output."},{"line_number":58,"context_line":"    provider\u003d$(openstack resource provider list -f value -c uuid)"},{"line_number":59,"context_line":"    provider\u003d$(echo $provider | cut -f1 -d \u0027 \u0027)"}],"source_content_type":"text/x-sh","patch_set":11,"id":"3f79a3b5_046dc22e","line":56,"range":{"start_line":56,"start_character":26,"end_line":56,"end_character":31},"updated":"2018-11-27 21:07:04.000000000","message":"The reason it was done this way was to limit the results to the first (in case this is a multi-node grenade job). What happens if we have 2 resource providers and we\u0027re just getting the UUID? It might work, but are we getting lucky with the returned sort order of the results? I see the run here is happy:\n\nhttp://logs.openstack.org/52/619552/11/check/neutron-grenade-multinode/f8f0170/logs/grenade.sh.txt.gz\n\nBut that\u0027s also py27 rather than py3 and I\u0027m not sure if the sort order will change in a py3 variant of the multi-node job.","commit_id":"bf7c33612be665024b88ad5af82656f9cb61a43b"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"b24ce358c606e9d4a0ea6080d234b5afdd320eb1","unresolved":false,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    key\u003d\"$1\""},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    # Get the uuid of the first resource provider, cut separately"},{"line_number":57,"context_line":"    # to avoid swallowing unexpected output."},{"line_number":58,"context_line":"    provider\u003d$(openstack resource provider list -f value -c uuid)"},{"line_number":59,"context_line":"    provider\u003d$(echo $provider | cut -f1 -d \u0027 \u0027)"}],"source_content_type":"text/x-sh","patch_set":11,"id":"3f79a3b5_a4664e26","line":56,"range":{"start_line":56,"start_character":26,"end_line":56,"end_character":31},"in_reply_to":"3f79a3b5_046dc22e","updated":"2018-11-27 21:12:43.000000000","message":"Right, but that same ordering problem is present either way you get the data, you\u0027re still listing two uuids, and the one you get first is not really in our control. In the old way we  are splitting on lines, in the new way we are splitting on words.","commit_id":"bf7c33612be665024b88ad5af82656f9cb61a43b"}]}
