)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"25761efa957c057eb4d6a2132cbf0ebc6d8ccc40","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4c0d23cf_b28d5d14","updated":"2025-02-26 02:32:37.000000000","message":"Sorry, left this in drafts","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"450d19b18c32b16b2178b97eba7e717ca759bab5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"69a7346f_139b7be0","updated":"2025-02-26 01:18:39.000000000","message":"nice work on this!","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"07c8ebd28eb3cff8e4b7b631573613e186d85928","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"57e3623b_6c689034","updated":"2025-02-25 15:43:15.000000000","message":"recheck - test_ip_access_to_server [AssertionError: False is not true]","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"2267b8e1323109912c47a67720e5bafce7491f89","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b8210e5f_dc16b0c1","updated":"2025-02-26 09:52:23.000000000","message":"thanks! This looks really promising, some comments inline","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"1209905934c50d1d9500b4c627c70828e2530ad5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6ad6aa71_264bc4e7","in_reply_to":"69a7346f_139b7be0","updated":"2025-02-26 14:15:26.000000000","message":"Acknowledged","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"1c65f8c9352dde77d18b76d0fa4af1c18ae27f4e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"66942e86_feed3897","updated":"2025-03-04 17:10:48.000000000","message":"recheck","commit_id":"36b5e11de61928cc3218ba9c43224653778ab5e0"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"431992a6ef8c6e49a42aace3db5921e25b27e62d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"73870bd0_ad2ab3ff","updated":"2025-03-03 08:06:17.000000000","message":"recheck - test_ip_access_to_server False","commit_id":"36b5e11de61928cc3218ba9c43224653778ab5e0"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"cbd49464003025e5745f9cd8f00da61440b24b13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"cede955f_2c6fa440","updated":"2025-02-28 07:32:46.000000000","message":"recheck - test_ip_access_to_server False","commit_id":"36b5e11de61928cc3218ba9c43224653778ab5e0"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"f5bfb12a4c5808bd0cd764a5cf5b44955dc9d91b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"90014974_e1694e22","updated":"2025-02-27 05:34:06.000000000","message":"recheck ironic-standalone-ipa-src which has passed previously","commit_id":"36b5e11de61928cc3218ba9c43224653778ab5e0"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"a47a7510e552ec63b5984b460f447d818195d51b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"9741ad96_2f12f45b","updated":"2025-02-26 21:40:09.000000000","message":"recheck ironic-standalone-ipa-src which has passed previously","commit_id":"36b5e11de61928cc3218ba9c43224653778ab5e0"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"cebad529228d3cb02a0c8a90557f108264d1746e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"0a033bd0_8b8a9bb3","updated":"2025-03-05 00:48:30.000000000","message":"recheck node failure","commit_id":"36b5e11de61928cc3218ba9c43224653778ab5e0"}],"tinyipa/build-tinyipa.sh":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"450d19b18c32b16b2178b97eba7e717ca759bab5","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"IPMITOOL_GIT_HASH\u003d\"19d78782d795d0cf4ceefe655f616210c9143e62\""},{"line_number":10,"context_line":"TC_RELEASE\u003d\"15.x\""},{"line_number":11,"context_line":"QEMU_RELEASE\u003d\"5.2.0-rc4\""},{"line_number":12,"context_line":"LSHW_RELEASE\u003d\"B.02.18\""},{"line_number":13,"context_line":"IPMITOOL_RELEASE\u003d\"1_8_19\" # points to the same SHA as IPMITOOL_GIT_HASH"},{"line_number":14,"context_line":"BIOSDEVNAME_RELEASE\u003d\"0.7.2\""}],"source_content_type":"text/x-sh","patch_set":4,"id":"7c020350_08b6f520","line":11,"updated":"2025-02-26 01:18:39.000000000","message":"why the switch to a release candidate?","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"1209905934c50d1d9500b4c627c70828e2530ad5","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"IPMITOOL_GIT_HASH\u003d\"19d78782d795d0cf4ceefe655f616210c9143e62\""},{"line_number":10,"context_line":"TC_RELEASE\u003d\"15.x\""},{"line_number":11,"context_line":"QEMU_RELEASE\u003d\"5.2.0-rc4\""},{"line_number":12,"context_line":"LSHW_RELEASE\u003d\"B.02.18\""},{"line_number":13,"context_line":"IPMITOOL_RELEASE\u003d\"1_8_19\" # points to the same SHA as IPMITOOL_GIT_HASH"},{"line_number":14,"context_line":"BIOSDEVNAME_RELEASE\u003d\"0.7.2\""}],"source_content_type":"text/x-sh","patch_set":4,"id":"968d159c_64816f9b","line":11,"in_reply_to":"44927170_d183ab29","updated":"2025-02-26 14:15:26.000000000","message":"Acknowledged","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"2267b8e1323109912c47a67720e5bafce7491f89","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"IPMITOOL_GIT_HASH\u003d\"19d78782d795d0cf4ceefe655f616210c9143e62\""},{"line_number":10,"context_line":"TC_RELEASE\u003d\"15.x\""},{"line_number":11,"context_line":"QEMU_RELEASE\u003d\"5.2.0-rc4\""},{"line_number":12,"context_line":"LSHW_RELEASE\u003d\"B.02.18\""},{"line_number":13,"context_line":"IPMITOOL_RELEASE\u003d\"1_8_19\" # points to the same SHA as IPMITOOL_GIT_HASH"},{"line_number":14,"context_line":"BIOSDEVNAME_RELEASE\u003d\"0.7.2\""}],"source_content_type":"text/x-sh","patch_set":4,"id":"a64da558_82b83d11","line":11,"in_reply_to":"7c020350_08b6f520","updated":"2025-02-26 09:52:23.000000000","message":"unless there\u0027s a huge reason we should not go from stable to RC\nalso not sure why moving the variables definitions here","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"dec7973ccf5aab9a6cb34f11a1505350b8f96b01","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"IPMITOOL_GIT_HASH\u003d\"19d78782d795d0cf4ceefe655f616210c9143e62\""},{"line_number":10,"context_line":"TC_RELEASE\u003d\"15.x\""},{"line_number":11,"context_line":"QEMU_RELEASE\u003d\"5.2.0-rc4\""},{"line_number":12,"context_line":"LSHW_RELEASE\u003d\"B.02.18\""},{"line_number":13,"context_line":"IPMITOOL_RELEASE\u003d\"1_8_19\" # points to the same SHA as IPMITOOL_GIT_HASH"},{"line_number":14,"context_line":"BIOSDEVNAME_RELEASE\u003d\"0.7.2\""}],"source_content_type":"text/x-sh","patch_set":4,"id":"44927170_d183ab29","line":11,"in_reply_to":"a64da558_82b83d11","updated":"2025-02-26 11:58:41.000000000","message":"The moving around of variable definitions was just so I can add my little comment, \"`# points to the same SHA as IPMITOOL_GIT_HASH`\" 😊. In light of your other comment, I have reverted that reshuffle.\n\n\nRe: release candidate: Acknowledged!","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"2267b8e1323109912c47a67720e5bafce7491f89","unresolved":true,"context_lines":[{"line_number":10,"context_line":"TC_RELEASE\u003d\"15.x\""},{"line_number":11,"context_line":"QEMU_RELEASE\u003d\"5.2.0-rc4\""},{"line_number":12,"context_line":"LSHW_RELEASE\u003d\"B.02.18\""},{"line_number":13,"context_line":"IPMITOOL_RELEASE\u003d\"1_8_19\" # points to the same SHA as IPMITOOL_GIT_HASH"},{"line_number":14,"context_line":"BIOSDEVNAME_RELEASE\u003d\"0.7.2\""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"CHROOT_CMD\u003d\"sudo chroot $BUILDDIR /usr/bin/env -i PATH\u003d$CHROOT_PATH http_proxy\u003d$http_proxy https_proxy\u003d$https_proxy no_proxy\u003d$no_proxy\""}],"source_content_type":"text/x-sh","patch_set":4,"id":"b9e77c9f_465254f0","line":13,"updated":"2025-02-26 09:52:23.000000000","message":"we use the hash because ipmitool releases are very rare\nI\u0027d rather keep the hash because ipmi tool releases are rare, and I\u0027m actually not sure it\u0027s developed anymore, so if we need to get more recent commits we\u0027ll have to go back to using the hash","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"dec7973ccf5aab9a6cb34f11a1505350b8f96b01","unresolved":false,"context_lines":[{"line_number":10,"context_line":"TC_RELEASE\u003d\"15.x\""},{"line_number":11,"context_line":"QEMU_RELEASE\u003d\"5.2.0-rc4\""},{"line_number":12,"context_line":"LSHW_RELEASE\u003d\"B.02.18\""},{"line_number":13,"context_line":"IPMITOOL_RELEASE\u003d\"1_8_19\" # points to the same SHA as IPMITOOL_GIT_HASH"},{"line_number":14,"context_line":"BIOSDEVNAME_RELEASE\u003d\"0.7.2\""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"CHROOT_CMD\u003d\"sudo chroot $BUILDDIR /usr/bin/env -i PATH\u003d$CHROOT_PATH http_proxy\u003d$http_proxy https_proxy\u003d$https_proxy no_proxy\u003d$no_proxy\""}],"source_content_type":"text/x-sh","patch_set":4,"id":"456b14d3_1f75747f","line":13,"in_reply_to":"b9e77c9f_465254f0","updated":"2025-02-26 11:58:41.000000000","message":"Acknowledged","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"450d19b18c32b16b2178b97eba7e717ca759bab5","unresolved":true,"context_lines":[{"line_number":37,"context_line":"        for ext in \"${!tar_flags[@]}\"; do"},{"line_number":38,"context_line":"            if [[ \"$source_url\" \u003d~ \\.${ext}$ ]]; then"},{"line_number":39,"context_line":"                mkdir -p \"$destination_path\""},{"line_number":40,"context_line":"                if wget --timeout\u003d30 --tries\u003d3 -O - \"$source_url\" | tar -x${tar_flags[$ext]} -C \"$destination_path\" --strip-components\u003d1 -f -; then"},{"line_number":41,"context_line":"                    echo \"Successfully downloaded $source_url on attempt $attempts\""},{"line_number":42,"context_line":"                    return 0"},{"line_number":43,"context_line":"                fi"}],"source_content_type":"text/x-sh","patch_set":4,"id":"94524f9c_be86b90c","line":40,"updated":"2025-02-26 01:18:39.000000000","message":"If your bash is new enough then your GNU tar is probably new enough to do auto-decompress with -xf then you don’t have to figure out the extension.","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"1209905934c50d1d9500b4c627c70828e2530ad5","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        for ext in \"${!tar_flags[@]}\"; do"},{"line_number":38,"context_line":"            if [[ \"$source_url\" \u003d~ \\.${ext}$ ]]; then"},{"line_number":39,"context_line":"                mkdir -p \"$destination_path\""},{"line_number":40,"context_line":"                if wget --timeout\u003d30 --tries\u003d3 -O - \"$source_url\" | tar -x${tar_flags[$ext]} -C \"$destination_path\" --strip-components\u003d1 -f -; then"},{"line_number":41,"context_line":"                    echo \"Successfully downloaded $source_url on attempt $attempts\""},{"line_number":42,"context_line":"                    return 0"},{"line_number":43,"context_line":"                fi"}],"source_content_type":"text/x-sh","patch_set":4,"id":"1295cab3_312184fe","line":40,"in_reply_to":"87434ad7_d9452407","updated":"2025-02-26 14:15:26.000000000","message":"Acknowledged","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"dec7973ccf5aab9a6cb34f11a1505350b8f96b01","unresolved":true,"context_lines":[{"line_number":37,"context_line":"        for ext in \"${!tar_flags[@]}\"; do"},{"line_number":38,"context_line":"            if [[ \"$source_url\" \u003d~ \\.${ext}$ ]]; then"},{"line_number":39,"context_line":"                mkdir -p \"$destination_path\""},{"line_number":40,"context_line":"                if wget --timeout\u003d30 --tries\u003d3 -O - \"$source_url\" | tar -x${tar_flags[$ext]} -C \"$destination_path\" --strip-components\u003d1 -f -; then"},{"line_number":41,"context_line":"                    echo \"Successfully downloaded $source_url on attempt $attempts\""},{"line_number":42,"context_line":"                    return 0"},{"line_number":43,"context_line":"                fi"}],"source_content_type":"text/x-sh","patch_set":4,"id":"87434ad7_d9452407","line":40,"in_reply_to":"94524f9c_be86b90c","updated":"2025-02-26 11:58:41.000000000","message":"I\u0027m assuming this is the case for our CI, and `-xf` is a lot simpler.\n\nUpdate: Just `-xf` did not work in devstack.\n\n```attempts\u003d2\n\u0027[\u0027 2 -le 5 \u0027]\u0027\n[[ https://download.qemu.org/qemu-5.2.0.tar.xz \u003d~ \\.tar\\.[a-z]+$ ]]\nmkdir -p /opt/stack/devstack/tinyipa/tinyipabuild/tmp/qemu\nwget --timeout\u003d30 --tries\u003d3 -O - https://download.qemu.org/qemu-5.2.0.tar.xz\ntar -xf - -C /opt/stack/devstack/tinyipa/tinyipabuild/tmp/qemu --strip-components\u003d1 --2025-02-26 11:35:07-- https://download.qemu.org/qemu-5.2.0.tar.xz Resolving download.qemu.org (download.qemu.org)... 156.146.53.35, 156.146.53.33, 2a02:6ea0:e600::13, ... Connecting to download.qemu.org (download.qemu.org)|156.146.53.35|:443... connected. HTTP request sent, awaiting response... 200 OK Length: 106902800 (102M) [application/x-tar] Saving to: ‘STDOUT’\n0%[ ] 0 --.-KB/s tar: Archive is compressed. Use -J option tar: Error is not recoverable: exiting now\n0%[ ] 15.57K --.-KB/s in 0.003s Cannot write to ‘-’ (Success).\n[[ ! https://download.qemu.org/qemu-5.2.0.tar.xz \u003d~ \\.tar\\.[a-z]+$ ]]\necho \u0027Download attempt 2 failed for https://download.qemu.org/qemu-5.2.0.tar.xz, retrying in 10 seconds...\u0027\nsleep 10 Download attempt 2 failed for https://download.qemu.org/qemu-5.2.0.tar.xz, retrying in 10 seconds...```","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"25761efa957c057eb4d6a2132cbf0ebc6d8ccc40","unresolved":true,"context_lines":[{"line_number":183,"context_line":""},{"line_number":184,"context_line":"while read line; do"},{"line_number":185,"context_line":"    sudo chroot --userspec\u003d$TC:$STAFF $BUILDDIR /usr/bin/env -i PATH\u003d$CHROOT_PATH http_proxy\u003d$http_proxy https_proxy\u003d$https_proxy no_proxy\u003d$no_proxy tce-load -wci $line"},{"line_number":186,"context_line":"done \u003c \u003c(paste $WORKDIR/build_files/$PY_REQS $WORKDIR/build_files/buildreqs.lst)"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"TINYIPA_PYTHON_EXE\u003d\"python3.9\""},{"line_number":189,"context_line":""}],"source_content_type":"text/x-sh","patch_set":4,"id":"50591957_d8cde7ef","line":186,"updated":"2025-02-26 02:32:37.000000000","message":"I think this construct is also downloading files. This is not blocking feedback as this change is still a massive improvement; just another place we could help on a follow-up.","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"dec7973ccf5aab9a6cb34f11a1505350b8f96b01","unresolved":false,"context_lines":[{"line_number":183,"context_line":""},{"line_number":184,"context_line":"while read line; do"},{"line_number":185,"context_line":"    sudo chroot --userspec\u003d$TC:$STAFF $BUILDDIR /usr/bin/env -i PATH\u003d$CHROOT_PATH http_proxy\u003d$http_proxy https_proxy\u003d$https_proxy no_proxy\u003d$no_proxy tce-load -wci $line"},{"line_number":186,"context_line":"done \u003c \u003c(paste $WORKDIR/build_files/$PY_REQS $WORKDIR/build_files/buildreqs.lst)"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"TINYIPA_PYTHON_EXE\u003d\"python3.9\""},{"line_number":189,"context_line":""}],"source_content_type":"text/x-sh","patch_set":4,"id":"06c69a53_3e5a1c93","line":186,"in_reply_to":"50591957_d8cde7ef","updated":"2025-02-26 11:58:41.000000000","message":"Acknowledged","commit_id":"dca4dd696464c6537443698a72e9e17bee8d3f2a"}]}