)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"81eedf77d418128ab1e25ba091bd68d38ea3bbed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"eb880a5e_56dc9663","updated":"2024-06-04 15:06:19.000000000","message":"Would bumping community.docker work?","commit_id":"8580c502c6e83035bf6b40eba87ecc0f34ec7728"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"671bd137ae5d262dd16b345feb6dd1d34a5ebf52","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"aadf5578_b87b086f","in_reply_to":"eb880a5e_56dc9663","updated":"2024-06-11 09:56:29.000000000","message":"not really, we cannot manage galaxy collections as ansible runs from venv on zuul executor servers, invoked from an isolated sandbox container.","commit_id":"8580c502c6e83035bf6b40eba87ecc0f34ec7728"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"06087d34a0b622f50fdca2bb8ab52cf5453fa370","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fd4c6aa0_757b1898","updated":"2024-06-11 10:11:20.000000000","message":"Hmm, if we can create, update and use one venv, why can\u0027t we also create a second one?\n\nBut anyway, since this is hopefully just a short-term workaround, let\u0027s get on with it.","commit_id":"5b6988acd8963641e60ed45df3d98c2bfb06b158"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"8f04e3d877c7dfc79c2b0f0a0b17a30bc6807c53","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"95ff8e69_fdeee350","in_reply_to":"fd4c6aa0_757b1898","updated":"2024-06-11 10:19:44.000000000","message":"We can create a second venv, but then we would need to run ansible in ansible to pick up the collections, seems like a bit more rework of the publish workflow.\nThis is temporary, so I guess we can survive for now ;-)","commit_id":"5b6988acd8963641e60ed45df3d98c2bfb06b158"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"c167465f55b435a4401873e6a92ee86637d1e3f9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5f560473_7afe5823","updated":"2024-06-11 10:58:41.000000000","message":"re-adding +2w, fixed typo in comment","commit_id":"4b00196410a6385cb3f17973f87c683b99ae4593"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"fb876374d4a3f459dc8df0c76c0a981d25d033dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f3d3d8ad_77e78e87","updated":"2024-06-11 11:07:56.000000000","message":"and typo in commit msg :)","commit_id":"428f37124986019bedc897469f9b1f3470b62082"}],"tests/playbooks/pre.yml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"483ff78fbe7f0fc999ef8e813de69b6072fffcb3","unresolved":true,"context_lines":[{"line_number":43,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install {{ zuul.project.src_dir }}\""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    - name: Install docker python library"},{"line_number":46,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install docker requests\u003c2.32\""},{"line_number":47,"context_line":"      when: container_engine \u003d\u003d \"docker\""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    - name: Install podman python library"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"c0c96bf2_6f820e0b","line":46,"updated":"2024-06-10 08:36:58.000000000","message":"I\u0027m not comfortable doing this globally only to fix the publishing jobs. Can we build a dedicated venv for that use case instead?","commit_id":"8580c502c6e83035bf6b40eba87ecc0f34ec7728"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"66bcf1a3f040d5c4058641776acf60ca142b7f52","unresolved":false,"context_lines":[{"line_number":43,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install {{ zuul.project.src_dir }}\""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    - name: Install docker python library"},{"line_number":46,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install docker requests\u003c2.32\""},{"line_number":47,"context_line":"      when: container_engine \u003d\u003d \"docker\""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    - name: Install podman python library"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7fdff739_26e6188e","line":46,"in_reply_to":"6ee234d9_e30f79d4","updated":"2024-06-11 09:56:39.000000000","message":"Done","commit_id":"8580c502c6e83035bf6b40eba87ecc0f34ec7728"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"671bd137ae5d262dd16b345feb6dd1d34a5ebf52","unresolved":true,"context_lines":[{"line_number":43,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install {{ zuul.project.src_dir }}\""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    - name: Install docker python library"},{"line_number":46,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install docker requests\u003c2.32\""},{"line_number":47,"context_line":"      when: container_engine \u003d\u003d \"docker\""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    - name: Install podman python library"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"481cd7ba_eeda0f4e","line":46,"in_reply_to":"6ee234d9_e30f79d4","updated":"2024-06-11 09:56:29.000000000","message":"to use dedicated venv we would need to migrate this playbook to run nested ansible, as we cannot manage galaxy collections here, because ansible runs from venv on zuul executor servers, invoked from an isolated sandbox container. I proposed different approach just to downgrade requests for publish playbook - which runs last - tested here https://review.opendev.org/c/openstack/kolla/+/921505","commit_id":"8580c502c6e83035bf6b40eba87ecc0f34ec7728"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"e5cd4bcd2ba22cd1deb3bd57280c23d0ad301cae","unresolved":true,"context_lines":[{"line_number":43,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install {{ zuul.project.src_dir }}\""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    - name: Install docker python library"},{"line_number":46,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install docker requests\u003c2.32\""},{"line_number":47,"context_line":"      when: container_engine \u003d\u003d \"docker\""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    - name: Install podman python library"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"6ee234d9_e30f79d4","line":46,"in_reply_to":"c0c96bf2_6f820e0b","updated":"2024-06-10 14:21:18.000000000","message":"that reads like a good idea imho.","commit_id":"8580c502c6e83035bf6b40eba87ecc0f34ec7728"}]}
