| opendevreview | OpenStack Proposal Bot proposed openstack/openstack-ansible master: Imported Translations from Zanata https://review.opendev.org/c/openstack/openstack-ansible/+/966384 | 04:23 |
|---|---|---|
| opendevreview | Merged openstack/openstack-ansible master: Imported Translations from Zanata https://review.opendev.org/c/openstack/openstack-ansible/+/966384 | 08:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-python_venv_build master: Define trusted host for wheels build https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/966404 | 08:56 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Copy mariabackup script instead of templating https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/960373 | 08:58 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-python_venv_build master: Define trusted host for wheels build https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/966404 | 10:05 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-python_venv_build master: Use uv for package installation https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/966338 | 10:11 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-python_venv_build master: Use uv for package installation https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/966338 | 10:12 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-python_venv_build master: Use uv for package installation https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/966338 | 10:13 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Use uv for install when wheels not build https://review.opendev.org/c/openstack/openstack-ansible/+/966409 | 10:25 |
| noonedeadpunk | Let's land backports for stable/2025.1 https://review.opendev.org/q/parentproject:openstack/openstack-ansible+branch:%5Estable/.*+status:open+label:Verified :) | 10:37 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-python_venv_build master: Use uv for package installation https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/966338 | 11:18 |
| noonedeadpunk | hm. I don't really see any noticable difference in CI though... | 11:22 |
| mgariepy | most of the time is spend downloading or synching to slow disk in most case. | 11:27 |
| noonedeadpunk | well... it's actually still 2 times faster... | 11:36 |
| noonedeadpunk | just does not move the needle at scale | 11:37 |
| noonedeadpunk | we totally need some more optimizations made to the python_venv_build I guess... | 11:40 |
| mgariepy | 2x for pkg install witch is what % of the overall test? | 11:59 |
| mgariepy | maybe we can do more stuff in parallel.. some service could be deployed at the same time.. | 11:59 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-python_venv_build master: Merge wheels verification under single task https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/966426 | 12:23 |
| noonedeadpunk | mgariepy: yeah, sure, this is smth I was thinking about as well, but this is doable only through some scripting, and not with ansible at all | 12:24 |
| noonedeadpunk | but python_venv_build takes a significant amount of time of overall deployment | 12:24 |
| noonedeadpunk | we also like run it 2 times for most of services - uwsgi and non-wsgi | 12:27 |
| noonedeadpunk | but then ubuntu upgrade job got best timing in a month: https://zuul.opendev.org/t/openstack/runtime?job_name=openstack-ansible-upgrade-aio_metal_tls-ubuntu-noble&project=openstack/openstack-ansible&branch=master&pipeline=check | 12:37 |
| mgariepy | ok | 12:44 |
| opendevreview | Merged openstack/openstack-ansible-os_keystone stable/2025.1: Fix URI for Shibboleth on EL https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/966206 | 12:47 |
| noonedeadpunk | but I agree that parallel execution of playbooks is smth which would improve overall performance a lot | 12:51 |
| mgariepy | even if we only do 2 batches most infra can be done at the same time. | 12:52 |
| mgariepy | but wouldn't work much on metal. | 12:52 |
| mgariepy | and on metal it might causes some conflicting status that wouldn't be fun to debug. | 12:53 |
| noonedeadpunk | um, depends. I'd expect that nova and neutron can run at same time. same for cinder and placement and glance for instance | 12:54 |
| mgariepy | until you install pkg. | 12:54 |
| noonedeadpunk | except, probably, apt part | 12:54 |
| noonedeadpunk | right | 12:54 |
| mgariepy | which will lock the apt stuff. | 12:54 |
| mgariepy | probably the same on dnf.. | 12:55 |
| noonedeadpunk | right... | 12:55 |
| mgariepy | and haproxy endoint manage stuff can cause race condition if multiple playbook decide that they need to do it right now. | 12:56 |
| noonedeadpunk | this is smth I'm least concerned about, tbh. as I guess haproxy can handle multiple commands almost together | 13:02 |
| noonedeadpunk | we're sending to socket more or les | 13:02 |
| noonedeadpunk | so as long as it's not the same frontend/backend it might be fine | 13:03 |
| mgariepy | still we could have some weird issue racing. | 13:03 |
| noonedeadpunk | yeah | 13:04 |
| mgariepy | maybe not haproxy but there are multiple places where it can race if not thought carefully. | 13:04 |
| noonedeadpunk | like ceph... | 13:04 |
| opendevreview | Merged openstack/openstack-ansible stable/2024.1: Do not require full git & autoconf on EL https://review.opendev.org/c/openstack/openstack-ansible/+/951622 | 13:35 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron master: Deprecate and remove neutron_use_uwsgi variable https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/961345 | 14:51 |
| opendevreview | Damian Dąbrowski proposed openstack/ansible-role-pki master: Add hashi_vault backend https://review.opendev.org/c/openstack/ansible-role-pki/+/948881 | 14:56 |
| opendevreview | Merged openstack/ansible-role-uwsgi stable/2025.1: Replace libssl3 with libssl3t64 https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/965342 | 15:32 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Use uv for python packages install https://review.opendev.org/c/openstack/openstack-ansible/+/966409 | 15:57 |
| opendevreview | Merged openstack/openstack-ansible-os_octavia stable/2024.1: Add default values to environment lookups https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/958941 | 16:49 |
| noonedeadpunk | It *feels* like we're getting ~5 mins benefit from uv.... | 17:54 |
| noonedeadpunk | would be really nice to also prepare venv with it, as it seemed to execute it faster as well | 17:56 |
| noonedeadpunk | but really hard to say due to discrepency between providers | 18:04 |
| OS-Keystone | Hi | 18:25 |
| opendevreview | Merged openstack/openstack-ansible stable/2025.1: Upgrade setuptools to version 75.8.2 https://review.opendev.org/c/openstack/openstack-ansible/+/966191 | 18:58 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!