)]}'
{"devstack-vm-gate-wrap.sh":[{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"d1f5af058f3dbc7508e33422b0fff895b6feddf7","unresolved":false,"context_lines":[{"line_number":480,"context_line":"    elif [ \"$ZUUL_BRANCH\" \u003d\u003d \"stable/havana\" ]; then"},{"line_number":481,"context_line":"        export GRENADE_OLD_BRANCH\u003d\"stable/havana\""},{"line_number":482,"context_line":"        export GRENADE_NEW_BRANCH\u003d\"master\""},{"line_number":483,"context_line":"    elif [ \"$ZUUL_BRANCH\" \u003d\u003d \"stable/icehouse\" ]; then"},{"line_number":484,"context_line":"        export GRENADE_OLD_BRANCH\u003d\"stable/icehouse\""},{"line_number":485,"context_line":"        export GRENADE_NEW_BRANCH\u003d\"master\""},{"line_number":486,"context_line":"    else # master"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAATn%2F%2F39c%3D","line":483,"updated":"2013-10-25 20:25:21.000000000","message":"I think you should not have stable/icehouse -\u003e master in here while you also have stable/havana -\u003e master.","commit_id":"6e85b9ec54de61956f3b4ce5c9b4120e0926944f"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"9cae3f364ca21eaa26d2cd8c435852fcfb1cdf0e","unresolved":false,"context_lines":[{"line_number":480,"context_line":"    elif [ \"$ZUUL_BRANCH\" \u003d\u003d \"stable/havana\" ]; then"},{"line_number":481,"context_line":"        export GRENADE_OLD_BRANCH\u003d\"stable/havana\""},{"line_number":482,"context_line":"        export GRENADE_NEW_BRANCH\u003d\"master\""},{"line_number":483,"context_line":"    elif [ \"$ZUUL_BRANCH\" \u003d\u003d \"stable/icehouse\" ]; then"},{"line_number":484,"context_line":"        export GRENADE_OLD_BRANCH\u003d\"stable/icehouse\""},{"line_number":485,"context_line":"        export GRENADE_NEW_BRANCH\u003d\"master\""},{"line_number":486,"context_line":"    else # master"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAATn%2F%2F38I%3D","line":483,"in_reply_to":"AAAATn%2F%2F39c%3D","updated":"2013-10-25 20:28:53.000000000","message":"sure, I was doing it for symmetry, easy to remove","commit_id":"6e85b9ec54de61956f3b4ce5c9b4120e0926944f"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"d1f5af058f3dbc7508e33422b0fff895b6feddf7","unresolved":false,"context_lines":[{"line_number":483,"context_line":"    elif [ \"$ZUUL_BRANCH\" \u003d\u003d \"stable/icehouse\" ]; then"},{"line_number":484,"context_line":"        export GRENADE_OLD_BRANCH\u003d\"stable/icehouse\""},{"line_number":485,"context_line":"        export GRENADE_NEW_BRANCH\u003d\"master\""},{"line_number":486,"context_line":"    else # master"},{"line_number":487,"context_line":"        export GRENADE_OLD_BRANCH\u003d\"stable/havana\""},{"line_number":488,"context_line":"        export GRENADE_NEW_BRANCH\u003d\"master\""},{"line_number":489,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAATn%2F%2F39Q%3D","line":486,"updated":"2013-10-25 20:25:21.000000000","message":"master shouldn\u0027t have a grenade-forward job. It\u0027s duplication.","commit_id":"6e85b9ec54de61956f3b4ce5c9b4120e0926944f"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"9cae3f364ca21eaa26d2cd8c435852fcfb1cdf0e","unresolved":false,"context_lines":[{"line_number":483,"context_line":"    elif [ \"$ZUUL_BRANCH\" \u003d\u003d \"stable/icehouse\" ]; then"},{"line_number":484,"context_line":"        export GRENADE_OLD_BRANCH\u003d\"stable/icehouse\""},{"line_number":485,"context_line":"        export GRENADE_NEW_BRANCH\u003d\"master\""},{"line_number":486,"context_line":"    else # master"},{"line_number":487,"context_line":"        export GRENADE_OLD_BRANCH\u003d\"stable/havana\""},{"line_number":488,"context_line":"        export GRENADE_NEW_BRANCH\u003d\"master\""},{"line_number":489,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"AAAATn%2F%2F374%3D","line":486,"in_reply_to":"AAAATn%2F%2F39Q%3D","updated":"2013-10-25 20:28:53.000000000","message":"sure, though this job should never be set on master, I was going to handle that in the jobs side. But this could be removed.","commit_id":"6e85b9ec54de61956f3b4ce5c9b4120e0926944f"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"c2162d8505b0e108241e99f1970f79228e8c4c6a","unresolved":false,"context_lines":[{"line_number":399,"context_line":"    export RE_EXEC\u003d\"true\""},{"line_number":400,"context_line":"    echo \"This build includes a change to the devstack gate; re-execing this script.\""},{"line_number":401,"context_line":"    exec $GATE_SCRIPT_DIR/devstack-vm-gate-wrap.sh"},{"line_number":402,"context_line":"fi"},{"line_number":403,"context_line":""},{"line_number":404,"context_line":"# Set to 1 to run the Tempest test suite"},{"line_number":405,"context_line":"export DEVSTACK_GATE_TEMPEST\u003d${DEVSTACK_GATE_TEMPEST:-0}"}],"source_content_type":"text/x-sh","patch_set":9,"id":"AAAATn%2F%2Fzwc%3D","side":"PARENT","line":402,"updated":"2013-10-28 18:09:10.000000000","message":"It is best to run the reexec as early as possible. Anything that happens before the reexec is not self tested when changes to d-g are proposed. Can we keep this chunk of code here?","commit_id":"e0df323af67940d6e2951828d3a2289591ab9301"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"f24b2090ebd55e52dc804eb45e10fc07622ccdef","unresolved":false,"context_lines":[{"line_number":399,"context_line":"    export RE_EXEC\u003d\"true\""},{"line_number":400,"context_line":"    echo \"This build includes a change to the devstack gate; re-execing this script.\""},{"line_number":401,"context_line":"    exec $GATE_SCRIPT_DIR/devstack-vm-gate-wrap.sh"},{"line_number":402,"context_line":"fi"},{"line_number":403,"context_line":""},{"line_number":404,"context_line":"# Set to 1 to run the Tempest test suite"},{"line_number":405,"context_line":"export DEVSTACK_GATE_TEMPEST\u003d${DEVSTACK_GATE_TEMPEST:-0}"}],"source_content_type":"text/x-sh","patch_set":9,"id":"AAAATn%2F%2Fzu0%3D","side":"PARENT","line":402,"in_reply_to":"AAAATn%2F%2Fzwc%3D","updated":"2013-10-28 18:13:07.000000000","message":"yes, this was a bad code merge actually, I hadn\u0027t intended to move it, but it got caught up in the refactor.","commit_id":"e0df323af67940d6e2951828d3a2289591ab9301"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"c2162d8505b0e108241e99f1970f79228e8c4c6a","unresolved":false,"context_lines":[{"line_number":57,"context_line":"      rsync -a ~/workspace-cache/ $DEST/"},{"line_number":58,"context_line":"    fi"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    echo \"Using branch: $BRANCH\""},{"line_number":61,"context_line":"    for PROJECT in $PROJECTS"},{"line_number":62,"context_line":"    do"},{"line_number":63,"context_line":"      BRANCH\u003d$branch"}],"source_content_type":"text/x-sh","patch_set":9,"id":"AAAATn%2F%2Fzw8%3D","line":60,"updated":"2013-10-28 18:09:10.000000000","message":"BRANCH is set on line 63 below. Should this be $branch?","commit_id":"4db8ac8d280b8d8bf4d5b2226cc2046ed1b93a9a"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"f24b2090ebd55e52dc804eb45e10fc07622ccdef","unresolved":false,"context_lines":[{"line_number":57,"context_line":"      rsync -a ~/workspace-cache/ $DEST/"},{"line_number":58,"context_line":"    fi"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    echo \"Using branch: $BRANCH\""},{"line_number":61,"context_line":"    for PROJECT in $PROJECTS"},{"line_number":62,"context_line":"    do"},{"line_number":63,"context_line":"      BRANCH\u003d$branch"}],"source_content_type":"text/x-sh","patch_set":9,"id":"AAAATn%2F%2FzvY%3D","line":60,"in_reply_to":"AAAATn%2F%2Fzw8%3D","updated":"2013-10-28 18:13:07.000000000","message":"yes, you are correct.","commit_id":"4db8ac8d280b8d8bf4d5b2226cc2046ed1b93a9a"}]}
