)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28413,"name":"yong hu","email":"yong.hu@intel.com","username":"yhu6"},"change_message_id":"f669ad07354dd6726033b92fe615566d652e72e6","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Add python2 based services to docker_image_build_cfg_file."},{"line_number":10,"context_line":"The config file could then be modified to individually shift"},{"line_number":11,"context_line":"images from python2 to python3 build method without having"},{"line_number":12,"context_line":"to tinker with cengn build scripts every time there is a change."},{"line_number":13,"context_line":"Add --python2-wheels opt for build-stx-images.sh to specify link"},{"line_number":14,"context_line":"to python2 based wheels tarball."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"bf51134e_0a3d59c3","line":12,"range":{"start_line":12,"start_character":35,"end_line":12,"end_character":45},"updated":"2020-06-24 05:41:42.000000000","message":"\"every time\" -\u003e \"when\"?","commit_id":"ea69c11ecae3bfbc85985cc4081e696fdc0a8b00"},{"author":{"_account_id":28413,"name":"yong hu","email":"yong.hu@intel.com","username":"yhu6"},"change_message_id":"f669ad07354dd6726033b92fe615566d652e72e6","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Add python2 based services to docker_image_build_cfg_file."},{"line_number":10,"context_line":"The config file could then be modified to individually shift"},{"line_number":11,"context_line":"images from python2 to python3 build method without having"},{"line_number":12,"context_line":"to tinker with cengn build scripts every time there is a change."},{"line_number":13,"context_line":"Add --python2-wheels opt for build-stx-images.sh to specify link"},{"line_number":14,"context_line":"to python2 based wheels tarball."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"bf51134e_aa242db0","line":12,"range":{"start_line":12,"start_character":3,"end_line":12,"end_character":9},"updated":"2020-06-24 05:41:42.000000000","message":"did you mean \"sync\"??","commit_id":"ea69c11ecae3bfbc85985cc4081e696fdc0a8b00"},{"author":{"_account_id":28570,"name":"zhipeng liu","email":"zhipengs.liu@intel.com","username":"zhipengl"},"change_message_id":"6e63fff5c09197051c6b8592ee32354542ed1514","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Add python2 based services to docker_image_build_cfg_file."},{"line_number":10,"context_line":"The config file could then be modified to individually shift"},{"line_number":11,"context_line":"images from python2 to python3 build method without having"},{"line_number":12,"context_line":"to tinker with cengn build scripts every time there is a change."},{"line_number":13,"context_line":"Add --python2-wheels opt for build-stx-images.sh to specify link"},{"line_number":14,"context_line":"to python2 based wheels tarball."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"bf51134e_8a2fa93d","line":12,"range":{"start_line":12,"start_character":3,"end_line":12,"end_character":9},"in_reply_to":"bf51134e_aa242db0","updated":"2020-06-24 06:09:13.000000000","message":"Done","commit_id":"ea69c11ecae3bfbc85985cc4081e696fdc0a8b00"}],"build-tools/build-docker-images/build-stx-images.sh":[{"author":{"_account_id":28410,"name":"Scott Little","email":"scott.little@windriver.com","username":"slittle1"},"change_message_id":"29dd0bdfd34bdb463dedd5ebabe50b68a05b8896","unresolved":false,"context_lines":[{"line_number":631,"context_line":"            WHEELS\u003d$2"},{"line_number":632,"context_line":"            shift 2"},{"line_number":633,"context_line":"            ;;"},{"line_number":634,"context_line":"        --python2-wheels)"},{"line_number":635,"context_line":"            WHEELS_PYTHON2\u003d$2"},{"line_number":636,"context_line":"            shift 2"},{"line_number":637,"context_line":"            ;;"},{"line_number":638,"context_line":"        --version)"}],"source_content_type":"text/x-sh","patch_set":3,"id":"bf51134e_3e534d19","line":635,"range":{"start_line":634,"start_character":0,"end_line":635,"end_character":29},"updated":"2020-06-23 14:07:03.000000000","message":"Drop this arg, read from config file instead.\n\nAlternative is to use this to override the config file value","commit_id":"246ffe59c44ad2da0a353873ee35fb2cbcae0b33"},{"author":{"_account_id":28570,"name":"zhipeng liu","email":"zhipengs.liu@intel.com","username":"zhipengl"},"change_message_id":"5de5f2d832e573319adcd7996c11bc73ad36ea9d","unresolved":false,"context_lines":[{"line_number":631,"context_line":"            WHEELS\u003d$2"},{"line_number":632,"context_line":"            shift 2"},{"line_number":633,"context_line":"            ;;"},{"line_number":634,"context_line":"        --python2-wheels)"},{"line_number":635,"context_line":"            WHEELS_PYTHON2\u003d$2"},{"line_number":636,"context_line":"            shift 2"},{"line_number":637,"context_line":"            ;;"},{"line_number":638,"context_line":"        --version)"}],"source_content_type":"text/x-sh","patch_set":3,"id":"bf51134e_5e644122","line":635,"range":{"start_line":634,"start_character":0,"end_line":635,"end_character":29},"in_reply_to":"bf51134e_3e534d19","updated":"2020-06-23 15:20:59.000000000","message":"I will keep it and use this one override config file value","commit_id":"246ffe59c44ad2da0a353873ee35fb2cbcae0b33"},{"author":{"_account_id":28410,"name":"Scott Little","email":"scott.little@windriver.com","username":"slittle1"},"change_message_id":"29dd0bdfd34bdb463dedd5ebabe50b68a05b8896","unresolved":false,"context_lines":[{"line_number":725,"context_line":"fi"},{"line_number":726,"context_line":""},{"line_number":727,"context_line":"# Get python2 based services from config file"},{"line_number":728,"context_line":"SERVICES_PYTHON2\u003d(`source ${IMAGE_BUILD_CFG_FILE} \u0026\u0026 echo ${SERVICES_PYTHON2} | sed s/,/\\ /g`)"},{"line_number":729,"context_line":""},{"line_number":730,"context_line":"if [ ${#SERVICES_PYTHON2[@]} -ne 0 ] \u0026\u0026 [ -z \"${WHEELS_PYTHON2}\" ]; then"},{"line_number":731,"context_line":"    echo \"Path to python2-wheels tarball must be specified with --python2-wheels option\"\\"}],"source_content_type":"text/x-sh","patch_set":3,"id":"bf51134e_9381b2fe","line":728,"range":{"start_line":728,"start_character":28,"end_line":728,"end_character":48},"updated":"2020-06-23 14:07:03.000000000","message":"test if file exists.  \n\nIf not, default to old behaviour\n\nRename to SERVICES_ALTERNATE\n\nUse SERVICES_ALTERNATE to populate first the ONLY, and later the SKIP arrays on two consecutive runs.","commit_id":"246ffe59c44ad2da0a353873ee35fb2cbcae0b33"},{"author":{"_account_id":28570,"name":"zhipeng liu","email":"zhipengs.liu@intel.com","username":"zhipengl"},"change_message_id":"5de5f2d832e573319adcd7996c11bc73ad36ea9d","unresolved":false,"context_lines":[{"line_number":725,"context_line":"fi"},{"line_number":726,"context_line":""},{"line_number":727,"context_line":"# Get python2 based services from config file"},{"line_number":728,"context_line":"SERVICES_PYTHON2\u003d(`source ${IMAGE_BUILD_CFG_FILE} \u0026\u0026 echo ${SERVICES_PYTHON2} | sed s/,/\\ /g`)"},{"line_number":729,"context_line":""},{"line_number":730,"context_line":"if [ ${#SERVICES_PYTHON2[@]} -ne 0 ] \u0026\u0026 [ -z \"${WHEELS_PYTHON2}\" ]; then"},{"line_number":731,"context_line":"    echo \"Path to python2-wheels tarball must be specified with --python2-wheels option\"\\"}],"source_content_type":"text/x-sh","patch_set":3,"id":"bf51134e_b9dad774","line":728,"range":{"start_line":728,"start_character":28,"end_line":728,"end_character":48},"in_reply_to":"bf51134e_9381b2fe","updated":"2020-06-23 15:20:59.000000000","message":"OK!\nBut it seems no need for two consecutive runs.\nOnly run it once is enough.\nJust check if the service is in alternate list, then use python2-wheel, otherwise use python3-wheel.","commit_id":"246ffe59c44ad2da0a353873ee35fb2cbcae0b33"},{"author":{"_account_id":28410,"name":"Scott Little","email":"scott.little@windriver.com","username":"slittle1"},"change_message_id":"29dd0bdfd34bdb463dedd5ebabe50b68a05b8896","unresolved":false,"context_lines":[{"line_number":727,"context_line":"# Get python2 based services from config file"},{"line_number":728,"context_line":"SERVICES_PYTHON2\u003d(`source ${IMAGE_BUILD_CFG_FILE} \u0026\u0026 echo ${SERVICES_PYTHON2} | sed s/,/\\ /g`)"},{"line_number":729,"context_line":""},{"line_number":730,"context_line":"if [ ${#SERVICES_PYTHON2[@]} -ne 0 ] \u0026\u0026 [ -z \"${WHEELS_PYTHON2}\" ]; then"},{"line_number":731,"context_line":"    echo \"Path to python2-wheels tarball must be specified with --python2-wheels option\"\\"},{"line_number":732,"context_line":"         \"if python2 based services need to be build!\" \u003e\u00262"},{"line_number":733,"context_line":"    exit 1"}],"source_content_type":"text/x-sh","patch_set":3,"id":"bf51134e_33954634","line":730,"range":{"start_line":730,"start_character":48,"end_line":730,"end_character":62},"updated":"2020-06-23 14:07:03.000000000","message":"Rename WHEELS_PYTHON2 to WHEELS_ALTERNATE.\n\nWHEELS_ALTERNATE needs to be read from the config file as well.","commit_id":"246ffe59c44ad2da0a353873ee35fb2cbcae0b33"},{"author":{"_account_id":28570,"name":"zhipeng liu","email":"zhipengs.liu@intel.com","username":"zhipengl"},"change_message_id":"5de5f2d832e573319adcd7996c11bc73ad36ea9d","unresolved":false,"context_lines":[{"line_number":727,"context_line":"# Get python2 based services from config file"},{"line_number":728,"context_line":"SERVICES_PYTHON2\u003d(`source ${IMAGE_BUILD_CFG_FILE} \u0026\u0026 echo ${SERVICES_PYTHON2} | sed s/,/\\ /g`)"},{"line_number":729,"context_line":""},{"line_number":730,"context_line":"if [ ${#SERVICES_PYTHON2[@]} -ne 0 ] \u0026\u0026 [ -z \"${WHEELS_PYTHON2}\" ]; then"},{"line_number":731,"context_line":"    echo \"Path to python2-wheels tarball must be specified with --python2-wheels option\"\\"},{"line_number":732,"context_line":"         \"if python2 based services need to be build!\" \u003e\u00262"},{"line_number":733,"context_line":"    exit 1"}],"source_content_type":"text/x-sh","patch_set":3,"id":"bf51134e_1974a3fd","line":730,"range":{"start_line":730,"start_character":48,"end_line":730,"end_character":62},"in_reply_to":"bf51134e_33954634","updated":"2020-06-23 15:20:59.000000000","message":"OK","commit_id":"246ffe59c44ad2da0a353873ee35fb2cbcae0b33"},{"author":{"_account_id":30728,"name":"YU CHENGDE","display_name":"Chant Yu","email":"chengde.yu@intel.com","username":"yuchengde"},"change_message_id":"3a6ae397d542f6b50d275b4d48c6d5368ae1b606","unresolved":false,"context_lines":[{"line_number":720,"context_line":""},{"line_number":721,"context_line":"# Get python2 based services from config file"},{"line_number":722,"context_line":"if [ -f ${IMAGE_BUILD_CFG_FILE} ]; then"},{"line_number":723,"context_line":"    SERVICES_ALTERNATE\u003d(`source $MY_REPO/build-tools/build-docker-images/$IMAGE_BUILD_CFG_FILE \u0026\u0026 echo ${SERVICES_ALTERNATE} | sed s/,/\\ /g`)"},{"line_number":724,"context_line":"    if [ -z \"${WHEELS_ALTERNATE}\" ]; then"},{"line_number":725,"context_line":"        WHEELS_ALTERNATE\u003d$(source $MY_REPO/build-tools/build-docker-images/$IMAGE_BUILD_CFG_FILE \u0026\u0026 echo ${WHEELS_ALTERNATE})"},{"line_number":726,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":6,"id":"bf51134e_846a0717","line":723,"range":{"start_line":723,"start_character":32,"end_line":723,"end_character":94},"updated":"2020-06-23 18:21:17.000000000","message":"$IMAGE_BUILD_CFG_FILE\n-\u003e\n$MY_REPO/build-tools/build-dockerimages/$IMAGE_BUILD_CFG_FILE","commit_id":"20e048d48b39a1686ab9a018d3fcb3c70560aa47"},{"author":{"_account_id":30728,"name":"YU CHENGDE","display_name":"Chant Yu","email":"chengde.yu@intel.com","username":"yuchengde"},"change_message_id":"3a6ae397d542f6b50d275b4d48c6d5368ae1b606","unresolved":false,"context_lines":[{"line_number":722,"context_line":"if [ -f ${IMAGE_BUILD_CFG_FILE} ]; then"},{"line_number":723,"context_line":"    SERVICES_ALTERNATE\u003d(`source $MY_REPO/build-tools/build-docker-images/$IMAGE_BUILD_CFG_FILE \u0026\u0026 echo ${SERVICES_ALTERNATE} | sed s/,/\\ /g`)"},{"line_number":724,"context_line":"    if [ -z \"${WHEELS_ALTERNATE}\" ]; then"},{"line_number":725,"context_line":"        WHEELS_ALTERNATE\u003d$(source $MY_REPO/build-tools/build-docker-images/$IMAGE_BUILD_CFG_FILE \u0026\u0026 echo ${WHEELS_ALTERNATE})"},{"line_number":726,"context_line":"    fi"},{"line_number":727,"context_line":"fi"},{"line_number":728,"context_line":""}],"source_content_type":"text/x-sh","patch_set":6,"id":"bf51134e_047f17e0","line":725,"range":{"start_line":725,"start_character":34,"end_line":725,"end_character":105},"updated":"2020-06-23 18:21:17.000000000","message":"$IMAGE_BUILD_CFG_FILE \n-\u003e\n$MY_REPO/build-tools/build-docker-images/$IMAGE_BUILD_CFG_FILE","commit_id":"20e048d48b39a1686ab9a018d3fcb3c70560aa47"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"2d81b529c27233c2b5a1b1296f9029b24d91d463","unresolved":false,"context_lines":[{"line_number":730,"context_line":"                        \u0026\u0026 echo ${SERVICES_ALTERNATE} | sed s/,/\\ /g`)"},{"line_number":731,"context_line":"    if [ -z \"${WHEELS_ALTERNATE}\" ]; then"},{"line_number":732,"context_line":"        WHEELS_ALTERNATE\u003d$(source $MY_REPO/build-tools/build-docker-images/$IMAGE_BUILD_CFG_FILE \\"},{"line_number":733,"context_line":"                           \u0026\u0026 echo ${WHEELS_ALTERNATE})"},{"line_number":734,"context_line":"    fi"},{"line_number":735,"context_line":"fi"},{"line_number":736,"context_line":""}],"source_content_type":"text/x-sh","patch_set":8,"id":"bf51134e_799b1517","line":733,"updated":"2020-06-24 00:31:26.000000000","message":"linters: E003 Indent not multiple of 4","commit_id":"20ee932998cb2afdf3cfaa5511854d820bbbb382"},{"author":{"_account_id":28413,"name":"yong hu","email":"yong.hu@intel.com","username":"yhu6"},"change_message_id":"f669ad07354dd6726033b92fe615566d652e72e6","unresolved":false,"context_lines":[{"line_number":53,"context_line":"    --stream:     Build stream, stable or dev (default: stable)"},{"line_number":54,"context_line":"    --base:       Specify base docker image (required option)"},{"line_number":55,"context_line":"    --wheels:     Specify path to wheels tarball or image, URL or docker tag (required option)"},{"line_number":56,"context_line":"    --wheels-alternate: Specify path to python2 based wheels tarball or image, URL or docker tag"},{"line_number":57,"context_line":"    --push:       Push to docker repo"},{"line_number":58,"context_line":"    --proxy:      Set proxy \u003cURL\u003e:\u003cPORT\u003e"},{"line_number":59,"context_line":"    --user:       Docker repo userid"}],"source_content_type":"text/x-sh","patch_set":9,"id":"bf51134e_8a4e896c","line":56,"range":{"start_line":56,"start_character":40,"end_line":56,"end_character":47},"updated":"2020-06-24 05:41:42.000000000","message":"does it have to be \"python2\"? could it be Python3?\nif it is only for Python2, why don\u0027t we name it \"--wheels-python2\"?","commit_id":"ea69c11ecae3bfbc85985cc4081e696fdc0a8b00"},{"author":{"_account_id":28570,"name":"zhipeng liu","email":"zhipengs.liu@intel.com","username":"zhipengl"},"change_message_id":"6e63fff5c09197051c6b8592ee32354542ed1514","unresolved":false,"context_lines":[{"line_number":53,"context_line":"    --stream:     Build stream, stable or dev (default: stable)"},{"line_number":54,"context_line":"    --base:       Specify base docker image (required option)"},{"line_number":55,"context_line":"    --wheels:     Specify path to wheels tarball or image, URL or docker tag (required option)"},{"line_number":56,"context_line":"    --wheels-alternate: Specify path to python2 based wheels tarball or image, URL or docker tag"},{"line_number":57,"context_line":"    --push:       Push to docker repo"},{"line_number":58,"context_line":"    --proxy:      Set proxy \u003cURL\u003e:\u003cPORT\u003e"},{"line_number":59,"context_line":"    --user:       Docker repo userid"}],"source_content_type":"text/x-sh","patch_set":9,"id":"bf51134e_2a165df2","line":56,"range":{"start_line":56,"start_character":40,"end_line":56,"end_character":47},"in_reply_to":"bf51134e_8a4e896c","updated":"2020-06-24 06:09:13.000000000","message":"Change it to alternate wheels","commit_id":"ea69c11ecae3bfbc85985cc4081e696fdc0a8b00"}],"build-tools/build-docker-images/docker-image-build.cfg":[{"author":{"_account_id":28410,"name":"Scott Little","email":"scott.little@windriver.com","username":"slittle1"},"change_message_id":"2a07a6784135d9c29233da99482136a3ceb842e6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"SERVICES_ALTERNATE\u003dstx-fm-rest-api,stx-keystone-api-proxy,stx-nova-api-proxy,stx-platformclients"},{"line_number":2,"context_line":"WHEELS_ALTERNATE\u003d"}],"source_content_type":"text/x-ttcn-cfg","patch_set":6,"id":"bf51134e_fafa7a40","line":2,"range":{"start_line":2,"start_character":16,"end_line":2,"end_character":17},"updated":"2020-06-23 20:06:28.000000000","message":"sigh\n\nThis is where we get in trouble isn\u0027t it.\n\nDo we generate a python2 wheels tarball, or do we download one from an old cengn build that is forever frozen in time.\n\nFor download we probably need an .lst file entry to download something like ...\n\nhttp://mirror.starlingx.cengn.ca/mirror/starlingx/master/centos/monolithic/latest_docker_image_build/outputs/wheels/stx-centos-stable-wheels.tar\n\nI\u0027d probably have to copy and rename it to ... \nhttp://mirror.starlingx.cengn.ca/mirror/starlingx/master/centos/stx-centos-py2_stable-wheels.tar\n... done\n\nMeanwhile WHEELS_ALTERNATE probably needs to reference $MY_REPO/stx/downloads/stx-centos-py2_stable-wheels.tar ...\nwhich might need an \u0027eval\u0027 to expand $MY_REPO","commit_id":"20e048d48b39a1686ab9a018d3fcb3c70560aa47"},{"author":{"_account_id":28570,"name":"zhipeng liu","email":"zhipengs.liu@intel.com","username":"zhipengl"},"change_message_id":"c71d6653c539b5ace5356c922170a14522633e7c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"SERVICES_ALTERNATE\u003dstx-fm-rest-api,stx-keystone-api-proxy,stx-nova-api-proxy,stx-platformclients"},{"line_number":2,"context_line":"WHEELS_ALTERNATE\u003d"}],"source_content_type":"text/x-ttcn-cfg","patch_set":6,"id":"bf51134e_3007f184","line":2,"range":{"start_line":2,"start_character":16,"end_line":2,"end_character":17},"in_reply_to":"bf51134e_fafa7a40","updated":"2020-06-23 21:30:59.000000000","message":"I add your python2 wheel tarball link here","commit_id":"20e048d48b39a1686ab9a018d3fcb3c70560aa47"}]}