)]}'
{"playbooks/ironic-python-agent-buildimage/run.yaml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ebc4b26c993919f79f00f19c5522e26a5615a6cb","unresolved":false,"context_lines":[{"line_number":40,"context_line":"                  if type_image \u003d\u003d \"dib\"; then"},{"line_number":41,"context_line":"                    export IPA_SOURCE_DIR\u003d{{ ansible_user_dir }}/src/opendev.org/openstack/ironic-python-agent"},{"line_number":42,"context_line":"                    pushd {{ ansible_user_dir }}/src/opendev.org/openstack/ironic-python-agent-builder/dib"},{"line_number":43,"context_line":"                    build"},{"line_number":44,"context_line":"                    mv dib*.tar.gz* \"$IPA_SOURCE_DIR/UPLOAD_TAR\""},{"line_number":45,"context_line":"                    mv dib*.* \"$IPA_SOURCE_DIR/UPLOAD_RAW\""},{"line_number":46,"context_line":"                    mv tiny-instance-uec*.tar.gz \"$IPA_SOURCE_DIR/UPLOAD_TAR\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_1ffaf8bb","line":43,"updated":"2019-08-06 08:50:23.000000000","message":"Maybe ./build.sh ?","commit_id":"01372336ab20acf05f1833ef2864c38b33ff1986"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ebc4b26c993919f79f00f19c5522e26a5615a6cb","unresolved":false,"context_lines":[{"line_number":41,"context_line":"                    export IPA_SOURCE_DIR\u003d{{ ansible_user_dir }}/src/opendev.org/openstack/ironic-python-agent"},{"line_number":42,"context_line":"                    pushd {{ ansible_user_dir }}/src/opendev.org/openstack/ironic-python-agent-builder/dib"},{"line_number":43,"context_line":"                    build"},{"line_number":44,"context_line":"                    mv dib*.tar.gz* \"$IPA_SOURCE_DIR/UPLOAD_TAR\""},{"line_number":45,"context_line":"                    mv dib*.* \"$IPA_SOURCE_DIR/UPLOAD_RAW\""},{"line_number":46,"context_line":"                    mv tiny-instance-uec*.tar.gz \"$IPA_SOURCE_DIR/UPLOAD_TAR\""},{"line_number":47,"context_line":"                    # NOTE(sambetts) Must clean up chroot directories before"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_3ff574cb","line":44,"updated":"2019-08-06 08:50:23.000000000","message":"I think the default name is different","commit_id":"01372336ab20acf05f1833ef2864c38b33ff1986"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ebc4b26c993919f79f00f19c5522e26a5615a6cb","unresolved":false,"context_lines":[{"line_number":43,"context_line":"                    build"},{"line_number":44,"context_line":"                    mv dib*.tar.gz* \"$IPA_SOURCE_DIR/UPLOAD_TAR\""},{"line_number":45,"context_line":"                    mv dib*.* \"$IPA_SOURCE_DIR/UPLOAD_RAW\""},{"line_number":46,"context_line":"                    mv tiny-instance-uec*.tar.gz \"$IPA_SOURCE_DIR/UPLOAD_TAR\""},{"line_number":47,"context_line":"                    # NOTE(sambetts) Must clean up chroot directories before"},{"line_number":48,"context_line":"                    # publisher rsync command tries and fails to read them"},{"line_number":49,"context_line":"                    make clean"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_dfff80a8","line":46,"updated":"2019-08-06 08:50:23.000000000","message":"This is from tinyipa","commit_id":"01372336ab20acf05f1833ef2864c38b33ff1986"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ebc4b26c993919f79f00f19c5522e26a5615a6cb","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                    mv tiny-instance-uec*.tar.gz \"$IPA_SOURCE_DIR/UPLOAD_TAR\""},{"line_number":47,"context_line":"                    # NOTE(sambetts) Must clean up chroot directories before"},{"line_number":48,"context_line":"                    # publisher rsync command tries and fails to read them"},{"line_number":49,"context_line":"                    make clean"},{"line_number":50,"context_line":"                    popd"},{"line_number":51,"context_line":"                  else"},{"line_number":52,"context_line":"                    export BUILD_AND_INSTALL_TINYIPA\u003dtrue"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_fffa7cb7","line":49,"updated":"2019-08-06 08:50:23.000000000","message":"I don\u0027t think we have a makefile for DIB","commit_id":"01372336ab20acf05f1833ef2864c38b33ff1986"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ebc4b26c993919f79f00f19c5522e26a5615a6cb","unresolved":false,"context_lines":[{"line_number":66,"context_line":"                  imagebuild/coreos/full_trusty_build.sh"},{"line_number":67,"context_line":"                  mv imagebuild/coreos/UPLOAD/coreos_production_pxe* UPLOAD_RAW"},{"line_number":68,"context_line":"                  mv ipa-coreos*.tar.gz* UPLOAD_TAR"},{"line_number":69,"context_line":"              ;;"},{"line_number":70,"context_line":"          esac"},{"line_number":71,"context_line":"          ERRORS\u003d0"},{"line_number":72,"context_line":"        executable: /bin/bash"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_9f2fc83d","line":69,"updated":"2019-08-06 08:50:23.000000000","message":"You have to add another case here, not inside of tinyipa case:\n\n dib)\n   ....\n ;;\n\nPlease check the bash switch statement syntax","commit_id":"01372336ab20acf05f1833ef2864c38b33ff1986"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bfedf7e521bb4a863f397cf2bd2e5ce36a1a5e1b","unresolved":false,"context_lines":[{"line_number":66,"context_line":"                  imagebuild/coreos/full_trusty_build.sh"},{"line_number":67,"context_line":"                  mv imagebuild/coreos/UPLOAD/coreos_production_pxe* UPLOAD_RAW"},{"line_number":68,"context_line":"                  mv ipa-coreos*.tar.gz* UPLOAD_TAR"},{"line_number":69,"context_line":"              ;;"},{"line_number":70,"context_line":"          esac"},{"line_number":71,"context_line":"          ERRORS\u003d0"},{"line_number":72,"context_line":"        executable: /bin/bash"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_4960a50e","line":69,"in_reply_to":"7faddb67_05d29a00","updated":"2019-08-07 07:21:05.000000000","message":"We only have image_name, image_type is not a thing. So, image_name can be \"dib\", \"tinyipa\" or \"coreos\".","commit_id":"01372336ab20acf05f1833ef2864c38b33ff1986"},{"author":{"_account_id":30099,"name":"Nisha Brahmankar","email":"nishagb017@gmail.com","username":"nishagb","status":"Currently, I work in the OpenStack Team of LY Corporation (formerly LINE Corp.) in Tokyo, Japan."},"change_message_id":"b1b37257477c0ddbdcdd8c3d3b57ec7e37167a54","unresolved":false,"context_lines":[{"line_number":66,"context_line":"                  imagebuild/coreos/full_trusty_build.sh"},{"line_number":67,"context_line":"                  mv imagebuild/coreos/UPLOAD/coreos_production_pxe* UPLOAD_RAW"},{"line_number":68,"context_line":"                  mv ipa-coreos*.tar.gz* UPLOAD_TAR"},{"line_number":69,"context_line":"              ;;"},{"line_number":70,"context_line":"          esac"},{"line_number":71,"context_line":"          ERRORS\u003d0"},{"line_number":72,"context_line":"        executable: /bin/bash"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_05d29a00","line":69,"in_reply_to":"7faddb67_9f2fc83d","updated":"2019-08-06 18:38:43.000000000","message":"So do you mean if the user has image_name\u003dtinyipa and image_type\u003ddib it will run both the if and the else statement that I have added?","commit_id":"01372336ab20acf05f1833ef2864c38b33ff1986"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"23aac5493abb2eba088b34ba633d97d5fd64eb65","unresolved":false,"context_lines":[{"line_number":55,"context_line":"              ;;"},{"line_number":56,"context_line":"          esac"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"          type_image\u003d\u0027{{ image_type }}\u0027"},{"line_number":59,"context_line":"          case $type_image in"},{"line_number":60,"context_line":"              dib)"},{"line_number":61,"context_line":"                  export IPA_SOURCE_DIR\u003d{{ ansible_user_dir }}/src/opendev.org/openstack/ironic-python-agent"},{"line_number":62,"context_line":"                  pushd {{ ansible_user_dir }}/src/opendev.org/openstack/ironic-python-agent-builder/dib"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7faddb67_296f69fd","line":59,"range":{"start_line":58,"start_character":0,"end_line":59,"end_character":29},"updated":"2019-08-07 07:24:00.000000000","message":"please use the existing image_name variable and make it a part of the switch above.","commit_id":"40c395fbf6cab6c0cf8848bd8d8df590587c3d62"},{"author":{"_account_id":30099,"name":"Nisha Brahmankar","email":"nishagb017@gmail.com","username":"nishagb","status":"Currently, I work in the OpenStack Team of LY Corporation (formerly LINE Corp.) in Tokyo, Japan."},"change_message_id":"0200588509726fd8e33927a28e7c6b3314755d7b","unresolved":false,"context_lines":[{"line_number":61,"context_line":"                  export IPA_SOURCE_DIR\u003d{{ ansible_user_dir }}/src/opendev.org/openstack/ironic-python-agent"},{"line_number":62,"context_line":"                  pushd {{ ansible_user_dir }}/src/opendev.org/openstack/ironic-python-agent-builder/dib"},{"line_number":63,"context_line":"                  ./build.sh"},{"line_number":64,"context_line":"                  mv dib*.tar.gz* \"$IPA_SOURCE_DIR/UPLOAD_TAR\""},{"line_number":65,"context_line":"                  mv dib*.* \"$IPA_SOURCE_DIR/UPLOAD_RAW\""},{"line_number":66,"context_line":"                  # NOTE(sambetts) Must clean up chroot directories before"},{"line_number":67,"context_line":"                  # publisher rsync command tries and fails to read them"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7faddb67_b7473d4c","line":64,"updated":"2019-08-06 19:02:44.000000000","message":"Need help with this. I am not sure about the default output name. How can I find that?","commit_id":"40c395fbf6cab6c0cf8848bd8d8df590587c3d62"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"23aac5493abb2eba088b34ba633d97d5fd64eb65","unresolved":false,"context_lines":[{"line_number":61,"context_line":"                  export IPA_SOURCE_DIR\u003d{{ ansible_user_dir }}/src/opendev.org/openstack/ironic-python-agent"},{"line_number":62,"context_line":"                  pushd {{ ansible_user_dir }}/src/opendev.org/openstack/ironic-python-agent-builder/dib"},{"line_number":63,"context_line":"                  ./build.sh"},{"line_number":64,"context_line":"                  mv dib*.tar.gz* \"$IPA_SOURCE_DIR/UPLOAD_TAR\""},{"line_number":65,"context_line":"                  mv dib*.* \"$IPA_SOURCE_DIR/UPLOAD_RAW\""},{"line_number":66,"context_line":"                  # NOTE(sambetts) Must clean up chroot directories before"},{"line_number":67,"context_line":"                  # publisher rsync command tries and fails to read them"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7faddb67_897d3db4","line":64,"in_reply_to":"7faddb67_b7473d4c","updated":"2019-08-07 07:24:00.000000000","message":"It goes after -o in https://github.com/openstack/ironic-python-agent-builder/blob/master/dib/build.sh#L10 (and by the way, we should probably remove .qcow from it).\n\nAlso we won\u0027t have a tarball, only the images themselves.","commit_id":"40c395fbf6cab6c0cf8848bd8d8df590587c3d62"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"fa7124aa9d0f97c694bea7773f35e86901e830f5","unresolved":false,"context_lines":[{"line_number":51,"context_line":"              coreos)"},{"line_number":52,"context_line":"                  imagebuild/coreos/full_trusty_build.sh"},{"line_number":53,"context_line":"                  mv imagebuild/coreos/UPLOAD/coreos_production_pxe* UPLOAD_RAW"},{"line_number":54,"context_line":"                  mv ipa-coreos*.tar.gz* UPLOAD_TAR"},{"line_number":55,"context_line":"              dib)"},{"line_number":56,"context_line":"                  export IPA_SOURCE_DIR\u003d{{ ansible_user_dir }}/src/opendev.org/openstack/ironic-python-agent"},{"line_number":57,"context_line":"                  pushd {{ ansible_user_dir }}/src/opendev.org/openstack/ironic-python-agent-builder/dib"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7faddb67_d75d0316","line":54,"updated":"2019-08-07 18:26:45.000000000","message":"you need ;; after the last instruction (as on lines 50 and 63). this is the weird bash syntax for switch statements.","commit_id":"1a91b7890d19736b5ad0a8d3cf55430def7a315c"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"fa7124aa9d0f97c694bea7773f35e86901e830f5","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                  ./build.sh"},{"line_number":59,"context_line":"                  mv ironic-python-agent* \"$IPA_SOURCE_DIR/UPLOAD_TAR\""},{"line_number":60,"context_line":"                  # NOTE(sambetts) Must clean up chroot directories before"},{"line_number":61,"context_line":"                  # publisher rsync command tries and fails to read them"},{"line_number":62,"context_line":"                  popd"},{"line_number":63,"context_line":"              ;;"},{"line_number":64,"context_line":"          esac"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7faddb67_7798cfcb","line":61,"updated":"2019-08-07 18:26:45.000000000","message":"this should be removed, we don\u0027t have a way to clean anything","commit_id":"1a91b7890d19736b5ad0a8d3cf55430def7a315c"}],"zuul.d/ironic-python-agent-jobs.yaml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"23aac5493abb2eba088b34ba633d97d5fd64eb65","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    name: ironic-python-agent-buildimage-tinyipa"},{"line_number":113,"context_line":"    parent: ironic-python-agent-buildimage-base"},{"line_number":114,"context_line":"    vars:"},{"line_number":115,"context_line":"      image_name: \u0027tinyipa\u0027"},{"line_number":116,"context_line":"      image_type: \u0027dib\u0027"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"# Experimental jobs"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7faddb67_2948c985","line":116,"range":{"start_line":115,"start_character":0,"end_line":116,"end_character":23},"updated":"2019-08-07 07:24:00.000000000","message":"tinyipa and dib are mutually exclusive, I\u0027m not sure why you\u0027re trying to have both. should be just\n\n image_name: \u0027dib\u0027","commit_id":"40c395fbf6cab6c0cf8848bd8d8df590587c3d62"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"fa7124aa9d0f97c694bea7773f35e86901e830f5","unresolved":false,"context_lines":[{"line_number":109,"context_line":"      image_name: \u0027tinyipa\u0027"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"- job:"},{"line_number":112,"context_line":"    name: ironic-python-agent-buildimage-tinyipa"},{"line_number":113,"context_line":"    parent: ironic-python-agent-buildimage-base"},{"line_number":114,"context_line":"    vars:"},{"line_number":115,"context_line":"      image_name: \u0027dib\u0027"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7faddb67_979d4bbd","line":112,"updated":"2019-08-07 18:26:45.000000000","message":"s/tinyipa/dib/\n\nand please add this job to zuul.d/project.yaml","commit_id":"1a91b7890d19736b5ad0a8d3cf55430def7a315c"}]}