)]}'
{"tools/gate_run.sh":[{"author":{"_account_id":24043,"name":"Rich Wellum","username":"rwellum","inactive":true},"change_message_id":"d574c15020a7e060e77dff5b191f81fbbdc5256b","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        REF_NAME\u003d$(echo $ZUUL_REFNAME | cut -d/ -f2)"},{"line_number":85,"context_line":"    fi"},{"line_number":86,"context_line":"    sudo mkdir \"images\""},{"line_number":87,"context_line":"    BRANCH\u003d$(echo \"$ZUUL_BRANCH\" | cut -d/ -f2)"},{"line_number":88,"context_line":"    FILENAME\u003d${BASE_DISTRO}-${INSTALL_TYPE}-registry-${BRANCH}.tar.gz"},{"line_number":89,"context_line":"    sudo docker stop registry"},{"line_number":90,"context_line":"    sudo tar -zcf \"images/$FILENAME\" -C /tmp/kolla_registry ."}],"source_content_type":"text/x-sh","patch_set":13,"id":"5f201791_9408f091","line":87,"updated":"2017-06-28 16:49:16.000000000","message":"Please add comment similar to the two above (# ZUUL_BRANCH\u003d...)","commit_id":"c2aacfaf9d32e4f2d042979d883d8e66a605335d"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"4abc96014ed4a3e4b1ad279372c6537a030c1fe7","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        REF_NAME\u003d$(echo $ZUUL_REFNAME | cut -d/ -f2)"},{"line_number":85,"context_line":"    fi"},{"line_number":86,"context_line":"    sudo mkdir \"images\""},{"line_number":87,"context_line":"    BRANCH\u003d$(echo \"$ZUUL_BRANCH\" | cut -d/ -f2)"},{"line_number":88,"context_line":"    FILENAME\u003d${BASE_DISTRO}-${INSTALL_TYPE}-registry-${BRANCH}.tar.gz"},{"line_number":89,"context_line":"    sudo docker stop registry"},{"line_number":90,"context_line":"    sudo tar -zcf \"images/$FILENAME\" -C /tmp/kolla_registry ."}],"source_content_type":"text/x-sh","patch_set":13,"id":"3f1d235d_94a82e8c","line":87,"in_reply_to":"3f1d235d_b4532a17","updated":"2017-07-05 18:48:29.000000000","message":"Also I don\u0027t think publisher pipeline will have REF_NAME. It didn\u0027t work before this change","commit_id":"c2aacfaf9d32e4f2d042979d883d8e66a605335d"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"2a99af08a801d1c6e01cb2f1375fa711edc01e68","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        REF_NAME\u003d$(echo $ZUUL_REFNAME | cut -d/ -f2)"},{"line_number":85,"context_line":"    fi"},{"line_number":86,"context_line":"    sudo mkdir \"images\""},{"line_number":87,"context_line":"    BRANCH\u003d$(echo \"$ZUUL_BRANCH\" | cut -d/ -f2)"},{"line_number":88,"context_line":"    FILENAME\u003d${BASE_DISTRO}-${INSTALL_TYPE}-registry-${BRANCH}.tar.gz"},{"line_number":89,"context_line":"    sudo docker stop registry"},{"line_number":90,"context_line":"    sudo tar -zcf \"images/$FILENAME\" -C /tmp/kolla_registry ."}],"source_content_type":"text/x-sh","patch_set":13,"id":"3f1d235d_b4532a17","line":87,"in_reply_to":"5f201791_1ba2c34c","updated":"2017-07-05 18:29:51.000000000","message":"We don\u0027t pack tags today, maybe in future we\u0027ll add that, but right now we don\u0027t have tags uploaded anywhere","commit_id":"c2aacfaf9d32e4f2d042979d883d8e66a605335d"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"b9714be4b06a95b27dc2ea9f20ed98bdd0cc035d","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        REF_NAME\u003d$(echo $ZUUL_REFNAME | cut -d/ -f2)"},{"line_number":85,"context_line":"    fi"},{"line_number":86,"context_line":"    sudo mkdir \"images\""},{"line_number":87,"context_line":"    BRANCH\u003d$(echo \"$ZUUL_BRANCH\" | cut -d/ -f2)"},{"line_number":88,"context_line":"    FILENAME\u003d${BASE_DISTRO}-${INSTALL_TYPE}-registry-${BRANCH}.tar.gz"},{"line_number":89,"context_line":"    sudo docker stop registry"},{"line_number":90,"context_line":"    sudo tar -zcf \"images/$FILENAME\" -C /tmp/kolla_registry ."}],"source_content_type":"text/x-sh","patch_set":13,"id":"5f201791_1ba2c34c","line":87,"in_reply_to":"5f201791_9408f091","updated":"2017-06-29 00:43:20.000000000","message":"REF_NAME is useless here.\n\nShouldn\u0027t we handle tag pipeline?","commit_id":"c2aacfaf9d32e4f2d042979d883d8e66a605335d"}],"tools/publisher.sh":[{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"00e4c456b06c8dbacd0cf0a5e589688d9c4f4507","unresolved":false,"context_lines":[{"line_number":27,"context_line":"fi"},{"line_number":28,"context_line":"if [[ $ACTION \u003d\u003d \"deploy-multinode\" ]]; then"},{"line_number":29,"context_line":"    if [[ $ORCH_ENGINE \u003d\u003d \"ansible\" ]]; then"},{"line_number":30,"context_line":"\tdeploy_ansible"},{"line_number":31,"context_line":"    fi"},{"line_number":32,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":5,"id":"5f201791_963e8127","line":30,"updated":"2017-06-21 16:16:05.000000000","message":"whitespace","commit_id":"9b610621495f7a84796838c795158c2255bde7fe"},{"author":{"_account_id":24043,"name":"Rich Wellum","username":"rwellum","inactive":true},"change_message_id":"d574c15020a7e060e77dff5b191f81fbbdc5256b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"env"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"export ZUUL_REF\u003d$ZUUL_REF"},{"line_number":4,"context_line":"export BRANCH\u003d$(echo \"$ZUUL_BRANCH\" | cut -d/ -f2)"},{"line_number":5,"context_line":"export TMP_REGISTRY\u003d\"tmp/\""}],"source_content_type":"text/x-sh","patch_set":13,"id":"5f201791_7419d4d4","line":2,"updated":"2017-06-28 16:49:16.000000000","message":"Might be cheeky - but I\u0027d love a file description at the top - seeing as it\u0027s a new script and you have the opportunity. I echo Surya\u0027s comments about explanations for a newb like me.","commit_id":"c2aacfaf9d32e4f2d042979d883d8e66a605335d"},{"author":{"_account_id":23717,"name":"Surya Prakash Singh","email":"singh.surya64mnnit@gmail.com","username":"SuryaPrakash"},"change_message_id":"7d5bbda009fbc1ea71e66bffac73ec7f2edf47c8","unresolved":false,"context_lines":[{"line_number":6,"context_line":"export PUBLISHER\u003d1"},{"line_number":7,"context_line":"export SIGNOFF_FILENAME\u003d${BASE_DISTRO}-${INSTALL_TYPE}-registry-${BRANCH}.txt"},{"line_number":8,"context_line":"export FILENAME\u003d${BASE_DISTRO}-${INSTALL_TYPE}-registry-${BRANCH}.tar.gz"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"function deploy_ansible {"},{"line_number":11,"context_line":"    export KOLLA_ANSIBLE_DIR\u003d$(mktemp -d)"},{"line_number":12,"context_line":"    cat \u003e /tmp/clonemap \u003c\u003cEOF"}],"source_content_type":"text/x-sh","patch_set":13,"id":"5f201791_ac49b7b7","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":0},"updated":"2017-06-26 15:37:50.000000000","message":"will be good if add some description about this function working","commit_id":"c2aacfaf9d32e4f2d042979d883d8e66a605335d"},{"author":{"_account_id":23717,"name":"Surya Prakash Singh","email":"singh.surya64mnnit@gmail.com","username":"SuryaPrakash"},"change_message_id":"7d5bbda009fbc1ea71e66bffac73ec7f2edf47c8","unresolved":false,"context_lines":[{"line_number":14,"context_line":" - name: openstack/kolla-ansible"},{"line_number":15,"context_line":"   dest: ${KOLLA_ANSIBLE_DIR}"},{"line_number":16,"context_line":"EOF"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"    /usr/zuul-env/bin/zuul-cloner -m /tmp/clonemap --workspace \"$(pwd)\" \\"},{"line_number":19,"context_line":"        --cache-dir /opt/git git://git.openstack.org openstack/kolla-ansible"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-sh","patch_set":13,"id":"5f201791_4ccea32d","line":17,"updated":"2017-06-26 15:37:50.000000000","message":"here too about saving them to temp space, with little description.","commit_id":"c2aacfaf9d32e4f2d042979d883d8e66a605335d"},{"author":{"_account_id":23717,"name":"Surya Prakash Singh","email":"singh.surya64mnnit@gmail.com","username":"SuryaPrakash"},"change_message_id":"7d5bbda009fbc1ea71e66bffac73ec7f2edf47c8","unresolved":false,"context_lines":[{"line_number":22,"context_line":"        tools/setup_gate.sh"},{"line_number":23,"context_line":"    popd"},{"line_number":24,"context_line":"}"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"function signoff {"},{"line_number":27,"context_line":"    mkdir -p images/"},{"line_number":28,"context_line":"    curl -o images/$SIGNOFF_FILENAME http://tarballs.openstack.org/kolla/images/tmp/$SIGNOFF_FILENAME"}],"source_content_type":"text/x-sh","patch_set":13,"id":"5f201791_ac8c17d7","line":25,"updated":"2017-06-26 15:37:50.000000000","message":"here too, will be good for new contributors in quick understanding.","commit_id":"c2aacfaf9d32e4f2d042979d883d8e66a605335d"},{"author":{"_account_id":24043,"name":"Rich Wellum","username":"rwellum","inactive":true},"change_message_id":"d574c15020a7e060e77dff5b191f81fbbdc5256b","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    exit 0"},{"line_number":39,"context_line":"fi"},{"line_number":40,"context_line":"if [[ $ACTION \u003d\u003d \"deploy-multinode\" ]]; then"},{"line_number":41,"context_line":"    if [[ $ORCH_ENGINE \u003d\u003d \"ansible\" ]]; then"},{"line_number":42,"context_line":"        deploy_ansible"},{"line_number":43,"context_line":"        signoff"},{"line_number":44,"context_line":"        exit 0"}],"source_content_type":"text/x-sh","patch_set":13,"id":"5f201791_7467944e","line":41,"updated":"2017-06-28 16:49:16.000000000","message":"Nit - you could combine these with an \u0026\u0026.","commit_id":"c2aacfaf9d32e4f2d042979d883d8e66a605335d"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"2a99af08a801d1c6e01cb2f1375fa711edc01e68","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    exit 0"},{"line_number":39,"context_line":"fi"},{"line_number":40,"context_line":"if [[ $ACTION \u003d\u003d \"deploy-multinode\" ]]; then"},{"line_number":41,"context_line":"    if [[ $ORCH_ENGINE \u003d\u003d \"ansible\" ]]; then"},{"line_number":42,"context_line":"        deploy_ansible"},{"line_number":43,"context_line":"        signoff"},{"line_number":44,"context_line":"        exit 0"}],"source_content_type":"text/x-sh","patch_set":13,"id":"5f201791_3e269669","line":41,"in_reply_to":"5f201791_7467944e","updated":"2017-07-05 18:29:51.000000000","message":"I did that on purpose as we\u0027ll add $ORCH_ENGINE \u003d\u003d kubernetes too;)","commit_id":"c2aacfaf9d32e4f2d042979d883d8e66a605335d"},{"author":{"_account_id":24043,"name":"Rich Wellum","username":"rwellum","inactive":true},"change_message_id":"d574c15020a7e060e77dff5b191f81fbbdc5256b","unresolved":false,"context_lines":[{"line_number":52,"context_line":"    gunzip /tmp/$FILENAME"},{"line_number":53,"context_line":"    tar -rf /tmp/${BASE_DISTRO}-${INSTALL_TYPE}-registry-${BRANCH}.tar /tmp/$SIGNOFF_FILENAME"},{"line_number":54,"context_line":"    gzip /tmp/${BASE_DISTRO}-${INSTALL_TYPE}-registry-${BRANCH}.tar"},{"line_number":55,"context_line":"    ls -la /tmp/"},{"line_number":56,"context_line":"    sudo mv /tmp/$FILENAME images/publisher-$FILENAME"},{"line_number":57,"context_line":"fi"},{"line_number":58,"context_line":""}],"source_content_type":"text/x-sh","patch_set":13,"id":"5f201791_d4a9c823","line":55,"updated":"2017-06-28 16:49:16.000000000","message":"Is this for debugging? I ask because /tmp fill up with a lot of stuff and can flood the output. Maybe need to be more specific about what you are looking for and then error out with appropriate message if needed?","commit_id":"c2aacfaf9d32e4f2d042979d883d8e66a605335d"}]}