)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"943310370a8db0d511e8356e31ed95cac46ea84b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ec6acdbe_7f13a618","updated":"2026-09-30 21:19:03.000000000","message":"recheck cloud.centos.org 502","commit_id":"d06953e76e27f2ff94bc30b2d8f2c70433ada389"}],"diskimage_builder/elements/yum/pre-install.d/01-00-centos-python3":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"5f25a786c8f6f60bba49fdc71acfcf9766664ed2","unresolved":true,"context_lines":[{"line_number":13,"context_line":"if [[ ${DISTRO_NAME} \u003d~ (centos|rhel) ]]; then"},{"line_number":14,"context_line":"    # Ensure the \"user\" python3 package is installed"},{"line_number":15,"context_line":"    # so we have /usr/bin/python3 and pyyaml."},{"line_number":16,"context_line":"    rpm -q python3 python3-pyyaml || dnf install -y python3 python3-pyyaml"},{"line_number":17,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"b1305daa_98db49b6","line":16,"updated":"2026-09-28 23:25:55.000000000","message":"Similar to my comment in https://review.opendev.org/c/openstack/diskimage-builder/+/1007796/1 I guess I\u0027m confused as to why this is necessary. It should effectively be a noop when dnf realizes the package is already isntalled?","commit_id":"ec04845ff98e2e976fee62c49e86ab6bd7601508"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"4ef21b4c59cb8156527b79ae2e5bae38f8b2bc14","unresolved":true,"context_lines":[{"line_number":13,"context_line":"if [[ ${DISTRO_NAME} \u003d~ (centos|rhel) ]]; then"},{"line_number":14,"context_line":"    # Ensure the \"user\" python3 package is installed"},{"line_number":15,"context_line":"    # so we have /usr/bin/python3 and pyyaml."},{"line_number":16,"context_line":"    rpm -q python3 python3-pyyaml || dnf install -y python3 python3-pyyaml"},{"line_number":17,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"29fe23d6_baf33215","line":16,"in_reply_to":"17ce58b4_b086a8bb","updated":"2026-09-29 23:14:29.000000000","message":"This is one of the rare package installs which doesn\u0027t go through the install-packages script, so I think extra defensiveness can be justified.","commit_id":"ec04845ff98e2e976fee62c49e86ab6bd7601508"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"2fef4be06dc1689d6e21f19da1f64ffa5bbb1268","unresolved":true,"context_lines":[{"line_number":13,"context_line":"if [[ ${DISTRO_NAME} \u003d~ (centos|rhel) ]]; then"},{"line_number":14,"context_line":"    # Ensure the \"user\" python3 package is installed"},{"line_number":15,"context_line":"    # so we have /usr/bin/python3 and pyyaml."},{"line_number":16,"context_line":"    rpm -q python3 python3-pyyaml || dnf install -y python3 python3-pyyaml"},{"line_number":17,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"17ce58b4_b086a8bb","line":16,"in_reply_to":"b1305daa_98db49b6","updated":"2026-09-29 23:12:18.000000000","message":"This would likely trigger a full rebuild of the repo cache based on configured repos, so network traffic and time to confirm something which could be confirmed at constant zero cost","commit_id":"ec04845ff98e2e976fee62c49e86ab6bd7601508"}]}