)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ecc223782724a4cd1c17066ed6ba435f5ba11538","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Enable GLOBAL_VENV\u003dTrue for all distros"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change removes the workaround for centos 9 stream"},{"line_number":10,"context_line":"and related distos which disabled the use of the GLOBAL_VENV"},{"line_number":11,"context_line":"due to a lack fo supprot for PEP-518 in openstack projects."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"67d8525f_0a4a1323","line":9,"range":{"start_line":9,"start_character":39,"end_line":9,"end_character":54},"updated":"2025-05-06 23:45:28.000000000","message":"c9s is mostly irrelvent at this point but removing the special casing and just have a single way to install (always using a gobal venv regardles of the disto) means this is still a a worth while goal.\n\nif we can eventually drop supprot for installing globally with sudo that would also be nice but im wonder how close we are to this.\n\ni suspect that in addtion to the pyproject.toml patches listed in depend on we need\nto also convert provide the \u003cservice\u003e/wsgi/\u003centrypoint\u003e.py module in each service\nand update devstack to use the module in uwsgi\n\nso i expect this to fail but we hopefully will be able to resolve those depencies over the next few days as each project fixes the issues realted to not having a pyproject.toml and makes progress on https://github.com/openstack/governance/blob/8cdd19ac689ec9a0ffddcd8dfd704df90ad72d2e/goals/proposed/migrate-from-wsgi-scripts-to-module-paths.rst#L26","commit_id":"332b98e0f01465831b4c84dcd0debd7bb94559ee"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"cbc7a3e5a58055fad923913d3324dfcc0fcf1b4d","unresolved":true,"context_lines":[{"line_number":39,"context_line":"for the PBR release provided the system has a new enough setuptools."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Closes-Bug: #2031639"},{"line_number":42,"context_line":"Depends-On: https://review.opendev.org/c/openstack/keystone/+/899519"},{"line_number":43,"context_line":"Depends-On: https://review.opendev.org/c/openstack/cinder/+/945558"},{"line_number":44,"context_line":"Depends-On: https://review.opendev.org/c/openstack/glance/+/899951"},{"line_number":45,"context_line":"Depends-On: https://review.opendev.org/c/openstack/neutron/+/899956"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"e57e9144_7085e539","line":42,"updated":"2025-05-13 11:25:26.000000000","message":"this is now merged so we can drop next time we need to respin","commit_id":"332b98e0f01465831b4c84dcd0debd7bb94559ee"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"cbc7a3e5a58055fad923913d3324dfcc0fcf1b4d","unresolved":true,"context_lines":[{"line_number":42,"context_line":"Depends-On: https://review.opendev.org/c/openstack/keystone/+/899519"},{"line_number":43,"context_line":"Depends-On: https://review.opendev.org/c/openstack/cinder/+/945558"},{"line_number":44,"context_line":"Depends-On: https://review.opendev.org/c/openstack/glance/+/899951"},{"line_number":45,"context_line":"Depends-On: https://review.opendev.org/c/openstack/neutron/+/899956"},{"line_number":46,"context_line":"Depends-On: https://review.opendev.org/c/openstack/horizon/+/899953"},{"line_number":47,"context_line":"Depends-On: https://review.opendev.org/c/openstack/swift/+/899958"},{"line_number":48,"context_line":"Change-Id: Ife6c97869d50cf59b595a5c7477d2d985b9cca38"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"67b1df47_39eb0e70","line":45,"updated":"2025-05-13 11:25:26.000000000","message":"same for neutron.","commit_id":"332b98e0f01465831b4c84dcd0debd7bb94559ee"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"cbc7a3e5a58055fad923913d3324dfcc0fcf1b4d","unresolved":true,"context_lines":[{"line_number":44,"context_line":"Depends-On: https://review.opendev.org/c/openstack/glance/+/899951"},{"line_number":45,"context_line":"Depends-On: https://review.opendev.org/c/openstack/neutron/+/899956"},{"line_number":46,"context_line":"Depends-On: https://review.opendev.org/c/openstack/horizon/+/899953"},{"line_number":47,"context_line":"Depends-On: https://review.opendev.org/c/openstack/swift/+/899958"},{"line_number":48,"context_line":"Change-Id: Ife6c97869d50cf59b595a5c7477d2d985b9cca38"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"a287a65c_6eee5e1d","line":47,"updated":"2025-05-13 11:25:26.000000000","message":"swift does not us pbr to provide the rest api as a freestandign wsgi applciaiton os this is should not be required to proceed but cinder glance and horizon +\n\nany projects that are installed via plugins would need to be updated.\n\nhaving said that they could alwasy disable it in there jobs i.e. GLOBAL_VENV\u003dFalse if they happen to run on centos which htey likely dont so if the base josb are passing we likely dont need to wait for the long tail.\n\nwe have proven the pattern of adding the wsgi module and updating the devstack plugin so there is an established pattern for them to follow to update there plugins.","commit_id":"332b98e0f01465831b4c84dcd0debd7bb94559ee"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"005e922565ce9cac8da9632e5865a8479dd5d115","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"79f9bc70_48f7d5fd","updated":"2023-10-27 02:10:29.000000000","message":"we chatted about the reports that enabling GLOBAL_VENV\u003dTrue did not work on centos9 stream\n\ni finally got around to trying to replicate this tonight and i think i have mostly figured out what going on and partly how to adress it but it appear that in newer pip version they have vendored wheel. i say this because with pip 23.x.y i can see we are building an editiabel wheel and then in stalling that but we dont have wheel installed (based on pip freeze). when using older pip we do not build a wheel and we generate the wsgi files in the vnev\u0027s bin dir.\n\nso we might need to fix this in pbr but we can temporally pin i belive to make it functional in devstack too.","commit_id":"814fa3b6877028ba2961b82865b173ada55ae487"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"003954f387125e4edc988188a674e3c1602b3bf5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b7bb5972_84930c2f","updated":"2024-05-21 15:35:47.000000000","message":"recheck fresh results","commit_id":"65cd909bef29dcf5fa6ed9dd9fc3ba7b756d9772"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"65e42d100e93bf6ce931be99c09722c83f180b1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d9466699_9b7a05b2","updated":"2023-11-08 09:34:06.000000000","message":"recheck pbr 6.0.0 has been released","commit_id":"65cd909bef29dcf5fa6ed9dd9fc3ba7b756d9772"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"392635466798252992228c09a4c3c9bfdd8e1b4b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a6bd4597_009f20e2","updated":"2024-08-20 15:05:11.000000000","message":"recheck to see the latest status","commit_id":"65cd909bef29dcf5fa6ed9dd9fc3ba7b756d9772"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0f36583748db364aca5a48d2bf88411f2d0cee25","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7c08f5ac_c25d8544","updated":"2024-08-20 15:48:45.000000000","message":"this need https://review.opendev.org/q/topic:%22remove-wsgi_scripts%22","commit_id":"65cd909bef29dcf5fa6ed9dd9fc3ba7b756d9772"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"aa0d6a6a8b8630e9ef2a5c63aa14589722b2bffa","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"486332a7_d401e8e1","in_reply_to":"a6bd4597_009f20e2","updated":"2024-08-20 15:47:02.000000000","message":"i do not expect this to pass on centos until we stop using pbr to generate teh wsgi script as it puts it in the worng place.","commit_id":"65cd909bef29dcf5fa6ed9dd9fc3ba7b756d9772"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"deafe7c4cb38ece11c6db8272004a16acd2f0391","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"93a0df63_92fa11b9","updated":"2025-05-13 06:52:19.000000000","message":"I really hope we can get this done this cycle, sadly most deps don\u0027t look good yet","commit_id":"332b98e0f01465831b4c84dcd0debd7bb94559ee"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"fb8fc6c4bbfc549fba4459aaa2adecc673149504","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"1221330c_dfcf5f1f","updated":"2025-05-13 03:11:31.000000000","message":"recheck 9-stream and rocky9 jobs should work now","commit_id":"332b98e0f01465831b4c84dcd0debd7bb94559ee"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"cd5b479eb2bd43ee8e0b98472e1b5626feedbcee","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5278831a_7bc12e91","in_reply_to":"6e7dfcb8_6688c0bc","updated":"2025-05-13 12:07:52.000000000","message":"yes, code cleanup good :)","commit_id":"332b98e0f01465831b4c84dcd0debd7bb94559ee"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"fd93f35e5f9373f637ee8c503092bf818713febb","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6e7dfcb8_6688c0bc","in_reply_to":"93a0df63_92fa11b9","updated":"2025-05-13 11:02:58.000000000","message":"ya im not goign to hold my breath but the fact we now have a green run with the depens on at least shows the pattern works.\n\nwhich si promising that it should eventurally be possible.\n\n\na followup perhaps next cycle would be to then clean up the supprot for not using a single global venv and removing all the logic that installs globally with sudo so we can have only one code path to maintain.\n\nwhile that not strictly requried it would be a logical next step to reduce the code size that need to be maintained.","commit_id":"332b98e0f01465831b4c84dcd0debd7bb94559ee"}],"functions-common":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"d8e43aa3b7be6ee00e2f6f5dca3402210b8c1641","unresolved":true,"context_lines":[{"line_number":2434,"context_line":"    if [[ \"$GLOBAL_VENV\" \u003d\u003d \"True\" ]]; then"},{"line_number":2435,"context_line":"        osc_cmd\u003d\"${DEVSTACK_VENV}/bin/openstack\""},{"line_number":2436,"context_line":"    fi"},{"line_number":2437,"context_line":"    out\u003d$(${osc_cmd} \"$@\")"},{"line_number":2438,"context_line":"    rc\u003d$?"},{"line_number":2439,"context_line":"    end\u003d$(date +%s%3N)"},{"line_number":2440,"context_line":"    echo $((end - start)) \u003e\u003e $OSCWRAP_TIMER_FILE"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"8aad7d05_48af2b63","line":2437,"updated":"2023-10-27 01:50:22.000000000","message":"i dont think this is actully required\n\ni broke /usr/local/bin by delting it instead of the file in /usr/local/bin and that i belive is why the simlinx created in stack.sh broke\n\n\nif [[ \"$GLOBAL_VENV\" \u003d\u003d \"True\" ]] ; then\n    # TODO(frickler): find a better solution for this\n    sudo ln -sf /opt/stack/data/venv/bin/cinder-rtstool /usr/local/bin\n    sudo ln -sf /opt/stack/data/venv/bin/glance /usr/local/bin\n    sudo ln -sf /opt/stack/data/venv/bin/nova-manage /usr/local/bin\n    sudo ln -sf /opt/stack/data/venv/bin/openstack /usr/local/bin\n    sudo ln -sf /opt/stack/data/venv/bin/privsep-helper /usr/local/bin\n    sudo ln -sf /opt/stack/data/venv/bin/rally /usr/local/bin\n    sudo ln -sf /opt/stack/data/venv/bin/tox /usr/local/bin\n\n    setup_devstack_virtualenv\nfi\n\nresulted in /usr/local/bin being tox","commit_id":"814fa3b6877028ba2961b82865b173ada55ae487"}],"inc/python":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"d8e43aa3b7be6ee00e2f6f5dca3402210b8c1641","unresolved":true,"context_lines":[{"line_number":41,"context_line":"        # This package is currently installed via the distro and not"},{"line_number":42,"context_line":"        # available on pypi."},{"line_number":43,"context_line":"        python$PYTHON3_VERSION -m venv --system-site-packages $DEVSTACK_VENV"},{"line_number":44,"context_line":"        pip_install -U pip\u003d\u003d21.1.3"},{"line_number":45,"context_line":"    fi"},{"line_number":46,"context_line":"    if [[ \":$PATH:\" !\u003d *\":$DEVSTACK_VENV/bin:\"* ]] ; then"},{"line_number":47,"context_line":"        export PATH\u003d\"$DEVSTACK_VENV/bin:$PATH\""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"2b409945_9a1f076a","line":44,"updated":"2023-10-27 01:50:22.000000000","message":"this i belive would fix the ci jobs.\n\nthe actual fix however will likely be to rework\n\nhttps://github.com/openstack/pbr/blob/d03d617c09e7ba8ddf62d1e53d71685cd708e2da/pbr/packaging.py#L493-L498\nto either always run or also run when building wheels","commit_id":"814fa3b6877028ba2961b82865b173ada55ae487"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f621deb1fd19647dcdd743af66cc9624c19b8223","unresolved":true,"context_lines":[{"line_number":41,"context_line":"        # This package is currently installed via the distro and not"},{"line_number":42,"context_line":"        # available on pypi."},{"line_number":43,"context_line":"        python$PYTHON3_VERSION -m venv --system-site-packages $DEVSTACK_VENV"},{"line_number":44,"context_line":"        pip_install -U pip\u003d\u003d21.1.3"},{"line_number":45,"context_line":"    fi"},{"line_number":46,"context_line":"    if [[ \":$PATH:\" !\u003d *\":$DEVSTACK_VENV/bin:\"* ]] ; then"},{"line_number":47,"context_line":"        export PATH\u003d\"$DEVSTACK_VENV/bin:$PATH\""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"bc61d04e_7d0c1e20","line":44,"in_reply_to":"2b409945_9a1f076a","updated":"2023-10-27 11:47:54.000000000","message":"ok so this fixed centos and ubuntu but broke debian becase\novs python bindings on deabin? requires PEP 517 support and this version of pep appreanly does not support that based on the error message.\n\n\n  ----------------------------------------\n  ERROR: Failed building wheel for ovs\n  Building wheel for logutils (setup.py): started\n  Building wheel for logutils (setup.py): finished with status \u0027done\u0027\n  Created wheel for logutils: filename\u003dlogutils-0.3.5-py3-none-any.whl size\u003d19936 sha256\u003d25fe740a2829f7599e9ab5da66924e9e74782922b12c4a52ba1894cc0b407676\n  Stored in directory: /opt/stack/.cache/pip/wheels/be/de/be/02af117b11596a23db9f3612b8d9d71edde2a587ab056082b5\nSuccessfully built ncclient logutils\nFailed to build ovs\nERROR: Could not build wheels for ovs which use PEP 517 and cannot be installed directly\nWARNING: You are using pip version 21.1.3; however, version 23.3.1 is available.\n\n\nso the bug i think is still in pbr and it does not fully support wsgi scripts in pep-517 mode.","commit_id":"814fa3b6877028ba2961b82865b173ada55ae487"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b3551e8fdadb7885a598881c36b2d29da5beb061","unresolved":true,"context_lines":[{"line_number":41,"context_line":"        # This package is currently installed via the distro and not"},{"line_number":42,"context_line":"        # available on pypi."},{"line_number":43,"context_line":"        python$PYTHON3_VERSION -m venv --system-site-packages $DEVSTACK_VENV"},{"line_number":44,"context_line":"        pip_install -U pip\u003d\u003d21.1.3"},{"line_number":45,"context_line":"    fi"},{"line_number":46,"context_line":"    if [[ \":$PATH:\" !\u003d *\":$DEVSTACK_VENV/bin:\"* ]] ; then"},{"line_number":47,"context_line":"        export PATH\u003d\"$DEVSTACK_VENV/bin:$PATH\""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"46058348_3b25adb4","line":44,"in_reply_to":"bc61d04e_7d0c1e20","updated":"2023-10-28 06:09:56.000000000","message":"ok i figure this out https://review.opendev.org/c/openstack/keystone/+/899519\nthis is the correct way to fix this i belvie","commit_id":"814fa3b6877028ba2961b82865b173ada55ae487"}]}