)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"e2d3d2e4a5c9961c5a4f1997e64b450f54c85ecd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"0eeee928_488f2822","updated":"2022-02-09 14:28:21.000000000","message":"recheck","commit_id":"ed3beda42d78d217654196d6383c32043df87030"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ff7e2a75def48785f7058b647950f2c5a0f78dcf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"891f48f6_e1596e27","updated":"2022-02-10 15:07:24.000000000","message":"recheck","commit_id":"ed3beda42d78d217654196d6383c32043df87030"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"bd8714073c66545af01f7b854865126c9482b464","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"e9bd06b5_365ecb44","updated":"2022-02-10 17:44:52.000000000","message":"recheck","commit_id":"ed3beda42d78d217654196d6383c32043df87030"}],"devstack/lib/ironic":[{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"05f7724b23c501cb200ab8a7917de87e8deb42b0","unresolved":true,"context_lines":[{"line_number":2863,"context_line":"        dest\u003d\"$IRONIC_DATA_DIR/$IRONIC_PARTITIONED_IMAGE_NAME.img\""},{"line_number":2864,"context_line":""},{"line_number":2865,"context_line":"        # Export some variables that the script is using."},{"line_number":2866,"context_line":"        CIRROR_ARCH\u003d$CIRROR_ARCH CIRROS_VERSION\u003d$CIRROS_VERSION \\"},{"line_number":2867,"context_line":"            IRONIC_TTY_DEV\u003d$IRONIC_TTY_DEV VERBOSE\u003d$VERBOSE \\"},{"line_number":2868,"context_line":"            $IRONIC_SCRIPTS_DIR/cirros-partition.sh \"$dest\""},{"line_number":2869,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":15,"id":"437e7e18_9f131746","line":2866,"range":{"start_line":2866,"start_character":8,"end_line":2866,"end_character":19},"updated":"2022-02-16 08:39:05.000000000","message":"nit: CIRROS ?","commit_id":"a14dd6d67b9807efb3bfb4084586c8ced36e5ece"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"b2b103f66d8071b72c5dd17c31fa0b0012330c04","unresolved":false,"context_lines":[{"line_number":2863,"context_line":"        dest\u003d\"$IRONIC_DATA_DIR/$IRONIC_PARTITIONED_IMAGE_NAME.img\""},{"line_number":2864,"context_line":""},{"line_number":2865,"context_line":"        # Export some variables that the script is using."},{"line_number":2866,"context_line":"        CIRROR_ARCH\u003d$CIRROR_ARCH CIRROS_VERSION\u003d$CIRROS_VERSION \\"},{"line_number":2867,"context_line":"            IRONIC_TTY_DEV\u003d$IRONIC_TTY_DEV VERBOSE\u003d$VERBOSE \\"},{"line_number":2868,"context_line":"            $IRONIC_SCRIPTS_DIR/cirros-partition.sh \"$dest\""},{"line_number":2869,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":15,"id":"11232df0_db6e627c","line":2866,"range":{"start_line":2866,"start_character":8,"end_line":2866,"end_character":19},"in_reply_to":"437e7e18_9f131746","updated":"2022-02-16 09:14:52.000000000","message":"Done","commit_id":"a14dd6d67b9807efb3bfb4084586c8ced36e5ece"}],"devstack/tools/ironic/scripts/cirros-partition.sh":[{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"05f7724b23c501cb200ab8a7917de87e8deb42b0","unresolved":true,"context_lines":[{"line_number":9,"context_line":"fi"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"CIRROS_VERSION\u003d${CIRROS_VERSION:-0.5.2}"},{"line_number":12,"context_line":"CIRROR_ARCH\u003d${CIRROR_ARCH:-x86_64}"},{"line_number":13,"context_line":"# TODO(dtantsur): use the image cached on infra images in the CI"},{"line_number":14,"context_line":"DISK_URL\u003dhttp://download.cirros-cloud.net/${CIRROS_VERSION}/cirros-${CIRROS_VERSION}-${CIRROR_ARCH}-disk.img"},{"line_number":15,"context_line":"OUT\u003d$(realpath ${1:-rootfs.img})"}],"source_content_type":"text/x-sh","patch_set":15,"id":"839de067_e1e3dcc1","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":11},"updated":"2022-02-16 08:39:05.000000000","message":"nit: CIRROS ?","commit_id":"a14dd6d67b9807efb3bfb4084586c8ced36e5ece"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"b2b103f66d8071b72c5dd17c31fa0b0012330c04","unresolved":false,"context_lines":[{"line_number":9,"context_line":"fi"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"CIRROS_VERSION\u003d${CIRROS_VERSION:-0.5.2}"},{"line_number":12,"context_line":"CIRROR_ARCH\u003d${CIRROR_ARCH:-x86_64}"},{"line_number":13,"context_line":"# TODO(dtantsur): use the image cached on infra images in the CI"},{"line_number":14,"context_line":"DISK_URL\u003dhttp://download.cirros-cloud.net/${CIRROS_VERSION}/cirros-${CIRROS_VERSION}-${CIRROR_ARCH}-disk.img"},{"line_number":15,"context_line":"OUT\u003d$(realpath ${1:-rootfs.img})"}],"source_content_type":"text/x-sh","patch_set":15,"id":"9ef11c53_c5470f0f","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":11},"in_reply_to":"839de067_e1e3dcc1","updated":"2022-02-16 09:14:52.000000000","message":"Done","commit_id":"a14dd6d67b9807efb3bfb4084586c8ced36e5ece"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"05f7724b23c501cb200ab8a7917de87e8deb42b0","unresolved":true,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"curl -Lf -o disk.qcow2 $DISK_URL"},{"line_number":31,"context_line":"qemu-img convert -O raw disk.qcow2 disk.img"},{"line_number":32,"context_line":"rm disk.qcow2"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# kpartx automatically allocates loop devices for all partitions in the image"},{"line_number":35,"context_line":"device\u003d$(sudo kpartx -av disk.img | grep -oE \u0027loop[0-9]+p\u0027 | head -1)"}],"source_content_type":"text/x-sh","patch_set":15,"id":"852c73f9_58d7d334","line":32,"updated":"2022-02-16 08:39:05.000000000","message":"put this in the clean_up function? also maybe add -f","commit_id":"a14dd6d67b9807efb3bfb4084586c8ced36e5ece"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"8c6d8c97a8de3f93700f28c6a3a15215348dc916","unresolved":true,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"curl -Lf -o disk.qcow2 $DISK_URL"},{"line_number":31,"context_line":"qemu-img convert -O raw disk.qcow2 disk.img"},{"line_number":32,"context_line":"rm disk.qcow2"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# kpartx automatically allocates loop devices for all partitions in the image"},{"line_number":35,"context_line":"device\u003d$(sudo kpartx -av disk.img | grep -oE \u0027loop[0-9]+p\u0027 | head -1)"}],"source_content_type":"text/x-sh","patch_set":15,"id":"cfed9fca_cdea4087","line":32,"in_reply_to":"852c73f9_58d7d334","updated":"2022-02-16 09:03:54.000000000","message":"No need for this, it\u0027s inside $workdir, which will be removed.","commit_id":"a14dd6d67b9807efb3bfb4084586c8ced36e5ece"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"b2b103f66d8071b72c5dd17c31fa0b0012330c04","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"curl -Lf -o disk.qcow2 $DISK_URL"},{"line_number":31,"context_line":"qemu-img convert -O raw disk.qcow2 disk.img"},{"line_number":32,"context_line":"rm disk.qcow2"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# kpartx automatically allocates loop devices for all partitions in the image"},{"line_number":35,"context_line":"device\u003d$(sudo kpartx -av disk.img | grep -oE \u0027loop[0-9]+p\u0027 | head -1)"}],"source_content_type":"text/x-sh","patch_set":15,"id":"1cbc08ca_37eaa537","line":32,"in_reply_to":"cfed9fca_cdea4087","updated":"2022-02-16 09:14:52.000000000","message":"Ack","commit_id":"a14dd6d67b9807efb3bfb4084586c8ced36e5ece"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"b22a48eeffcc878890271fded9119da6e941078f","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"curl -Lf -o disk.qcow2 $DISK_URL"},{"line_number":31,"context_line":"qemu-img convert -O raw disk.qcow2 disk.img"},{"line_number":32,"context_line":"rm disk.qcow2"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# kpartx automatically allocates loop devices for all partitions in the image"},{"line_number":35,"context_line":"device\u003d$(sudo kpartx -av disk.img | grep -oE \u0027loop[0-9]+p\u0027 | head -1)"}],"source_content_type":"text/x-sh","patch_set":15,"id":"58203828_e1997a2c","line":32,"in_reply_to":"cfed9fca_cdea4087","updated":"2022-02-16 09:16:42.000000000","message":"Ack","commit_id":"a14dd6d67b9807efb3bfb4084586c8ced36e5ece"}]}