)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"fce236ff4fd2a18b8959d0f50d90266e8c0c614e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"replace the use of zuul-cloner with a shell script"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Under zuulv3 the behavior of zuul-cloner has changed significantly to"},{"line_number":10,"context_line":"now *require* a local cache. As soon as zuulv3 is released, our jobs"},{"line_number":11,"context_line":"relying on the old fallback behavior of checking out from a cache when"},{"line_number":12,"context_line":"present but using the upstream server in other situations will"},{"line_number":13,"context_line":"fail."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f4e5783_cab36ce8","line":10,"updated":"2017-10-17 17:53:50.000000000","message":"Zuul v3 doesn\u0027t so much \"require\" a local cache as it requires you to specify in advance which repositories you\u0027ll be using if you want to rely on it to set them up in the (perhaps speculative future) state your jobs need, and zuul-cloner has been replaced by a shim to simply consume the repository states Zuul now provides through that mechanism.","commit_id":"c129161e6eeb5b7cee3b642d88b75567a93ac126"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"7b27e2dc272aac0def21cffc4e542443bcd84879","unresolved":false,"context_lines":[{"line_number":7,"context_line":"replace the use of zuul-cloner with a shell script"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Under zuulv3 the behavior of zuul-cloner has changed significantly to"},{"line_number":10,"context_line":"now *require* a local cache. As soon as zuulv3 is released, our jobs"},{"line_number":11,"context_line":"relying on the old fallback behavior of checking out from a cache when"},{"line_number":12,"context_line":"present but using the upstream server in other situations will"},{"line_number":13,"context_line":"fail."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f4e5783_ca15ac4a","line":10,"in_reply_to":"5f4e5783_cab36ce8","updated":"2017-10-17 18:07:14.000000000","message":"Done","commit_id":"c129161e6eeb5b7cee3b642d88b75567a93ac126"}],"tools/clone_repo.sh":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"2f6fd552e92273e7b81adea4bb2c11874af0bfba","unresolved":false,"context_lines":[{"line_number":142,"context_line":"# Clone"},{"line_number":143,"context_line":"if [ -d $local_dir ]; then"},{"line_number":144,"context_line":"    echo \"Already have a local copy of $REPO in $local_dir\""},{"line_number":145,"context_line":"    (cd $local_dir \u0026\u0026 git remote update)"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"elif [ ! -z \"$cache_remote\" ]; then"},{"line_number":148,"context_line":"    (cd $WORKSPACE \u0026\u0026"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5f4e5783_6ac8f872","line":145,"updated":"2017-10-17 17:50:35.000000000","message":"Because \u0027git remote update\u0027 doesn\u0027t always update tags, zuul-cloner used to do:\n\n  git fetch origin --tags\n\n(if git was older than 1.9 it would do:\n  git fetch origin\n  git fetch origin --tags\n)\n\nYou might consider dropping the git remote update here and in the next stanza...","commit_id":"c129161e6eeb5b7cee3b642d88b75567a93ac126"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"7b27e2dc272aac0def21cffc4e542443bcd84879","unresolved":false,"context_lines":[{"line_number":142,"context_line":"# Clone"},{"line_number":143,"context_line":"if [ -d $local_dir ]; then"},{"line_number":144,"context_line":"    echo \"Already have a local copy of $REPO in $local_dir\""},{"line_number":145,"context_line":"    (cd $local_dir \u0026\u0026 git remote update)"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"elif [ ! -z \"$cache_remote\" ]; then"},{"line_number":148,"context_line":"    (cd $WORKSPACE \u0026\u0026"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5f4e5783_6a2d98a6","line":145,"in_reply_to":"5f4e5783_6ac8f872","updated":"2017-10-17 18:07:14.000000000","message":"Done","commit_id":"c129161e6eeb5b7cee3b642d88b75567a93ac126"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"2f6fd552e92273e7b81adea4bb2c11874af0bfba","unresolved":false,"context_lines":[{"line_number":154,"context_line":""},{"line_number":155,"context_line":"else"},{"line_number":156,"context_line":"    (cd $WORKSPACE \u0026\u0026 git clone $upstream_remote $REPO)"},{"line_number":157,"context_line":"fi"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"if [ ! -z \"$REF\" ]; then"},{"line_number":160,"context_line":"    # Check out the specified reference."}],"source_content_type":"text/x-sh","patch_set":1,"id":"5f4e5783_4ac3fc57","line":157,"updated":"2017-10-17 17:50:35.000000000","message":"... and putting here:\n\n  (cd $local_dir \u0026\u0026 git fetch origin --tags)\n\nI\u0027m guessing you don\u0027t need to worry about git \u003c 1.9 and can omit the bare \u0027git fetch origin\u0027.","commit_id":"c129161e6eeb5b7cee3b642d88b75567a93ac126"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"7b27e2dc272aac0def21cffc4e542443bcd84879","unresolved":false,"context_lines":[{"line_number":154,"context_line":""},{"line_number":155,"context_line":"else"},{"line_number":156,"context_line":"    (cd $WORKSPACE \u0026\u0026 git clone $upstream_remote $REPO)"},{"line_number":157,"context_line":"fi"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"if [ ! -z \"$REF\" ]; then"},{"line_number":160,"context_line":"    # Check out the specified reference."}],"source_content_type":"text/x-sh","patch_set":1,"id":"5f4e5783_2a1d6073","line":157,"in_reply_to":"5f4e5783_4ac3fc57","updated":"2017-10-17 18:07:14.000000000","message":"Done","commit_id":"c129161e6eeb5b7cee3b642d88b75567a93ac126"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"2f6fd552e92273e7b81adea4bb2c11874af0bfba","unresolved":false,"context_lines":[{"line_number":163,"context_line":"    # Check out the expected branch (master is the default, but if the"},{"line_number":164,"context_line":"    # directory already existed we might have checked out something else"},{"line_number":165,"context_line":"    # before so just do it again)."},{"line_number":166,"context_line":"    (cd $local_dir \u0026\u0026 git checkout $BRANCH \u0026\u0026 git pull)"},{"line_number":167,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5f4e5783_2aa440b9","line":166,"updated":"2017-10-17 17:50:35.000000000","message":"Maybe \u0027git pull --ff-only\u0027 ?\nOr you could \u0027git reset --hard origin/$BRANCH\u0027.","commit_id":"c129161e6eeb5b7cee3b642d88b75567a93ac126"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"7b27e2dc272aac0def21cffc4e542443bcd84879","unresolved":false,"context_lines":[{"line_number":163,"context_line":"    # Check out the expected branch (master is the default, but if the"},{"line_number":164,"context_line":"    # directory already existed we might have checked out something else"},{"line_number":165,"context_line":"    # before so just do it again)."},{"line_number":166,"context_line":"    (cd $local_dir \u0026\u0026 git checkout $BRANCH \u0026\u0026 git pull)"},{"line_number":167,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5f4e5783_0a2064ab","line":166,"in_reply_to":"5f4e5783_2aa440b9","updated":"2017-10-17 18:07:14.000000000","message":"Done","commit_id":"c129161e6eeb5b7cee3b642d88b75567a93ac126"}],"tools/list_unreleased_changes.sh":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"33d3983077e7b60fa8c3c2abdca599e9064eb77d","unresolved":false,"context_lines":[{"line_number":28,"context_line":"BASEDIR\u003d$(dirname $TOOLSDIR)"},{"line_number":29,"context_line":"source $TOOLSDIR/functions"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"if [[ -z \"$VIRTUAL_ENV\" ]]; then"},{"line_number":32,"context_line":"    if [[ ! -d $BASEDIR/.tox/venv ]]; then"},{"line_number":33,"context_line":"        (cd $BASEDIR \u0026\u0026 tox -e venv --notest)"},{"line_number":34,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5f4e5783_0aef441e","side":"PARENT","line":31,"updated":"2017-10-17 17:41:31.000000000","message":"Was this intended to be removed as part of this patch?","commit_id":"2bf39ddf034397018c262c07e9d845d0235e2ee6"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"7b27e2dc272aac0def21cffc4e542443bcd84879","unresolved":false,"context_lines":[{"line_number":28,"context_line":"BASEDIR\u003d$(dirname $TOOLSDIR)"},{"line_number":29,"context_line":"source $TOOLSDIR/functions"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"if [[ -z \"$VIRTUAL_ENV\" ]]; then"},{"line_number":32,"context_line":"    if [[ ! -d $BASEDIR/.tox/venv ]]; then"},{"line_number":33,"context_line":"        (cd $BASEDIR \u0026\u0026 tox -e venv --notest)"},{"line_number":34,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":1,"id":"5f4e5783_8ace9457","side":"PARENT","line":31,"in_reply_to":"5f4e5783_0aef441e","updated":"2017-10-17 18:07:14.000000000","message":"Yes, it was only needed for clone_repo to be able to use zuul-cloner.","commit_id":"2bf39ddf034397018c262c07e9d845d0235e2ee6"}]}