)]}'
{"diskimage_builder/elements/centos/root.d/10-centos-cloud-image":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"076df2b9d53c554933d568e532b65dd3ef100b0e","unresolved":true,"context_lines":[{"line_number":42,"context_line":"    else"},{"line_number":43,"context_line":"        dib_release_path\u003d${DIB_RELEASE}"},{"line_number":44,"context_line":"    fi"},{"line_number":45,"context_line":"    BASE_IMAGE_FILE\u003d${BASE_IMAGE_FILE:-$(head -1 \u003c \u003c(curl -s https://cloud.centos.org/centos/${dib_release_path}/${ARCH}/images/ | grep -v \"CentOS-.[^\u003e]*-x86_64-\" | grep -o \"CentOS-.[^\u003e]*${DIB_FLAVOR}-.[^\u003e]*.qcow2\" | sort -r))}"},{"line_number":46,"context_line":"    BASE_IMAGE_TAR\u003d$BASE_IMAGE_FILE.tgz"},{"line_number":47,"context_line":"    IMAGE_LOCATION\u003d$DIB_CLOUD_IMAGES/$BASE_IMAGE_FILE"},{"line_number":48,"context_line":"    CACHED_IMAGE\u003d$DIB_IMAGE_CACHE/$BASE_IMAGE_FILE"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"a5c7e636_771686b5","line":45,"updated":"2026-09-25 16:20:52.000000000","message":"I guess my only concern here is we\u0027re locking the architecture here. Is there a clean pattern to do it without including x86_64 explicitly, or to map the requested architecture for DIB to operate with to the image downloaded?","commit_id":"73764519c23e338f88b9daf642d80376c416ae75"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"03712059a28391242b3a86ec3041758de91b2c5f","unresolved":true,"context_lines":[{"line_number":42,"context_line":"    else"},{"line_number":43,"context_line":"        dib_release_path\u003d${DIB_RELEASE}"},{"line_number":44,"context_line":"    fi"},{"line_number":45,"context_line":"    BASE_IMAGE_FILE\u003d${BASE_IMAGE_FILE:-$(head -1 \u003c \u003c(curl -s https://cloud.centos.org/centos/${dib_release_path}/${ARCH}/images/ | grep -v \"CentOS-.[^\u003e]*-x86_64-\" | grep -o \"CentOS-.[^\u003e]*${DIB_FLAVOR}-.[^\u003e]*.qcow2\" | sort -r))}"},{"line_number":46,"context_line":"    BASE_IMAGE_TAR\u003d$BASE_IMAGE_FILE.tgz"},{"line_number":47,"context_line":"    IMAGE_LOCATION\u003d$DIB_CLOUD_IMAGES/$BASE_IMAGE_FILE"},{"line_number":48,"context_line":"    CACHED_IMAGE\u003d$DIB_IMAGE_CACHE/$BASE_IMAGE_FILE"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"3c2f228d_0cc51354","line":45,"in_reply_to":"08f9d35c_b86432eb","updated":"2026-09-25 19:29:42.000000000","message":"oh, right. the -v didn\u0027t register for me. That works, but also didn\u0027t need the expansion either. That being said, this change passed CI and for another change recently didn\u0027t, so... better than nothing.","commit_id":"73764519c23e338f88b9daf642d80376c416ae75"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"aa247568e2e660f7480db0d8b3f332cbb0ff573c","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    else"},{"line_number":43,"context_line":"        dib_release_path\u003d${DIB_RELEASE}"},{"line_number":44,"context_line":"    fi"},{"line_number":45,"context_line":"    BASE_IMAGE_FILE\u003d${BASE_IMAGE_FILE:-$(head -1 \u003c \u003c(curl -s https://cloud.centos.org/centos/${dib_release_path}/${ARCH}/images/ | grep -v \"CentOS-.[^\u003e]*-x86_64-\" | grep -o \"CentOS-.[^\u003e]*${DIB_FLAVOR}-.[^\u003e]*.qcow2\" | sort -r))}"},{"line_number":46,"context_line":"    BASE_IMAGE_TAR\u003d$BASE_IMAGE_FILE.tgz"},{"line_number":47,"context_line":"    IMAGE_LOCATION\u003d$DIB_CLOUD_IMAGES/$BASE_IMAGE_FILE"},{"line_number":48,"context_line":"    CACHED_IMAGE\u003d$DIB_IMAGE_CACHE/$BASE_IMAGE_FILE"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"764864c3_bb920e8c","line":45,"in_reply_to":"3c2f228d_0cc51354","updated":"2026-09-25 21:01:30.000000000","message":"Let\u0027s make it resolved.","commit_id":"73764519c23e338f88b9daf642d80376c416ae75"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"bf5f40e22cb62842b6ee7bdbbfa1d40bad91bd28","unresolved":true,"context_lines":[{"line_number":42,"context_line":"    else"},{"line_number":43,"context_line":"        dib_release_path\u003d${DIB_RELEASE}"},{"line_number":44,"context_line":"    fi"},{"line_number":45,"context_line":"    BASE_IMAGE_FILE\u003d${BASE_IMAGE_FILE:-$(head -1 \u003c \u003c(curl -s https://cloud.centos.org/centos/${dib_release_path}/${ARCH}/images/ | grep -v \"CentOS-.[^\u003e]*-x86_64-\" | grep -o \"CentOS-.[^\u003e]*${DIB_FLAVOR}-.[^\u003e]*.qcow2\" | sort -r))}"},{"line_number":46,"context_line":"    BASE_IMAGE_TAR\u003d$BASE_IMAGE_FILE.tgz"},{"line_number":47,"context_line":"    IMAGE_LOCATION\u003d$DIB_CLOUD_IMAGES/$BASE_IMAGE_FILE"},{"line_number":48,"context_line":"    CACHED_IMAGE\u003d$DIB_IMAGE_CACHE/$BASE_IMAGE_FILE"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"08f9d35c_b86432eb","line":45,"in_reply_to":"a5c7e636_771686b5","updated":"2026-09-25 16:41:16.000000000","message":"The added grep is excluding `-x86_64-` - I think for aarch64 ppc64le this will still be ok. The additional `grep -v \"CentOS-.[^\u003e]*-x86_64-\"` wont match/filter anything in the other architectures? (Only x86_64 had two images, one BIOS only that used the \"New\" naming schema already, and the one with -86_64- could do both BIOS and UEFI.","commit_id":"73764519c23e338f88b9daf642d80376c416ae75"}]}