)]}'
{"/COMMIT_MSG":[{"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":"e21b41445b94b86b3646f6302372da05de835971","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     cid \u003cafonnepaulc@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-05-29 12:49:45 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"wip: Provision aarch64 fake-bare-metal-vms"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Steps:"},{"line_number":10,"context_line":"- Ironic devstack plugin loads the right arch disk images."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":45,"id":"7aa97068_43fe579a","line":7,"updated":"2024-05-30 11:14:50.000000000","message":"If you believe this is ready for review, make sure to revise this to remove wip.","commit_id":"29e2984cba9ad82f1a33f22c440d2ba8b8c3593a"},{"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":"efc02ddad2fb9fc81366e958b493bba9d058aed3","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     cid \u003cafonnepaulc@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-05-29 12:49:45 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"wip: Provision aarch64 fake-bare-metal-vms"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Steps:"},{"line_number":10,"context_line":"- Ironic devstack plugin loads the right arch disk images."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":45,"id":"9c4a8309_3d65a767","line":7,"in_reply_to":"7aa97068_43fe579a","updated":"2024-05-30 17:31:04.000000000","message":"Acknowledged","commit_id":"29e2984cba9ad82f1a33f22c440d2ba8b8c3593a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"3f09cafdafe58266f82932b52cbaf34277893945","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"c8a2cba4_ebab4dc3","updated":"2024-04-29 14:27:24.000000000","message":"Hey cid!\n\nthe issue is actually in devstack/tools/ironic/templates/vm.xml\n\nHere is a diff:\n\n+++ b/devstack/tools/ironic/templates/vm.xml\n@@ -14,7 +14,9 @@\n         {% endif %}\n     {% endif %}\n     \u003cbootmenu enable\u003d\u0027no\u0027/\u003e\n+    {% if arch !\u003d \u0027aarch64\u0027 %}\n     \u003cbios useserial\u003d\u0027yes\u0027/\u003e\n+    {% endif %}\n   \u003c/os\u003e\n   {% if engine \u003d\u003d \u0027kvm\u0027 %}\n   \u003ccpu mode\u003d\u0027host-passthrough\u0027/\u003e\n\n\nThe issue is that the error is from the bios definition which is an x86 feature called the Serial Graphics Adapter. This is an interface which basically identifies text and treats it as a serial port output. This means our use will likely need to be set to *explicitly* log to serial console. Most likely. Anyway, hopefully this unblocks you. Have a great day!","commit_id":"351e064c5e3b5b03b3ad6d8cad9b256f5cd748a5"},{"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":"71123537daf2d2c877e1c9a3b40aca02b659fe82","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"5120abbb_3ac6467b","updated":"2024-05-07 15:02:20.000000000","message":"Recheck","commit_id":"d59ad3ef812f576e47137d7dec818bb3c9af74b1"},{"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":"e21b41445b94b86b3646f6302372da05de835971","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":45,"id":"e0eb3726_945c9541","updated":"2024-05-30 11:14:50.000000000","message":"This is extremely close.\n\nA couple of pieces missing, most of which can be added in a follow-up:\n- We need to document ARM devstack configurations for contributors, probably with another configuration (or some commented out options like what we do to teach folks how to swap from ipmi to redfish) in https://opendev.org/openstack/ironic/src/branch/master/doc/source/contributor/devstack-guide.rst -- it may even be valuable to add a completely new document on ARM+devstack, which would allow you to document some of the existing limitations; but this is potentially overkill\n- We need to ensure to put some safeguards in place around things that won\u0027t work: e.g. IRONIC_HW_ARCH\u003daarch64 + IRONIC_BUILD_DEPLOY_RAMDISK is going to be a bad mix right now (and making this work would be an additional follow-up that would enable ARM IPA tempest testing)","commit_id":"29e2984cba9ad82f1a33f22c440d2ba8b8c3593a"},{"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":"e8313430a17ea6139fa2af33bf646eac9e89c012","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":48,"id":"23693f89_5bbc3573","updated":"2024-05-31 15:52:07.000000000","message":"recheck nodes strangely booted to grub without ever netbooting IPA, there\u0027s nothing in this change AFAICT that would cause this","commit_id":"109f9b4048dd9a185e316924b2ba357d2fe12271"},{"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":"d293cb721258e68c74890a1a475cd8d676f3250f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":49,"id":"f2fff422_905a9a9f","updated":"2024-06-07 17:54:54.000000000","message":"Evaluating...","commit_id":"484b96a5d5be4c024c3edeeb3b75b05a2126a37b"},{"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":"2baab6f4fae1e503a4674c95f5496e665301a50e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":49,"id":"9f119371_3fa99e35","updated":"2024-06-07 16:12:58.000000000","message":"Recheck to ensure this is indeed a failure resulting from this change!","commit_id":"484b96a5d5be4c024c3edeeb3b75b05a2126a37b"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"5edd24273d88fd50c1431bbd6f7d105e409c4112","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":52,"id":"bbbd2564_f25d5f8c","updated":"2024-06-11 20:36:02.000000000","message":"recheck ci hiccup?","commit_id":"4a80df586b41605fe7c7aca4d2ca747850ca0f8b"}],"devstack/common_settings":[{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"434c6de2773cac699033e479e2a28184d1f214b0","unresolved":true,"context_lines":[{"line_number":35,"context_line":"# Do not restrict downloading image only for specific case. Download both disk and uec images."},{"line_number":36,"context_line":"# NOTE (vdrok): Here the images are actually pre-cached by devstack, in"},{"line_number":37,"context_line":"# the files folder, so they won\u0027t be downloaded again."},{"line_number":38,"context_line":"if [[ $IRONIC_HW_ARCH !\u003d \"aarch64\" ]]; then"},{"line_number":39,"context_line":"    add_image_link http://download.cirros-cloud.net/${CIRROS_VERSION}/cirros-${CIRROS_VERSION}-x86_64-uec.tar.gz"},{"line_number":40,"context_line":"    add_image_link http://download.cirros-cloud.net/${CIRROS_VERSION}/cirros-${CIRROS_VERSION}-x86_64-disk.img"},{"line_number":41,"context_line":"else"}],"source_content_type":"application/x-shellscript","patch_set":52,"id":"3d4f9362_3fdf49dc","line":38,"updated":"2024-06-12 07:57:48.000000000","message":"nit: I would rather use a positive check than a negative one, easier to read IMHO","commit_id":"4a80df586b41605fe7c7aca4d2ca747850ca0f8b"},{"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":"22b0aeb303e0327005fbbd85d441da55ab36cf58","unresolved":false,"context_lines":[{"line_number":35,"context_line":"# Do not restrict downloading image only for specific case. Download both disk and uec images."},{"line_number":36,"context_line":"# NOTE (vdrok): Here the images are actually pre-cached by devstack, in"},{"line_number":37,"context_line":"# the files folder, so they won\u0027t be downloaded again."},{"line_number":38,"context_line":"if [[ $IRONIC_HW_ARCH !\u003d \"aarch64\" ]]; then"},{"line_number":39,"context_line":"    add_image_link http://download.cirros-cloud.net/${CIRROS_VERSION}/cirros-${CIRROS_VERSION}-x86_64-uec.tar.gz"},{"line_number":40,"context_line":"    add_image_link http://download.cirros-cloud.net/${CIRROS_VERSION}/cirros-${CIRROS_VERSION}-x86_64-disk.img"},{"line_number":41,"context_line":"else"}],"source_content_type":"application/x-shellscript","patch_set":52,"id":"a55beb00_b0053c39","line":38,"in_reply_to":"3d4f9362_3fdf49dc","updated":"2024-06-12 11:46:20.000000000","message":"Acknowledged","commit_id":"4a80df586b41605fe7c7aca4d2ca747850ca0f8b"}],"devstack/lib/ironic":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"4730dc75080f84ce6b419461c727b51a01e65981","unresolved":true,"context_lines":[{"line_number":151,"context_line":"IRONIC_VM_COUNT\u003d${IRONIC_VM_COUNT:-1}"},{"line_number":152,"context_line":"IRONIC_VM_SPECS_CPU\u003d${IRONIC_VM_SPECS_CPU:-1}"},{"line_number":153,"context_line":"IRONIC_VM_SPECS_RAM\u003d${IRONIC_VM_SPECS_RAM:-3072}"},{"line_number":154,"context_line":"IRONIC_VM_SPECS_CPU_ARCH\u003d${IRONIC_VM_SPECS_CPU_ARCH:-\u0027x86_64\u0027}"},{"line_number":155,"context_line":"IRONIC_VM_SPECS_DISK\u003d${IRONIC_VM_SPECS_DISK:-10}"},{"line_number":156,"context_line":"IRONIC_VM_SPECS_DISK_FORMAT\u003d${IRONIC_VM_SPECS_DISK_FORMAT:-qcow2}"},{"line_number":157,"context_line":"IRONIC_VM_EPHEMERAL_DISK\u003d${IRONIC_VM_EPHEMERAL_DISK:-0}"}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"cb5bf449_ce1ce75e","side":"PARENT","line":154,"updated":"2024-04-24 19:10:41.000000000","message":"We likely want to keep this as-is, and just create a single job or two which leverages arm.","commit_id":"c1f3daf7b0006bd555b950a0cd0dfe8a04878ec7"},{"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":"1c851b60a5fef764b311c701163c06b638f4001d","unresolved":false,"context_lines":[{"line_number":151,"context_line":"IRONIC_VM_COUNT\u003d${IRONIC_VM_COUNT:-1}"},{"line_number":152,"context_line":"IRONIC_VM_SPECS_CPU\u003d${IRONIC_VM_SPECS_CPU:-1}"},{"line_number":153,"context_line":"IRONIC_VM_SPECS_RAM\u003d${IRONIC_VM_SPECS_RAM:-3072}"},{"line_number":154,"context_line":"IRONIC_VM_SPECS_CPU_ARCH\u003d${IRONIC_VM_SPECS_CPU_ARCH:-\u0027x86_64\u0027}"},{"line_number":155,"context_line":"IRONIC_VM_SPECS_DISK\u003d${IRONIC_VM_SPECS_DISK:-10}"},{"line_number":156,"context_line":"IRONIC_VM_SPECS_DISK_FORMAT\u003d${IRONIC_VM_SPECS_DISK_FORMAT:-qcow2}"},{"line_number":157,"context_line":"IRONIC_VM_EPHEMERAL_DISK\u003d${IRONIC_VM_EPHEMERAL_DISK:-0}"}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"73cc2f3b_6ea79690","side":"PARENT","line":154,"in_reply_to":"cb5bf449_ce1ce75e","updated":"2024-05-22 09:10:35.000000000","message":"Done","commit_id":"c1f3daf7b0006bd555b950a0cd0dfe8a04878ec7"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"4730dc75080f84ce6b419461c727b51a01e65981","unresolved":true,"context_lines":[{"line_number":155,"context_line":"IRONIC_VM_SPECS_DISK\u003d${IRONIC_VM_SPECS_DISK:-10}"},{"line_number":156,"context_line":"IRONIC_VM_SPECS_DISK_FORMAT\u003d${IRONIC_VM_SPECS_DISK_FORMAT:-qcow2}"},{"line_number":157,"context_line":"IRONIC_VM_EPHEMERAL_DISK\u003d${IRONIC_VM_EPHEMERAL_DISK:-0}"},{"line_number":158,"context_line":"IRONIC_VM_EMULATOR\u003d${IRONIC_VM_EMULATOR:-\u0027/usr/bin/qemu-efi-aarch64\u0027}"},{"line_number":159,"context_line":"IRONIC_VM_ENGINE\u003d${IRONIC_VM_ENGINE:-qemu}"},{"line_number":160,"context_line":"IRONIC_VM_NETWORK_BRIDGE\u003d${IRONIC_VM_NETWORK_BRIDGE:-brbm}"},{"line_number":161,"context_line":"IRONIC_VM_INTERFACE_COUNT\u003d${IRONIC_VM_INTERFACE_COUNT:-2}"}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"fc473ac5_06a4fdf9","line":158,"updated":"2024-04-24 19:10:41.000000000","message":"I\u0027m not entirely sure this is the right parameter, at *least* on debian.\n\nAfter installing the packages: \n- qemu-efi-aarch64\n- qemu-system-arm\n\n\nI get /usr/bin/qemu-system-aarch64 as the only package which provides a runtime.\n\nI guess the secret is the arm64 architecture is it should all be EFI\u0027ed.","commit_id":"a2f1a60f21f133cae6e02cc582b50f5da95c72de"},{"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":"bc5cf7e5ff86f010899cba54003394722258d7dc","unresolved":false,"context_lines":[{"line_number":155,"context_line":"IRONIC_VM_SPECS_DISK\u003d${IRONIC_VM_SPECS_DISK:-10}"},{"line_number":156,"context_line":"IRONIC_VM_SPECS_DISK_FORMAT\u003d${IRONIC_VM_SPECS_DISK_FORMAT:-qcow2}"},{"line_number":157,"context_line":"IRONIC_VM_EPHEMERAL_DISK\u003d${IRONIC_VM_EPHEMERAL_DISK:-0}"},{"line_number":158,"context_line":"IRONIC_VM_EMULATOR\u003d${IRONIC_VM_EMULATOR:-\u0027/usr/bin/qemu-efi-aarch64\u0027}"},{"line_number":159,"context_line":"IRONIC_VM_ENGINE\u003d${IRONIC_VM_ENGINE:-qemu}"},{"line_number":160,"context_line":"IRONIC_VM_NETWORK_BRIDGE\u003d${IRONIC_VM_NETWORK_BRIDGE:-brbm}"},{"line_number":161,"context_line":"IRONIC_VM_INTERFACE_COUNT\u003d${IRONIC_VM_INTERFACE_COUNT:-2}"}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"22c5efb7_ce09d59d","line":158,"in_reply_to":"fc473ac5_06a4fdf9","updated":"2024-05-22 13:03:58.000000000","message":"Acknowledged","commit_id":"a2f1a60f21f133cae6e02cc582b50f5da95c72de"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"d076e20ca0aeb45bd392cda464ab35240bf90853","unresolved":true,"context_lines":[{"line_number":673,"context_line":"        install_package shim"},{"line_number":674,"context_line":"        install_package grub-efi-amd64-signed shim-signed"},{"line_number":675,"context_line":"    elif [[ $IRONIC_HW_ARCH \u003d\u003d \"aarch64\" ]]; then"},{"line_number":676,"context_line":"        install_package qemu-system-arm qemu-efi qemu-efi-aarch64 qemu-efi-aarch64-signed shim-signed"},{"line_number":677,"context_line":"    fi"},{"line_number":678,"context_line":"elif is_fedora; then"},{"line_number":679,"context_line":"    if [[ $IRONIC_HW_ARCH \u003d\u003d \"x86_64\" ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":10,"id":"d1078bbb_90a4c67f","line":676,"updated":"2024-04-26 12:29:20.000000000","message":"there\u0027s no qemu-efi-aarch64-signed package\nI think you need grub-efi-arm64-signed instead","commit_id":"e774c02d570fbe999ab70a1443f64148a1182b7b"},{"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":"71123537daf2d2c877e1c9a3b40aca02b659fe82","unresolved":false,"context_lines":[{"line_number":673,"context_line":"        install_package shim"},{"line_number":674,"context_line":"        install_package grub-efi-amd64-signed shim-signed"},{"line_number":675,"context_line":"    elif [[ $IRONIC_HW_ARCH \u003d\u003d \"aarch64\" ]]; then"},{"line_number":676,"context_line":"        install_package qemu-system-arm qemu-efi qemu-efi-aarch64 qemu-efi-aarch64-signed shim-signed"},{"line_number":677,"context_line":"    fi"},{"line_number":678,"context_line":"elif is_fedora; then"},{"line_number":679,"context_line":"    if [[ $IRONIC_HW_ARCH \u003d\u003d \"x86_64\" ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":10,"id":"82d3d68b_5e66396e","line":676,"in_reply_to":"59887041_affa6e6e","updated":"2024-05-07 15:02:20.000000000","message":"Acknowledged","commit_id":"e774c02d570fbe999ab70a1443f64148a1182b7b"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"f689f65d8ef94ae0e8e1f221431c67906237a719","unresolved":true,"context_lines":[{"line_number":673,"context_line":"        install_package shim"},{"line_number":674,"context_line":"        install_package grub-efi-amd64-signed shim-signed"},{"line_number":675,"context_line":"    elif [[ $IRONIC_HW_ARCH \u003d\u003d \"aarch64\" ]]; then"},{"line_number":676,"context_line":"        install_package qemu-system-arm qemu-efi qemu-efi-aarch64 qemu-efi-aarch64-signed shim-signed"},{"line_number":677,"context_line":"    fi"},{"line_number":678,"context_line":"elif is_fedora; then"},{"line_number":679,"context_line":"    if [[ $IRONIC_HW_ARCH \u003d\u003d \"x86_64\" ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":10,"id":"59887041_affa6e6e","line":676,"in_reply_to":"d1078bbb_90a4c67f","updated":"2024-04-26 13:42:32.000000000","message":"++\n\nError from the logs:\n\n2024-04-26 09:31:28.980928 | controller | Reading state information...\n2024-04-26 09:31:29.000933 | controller | E: Unable to locate package qemu-efi-aarch64-signed\n2024-04-26 09:31:29.006746 | controller | + functions-common:real_install_package:1  :   exit_trap\n2024-04-26 09:31:29.009406 | controller | + ./stack.sh:exit_trap:549                 :   local r\u003d100","commit_id":"e774c02d570fbe999ab70a1443f64148a1182b7b"},{"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":"b602fb0e645614e43a89d85afb2d9d624216ed5e","unresolved":true,"context_lines":[{"line_number":354,"context_line":"            IRONIC_AGENT_RAMDISK_FILE\u003dipa-${IRONIC_DIB_RAMDISK_OS}-${IPA_DOWNLOAD_BRANCH}.initramfs"},{"line_number":355,"context_line":"        ;;"},{"line_number":356,"context_line":"    esac"},{"line_number":357,"context_line":"    IRONIC_AGENT_KERNEL_URL\u003dhttps://tarballs.openstack.org/${$IPA_DOWNLOAD_SOURCE}/${IRONIC_RAMDISK_TYPE}/files/${IRONIC_AGENT_KERNEL_FILE}"},{"line_number":358,"context_line":"    IRONIC_AGENT_RAMDISK_URL\u003dhttps://tarballs.openstack.org/${$IPA_DOWNLOAD_SOURCE}/${IRONIC_RAMDISK_TYPE}/files/${IRONIC_AGENT_RAMDISK_FILE}"},{"line_number":359,"context_line":"fi"},{"line_number":360,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":26,"id":"edf10387_ece3e4a6","line":357,"updated":"2024-05-16 01:57:05.000000000","message":"${IPA_DOWNLOAD_SOURCE}\n\nThere is a duplicated $ inside the curly braces. Same in the line below. This is your CI failure","commit_id":"4cc77d5dd36905f8d7bd16aeb74382cc24572aa5"},{"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":"bc5cf7e5ff86f010899cba54003394722258d7dc","unresolved":false,"context_lines":[{"line_number":354,"context_line":"            IRONIC_AGENT_RAMDISK_FILE\u003dipa-${IRONIC_DIB_RAMDISK_OS}-${IPA_DOWNLOAD_BRANCH}.initramfs"},{"line_number":355,"context_line":"        ;;"},{"line_number":356,"context_line":"    esac"},{"line_number":357,"context_line":"    IRONIC_AGENT_KERNEL_URL\u003dhttps://tarballs.openstack.org/${$IPA_DOWNLOAD_SOURCE}/${IRONIC_RAMDISK_TYPE}/files/${IRONIC_AGENT_KERNEL_FILE}"},{"line_number":358,"context_line":"    IRONIC_AGENT_RAMDISK_URL\u003dhttps://tarballs.openstack.org/${$IPA_DOWNLOAD_SOURCE}/${IRONIC_RAMDISK_TYPE}/files/${IRONIC_AGENT_RAMDISK_FILE}"},{"line_number":359,"context_line":"fi"},{"line_number":360,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":26,"id":"41aecaf0_e09e0ba2","line":357,"in_reply_to":"edf10387_ece3e4a6","updated":"2024-05-22 13:03:58.000000000","message":"Acknowledged","commit_id":"4cc77d5dd36905f8d7bd16aeb74382cc24572aa5"},{"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":"d09be67d4842cefe3312d21375c49119b88a165d","unresolved":true,"context_lines":[{"line_number":1365,"context_line":"                    fi"},{"line_number":1366,"context_line":"                elif [[ $IRONIC_HW_ARCH \u003d\u003d \"aarch64\" ]]; then"},{"line_number":1367,"context_line":"                    # NOTE(cid): Ensure ARM-specific binaries are prepared if they don\u0027t already exist"},{"line_number":1368,"context_line":"                    if [[ ! -f /usr/lib/ipxe/arm/snponly.efi ]]; then"},{"line_number":1369,"context_line":"                        git clone https://github.com/ipxe/ipxe.git"},{"line_number":1370,"context_line":"                        cd ipxe/src"},{"line_number":1371,"context_line":"                        make bin-arm64-efi/snponly.efi"}],"source_content_type":"application/x-shellscript","patch_set":32,"id":"eec8c085_f2ce6720","line":1368,"updated":"2024-05-20 19:55:15.000000000","message":"This isn\u0027t really going to work well in devstack because you\u0027d have to install a cross-compilation toolkit (the compilers on an x86 VM won\u0027t be able to build ARM without special effort).\n\nThis is not a long-term fix, but if you want to get past this problem, it looks like there are prebuilt binaries here: http://boot.ipxe.org/arm64-efi/ you could use to get closer to a prototype. We\u0027ll have to update it to not pull from an unknown source before it lands (probably), but it\u0027s valuable to get to a prototype faster.","commit_id":"9eb687651963ba8884d6eb693bd8f322ea51db0a"},{"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":"1c851b60a5fef764b311c701163c06b638f4001d","unresolved":false,"context_lines":[{"line_number":1365,"context_line":"                    fi"},{"line_number":1366,"context_line":"                elif [[ $IRONIC_HW_ARCH \u003d\u003d \"aarch64\" ]]; then"},{"line_number":1367,"context_line":"                    # NOTE(cid): Ensure ARM-specific binaries are prepared if they don\u0027t already exist"},{"line_number":1368,"context_line":"                    if [[ ! -f /usr/lib/ipxe/arm/snponly.efi ]]; then"},{"line_number":1369,"context_line":"                        git clone https://github.com/ipxe/ipxe.git"},{"line_number":1370,"context_line":"                        cd ipxe/src"},{"line_number":1371,"context_line":"                        make bin-arm64-efi/snponly.efi"}],"source_content_type":"application/x-shellscript","patch_set":32,"id":"6c7f0320_8a4b476b","line":1368,"in_reply_to":"eec8c085_f2ce6720","updated":"2024-05-22 09:10:35.000000000","message":"Acknowledged","commit_id":"9eb687651963ba8884d6eb693bd8f322ea51db0a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"a06978af37893b8a49e65ed3b2c60eaefddeeb09","unresolved":true,"context_lines":[{"line_number":304,"context_line":"# NOTE(TheJulia): If we ever run any arm64, we will need to consider doing"},{"line_number":305,"context_line":"# the same. Nested virt is not a thing there."},{"line_number":306,"context_line":"# Prevent a case that will likely result in a failure."},{"line_number":307,"context_line":"if [[ $IRONIC_HW_ARCH !\u003d \"aarch64\" ]]; then"},{"line_number":308,"context_line":"    if [[ \"$hostdomain\" \u003d~ \"rax\" ]] || [[ \"$hostdomain\" \u003d~ \"iweb\" ]] || ! $(grep -q vmx /proc/cpuinfo) ; then"},{"line_number":309,"context_line":"        if [[ \"$IRONIC_RAMDISK_TYPE\" \u003d\u003d \"dib\" ]]; then"},{"line_number":310,"context_line":"            echo \"** WARNING ** - DIB based IPA images have been defined, however we are running devstack on an environment which does not support nested VMs. Due to virtualization constraints, we are automatically falling back to TinyIPA to ensure CI job passage.\""}],"source_content_type":"application/x-shellscript","patch_set":45,"id":"ce1924be_8f972cac","line":307,"updated":"2024-05-29 17:24:36.000000000","message":"I\u0027m slightly worried, just from past experience, that we might need to not exclude this logic. That being said, it makes sense in this case as well, I guess time will tell.","commit_id":"29e2984cba9ad82f1a33f22c440d2ba8b8c3593a"},{"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":"cc7b3cca154ebf8c1201288442c91ee049610e5f","unresolved":true,"context_lines":[{"line_number":304,"context_line":"# NOTE(TheJulia): If we ever run any arm64, we will need to consider doing"},{"line_number":305,"context_line":"# the same. Nested virt is not a thing there."},{"line_number":306,"context_line":"# Prevent a case that will likely result in a failure."},{"line_number":307,"context_line":"if [[ $IRONIC_HW_ARCH !\u003d \"aarch64\" ]]; then"},{"line_number":308,"context_line":"    if [[ \"$hostdomain\" \u003d~ \"rax\" ]] || [[ \"$hostdomain\" \u003d~ \"iweb\" ]] || ! $(grep -q vmx /proc/cpuinfo) ; then"},{"line_number":309,"context_line":"        if [[ \"$IRONIC_RAMDISK_TYPE\" \u003d\u003d \"dib\" ]]; then"},{"line_number":310,"context_line":"            echo \"** WARNING ** - DIB based IPA images have been defined, however we are running devstack on an environment which does not support nested VMs. Due to virtualization constraints, we are automatically falling back to TinyIPA to ensure CI job passage.\""}],"source_content_type":"application/x-shellscript","patch_set":45,"id":"8c810fec_55fe0e71","line":307,"in_reply_to":"6db267f4_dde06a35","updated":"2024-06-05 18:27:48.000000000","message":"Erm, that will actually change everything. So, maybe not!","commit_id":"29e2984cba9ad82f1a33f22c440d2ba8b8c3593a"},{"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":"e21b41445b94b86b3646f6302372da05de835971","unresolved":true,"context_lines":[{"line_number":304,"context_line":"# NOTE(TheJulia): If we ever run any arm64, we will need to consider doing"},{"line_number":305,"context_line":"# the same. Nested virt is not a thing there."},{"line_number":306,"context_line":"# Prevent a case that will likely result in a failure."},{"line_number":307,"context_line":"if [[ $IRONIC_HW_ARCH !\u003d \"aarch64\" ]]; then"},{"line_number":308,"context_line":"    if [[ \"$hostdomain\" \u003d~ \"rax\" ]] || [[ \"$hostdomain\" \u003d~ \"iweb\" ]] || ! $(grep -q vmx /proc/cpuinfo) ; then"},{"line_number":309,"context_line":"        if [[ \"$IRONIC_RAMDISK_TYPE\" \u003d\u003d \"dib\" ]]; then"},{"line_number":310,"context_line":"            echo \"** WARNING ** - DIB based IPA images have been defined, however we are running devstack on an environment which does not support nested VMs. Due to virtualization constraints, we are automatically falling back to TinyIPA to ensure CI job passage.\""}],"source_content_type":"application/x-shellscript","patch_set":45,"id":"fef2ee6f_4a60686b","line":307,"in_reply_to":"ce1924be_8f972cac","updated":"2024-05-30 11:14:50.000000000","message":"There is no ARM-based TinyIPA. That\u0027s the reason we need to exclude this code. We can trivially make an ARM-based gentoo ramdisk if/when we head down that path.","commit_id":"29e2984cba9ad82f1a33f22c440d2ba8b8c3593a"},{"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":"07721c9ed2e064ce8f6ebe1a0899da6de2f3fb51","unresolved":true,"context_lines":[{"line_number":304,"context_line":"# NOTE(TheJulia): If we ever run any arm64, we will need to consider doing"},{"line_number":305,"context_line":"# the same. Nested virt is not a thing there."},{"line_number":306,"context_line":"# Prevent a case that will likely result in a failure."},{"line_number":307,"context_line":"if [[ $IRONIC_HW_ARCH !\u003d \"aarch64\" ]]; then"},{"line_number":308,"context_line":"    if [[ \"$hostdomain\" \u003d~ \"rax\" ]] || [[ \"$hostdomain\" \u003d~ \"iweb\" ]] || ! $(grep -q vmx /proc/cpuinfo) ; then"},{"line_number":309,"context_line":"        if [[ \"$IRONIC_RAMDISK_TYPE\" \u003d\u003d \"dib\" ]]; then"},{"line_number":310,"context_line":"            echo \"** WARNING ** - DIB based IPA images have been defined, however we are running devstack on an environment which does not support nested VMs. Due to virtualization constraints, we are automatically falling back to TinyIPA to ensure CI job passage.\""}],"source_content_type":"application/x-shellscript","patch_set":45,"id":"6db267f4_dde06a35","line":307,"in_reply_to":"fef2ee6f_4a60686b","updated":"2024-06-05 18:19:40.000000000","message":"I suspect the current failure of the ``ironic-tempest-ipa-partition-uefi-pxe-grub2`` job may be from this. Not sure.\n\nSo, I\u0027m going to re-upload the change without this and see.","commit_id":"29e2984cba9ad82f1a33f22c440d2ba8b8c3593a"},{"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":"9f883898041a309c34b1c2c9ca297e15ca6c0e43","unresolved":true,"context_lines":[{"line_number":74,"context_line":"IRONIC_AUTH_CACHE_DIR\u003d${IRONIC_AUTH_CACHE_DIR:-/var/cache/ironic}"},{"line_number":75,"context_line":"IRONIC_CONF_DIR\u003d${IRONIC_CONF_DIR:-/etc/ironic}"},{"line_number":76,"context_line":"IRONIC_CONF_FILE\u003d$IRONIC_CONF_DIR/ironic.conf"},{"line_number":77,"context_line":"IRONIC_ARCH_MACHINE_TYPE\u003d${IRONIC_ARCH_MACHINE_TYPE:-q35}"},{"line_number":78,"context_line":"# Deploy Ironic API under uwsgi (NOT mod_wsgi) server."},{"line_number":79,"context_line":"# Devstack aims to remove mod_wsgi support, so ironic shouldn\u0027t use it too."},{"line_number":80,"context_line":"# If set to False that will fall back to use the eventlet server that"}],"source_content_type":"application/x-shellscript","patch_set":51,"id":"589fd497_5c83596a","line":77,"updated":"2024-06-11 14:54:11.000000000","message":"nit: you probably want this to be IRONIC_AARCH_MACHINE_TYPE or IRONIC_AARCH64_MACHINE_TYPE? (throughout)","commit_id":"88ffc60dd8a069472ad6b279013f8e08103bbb13"},{"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":"7ee7d642eddc9d0788f6454d948556aa6a5eeadc","unresolved":false,"context_lines":[{"line_number":74,"context_line":"IRONIC_AUTH_CACHE_DIR\u003d${IRONIC_AUTH_CACHE_DIR:-/var/cache/ironic}"},{"line_number":75,"context_line":"IRONIC_CONF_DIR\u003d${IRONIC_CONF_DIR:-/etc/ironic}"},{"line_number":76,"context_line":"IRONIC_CONF_FILE\u003d$IRONIC_CONF_DIR/ironic.conf"},{"line_number":77,"context_line":"IRONIC_ARCH_MACHINE_TYPE\u003d${IRONIC_ARCH_MACHINE_TYPE:-q35}"},{"line_number":78,"context_line":"# Deploy Ironic API under uwsgi (NOT mod_wsgi) server."},{"line_number":79,"context_line":"# Devstack aims to remove mod_wsgi support, so ironic shouldn\u0027t use it too."},{"line_number":80,"context_line":"# If set to False that will fall back to use the eventlet server that"}],"source_content_type":"application/x-shellscript","patch_set":51,"id":"32772aa4_1b4e80d4","line":77,"in_reply_to":"589fd497_5c83596a","updated":"2024-06-11 15:14:19.000000000","message":"Acknowledged","commit_id":"88ffc60dd8a069472ad6b279013f8e08103bbb13"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"434c6de2773cac699033e479e2a28184d1f214b0","unresolved":true,"context_lines":[{"line_number":729,"context_line":"                IRONIC_GRUB2_FILE\u003d/boot/efi/EFI/fedora/grubaa64.efi"},{"line_number":730,"context_line":"                IRONIC_GRUB2_NETWORK_FILE\u003d/boot/efi/EFI/fedora/grubaa64.efi"},{"line_number":731,"context_line":"            fi"},{"line_number":732,"context_line":"        fi"},{"line_number":733,"context_line":"    fi"},{"line_number":734,"context_line":""},{"line_number":735,"context_line":"    if [[ \"$IRONIC_IPXE_ENABLED\" \u003d\u003d \"False\" ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":52,"id":"3bb0597c_e2ea279f","line":732,"updated":"2024-06-12 07:57:48.000000000","message":"since we\u0027re explicitely mentioning 2 architectures, should we add an \"unsupported\" architecture message and exit?\nmaybe not here but at the very beginning right after IRONIC_HW_ARCH is defined","commit_id":"4a80df586b41605fe7c7aca4d2ca747850ca0f8b"},{"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":"22b0aeb303e0327005fbbd85d441da55ab36cf58","unresolved":true,"context_lines":[{"line_number":729,"context_line":"                IRONIC_GRUB2_FILE\u003d/boot/efi/EFI/fedora/grubaa64.efi"},{"line_number":730,"context_line":"                IRONIC_GRUB2_NETWORK_FILE\u003d/boot/efi/EFI/fedora/grubaa64.efi"},{"line_number":731,"context_line":"            fi"},{"line_number":732,"context_line":"        fi"},{"line_number":733,"context_line":"    fi"},{"line_number":734,"context_line":""},{"line_number":735,"context_line":"    if [[ \"$IRONIC_IPXE_ENABLED\" \u003d\u003d \"False\" ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":52,"id":"91ad2acb_ae94cafc","line":732,"in_reply_to":"3bb0597c_e2ea279f","updated":"2024-06-12 11:46:20.000000000","message":"Right!","commit_id":"4a80df586b41605fe7c7aca4d2ca747850ca0f8b"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":74,"context_line":"IRONIC_AUTH_CACHE_DIR\u003d${IRONIC_AUTH_CACHE_DIR:-/var/cache/ironic}"},{"line_number":75,"context_line":"IRONIC_CONF_DIR\u003d${IRONIC_CONF_DIR:-/etc/ironic}"},{"line_number":76,"context_line":"IRONIC_CONF_FILE\u003d$IRONIC_CONF_DIR/ironic.conf"},{"line_number":77,"context_line":"IRONIC_AARCH64_MACHINE_TYPE\u003d${IRONIC_AARCH64_MACHINE_TYPE:-q35}"},{"line_number":78,"context_line":"# Deploy Ironic API under uwsgi (NOT mod_wsgi) server."},{"line_number":79,"context_line":"# Devstack aims to remove mod_wsgi support, so ironic shouldn\u0027t use it too."},{"line_number":80,"context_line":"# If set to False that will fall back to use the eventlet server that"}],"source_content_type":"application/x-shellscript","patch_set":54,"id":"81a9b59c_f09a7f63","line":77,"updated":"2024-06-17 08:41:37.000000000","message":"Does q35 even work for aarch? I thought you need to use \"virt\" or \"virt-x.y\".","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"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":"0cc728a0413467f7ddd98e4f117f0fda9e067b02","unresolved":true,"context_lines":[{"line_number":74,"context_line":"IRONIC_AUTH_CACHE_DIR\u003d${IRONIC_AUTH_CACHE_DIR:-/var/cache/ironic}"},{"line_number":75,"context_line":"IRONIC_CONF_DIR\u003d${IRONIC_CONF_DIR:-/etc/ironic}"},{"line_number":76,"context_line":"IRONIC_CONF_FILE\u003d$IRONIC_CONF_DIR/ironic.conf"},{"line_number":77,"context_line":"IRONIC_AARCH64_MACHINE_TYPE\u003d${IRONIC_AARCH64_MACHINE_TYPE:-q35}"},{"line_number":78,"context_line":"# Deploy Ironic API under uwsgi (NOT mod_wsgi) server."},{"line_number":79,"context_line":"# Devstack aims to remove mod_wsgi support, so ironic shouldn\u0027t use it too."},{"line_number":80,"context_line":"# If set to False that will fall back to use the eventlet server that"}],"source_content_type":"application/x-shellscript","patch_set":54,"id":"8a849da8_8984fbe1","line":77,"in_reply_to":"81a9b59c_f09a7f63","updated":"2024-06-17 12:32:13.000000000","message":"I think the naming might be confusing because we still need x86_64-compatible defaults like the q35 as is. I\u0027ll probably return it to the previous state.\n\nFYI: I did s/IRONIC_MACHINE_TYPE/IRONIC_AARCH64_MACHINE_TYPE/g","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":268,"context_line":"        echo \"WARNING: Switching to QEMU\""},{"line_number":269,"context_line":"        IRONIC_VM_ENGINE\u003dqemu"},{"line_number":270,"context_line":"        if [[ -z \"$IRONIC_VM_EMULATOR\" ]]; then"},{"line_number":271,"context_line":"            if [[ $IRONIC_HW_ARCH \u003d\u003d \"aarch64\" ]]; then"},{"line_number":272,"context_line":"                IRONIC_VM_EMULATOR\u003d\u0027/usr/bin/qemu-system-aarch64\u0027"},{"line_number":273,"context_line":"            else"},{"line_number":274,"context_line":"                IRONIC_VM_EMULATOR\u003d\u0027/usr/bin/qemu-system-x86_64\u0027"}],"source_content_type":"application/x-shellscript","patch_set":54,"id":"fe77eaf4_db67c95b","line":271,"updated":"2024-06-17 08:41:37.000000000","message":"IRONIC_VM_EMULATOR\u003d\"/usr/bin/qemu-system-${IRONIC_HW_ARCH:-x86_64}\"\n\n(many similar cases below)","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":691,"context_line":"        install_package grub2 grub2-efi"},{"line_number":692,"context_line":"    fi"},{"line_number":693,"context_line":"elif [[ $IRONIC_HW_ARCH \u003d\u003d \"aarch64\" ]]; then"},{"line_number":694,"context_line":"    install_package shim"},{"line_number":695,"context_line":"    if is_ubuntu; then"},{"line_number":696,"context_line":"        install_package qemu-system-arm qemu-efi qemu-efi-aarch64 shim-signed"},{"line_number":697,"context_line":"    elif is_fedora; then"}],"source_content_type":"application/x-shellscript","patch_set":54,"id":"185bf9d3_83d8b2f6","line":694,"updated":"2024-06-17 08:41:37.000000000","message":"nit: This command is common for both cases","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":1184,"context_line":"    local ironic_node_arch\u003d\"x86_64\""},{"line_number":1185,"context_line":"    if [[ $IRONIC_HW_ARCH \u003d\u003d \"aarch64\" ]]; then"},{"line_number":1186,"context_line":"        ironic_node_arch\u003d\"aarch64\""},{"line_number":1187,"context_line":"    fi"},{"line_number":1188,"context_line":""},{"line_number":1189,"context_line":"    cat - \u003c\u003cEOF \u003e $IRONIC_REDFISH_EMULATOR_CONFIG"},{"line_number":1190,"context_line":"SUSHY_EMULATOR_BOOT_LOADER_MAP \u003d {"}],"source_content_type":"application/x-shellscript","patch_set":54,"id":"a79780e9_31269780","line":1187,"updated":"2024-06-17 08:41:37.000000000","message":"It looks like this and many other conditions in this patch could be replaced by a variable substitution of ${IRONIC_HW_ARCH:-x86_64}","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":3084,"context_line":"        IRONIC_PARTITIONED_IMAGE_NAME\u003dcirros-${CIRROS_VERSION}-x86_64-partition"},{"line_number":3085,"context_line":"        if [[ $IRONIC_HW_ARCH \u003d\u003d \"aarch64\" ]]; then"},{"line_number":3086,"context_line":"            IRONIC_PARTITIONED_IMAGE_NAME\u003dcirros-${CIRROS_VERSION}-aarch64-partition"},{"line_number":3087,"context_line":"        fi"},{"line_number":3088,"context_line":"        dest\u003d\"$IRONIC_DATA_DIR/$IRONIC_PARTITIONED_IMAGE_NAME.img\""},{"line_number":3089,"context_line":""},{"line_number":3090,"context_line":"        # Export some variables that the script is using."}],"source_content_type":"application/x-shellscript","patch_set":54,"id":"1be3e031_662cc1f8","line":3087,"updated":"2024-06-17 08:41:37.000000000","message":"Like here as well","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":3150,"context_line":""},{"line_number":3151,"context_line":"        if [[ $IRONIC_HW_ARCH \u003d\u003d \"aarch64\" ]]; then"},{"line_number":3152,"context_line":"            if [[ \"$IRONIC_GRUB2_SHIM_FILE\" \u003d~ \"http\".* ]]; then"},{"line_number":3153,"context_line":"                sudo wget \"$IRONIC_GRUB2_SHIM_FILE\" -O $efiboot_mount/efi/boot/bootaa64.efi"},{"line_number":3154,"context_line":"            else"},{"line_number":3155,"context_line":"                sudo cp \"$IRONIC_GRUB2_SHIM_FILE\" $efiboot_mount/efi/boot/bootaa64.efi"},{"line_number":3156,"context_line":"            fi"}],"source_content_type":"application/x-shellscript","patch_set":54,"id":"999f56f8_0c301b82","line":3153,"updated":"2024-06-17 08:41:37.000000000","message":"In the similar spirit, you can define a new variable like GRUB_ARCH\u003dx64/aa64 and then use it instead of nested conditions.","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"}],"devstack/tools/ironic/scripts/configure-vm.py":[{"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":"05f62029766d453ae276158d80c5b71bdcf3e7e3","unresolved":true,"context_lines":[{"line_number":38,"context_line":"    \u003c/serial\u003e"},{"line_number":39,"context_line":"    \u003cconsole type\u003d\u0027file\u0027\u003e"},{"line_number":40,"context_line":"      \u003csource path\u003d\u0027%(console_log)s\u0027/\u003e"},{"line_number":41,"context_line":"      \u003ctarget type\u003d\u0027serial\u0027 port\u003d\u00270\u0027/\u003e"},{"line_number":42,"context_line":"      \u003calias name\u003d\u0027serial0\u0027/\u003e"},{"line_number":43,"context_line":"    \u003c/console\u003e"},{"line_number":44,"context_line":"\"\"\""}],"source_content_type":"text/x-python","patch_set":14,"id":"ebe1582c_01c7d53b","line":41,"updated":"2024-04-27 20:11:43.000000000","message":"the 3 consoles here are serial consoles","commit_id":"1c50343b14719bd34f1f95070ded7fce528ae2a2"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"a06978af37893b8a49e65ed3b2c60eaefddeeb09","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    \u003c/serial\u003e"},{"line_number":39,"context_line":"    \u003cconsole type\u003d\u0027file\u0027\u003e"},{"line_number":40,"context_line":"      \u003csource path\u003d\u0027%(console_log)s\u0027/\u003e"},{"line_number":41,"context_line":"      \u003ctarget type\u003d\u0027serial\u0027 port\u003d\u00270\u0027/\u003e"},{"line_number":42,"context_line":"      \u003calias name\u003d\u0027serial0\u0027/\u003e"},{"line_number":43,"context_line":"    \u003c/console\u003e"},{"line_number":44,"context_line":"\"\"\""}],"source_content_type":"text/x-python","patch_set":14,"id":"f923f066_76ee84e1","line":41,"in_reply_to":"ebe1582c_01c7d53b","updated":"2024-05-29 17:24:36.000000000","message":"I suspect this has been resolved at this point.","commit_id":"1c50343b14719bd34f1f95070ded7fce528ae2a2"},{"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":"efc02ddad2fb9fc81366e958b493bba9d058aed3","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    \u003c/serial\u003e"},{"line_number":39,"context_line":"    \u003cconsole type\u003d\u0027file\u0027\u003e"},{"line_number":40,"context_line":"      \u003csource path\u003d\u0027%(console_log)s\u0027/\u003e"},{"line_number":41,"context_line":"      \u003ctarget type\u003d\u0027serial\u0027 port\u003d\u00270\u0027/\u003e"},{"line_number":42,"context_line":"      \u003calias name\u003d\u0027serial0\u0027/\u003e"},{"line_number":43,"context_line":"    \u003c/console\u003e"},{"line_number":44,"context_line":"\"\"\""}],"source_content_type":"text/x-python","patch_set":14,"id":"c9a7d4dd_afbb4cb0","line":41,"in_reply_to":"f923f066_76ee84e1","updated":"2024-05-30 17:31:04.000000000","message":"Yes","commit_id":"1c50343b14719bd34f1f95070ded7fce528ae2a2"},{"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":"e21b41445b94b86b3646f6302372da05de835971","unresolved":true,"context_lines":[{"line_number":49,"context_line":"      \u003ctarget port\u003d\u00270\u0027/\u003e"},{"line_number":50,"context_line":"    \u003c/serial\u003e"},{"line_number":51,"context_line":"    \u003cconsole type\u003d\u0027pty\u0027\u003e"},{"line_number":52,"context_line":"      \u003ctarget port\u003d\u00270\u0027/\u003e"},{"line_number":53,"context_line":"    \u003c/console\u003e"},{"line_number":54,"context_line":"\"\"\""},{"line_number":55,"context_line":""}],"source_content_type":"text/x-python","patch_set":45,"id":"2bf2fedc_b43f64fc","line":52,"updated":"2024-05-30 11:14:50.000000000","message":"Is this change a no-op on x86 machines? I assume so?","commit_id":"29e2984cba9ad82f1a33f22c440d2ba8b8c3593a"},{"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":"efc02ddad2fb9fc81366e958b493bba9d058aed3","unresolved":true,"context_lines":[{"line_number":49,"context_line":"      \u003ctarget port\u003d\u00270\u0027/\u003e"},{"line_number":50,"context_line":"    \u003c/serial\u003e"},{"line_number":51,"context_line":"    \u003cconsole type\u003d\u0027pty\u0027\u003e"},{"line_number":52,"context_line":"      \u003ctarget port\u003d\u00270\u0027/\u003e"},{"line_number":53,"context_line":"    \u003c/console\u003e"},{"line_number":54,"context_line":"\"\"\""},{"line_number":55,"context_line":""}],"source_content_type":"text/x-python","patch_set":45,"id":"da0c2ad9_d0a7f361","line":52,"in_reply_to":"2bf2fedc_b43f64fc","updated":"2024-05-30 17:31:04.000000000","message":"Yes. Libvirt automatically applies the default (serial) on x86_64, the same as on aarch64.","commit_id":"29e2984cba9ad82f1a33f22c440d2ba8b8c3593a"},{"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":"f286cfda795743bfad1eca082a79d576e44ca90a","unresolved":false,"context_lines":[{"line_number":49,"context_line":"      \u003ctarget port\u003d\u00270\u0027/\u003e"},{"line_number":50,"context_line":"    \u003c/serial\u003e"},{"line_number":51,"context_line":"    \u003cconsole type\u003d\u0027pty\u0027\u003e"},{"line_number":52,"context_line":"      \u003ctarget port\u003d\u00270\u0027/\u003e"},{"line_number":53,"context_line":"    \u003c/console\u003e"},{"line_number":54,"context_line":"\"\"\""},{"line_number":55,"context_line":""}],"source_content_type":"text/x-python","patch_set":45,"id":"57a4f225_43979ff6","line":52,"in_reply_to":"da0c2ad9_d0a7f361","updated":"2024-05-31 10:25:38.000000000","message":"Acknowledged","commit_id":"29e2984cba9ad82f1a33f22c440d2ba8b8c3593a"}],"devstack/tools/ironic/templates/vm.xml":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"1b8de5df5d1c68127f64e7228619d5e0219ad6eb","unresolved":true,"context_lines":[{"line_number":18,"context_line":"      \u003cbios useserial\u003d\u0027yes\u0027/\u003e"},{"line_number":19,"context_line":"    {% endif %}"},{"line_number":20,"context_line":"  \u003c/os\u003e"},{"line_number":21,"context_line":"  {% if engine \u003d\u003d \u0027kvm\u0027 %}"},{"line_number":22,"context_line":"  \u003ccpu mode\u003d\u0027host-passthrough\u0027/\u003e"},{"line_number":23,"context_line":"  {% else %}"},{"line_number":24,"context_line":"  \u003ccpu mode\u003d\u0027host-model\u0027/\u003e"},{"line_number":25,"context_line":"  {% endif %}"},{"line_number":26,"context_line":"  \u003cfeatures\u003e"},{"line_number":27,"context_line":"    \u003cacpi/\u003e"},{"line_number":28,"context_line":"    \u003capic/\u003e"}],"source_content_type":"application/xml","patch_set":20,"id":"7219a645_12477801","line":25,"range":{"start_line":21,"start_character":0,"end_line":25,"end_character":13},"updated":"2024-05-07 20:21:59.000000000","message":"I suspect we can\u0027t have these options on Aarch64.\n\nhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_dd1/915441/20/check/ironic-standalone-aarch64/dd12919/controller/logs/screen-virtualbmc.txt\n\nError powering on the domain node-2. Error: unsupported configuration: CPU mode \u0027host-model\u0027 for aarch64 qemu domain on x86_64 host is not supported by hypervisor","commit_id":"d59ad3ef812f576e47137d7dec818bb3c9af74b1"},{"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":"1c851b60a5fef764b311c701163c06b638f4001d","unresolved":false,"context_lines":[{"line_number":18,"context_line":"      \u003cbios useserial\u003d\u0027yes\u0027/\u003e"},{"line_number":19,"context_line":"    {% endif %}"},{"line_number":20,"context_line":"  \u003c/os\u003e"},{"line_number":21,"context_line":"  {% if engine \u003d\u003d \u0027kvm\u0027 %}"},{"line_number":22,"context_line":"  \u003ccpu mode\u003d\u0027host-passthrough\u0027/\u003e"},{"line_number":23,"context_line":"  {% else %}"},{"line_number":24,"context_line":"  \u003ccpu mode\u003d\u0027host-model\u0027/\u003e"},{"line_number":25,"context_line":"  {% endif %}"},{"line_number":26,"context_line":"  \u003cfeatures\u003e"},{"line_number":27,"context_line":"    \u003cacpi/\u003e"},{"line_number":28,"context_line":"    \u003capic/\u003e"}],"source_content_type":"application/xml","patch_set":20,"id":"f459ae96_15e14425","line":25,"range":{"start_line":21,"start_character":0,"end_line":25,"end_character":13},"in_reply_to":"7219a645_12477801","updated":"2024-05-22 09:10:35.000000000","message":"Acknowledged","commit_id":"d59ad3ef812f576e47137d7dec818bb3c9af74b1"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"1b8de5df5d1c68127f64e7228619d5e0219ad6eb","unresolved":true,"context_lines":[{"line_number":23,"context_line":"  {% else %}"},{"line_number":24,"context_line":"  \u003ccpu mode\u003d\u0027host-model\u0027/\u003e"},{"line_number":25,"context_line":"  {% endif %}"},{"line_number":26,"context_line":"  \u003cfeatures\u003e"},{"line_number":27,"context_line":"    \u003cacpi/\u003e"},{"line_number":28,"context_line":"    \u003capic/\u003e"},{"line_number":29,"context_line":"    \u003cpae/\u003e"},{"line_number":30,"context_line":"  \u003c/features\u003e"},{"line_number":31,"context_line":"  \u003cclock offset\u003d\u0027utc\u0027/\u003e"},{"line_number":32,"context_line":"  \u003con_poweroff\u003edestroy\u003c/on_poweroff\u003e"},{"line_number":33,"context_line":"  \u003con_reboot\u003edestroy\u003c/on_reboot\u003e"}],"source_content_type":"application/xml","patch_set":20,"id":"5504a0f6_48adab3d","line":30,"range":{"start_line":26,"start_character":0,"end_line":30,"end_character":13},"updated":"2024-05-07 20:21:59.000000000","message":"These, also, may not be viable on aarch64...","commit_id":"d59ad3ef812f576e47137d7dec818bb3c9af74b1"},{"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":"1c851b60a5fef764b311c701163c06b638f4001d","unresolved":false,"context_lines":[{"line_number":23,"context_line":"  {% else %}"},{"line_number":24,"context_line":"  \u003ccpu mode\u003d\u0027host-model\u0027/\u003e"},{"line_number":25,"context_line":"  {% endif %}"},{"line_number":26,"context_line":"  \u003cfeatures\u003e"},{"line_number":27,"context_line":"    \u003cacpi/\u003e"},{"line_number":28,"context_line":"    \u003capic/\u003e"},{"line_number":29,"context_line":"    \u003cpae/\u003e"},{"line_number":30,"context_line":"  \u003c/features\u003e"},{"line_number":31,"context_line":"  \u003cclock offset\u003d\u0027utc\u0027/\u003e"},{"line_number":32,"context_line":"  \u003con_poweroff\u003edestroy\u003c/on_poweroff\u003e"},{"line_number":33,"context_line":"  \u003con_reboot\u003edestroy\u003c/on_reboot\u003e"}],"source_content_type":"application/xml","patch_set":20,"id":"d7c3b9ed_f80c1683","line":30,"range":{"start_line":26,"start_character":0,"end_line":30,"end_character":13},"in_reply_to":"5504a0f6_48adab3d","updated":"2024-05-22 09:10:35.000000000","message":"Acknowledged","commit_id":"d59ad3ef812f576e47137d7dec818bb3c9af74b1"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":24,"context_line":"    {% endif %}"},{"line_number":25,"context_line":"    {% if arch \u003d\u003d \u0027aarch64\u0027 %}"},{"line_number":26,"context_line":"    \u003ccpu mode\u003d\u0027custom\u0027 match\u003d\u0027exact\u0027 check\u003d\u0027none\u0027\u003e"},{"line_number":27,"context_line":"      \u003cmodel fallback\u003d\u0027allow\u0027\u003ecortex-a53\u003c/model\u003e"},{"line_number":28,"context_line":"    \u003c/cpu\u003e"},{"line_number":29,"context_line":"    {% endif %}"},{"line_number":30,"context_line":"  {% else %}"}],"source_content_type":"application/xml","patch_set":54,"id":"c4146705_64d40006","line":27,"updated":"2024-06-17 08:41:37.000000000","message":"I\u0027m not sure whether and why we\u0027d want to emulate A53 specifically, we could have better results with the \"maximum\" model (or how it\u0027s called)","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"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":"9c29d2050b544d1d92ee4edee345a1e3168090c2","unresolved":true,"context_lines":[{"line_number":24,"context_line":"    {% endif %}"},{"line_number":25,"context_line":"    {% if arch \u003d\u003d \u0027aarch64\u0027 %}"},{"line_number":26,"context_line":"    \u003ccpu mode\u003d\u0027custom\u0027 match\u003d\u0027exact\u0027 check\u003d\u0027none\u0027\u003e"},{"line_number":27,"context_line":"      \u003cmodel fallback\u003d\u0027allow\u0027\u003ecortex-a53\u003c/model\u003e"},{"line_number":28,"context_line":"    \u003c/cpu\u003e"},{"line_number":29,"context_line":"    {% endif %}"},{"line_number":30,"context_line":"  {% else %}"}],"source_content_type":"application/xml","patch_set":54,"id":"b962cfc4_3d96c7be","line":27,"in_reply_to":"a9f3d616_51925013","updated":"2024-06-18 18:58:09.000000000","message":"Wouldn\u0027t hurt to test that.","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"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":"82351e85ab6a6b136737cb3474d97c4ac2ecc533","unresolved":true,"context_lines":[{"line_number":24,"context_line":"    {% endif %}"},{"line_number":25,"context_line":"    {% if arch \u003d\u003d \u0027aarch64\u0027 %}"},{"line_number":26,"context_line":"    \u003ccpu mode\u003d\u0027custom\u0027 match\u003d\u0027exact\u0027 check\u003d\u0027none\u0027\u003e"},{"line_number":27,"context_line":"      \u003cmodel fallback\u003d\u0027allow\u0027\u003ecortex-a53\u003c/model\u003e"},{"line_number":28,"context_line":"    \u003c/cpu\u003e"},{"line_number":29,"context_line":"    {% endif %}"},{"line_number":30,"context_line":"  {% else %}"}],"source_content_type":"application/xml","patch_set":54,"id":"a9f3d616_51925013","line":27,"in_reply_to":"c4146705_64d40006","updated":"2024-06-17 21:45:05.000000000","message":"To some degree, I\u0027d land on \"it works and shows console output\" and suggest enhancements like this be kept for follow-up? (So CID doesn\u0027t have to rebase this patch ~forever)","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"}],"doc/source/contributor/arm-devstack-guide.rst":[{"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":"f286cfda795743bfad1eca082a79d576e44ca90a","unresolved":true,"context_lines":[{"line_number":8,"context_line":"`Deploying Ironic with DevStack \u003chttps://docs.openstack.org/ironic/latest/contributor/devstack-guide.html\u003e`_"},{"line_number":9,"context_line":"on an ARM64 architecture."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":".. _configurations:"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Configurations"},{"line_number":14,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":46,"id":"9ee72ac6_20318d88","line":11,"updated":"2024-05-31 10:25:38.000000000","message":"I think your docs failure is due to these keys needing to be globally unique (and this one is also in devstack-guide.rst). We also should ensure the arm devstack configurations are linked from the existing devstack-guide.","commit_id":"73584b5a0cb6ab5535fa04fd4b640890b4d6eadc"},{"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":"2721385561d48a0cbfa365724f649f26010c0cfc","unresolved":false,"context_lines":[{"line_number":8,"context_line":"`Deploying Ironic with DevStack \u003chttps://docs.openstack.org/ironic/latest/contributor/devstack-guide.html\u003e`_"},{"line_number":9,"context_line":"on an ARM64 architecture."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":".. _configurations:"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Configurations"},{"line_number":14,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":46,"id":"ef42e7e7_cf1b5bd1","line":11,"in_reply_to":"9ee72ac6_20318d88","updated":"2024-05-31 11:16:24.000000000","message":"Done","commit_id":"73584b5a0cb6ab5535fa04fd4b640890b4d6eadc"},{"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":"f286cfda795743bfad1eca082a79d576e44ca90a","unresolved":true,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    # The parameters below represent the minimum possible values to create"},{"line_number":41,"context_line":"    # functional aarch64-based nodes."},{"line_number":42,"context_line":"    IRONIC_VM_SPECS_RAM: 4096"},{"line_number":43,"context_line":"    IRONIC_VM_SPECS_DISK\u003d3"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    IRONIC_VM_SPECS_CPU: 1"}],"source_content_type":"text/x-rst","patch_set":46,"id":"44e11854_2940245c","line":42,"updated":"2024-05-31 10:25:38.000000000","message":"throughout this doc you mix the proper bash (KEY\u003dvalue) format with yaml-style keys CI uses (key: value)","commit_id":"73584b5a0cb6ab5535fa04fd4b640890b4d6eadc"},{"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":"2721385561d48a0cbfa365724f649f26010c0cfc","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    # The parameters below represent the minimum possible values to create"},{"line_number":41,"context_line":"    # functional aarch64-based nodes."},{"line_number":42,"context_line":"    IRONIC_VM_SPECS_RAM: 4096"},{"line_number":43,"context_line":"    IRONIC_VM_SPECS_DISK\u003d3"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    IRONIC_VM_SPECS_CPU: 1"}],"source_content_type":"text/x-rst","patch_set":46,"id":"9b8d02d6_d93827c6","line":42,"in_reply_to":"44e11854_2940245c","updated":"2024-05-31 11:16:24.000000000","message":"Acknowledged","commit_id":"73584b5a0cb6ab5535fa04fd4b640890b4d6eadc"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":1,"context_line":".. _`deploy_devstack_on_arm`:"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":"Deploying Ironic on ARM64 with DevStack"}],"source_content_type":"text/x-rst","patch_set":54,"id":"d782acb1_4cc74b37","line":1,"updated":"2024-06-17 08:41:37.000000000","message":"A reference on the first line of a file is never needed, you can always refer to a file","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":72,"context_line":"    IRONIC_AUTOMATED_CLEAN_ENABLED\u003dFalse"},{"line_number":73,"context_line":"    IRONIC_BOOT_MODE\u003duefi"},{"line_number":74,"context_line":"    IRONIC_CALLBACK_TIMEOUT\u003d800"},{"line_number":75,"context_line":"    IRONIC_RAMDISK_IMAGE\u003dhttp://dl-cdn.alpinelinux.org/alpine/v3.13/releases/aarch64/alpine-standard-3.13.5-aarch64.iso"},{"line_number":76,"context_line":"    IRONIC_GRUB2_SHIM_FILE\u003dhttps://mirror.stream.centos.org/9-stream/BaseOS/aarch64/os/EFI/BOOT/BOOTAA64.EFI"},{"line_number":77,"context_line":"    IRONIC_GRUB2_FILE\u003dhttps://mirror.stream.centos.org/9-stream/BaseOS/aarch64/os/EFI/BOOT/grubaa64.efi"},{"line_number":78,"context_line":"    IRONIC_HW_ARCH\u003daarch64"}],"source_content_type":"text/x-rst","patch_set":54,"id":"f71d4e1f_2c85a251","line":75,"updated":"2024-06-17 08:41:37.000000000","message":"This is not a ramdisk image? At least not our ramdisk?","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"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":"0cc728a0413467f7ddd98e4f117f0fda9e067b02","unresolved":true,"context_lines":[{"line_number":72,"context_line":"    IRONIC_AUTOMATED_CLEAN_ENABLED\u003dFalse"},{"line_number":73,"context_line":"    IRONIC_BOOT_MODE\u003duefi"},{"line_number":74,"context_line":"    IRONIC_CALLBACK_TIMEOUT\u003d800"},{"line_number":75,"context_line":"    IRONIC_RAMDISK_IMAGE\u003dhttp://dl-cdn.alpinelinux.org/alpine/v3.13/releases/aarch64/alpine-standard-3.13.5-aarch64.iso"},{"line_number":76,"context_line":"    IRONIC_GRUB2_SHIM_FILE\u003dhttps://mirror.stream.centos.org/9-stream/BaseOS/aarch64/os/EFI/BOOT/BOOTAA64.EFI"},{"line_number":77,"context_line":"    IRONIC_GRUB2_FILE\u003dhttps://mirror.stream.centos.org/9-stream/BaseOS/aarch64/os/EFI/BOOT/grubaa64.efi"},{"line_number":78,"context_line":"    IRONIC_HW_ARCH\u003daarch64"}],"source_content_type":"text/x-rst","patch_set":54,"id":"0618333c_9e698332","line":75,"in_reply_to":"f71d4e1f_2c85a251","updated":"2024-06-17 12:32:13.000000000","message":"Yea. I used that for testing!","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":76,"context_line":"    IRONIC_GRUB2_SHIM_FILE\u003dhttps://mirror.stream.centos.org/9-stream/BaseOS/aarch64/os/EFI/BOOT/BOOTAA64.EFI"},{"line_number":77,"context_line":"    IRONIC_GRUB2_FILE\u003dhttps://mirror.stream.centos.org/9-stream/BaseOS/aarch64/os/EFI/BOOT/grubaa64.efi"},{"line_number":78,"context_line":"    IRONIC_HW_ARCH\u003daarch64"},{"line_number":79,"context_line":"    IPA_DOWNLOAD_SOURCE\u003dironic-python-agent-builder"},{"line_number":80,"context_line":"    IRONIC_AARCH64_MACHINE_TYPE\u003dvirt-6.2"},{"line_number":81,"context_line":"    IRONIC_JSON_RPC_AUTH_STRATEGY\u003d\u0027http_basic\u0027"},{"line_number":82,"context_line":"    IRONIC_DIB_RAMDISK_OS\u003ddebian-arm64"}],"source_content_type":"text/x-rst","patch_set":54,"id":"f01c1360_a406f913","line":79,"updated":"2024-06-17 08:41:37.000000000","message":"Can we just deduce this from the architecture? It\u0027s something we know, not something we need the developers to tell us.","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":77,"context_line":"    IRONIC_GRUB2_FILE\u003dhttps://mirror.stream.centos.org/9-stream/BaseOS/aarch64/os/EFI/BOOT/grubaa64.efi"},{"line_number":78,"context_line":"    IRONIC_HW_ARCH\u003daarch64"},{"line_number":79,"context_line":"    IPA_DOWNLOAD_SOURCE\u003dironic-python-agent-builder"},{"line_number":80,"context_line":"    IRONIC_AARCH64_MACHINE_TYPE\u003dvirt-6.2"},{"line_number":81,"context_line":"    IRONIC_JSON_RPC_AUTH_STRATEGY\u003d\u0027http_basic\u0027"},{"line_number":82,"context_line":"    IRONIC_DIB_RAMDISK_OS\u003ddebian-arm64"},{"line_number":83,"context_line":"    IRONIC_VM_EMULATOR\u003d/usr/bin/qemu-system-aarch64"}],"source_content_type":"text/x-rst","patch_set":54,"id":"d8acb0d5_bf2d403e","line":80,"updated":"2024-06-17 08:41:37.000000000","message":"This should not be required if you provide a good default instead of q35 (i.e. \"virt\"). \"virt-6.2\" is yet another unclear bit of information the users will get wrong.","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":80,"context_line":"    IRONIC_AARCH64_MACHINE_TYPE\u003dvirt-6.2"},{"line_number":81,"context_line":"    IRONIC_JSON_RPC_AUTH_STRATEGY\u003d\u0027http_basic\u0027"},{"line_number":82,"context_line":"    IRONIC_DIB_RAMDISK_OS\u003ddebian-arm64"},{"line_number":83,"context_line":"    IRONIC_VM_EMULATOR\u003d/usr/bin/qemu-system-aarch64"},{"line_number":84,"context_line":"    IRONIC_VM_SPECS_CPU_ARCH\u003daarch64"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    INSTALL_TEMPEST\u003dFalse"}],"source_content_type":"text/x-rst","patch_set":54,"id":"b87aedab_a28d3b73","line":83,"updated":"2024-06-17 08:41:37.000000000","message":"Not needed, we detect this value.","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"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":"9c29d2050b544d1d92ee4edee345a1e3168090c2","unresolved":false,"context_lines":[{"line_number":80,"context_line":"    IRONIC_AARCH64_MACHINE_TYPE\u003dvirt-6.2"},{"line_number":81,"context_line":"    IRONIC_JSON_RPC_AUTH_STRATEGY\u003d\u0027http_basic\u0027"},{"line_number":82,"context_line":"    IRONIC_DIB_RAMDISK_OS\u003ddebian-arm64"},{"line_number":83,"context_line":"    IRONIC_VM_EMULATOR\u003d/usr/bin/qemu-system-aarch64"},{"line_number":84,"context_line":"    IRONIC_VM_SPECS_CPU_ARCH\u003daarch64"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    INSTALL_TEMPEST\u003dFalse"}],"source_content_type":"text/x-rst","patch_set":54,"id":"b5254ad2_f44dcee9","line":83,"in_reply_to":"b87aedab_a28d3b73","updated":"2024-06-18 18:58:09.000000000","message":"Acknowledged","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":81,"context_line":"    IRONIC_JSON_RPC_AUTH_STRATEGY\u003d\u0027http_basic\u0027"},{"line_number":82,"context_line":"    IRONIC_DIB_RAMDISK_OS\u003ddebian-arm64"},{"line_number":83,"context_line":"    IRONIC_VM_EMULATOR\u003d/usr/bin/qemu-system-aarch64"},{"line_number":84,"context_line":"    IRONIC_VM_SPECS_CPU_ARCH\u003daarch64"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    INSTALL_TEMPEST\u003dFalse"},{"line_number":87,"context_line":"    VIRT_DRIVER\u003dironic"}],"source_content_type":"text/x-rst","patch_set":54,"id":"13f05289_5b1e5f80","line":84,"updated":"2024-06-17 08:41:37.000000000","message":"How is this different from IRONIC_HW_ARCH which you actually use?","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"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":"9c29d2050b544d1d92ee4edee345a1e3168090c2","unresolved":true,"context_lines":[{"line_number":81,"context_line":"    IRONIC_JSON_RPC_AUTH_STRATEGY\u003d\u0027http_basic\u0027"},{"line_number":82,"context_line":"    IRONIC_DIB_RAMDISK_OS\u003ddebian-arm64"},{"line_number":83,"context_line":"    IRONIC_VM_EMULATOR\u003d/usr/bin/qemu-system-aarch64"},{"line_number":84,"context_line":"    IRONIC_VM_SPECS_CPU_ARCH\u003daarch64"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    INSTALL_TEMPEST\u003dFalse"},{"line_number":87,"context_line":"    VIRT_DRIVER\u003dironic"}],"source_content_type":"text/x-rst","patch_set":54,"id":"81a04ceb_ef7178fb","line":84,"in_reply_to":"13f05289_5b1e5f80","updated":"2024-06-18 18:58:09.000000000","message":"Fixed","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"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":"82351e85ab6a6b136737cb3474d97c4ac2ecc533","unresolved":true,"context_lines":[{"line_number":40,"context_line":"    IRONIC_VM_SPECS_RAM\u003d4096"},{"line_number":41,"context_line":"    IRONIC_VM_SPECS_DISK\u003d3"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    IRONIC_VM_EMULATOR: /usr/bin/qemu-system-aarch64"},{"line_number":44,"context_line":"    IRONIC_VM_SPECS_CPU\u003d1"},{"line_number":45,"context_line":"    IRONIC_VM_VOLUME_COUNT\u003d2"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-rst","patch_set":59,"id":"c805c70d_f19969dd","line":43,"updated":"2024-06-17 21:45:05.000000000","message":"This is deduced, per dtantsur\u0027s comment, and is in YAML format even if we did need it.","commit_id":"dca405efd302b2a4a96effc6cac1c1791d81590b"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"e12ea2dabff44a8811d5c1a809b68e2e3945b88f","unresolved":true,"context_lines":[{"line_number":57,"context_line":"    # Specify deploy driver. This driver should be in the enabled list above."},{"line_number":58,"context_line":"    IRONIC_DEPLOY_DRIVER\u003dredfish"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    CIRROS_VERSION\u003d0.6.1"},{"line_number":61,"context_line":"    CIRROS_ARCH\u003daarch64"},{"line_number":62,"context_line":"    DEFAULT_IMAGE_NAME\u003dcirros-0.6.1-aarch64-uec"},{"line_number":63,"context_line":""}],"source_content_type":"text/x-rst","patch_set":61,"id":"a44b1786_42407ac8","line":60,"updated":"2024-06-19 15:31:26.000000000","message":"I\u0027m worried we\u0027ll forget to update this when bumping the version everywhere. Isn\u0027t it the default?","commit_id":"209ba7973294d9b9fac5de8d302396c460e7969e"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"e12ea2dabff44a8811d5c1a809b68e2e3945b88f","unresolved":true,"context_lines":[{"line_number":58,"context_line":"    IRONIC_DEPLOY_DRIVER\u003dredfish"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    CIRROS_VERSION\u003d0.6.1"},{"line_number":61,"context_line":"    CIRROS_ARCH\u003daarch64"},{"line_number":62,"context_line":"    DEFAULT_IMAGE_NAME\u003dcirros-0.6.1-aarch64-uec"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    FORCE_CONFIG_DRIVE\u003dFalse"}],"source_content_type":"text/x-rst","patch_set":61,"id":"6ef80ee0_e1785343","line":61,"updated":"2024-06-19 15:31:26.000000000","message":"This can be derived from IRONIC_HW_ARCH","commit_id":"209ba7973294d9b9fac5de8d302396c460e7969e"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"e12ea2dabff44a8811d5c1a809b68e2e3945b88f","unresolved":true,"context_lines":[{"line_number":73,"context_line":"    IRONIC_GRUB2_SHIM_FILE\u003dhttps://mirror.stream.centos.org/9-stream/BaseOS/aarch64/os/EFI/BOOT/BOOTAA64.EFI"},{"line_number":74,"context_line":"    IRONIC_GRUB2_FILE\u003dhttps://mirror.stream.centos.org/9-stream/BaseOS/aarch64/os/EFI/BOOT/grubaa64.efi"},{"line_number":75,"context_line":"    IRONIC_HW_ARCH\u003daarch64"},{"line_number":76,"context_line":"    IRONIC_MACHINE_TYPE\u003dvirt-6.2"},{"line_number":77,"context_line":"    IRONIC_JSON_RPC_AUTH_STRATEGY\u003d\u0027http_basic\u0027"},{"line_number":78,"context_line":"    IRONIC_DIB_RAMDISK_OS\u003ddebian-arm64"},{"line_number":79,"context_line":""}],"source_content_type":"text/x-rst","patch_set":61,"id":"35b8a2d6_1b2a8dee","line":76,"updated":"2024-06-19 15:31:26.000000000","message":"This is a magical string from the user\u0027s perspective, can we not default to something working, even if just \"virt\"?","commit_id":"209ba7973294d9b9fac5de8d302396c460e7969e"}],"zuul.d/ironic-jobs.yaml":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fdc05ef597832a83ba180cf6cfabffb82ccfadab","unresolved":true,"context_lines":[{"line_number":120,"context_line":"      - ^tools/.*$"},{"line_number":121,"context_line":"      - ^tox.ini$"},{"line_number":122,"context_line":"    vars:"},{"line_number":123,"context_line":"      tempest_test_regex: BaremetalPxeIpmitoolWholedisk"},{"line_number":124,"context_line":"      tempest_concurrency: 2"},{"line_number":125,"context_line":"      devstack_localrc:"},{"line_number":126,"context_line":"        FORCE_CONFIG_DRIVE: False"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"c4d2df39_56bab3ae","line":123,"range":{"start_line":123,"start_character":26,"end_line":123,"end_character":55},"updated":"2024-05-01 20:42:51.000000000","message":"So here is the problem:\n\nThis string matches tests which have a driver name of \u0027pxe_ipmitool\u0027.\n\nWhich is a legacy driver name, we removed it from Ironic *years* ago.\n\nA better test name is: BaremetalDriverDirectWholedisk","commit_id":"74c0627cbceb3d4116f3e864c7564b3abd14c296"},{"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":"71123537daf2d2c877e1c9a3b40aca02b659fe82","unresolved":false,"context_lines":[{"line_number":120,"context_line":"      - ^tools/.*$"},{"line_number":121,"context_line":"      - ^tox.ini$"},{"line_number":122,"context_line":"    vars:"},{"line_number":123,"context_line":"      tempest_test_regex: BaremetalPxeIpmitoolWholedisk"},{"line_number":124,"context_line":"      tempest_concurrency: 2"},{"line_number":125,"context_line":"      devstack_localrc:"},{"line_number":126,"context_line":"        FORCE_CONFIG_DRIVE: False"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"ee08a390_e3e90ebb","line":123,"range":{"start_line":123,"start_character":26,"end_line":123,"end_character":55},"in_reply_to":"30b6fac2_31e240b3","updated":"2024-05-07 15:02:20.000000000","message":"Acknowledged","commit_id":"74c0627cbceb3d4116f3e864c7564b3abd14c296"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"25a165bfb9f56d5af309f391ba1ba9f118f108bd","unresolved":true,"context_lines":[{"line_number":120,"context_line":"      - ^tools/.*$"},{"line_number":121,"context_line":"      - ^tox.ini$"},{"line_number":122,"context_line":"    vars:"},{"line_number":123,"context_line":"      tempest_test_regex: BaremetalPxeIpmitoolWholedisk"},{"line_number":124,"context_line":"      tempest_concurrency: 2"},{"line_number":125,"context_line":"      devstack_localrc:"},{"line_number":126,"context_line":"        FORCE_CONFIG_DRIVE: False"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"30b6fac2_31e240b3","line":123,"range":{"start_line":123,"start_character":26,"end_line":123,"end_character":55},"in_reply_to":"c4d2df39_56bab3ae","updated":"2024-05-01 20:48:19.000000000","message":"https://github.com/openstack/ironic-tempest-plugin/blob/5fd53b282decdb765f2725117badb78fa3be7541/ironic_tempest_plugin/tests/scenario/ironic_standalone/test_basic_ops.py#L75","commit_id":"74c0627cbceb3d4116f3e864c7564b3abd14c296"},{"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":"90f45e40b82a0d449e8d18dcd503f3e6daa1e4fe","unresolved":true,"context_lines":[{"line_number":131,"context_line":"        IRONIC_ENABLED_BOOT_INTERFACES: \"ipxe,redfish-virtual-media,http-ipxe,pxe,http\""},{"line_number":132,"context_line":"        IRONIC_ENABLED_HARDWARE_TYPES: redfish"},{"line_number":133,"context_line":"        IRONIC_ENABLED_MANAGEMENT_INTERFACES: redfish"},{"line_number":134,"context_line":"        IRONIC_ENABLED_DEPLOY_INTERFACES: \"direct,ansible,ramdisk\""},{"line_number":135,"context_line":"        IRONIC_ENABLED_RESCUE_INTERFACES: \"agent,no-rescue\""},{"line_number":136,"context_line":"        IRONIC_BOOT_MODE: uefi"},{"line_number":137,"context_line":"        IRONIC_CALLBACK_TIMEOUT: 800"}],"source_content_type":"text/x-yaml","patch_set":43,"id":"8ce4125b_41f2a113","line":134,"updated":"2024-05-28 23:08:31.000000000","message":"remove ansible from this list?","commit_id":"341edd748236447d9e9a45f0b4ec8c7a4436ba7f"},{"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":"e21b41445b94b86b3646f6302372da05de835971","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        IRONIC_ENABLED_BOOT_INTERFACES: \"ipxe,redfish-virtual-media,http-ipxe,pxe,http\""},{"line_number":132,"context_line":"        IRONIC_ENABLED_HARDWARE_TYPES: redfish"},{"line_number":133,"context_line":"        IRONIC_ENABLED_MANAGEMENT_INTERFACES: redfish"},{"line_number":134,"context_line":"        IRONIC_ENABLED_DEPLOY_INTERFACES: \"direct,ansible,ramdisk\""},{"line_number":135,"context_line":"        IRONIC_ENABLED_RESCUE_INTERFACES: \"agent,no-rescue\""},{"line_number":136,"context_line":"        IRONIC_BOOT_MODE: uefi"},{"line_number":137,"context_line":"        IRONIC_CALLBACK_TIMEOUT: 800"}],"source_content_type":"text/x-yaml","patch_set":43,"id":"49555c4a_84bffec1","line":134,"in_reply_to":"8ce4125b_41f2a113","updated":"2024-05-30 11:14:50.000000000","message":"Done","commit_id":"341edd748236447d9e9a45f0b4ec8c7a4436ba7f"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"a06978af37893b8a49e65ed3b2c60eaefddeeb09","unresolved":true,"context_lines":[{"line_number":120,"context_line":"      - ^tools/.*$"},{"line_number":121,"context_line":"      - ^tox.ini$"},{"line_number":122,"context_line":"    vars:"},{"line_number":123,"context_line":"      tempest_test_regex: BaremetalDriverDirectWholedisk"},{"line_number":124,"context_line":"      tempest_concurrency: 2"},{"line_number":125,"context_line":"      devstack_localrc:"},{"line_number":126,"context_line":"        CIRROS_VERSION: 0.6.1"}],"source_content_type":"text/x-yaml","patch_set":45,"id":"2afdd945_fd52d2b1","line":123,"updated":"2024-05-29 17:24:36.000000000","message":"We might want to change this, but I\u0027m not sure of what would be better. We\u0027re just trying to use a pinpointed test, so I would be okay with dialing back the original change which special cased it to redfish only.","commit_id":"29e2984cba9ad82f1a33f22c440d2ba8b8c3593a"},{"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":"e21b41445b94b86b3646f6302372da05de835971","unresolved":true,"context_lines":[{"line_number":120,"context_line":"      - ^tools/.*$"},{"line_number":121,"context_line":"      - ^tox.ini$"},{"line_number":122,"context_line":"    vars:"},{"line_number":123,"context_line":"      tempest_test_regex: BaremetalDriverDirectWholedisk"},{"line_number":124,"context_line":"      tempest_concurrency: 2"},{"line_number":125,"context_line":"      devstack_localrc:"},{"line_number":126,"context_line":"        CIRROS_VERSION: 0.6.1"}],"source_content_type":"text/x-yaml","patch_set":45,"id":"c374414d_f599315a","line":123,"in_reply_to":"2afdd945_fd52d2b1","updated":"2024-05-30 11:14:50.000000000","message":"I think just \"did we boot a thing we can ping\" is sufficient hello world to validate ARM functionality vs x86 functionality at least until we get the ability to do more involved validation (e.g. an arm-based inspection test would be interesting to see results of, but MUCH less impactful than just this)","commit_id":"29e2984cba9ad82f1a33f22c440d2ba8b8c3593a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"54be02300f6d9786a58d5487a2f4674136f32c4d","unresolved":true,"context_lines":[{"line_number":148,"context_line":"        IRONIC_DIB_RAMDISK_OS: debian-arm64"},{"line_number":149,"context_line":"        IRONIC_RAMDISK_TYPE: dib"},{"line_number":150,"context_line":"        IRONIC_RPC_TRANSPORT: json-rpc"},{"line_number":151,"context_line":"        IRONIC_VM_EMULATOR: /usr/bin/qemu-system-aarch64"},{"line_number":152,"context_line":"        IRONIC_VM_SPECS_CPU_ARCH: aarch64"},{"line_number":153,"context_line":"        IRONIC_VM_SPECS_RAM: 4096"},{"line_number":154,"context_line":"        IRONIC_VM_SPECS_CPU: 1"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"b48008db_25f0bc99","line":151,"updated":"2024-06-17 08:41:37.000000000","message":"See my comment in the contributors docs: this configuration has many parameters that we should deduce automatically (insert a comment from Jay about needing a decoder ring).","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"},{"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":"82351e85ab6a6b136737cb3474d97c4ac2ecc533","unresolved":true,"context_lines":[{"line_number":148,"context_line":"        IRONIC_DIB_RAMDISK_OS: debian-arm64"},{"line_number":149,"context_line":"        IRONIC_RAMDISK_TYPE: dib"},{"line_number":150,"context_line":"        IRONIC_RPC_TRANSPORT: json-rpc"},{"line_number":151,"context_line":"        IRONIC_VM_EMULATOR: /usr/bin/qemu-system-aarch64"},{"line_number":152,"context_line":"        IRONIC_VM_SPECS_CPU_ARCH: aarch64"},{"line_number":153,"context_line":"        IRONIC_VM_SPECS_RAM: 4096"},{"line_number":154,"context_line":"        IRONIC_VM_SPECS_CPU: 1"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"37f419dc_23c4d92d","line":151,"in_reply_to":"b48008db_25f0bc99","updated":"2024-06-17 21:45:05.000000000","message":"++ We should set the minimum needed set of things","commit_id":"d8d1a786841a81a883ea30beb54485466ee9542d"}],"zuul.d/project.yaml":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"a06978af37893b8a49e65ed3b2c60eaefddeeb09","unresolved":true,"context_lines":[{"line_number":73,"context_line":"            voting: false"},{"line_number":74,"context_line":"    gate:"},{"line_number":75,"context_line":"      jobs:"},{"line_number":76,"context_line":"        - ironic-tox-bandit"},{"line_number":77,"context_line":"        - ironic-tox-codespell"},{"line_number":78,"context_line":"        - ironic-tox-unit-mysql-migrations"},{"line_number":79,"context_line":"        - ironic-tox-unit-with-driver-libs"},{"line_number":80,"context_line":"        - ironic-tempest-functional-python3"},{"line_number":81,"context_line":"        # NOTE(rpittau) disabled until we fix the tests"},{"line_number":82,"context_line":"        # see also https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/882312"},{"line_number":83,"context_line":"        # - ironic-tempest-functional-rbac-scope-enforced"},{"line_number":84,"context_line":"        - ironic-grenade"},{"line_number":85,"context_line":"        # NOTE(JayF): Disabling standalone jobs on 20240228 from voting, there\u0027s a"},{"line_number":86,"context_line":"        # dnsmasq bug only exposed on these jobs."},{"line_number":87,"context_line":"        #- ironic-standalone-redfish"},{"line_number":88,"context_line":"        - ironic-tempest-bios-redfish-pxe"},{"line_number":89,"context_line":"        - ironic-tempest-uefi-redfish-vmedia"},{"line_number":90,"context_line":"        - ironic-tempest-ramdisk-bios-snmp-pxe"},{"line_number":91,"context_line":"        - ironic-tempest-uefi-redfish-https"},{"line_number":92,"context_line":"        - ironic-tempest-ovn-uefi-ipmi-pxe"},{"line_number":93,"context_line":"        # NOTE(TheJulia): Disabled multinode on 20210311 due to Limestone"},{"line_number":94,"context_line":"        # seeming to be"},{"line_number":95,"context_line":"        # - ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode"},{"line_number":96,"context_line":"        - ironic-tempest-bios-ipmi-direct-tinyipa"},{"line_number":97,"context_line":"        - ironic-tempest-bfv"},{"line_number":98,"context_line":"        - ironic-tempest-ipa-partition-uefi-pxe-grub2"},{"line_number":99,"context_line":"        # NOTE(rpittau): Currently broken because of an issue with parted"},{"line_number":100,"context_line":"        #- metalsmith-integration-glance-centos9-legacy"},{"line_number":101,"context_line":"        # NOTE(TheJulia): At present, metal3 doesn\u0027t leverage"},{"line_number":102,"context_line":"        # stable branches, and as far as we are aware these jobs"},{"line_number":103,"context_line":"        # can be removed once this branch is made stable."},{"line_number":104,"context_line":"        - metal3-integration"},{"line_number":105,"context_line":"    experimental:"},{"line_number":106,"context_line":"      jobs:"},{"line_number":107,"context_line":"        # TODO(dtantsur): these jobs are useful but currently hopelessly"},{"line_number":108,"context_line":"        # broken. Fix them and bring back to the gate."},{"line_number":109,"context_line":"        - ironic-grenade-multinode-multitenant:"},{"line_number":110,"context_line":"            voting: false"},{"line_number":111,"context_line":"        - ironic-inspector-tempest-discovery-fast-track:"},{"line_number":112,"context_line":"            voting: false"}],"source_content_type":"text/x-yaml","patch_set":45,"id":"b8eacfb5_e2057508","side":"PARENT","line":112,"range":{"start_line":76,"start_character":0,"end_line":112,"end_character":25},"updated":"2024-05-29 17:24:36.000000000","message":"So lets restore the job configuration. Instead of putting this in experimental, I\u0027d almost just start with it as non-voting on the check list. That way we can get some data points to figure out the next step.","commit_id":"0d2a2e4bf4069cb5961bdbccb67950578cb0c2d1"},{"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":"e21b41445b94b86b3646f6302372da05de835971","unresolved":true,"context_lines":[{"line_number":73,"context_line":"            voting: false"},{"line_number":74,"context_line":"    gate:"},{"line_number":75,"context_line":"      jobs:"},{"line_number":76,"context_line":"        - ironic-tox-bandit"},{"line_number":77,"context_line":"        - ironic-tox-codespell"},{"line_number":78,"context_line":"        - ironic-tox-unit-mysql-migrations"},{"line_number":79,"context_line":"        - ironic-tox-unit-with-driver-libs"},{"line_number":80,"context_line":"        - ironic-tempest-functional-python3"},{"line_number":81,"context_line":"        # NOTE(rpittau) disabled until we fix the tests"},{"line_number":82,"context_line":"        # see also https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/882312"},{"line_number":83,"context_line":"        # - ironic-tempest-functional-rbac-scope-enforced"},{"line_number":84,"context_line":"        - ironic-grenade"},{"line_number":85,"context_line":"        # NOTE(JayF): Disabling standalone jobs on 20240228 from voting, there\u0027s a"},{"line_number":86,"context_line":"        # dnsmasq bug only exposed on these jobs."},{"line_number":87,"context_line":"        #- ironic-standalone-redfish"},{"line_number":88,"context_line":"        - ironic-tempest-bios-redfish-pxe"},{"line_number":89,"context_line":"        - ironic-tempest-uefi-redfish-vmedia"},{"line_number":90,"context_line":"        - ironic-tempest-ramdisk-bios-snmp-pxe"},{"line_number":91,"context_line":"        - ironic-tempest-uefi-redfish-https"},{"line_number":92,"context_line":"        - ironic-tempest-ovn-uefi-ipmi-pxe"},{"line_number":93,"context_line":"        # NOTE(TheJulia): Disabled multinode on 20210311 due to Limestone"},{"line_number":94,"context_line":"        # seeming to be"},{"line_number":95,"context_line":"        # - ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode"},{"line_number":96,"context_line":"        - ironic-tempest-bios-ipmi-direct-tinyipa"},{"line_number":97,"context_line":"        - ironic-tempest-bfv"},{"line_number":98,"context_line":"        - ironic-tempest-ipa-partition-uefi-pxe-grub2"},{"line_number":99,"context_line":"        # NOTE(rpittau): Currently broken because of an issue with parted"},{"line_number":100,"context_line":"        #- metalsmith-integration-glance-centos9-legacy"},{"line_number":101,"context_line":"        # NOTE(TheJulia): At present, metal3 doesn\u0027t leverage"},{"line_number":102,"context_line":"        # stable branches, and as far as we are aware these jobs"},{"line_number":103,"context_line":"        # can be removed once this branch is made stable."},{"line_number":104,"context_line":"        - metal3-integration"},{"line_number":105,"context_line":"    experimental:"},{"line_number":106,"context_line":"      jobs:"},{"line_number":107,"context_line":"        # TODO(dtantsur): these jobs are useful but currently hopelessly"},{"line_number":108,"context_line":"        # broken. Fix them and bring back to the gate."},{"line_number":109,"context_line":"        - ironic-grenade-multinode-multitenant:"},{"line_number":110,"context_line":"            voting: false"},{"line_number":111,"context_line":"        - ironic-inspector-tempest-discovery-fast-track:"},{"line_number":112,"context_line":"            voting: false"}],"source_content_type":"text/x-yaml","patch_set":45,"id":"c63f3f99_bd6f8e6d","side":"PARENT","line":112,"range":{"start_line":76,"start_character":0,"end_line":112,"end_character":25},"in_reply_to":"b8eacfb5_e2057508","updated":"2024-05-30 11:14:50.000000000","message":"++\n\nBasically revert changes to this file, and add ironic-standalone-aarch64 as a voting: false job under check","commit_id":"0d2a2e4bf4069cb5961bdbccb67950578cb0c2d1"},{"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":"efc02ddad2fb9fc81366e958b493bba9d058aed3","unresolved":false,"context_lines":[{"line_number":73,"context_line":"            voting: false"},{"line_number":74,"context_line":"    gate:"},{"line_number":75,"context_line":"      jobs:"},{"line_number":76,"context_line":"        - ironic-tox-bandit"},{"line_number":77,"context_line":"        - ironic-tox-codespell"},{"line_number":78,"context_line":"        - ironic-tox-unit-mysql-migrations"},{"line_number":79,"context_line":"        - ironic-tox-unit-with-driver-libs"},{"line_number":80,"context_line":"        - ironic-tempest-functional-python3"},{"line_number":81,"context_line":"        # NOTE(rpittau) disabled until we fix the tests"},{"line_number":82,"context_line":"        # see also https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/882312"},{"line_number":83,"context_line":"        # - ironic-tempest-functional-rbac-scope-enforced"},{"line_number":84,"context_line":"        - ironic-grenade"},{"line_number":85,"context_line":"        # NOTE(JayF): Disabling standalone jobs on 20240228 from voting, there\u0027s a"},{"line_number":86,"context_line":"        # dnsmasq bug only exposed on these jobs."},{"line_number":87,"context_line":"        #- ironic-standalone-redfish"},{"line_number":88,"context_line":"        - ironic-tempest-bios-redfish-pxe"},{"line_number":89,"context_line":"        - ironic-tempest-uefi-redfish-vmedia"},{"line_number":90,"context_line":"        - ironic-tempest-ramdisk-bios-snmp-pxe"},{"line_number":91,"context_line":"        - ironic-tempest-uefi-redfish-https"},{"line_number":92,"context_line":"        - ironic-tempest-ovn-uefi-ipmi-pxe"},{"line_number":93,"context_line":"        # NOTE(TheJulia): Disabled multinode on 20210311 due to Limestone"},{"line_number":94,"context_line":"        # seeming to be"},{"line_number":95,"context_line":"        # - ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode"},{"line_number":96,"context_line":"        - ironic-tempest-bios-ipmi-direct-tinyipa"},{"line_number":97,"context_line":"        - ironic-tempest-bfv"},{"line_number":98,"context_line":"        - ironic-tempest-ipa-partition-uefi-pxe-grub2"},{"line_number":99,"context_line":"        # NOTE(rpittau): Currently broken because of an issue with parted"},{"line_number":100,"context_line":"        #- metalsmith-integration-glance-centos9-legacy"},{"line_number":101,"context_line":"        # NOTE(TheJulia): At present, metal3 doesn\u0027t leverage"},{"line_number":102,"context_line":"        # stable branches, and as far as we are aware these jobs"},{"line_number":103,"context_line":"        # can be removed once this branch is made stable."},{"line_number":104,"context_line":"        - metal3-integration"},{"line_number":105,"context_line":"    experimental:"},{"line_number":106,"context_line":"      jobs:"},{"line_number":107,"context_line":"        # TODO(dtantsur): these jobs are useful but currently hopelessly"},{"line_number":108,"context_line":"        # broken. Fix them and bring back to the gate."},{"line_number":109,"context_line":"        - ironic-grenade-multinode-multitenant:"},{"line_number":110,"context_line":"            voting: false"},{"line_number":111,"context_line":"        - ironic-inspector-tempest-discovery-fast-track:"},{"line_number":112,"context_line":"            voting: false"}],"source_content_type":"text/x-yaml","patch_set":45,"id":"9e095973_e4bbae1b","side":"PARENT","line":112,"range":{"start_line":76,"start_character":0,"end_line":112,"end_character":25},"in_reply_to":"c63f3f99_bd6f8e6d","updated":"2024-05-30 17:31:04.000000000","message":"Acknowledged","commit_id":"0d2a2e4bf4069cb5961bdbccb67950578cb0c2d1"}]}
