)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"112ac8ae868cc1f8bc87177f091caa7e305d9835","unresolved":true,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Depends-On: https://review.opendev.org/c/openstack/devstack/+/946763"},{"line_number":21,"context_line":"Depends-On: https://review.opendev.org/c/openstack/devstack/+/948558"},{"line_number":22,"context_line":"Depends-On: https://review.opendev.org/c/openstack/devstack/+/948786"},{"line_number":23,"context_line":"Depends-On: https://review.opendev.org/c/openstack/devstack/+/948797"},{"line_number":24,"context_line":"Related-Bug: #2109591"},{"line_number":25,"context_line":"Change-Id: I80d4c65ba0728c3e4b18738c6b0d539409f19976"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"71b80fb9_1ebe6ab8","line":22,"range":{"start_line":22,"start_character":62,"end_line":22,"end_character":68},"updated":"2025-05-12 17:07:51.000000000","message":"nit: so https://review.opendev.org/c/openstack/devstack/+/948786 is the only one still peneing but its approved.\n\nso i guess this is ready to proceed.\n\nif this need a respin after that merges you coudl drop al lthe depends on links.","commit_id":"f5b9596f24c03079c6ea2bcac29b1973bf5fe589"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c7ab065b5a1a288999ebd35bed946e424cab7e38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e86b3a42_db15f35a","updated":"2025-05-06 15:29:17.000000000","message":"lgtm","commit_id":"36434ca7f84e8939666a2ad03072686b8f27945b"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"0fc1e4d1bc814566464761d27d54a282021d147c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b3f36c03_387b8852","updated":"2025-05-05 14:46:45.000000000","message":"recheck unrelated jammy failure","commit_id":"36434ca7f84e8939666a2ad03072686b8f27945b"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"75031a95c6e02ba8df63671c93a19676f42e3604","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2656feb5_43e1363a","updated":"2025-05-12 15:35:24.000000000","message":"I\u0027ll leave this to some of the other devstack cores (perhaps frickler?) but this doesn\u0027t feel right to me.","commit_id":"fd75e29d0d48df5689d95429f0eafaf93bc4fbe4"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"d2904e0eb2ddb22de4f9cab4bc4d355d6caa00bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7c9f8cb7_9ff52cf4","updated":"2025-05-09 07:15:37.000000000","message":"recheck unrelated ceph multistore failure","commit_id":"fd75e29d0d48df5689d95429f0eafaf93bc4fbe4"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b400f9a36a4a5f3e8b34b7962d53ee58ebc6edb8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"50452b42_5d834c1b","updated":"2025-05-12 20:14:32.000000000","message":"thanks for updates. I have abandon mine change, jobs are passing now.","commit_id":"f5b9596f24c03079c6ea2bcac29b1973bf5fe589"}],"functions-common":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"c29d648d100ca747180f995a2fd17594416b3f89","unresolved":true,"context_lines":[{"line_number":2282,"context_line":"    if [[ -x $(which $interp 2\u003e /dev/null) ]]; then"},{"line_number":2283,"context_line":"        version\u003d$($interp -c \u0027import sys; print(\"%s.%s\" % sys.version_info[0:2])\u0027)"},{"line_number":2284,"context_line":"    fi"},{"line_number":2285,"context_line":"    if [[ $DISTRO \u003d~ rhel9 ]]; then"},{"line_number":2286,"context_line":"        echo 3.11"},{"line_number":2287,"context_line":"    else"},{"line_number":2288,"context_line":"        echo ${version}"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"93912374_a50cf445","line":2285,"updated":"2025-05-08 14:40:07.000000000","message":"This seems like an oddly deep place to override the version that people wouldn\u0027t expect. It also ties the distro, the _current_ state of requirements, and devstack all together. Can we not set this in a `local.conf` for those jobs or something?","commit_id":"36434ca7f84e8939666a2ad03072686b8f27945b"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"689d549d65f6b50491c89943f1bf69ac95de073c","unresolved":true,"context_lines":[{"line_number":2282,"context_line":"    if [[ -x $(which $interp 2\u003e /dev/null) ]]; then"},{"line_number":2283,"context_line":"        version\u003d$($interp -c \u0027import sys; print(\"%s.%s\" % sys.version_info[0:2])\u0027)"},{"line_number":2284,"context_line":"    fi"},{"line_number":2285,"context_line":"    if [[ $DISTRO \u003d~ rhel9 ]]; then"},{"line_number":2286,"context_line":"        echo 3.11"},{"line_number":2287,"context_line":"    else"},{"line_number":2288,"context_line":"        echo ${version}"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"a6ed4980_7ada3d1d","line":2285,"in_reply_to":"93912374_a50cf445","updated":"2025-05-08 14:46:13.000000000","message":"Actually i did here considering these changes temporary until centos 10 stream support is ready and also to work out of the box on 9-stream outside of CI as currently it\u0027s not possible to deploy it.","commit_id":"36434ca7f84e8939666a2ad03072686b8f27945b"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e367e288b33fa8f57ef7611e7be7a67230c59c91","unresolved":true,"context_lines":[{"line_number":2282,"context_line":"    if [[ -x $(which $interp 2\u003e /dev/null) ]]; then"},{"line_number":2283,"context_line":"        version\u003d$($interp -c \u0027import sys; print(\"%s.%s\" % sys.version_info[0:2])\u0027)"},{"line_number":2284,"context_line":"    fi"},{"line_number":2285,"context_line":"    if [[ $DISTRO \u003d~ rhel9 ]]; then"},{"line_number":2286,"context_line":"        echo 3.11"},{"line_number":2287,"context_line":"    else"},{"line_number":2288,"context_line":"        echo ${version}"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"bfb7459b_e5179d36","line":2285,"in_reply_to":"a6ed4980_7ada3d1d","updated":"2025-05-08 19:50:03.000000000","message":"i woudl have done it here\nhttps://github.com/openstack/devstack/blob/master/stackrc#L129-L132\n\nthe same way we set the global_venv  based on the distro\n\nhttps://github.com/openstack/devstack/blob/master/stackrc#L180","commit_id":"36434ca7f84e8939666a2ad03072686b8f27945b"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"f7f943e36f369df25d15a74bae30ccf25784fe6a","unresolved":true,"context_lines":[{"line_number":2282,"context_line":"    if [[ -x $(which $interp 2\u003e /dev/null) ]]; then"},{"line_number":2283,"context_line":"        version\u003d$($interp -c \u0027import sys; print(\"%s.%s\" % sys.version_info[0:2])\u0027)"},{"line_number":2284,"context_line":"    fi"},{"line_number":2285,"context_line":"    if [[ $DISTRO \u003d~ rhel9 ]]; then"},{"line_number":2286,"context_line":"        echo 3.11"},{"line_number":2287,"context_line":"    else"},{"line_number":2288,"context_line":"        echo ${version}"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"cc9d43ba_7c474e46","line":2285,"in_reply_to":"bfb7459b_e5179d36","updated":"2025-05-09 05:16:04.000000000","message":"Ok thx, will update and add it to stackrc.","commit_id":"36434ca7f84e8939666a2ad03072686b8f27945b"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"8354c54075be1336f0e01c5dc97761aa9345bf6a","unresolved":false,"context_lines":[{"line_number":2282,"context_line":"    if [[ -x $(which $interp 2\u003e /dev/null) ]]; then"},{"line_number":2283,"context_line":"        version\u003d$($interp -c \u0027import sys; print(\"%s.%s\" % sys.version_info[0:2])\u0027)"},{"line_number":2284,"context_line":"    fi"},{"line_number":2285,"context_line":"    if [[ $DISTRO \u003d~ rhel9 ]]; then"},{"line_number":2286,"context_line":"        echo 3.11"},{"line_number":2287,"context_line":"    else"},{"line_number":2288,"context_line":"        echo ${version}"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"5ce3b587_2e748dbd","line":2285,"in_reply_to":"cc9d43ba_7c474e46","updated":"2025-05-09 05:28:21.000000000","message":"Done","commit_id":"36434ca7f84e8939666a2ad03072686b8f27945b"}],"stackrc":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d15ea00886f4fb65a8caf0794d9e4a614f4b3119","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    # https://review.opendev.org/c/openstack/devstack/+/937251"},{"line_number":137,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-3.11}"},{"line_number":138,"context_line":"else"},{"line_number":139,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-${_DEFAULT_PYTHON3_VERSION:-3}}"},{"line_number":140,"context_line":"fi"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"# Create a virtualenv with this"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"74bd1f60_32f8bc0d","line":139,"range":{"start_line":139,"start_character":74,"end_line":139,"end_character":75},"updated":"2025-05-09 17:35:46.000000000","message":"maybe later but we should keep bumping this also as per min supported python version for that cycle unless some jobs testing old distro/old python.\n\nanwyays, this is unrelated to this change so i m ok with current proposal.","commit_id":"fd75e29d0d48df5689d95429f0eafaf93bc4fbe4"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ebc6d6574d39091b430dde1aa34bfa9dc59f3f2d","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    # https://review.opendev.org/c/openstack/devstack/+/937251"},{"line_number":137,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-3.11}"},{"line_number":138,"context_line":"else"},{"line_number":139,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-${_DEFAULT_PYTHON3_VERSION:-3}}"},{"line_number":140,"context_line":"fi"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"# Create a virtualenv with this"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"e829d8d2_1b4e8106","line":139,"range":{"start_line":139,"start_character":74,"end_line":139,"end_character":75},"in_reply_to":"58f9b2c3_db4b0fd8","updated":"2025-05-12 16:02:54.000000000","message":"I am usually ok with this but cannot diagree with Dan concern which is valid. I think doing it in job side is better and will be clear if anyone update/fix/look into those jobs for newer python. keeping the hack here will be hidden for many people.\n\nOn multiple place fixes, you can set the same in devstack in-tree defined base jobs. I hope those might be the base for many of c9s/rocky jobs and might not require fixes other places but if required, let\u0027s do it on job side.\n\nFYI, I propose to stop running those jobs as they are 100% failing, once you pin the pythin version, i can abandon mine - https://review.opendev.org/c/openstack/devstack/+/948592","commit_id":"fd75e29d0d48df5689d95429f0eafaf93bc4fbe4"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"25ce9959deed2638b34f1af6051c63107451d20a","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    # https://review.opendev.org/c/openstack/devstack/+/937251"},{"line_number":137,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-3.11}"},{"line_number":138,"context_line":"else"},{"line_number":139,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-${_DEFAULT_PYTHON3_VERSION:-3}}"},{"line_number":140,"context_line":"fi"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"# Create a virtualenv with this"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"58f9b2c3_db4b0fd8","line":139,"range":{"start_line":139,"start_character":74,"end_line":139,"end_character":75},"in_reply_to":"6e9bb87f_a58b6b8c","updated":"2025-05-12 15:44:15.000000000","message":"The python version was choosen as provided by rhel9 distros and supported with 2025.2, as can see both 9-stream and rocky9 jobs green with this patch.\nAlso the same variable takes care of installing it and the hack was basically to make devstack work out of the box on rhel9 hosts irrespective of CI or non CI.\n\nBut if others feel the same I can split this change and let respective projects fix their CI jobs. Even with that local devstack setup will not work out of the box.","commit_id":"fd75e29d0d48df5689d95429f0eafaf93bc4fbe4"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"fb62b5fd223e721a1edf619a711bc7df1f3c8ae3","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    # https://review.opendev.org/c/openstack/devstack/+/937251"},{"line_number":137,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-3.11}"},{"line_number":138,"context_line":"else"},{"line_number":139,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-${_DEFAULT_PYTHON3_VERSION:-3}}"},{"line_number":140,"context_line":"fi"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"# Create a virtualenv with this"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"b270ef3b_e7842293","line":139,"range":{"start_line":139,"start_character":74,"end_line":139,"end_character":75},"in_reply_to":"74bd1f60_32f8bc0d","updated":"2025-05-12 15:24:06.000000000","message":"I still don\u0027t understand why we\u0027re encoding 3.11 here in devstack instead of setting PYTHON3_VERSION in *jobs* that are specific to rhel9. This is better than where it was being done before, but it still seems wrong to mention a specific version here. Are there other examples of us doing this in devstack?","commit_id":"fd75e29d0d48df5689d95429f0eafaf93bc4fbe4"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"7e0f31f29f8d5be50404bb7909efb8859a990945","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    # https://review.opendev.org/c/openstack/devstack/+/937251"},{"line_number":137,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-3.11}"},{"line_number":138,"context_line":"else"},{"line_number":139,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-${_DEFAULT_PYTHON3_VERSION:-3}}"},{"line_number":140,"context_line":"fi"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"# Create a virtualenv with this"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"b36835a9_33ccfb34","line":139,"range":{"start_line":139,"start_character":74,"end_line":139,"end_character":75},"in_reply_to":"b270ef3b_e7842293","updated":"2025-05-12 15:29:13.000000000","message":"@dan actually jobs are distributed across repos so no single change can do that, and also it\u0027s not just job specific but distro specific like local setup also broken so changing it in devstack itself.\nAlso this change temporary so we revert once centos 10-stream support get\u0027s ready.\n\nOne other example for distro specific things is setting GLOBAL_VENV for rhel 9 distros as done in https://github.com/openstack/devstack/blob/master/stackrc#L180","commit_id":"fd75e29d0d48df5689d95429f0eafaf93bc4fbe4"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"75031a95c6e02ba8df63671c93a19676f42e3604","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    # https://review.opendev.org/c/openstack/devstack/+/937251"},{"line_number":137,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-3.11}"},{"line_number":138,"context_line":"else"},{"line_number":139,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-${_DEFAULT_PYTHON3_VERSION:-3}}"},{"line_number":140,"context_line":"fi"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"# Create a virtualenv with this"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"6e9bb87f_a58b6b8c","line":139,"range":{"start_line":139,"start_character":74,"end_line":139,"end_character":75},"in_reply_to":"b36835a9_33ccfb34","updated":"2025-05-12 15:35:24.000000000","message":"I didn\u0027t say \"a single change\" would do it, it just feels like the job config is the proper place and not devstack. Devstack should be as timeless as possible (IMHO) and not need modifications to run because of specific current conditions. These sorts of hacks make it hard(er) to run devstack on older releases when you need to do a backport or something.\n\nThe example you cite is totally different - it\u0027s setting a flag for behavior based on the distro version. That\u0027s entirely different and much less of a concern than picking an arbitrary python version for a given distro, for a short period if time, and which isn\u0027t even installed by/for the distro itself.","commit_id":"fd75e29d0d48df5689d95429f0eafaf93bc4fbe4"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"e132aec59b8f7fa66d815a3595287c5bfc39b957","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    # https://review.opendev.org/c/openstack/devstack/+/937251"},{"line_number":137,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-3.11}"},{"line_number":138,"context_line":"else"},{"line_number":139,"context_line":"    export PYTHON3_VERSION\u003d${PYTHON3_VERSION:-${_DEFAULT_PYTHON3_VERSION:-3}}"},{"line_number":140,"context_line":"fi"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"# Create a virtualenv with this"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"966f7cc0_6d69c254","line":139,"range":{"start_line":139,"start_character":74,"end_line":139,"end_character":75},"in_reply_to":"e829d8d2_1b4e8106","updated":"2025-05-12 16:05:16.000000000","message":"Ok thx, updated it on job side.","commit_id":"fd75e29d0d48df5689d95429f0eafaf93bc4fbe4"}]}