)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"1f970f3f5ff713bccbac9ce671b3335de1e83d36","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"trigger devstack-gate test as early as possible"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"if we have changes to devstack-gate we need need to run this as"},{"line_number":10,"context_line":"early as possible. So do this the second we\u0027ve decided that"},{"line_number":11,"context_line":"we need to run devstack gate in our project list."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATn%2F%2FcqI%3D","line":9,"updated":"2013-11-06 01:43:05.000000000","message":"Nit: Capitals here and in the summary would be great.","commit_id":"8234e25fa857efafe2cadc72d0a931bb603721b9"}],"devstack-vm-gate-wrap.sh":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"0e7edbc62f00b515b1b5ca00972e67719a098ea8","unresolved":false,"context_lines":[{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    fix_etc_hosts"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    fix_disk_layout"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"    sudo mkdir -p $DEST"},{"line_number":175,"context_line":"    sudo chown -R jenkins:jenkins $DEST"}],"source_content_type":"text/x-sh","patch_set":4,"id":"AAAATn%2F%2FUxY%3D","line":172,"updated":"2013-11-10 21:20:16.000000000","message":"This second fix_disk_layout should be safe due to the swap check in fix_disk_layout. However, that check doesn\u0027t check for all of the changes that fix_disk_layout is making. I think we should make the check more robust, but since this is an existing issue we can do that in a subsequent patch. Thoughts?","commit_id":"8234e25fa857efafe2cadc72d0a931bb603721b9"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"303fbcccad27bfad3273e22a947fe0367521c617","unresolved":false,"context_lines":[{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    fix_etc_hosts"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    fix_disk_layout"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"    sudo mkdir -p $DEST"},{"line_number":175,"context_line":"    sudo chown -R jenkins:jenkins $DEST"}],"source_content_type":"text/x-sh","patch_set":4,"id":"AAAATn%2F%2FUwc%3D","line":172,"in_reply_to":"AAAATn%2F%2FUxY%3D","updated":"2013-11-10 21:42:26.000000000","message":"so in every grenade run we are running fix_disk_layout twice already, so it\u0027s definitely repeatable. I do agree we probably should do more checks on this.","commit_id":"8234e25fa857efafe2cadc72d0a931bb603721b9"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"7876cbd5fbb8015e0ec10208e1a918c8e0379af8","unresolved":false,"context_lines":[{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    mkdir -p $BASE/new"},{"line_number":167,"context_line":"    cd $BASE/new"},{"line_number":168,"context_line":"    setup_project openstack-infra/devstack-gate master"},{"line_number":169,"context_line":"    exec $GATE_SCRIPT_DIR/devstack-vm-gate-wrap.sh"},{"line_number":170,"context_line":"}"},{"line_number":171,"context_line":""}],"source_content_type":"text/x-sh","patch_set":6,"id":"AAAATn%2F%2FMo8%3D","line":168,"updated":"2013-11-13 02:58:24.000000000","message":"Hardcoding master here is fine because d-g doesn\u0027t use any other branches. We should keep in mind that this will need to be updated if we ever move to using branches with d-g.","commit_id":"46d2193b00ca70619b174a5e5dc86f3f44e1980b"}]}