| opendevreview | Merged openstack/openstack-ansible master: Upgrade setuptools to version 75.8.2 https://review.opendev.org/c/openstack/openstack-ansible/+/964773 | 00:09 |
|---|---|---|
| opendevreview | Merged openstack/openstack-ansible master: [doc] Document Debian 13 (Trixie) support https://review.opendev.org/c/openstack/openstack-ansible/+/965328 | 00:10 |
| opendevreview | Merged openstack/openstack-ansible master: [doc] Move EOL releases from compatability matrix https://review.opendev.org/c/openstack/openstack-ansible/+/965333 | 00:10 |
| noonedeadpunk | hm, so why in the world https://zuul.opendev.org/t/openstack/build/1e325cc85d024f4aa5afc2039dc51223 is failing.... | 08:52 |
| noonedeadpunk | another vote on https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/965115 would be nice before I do bumps for the release | 08:59 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins stable/2025.1: [DNM] Check molecule out https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966150 | 10:10 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Split glusterd tasks to task files https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966116 | 10:35 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins stable/2025.1: [DNM] Check molecule out https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966150 | 10:38 |
| noonedeadpunk | I think this is actually a reason why we might wanna have default ansible tmp dir to /tmp.... | 10:39 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Set ANSIBLE_LOCAL_TEMP to /tmp to prevent remote_tmp warnings https://review.opendev.org/c/openstack/openstack-ansible/+/953218 | 10:50 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Set ANSIBLE_LOCAL_TEMP to /tmp to prevent remote_tmp warnings https://review.opendev.org/c/openstack/openstack-ansible/+/953218 | 10:50 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins stable/2025.1: Set Ansible config directory to /tmp for molecule https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966150 | 10:51 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins stable/2025.1: Install setuptools for EL10 https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/965941 | 10:51 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Set ANSIBLE_LOCAL_TEMP to /tmp to prevent remote_tmp warnings https://review.opendev.org/c/openstack/openstack-ansible/+/953218 | 10:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: [DNM] Clean-up stale gluster peers https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966117 | 10:56 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Allow to skip pacemaker/corosync setup https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966065 | 10:56 |
| noonedeadpunk | damiandabrowski: I have not yet checked defaults, but posted what I already had for https://review.opendev.org/c/openstack/ansible-role-pki/+/948881 | 11:18 |
| noonedeadpunk | and I also spotted that these jobs are not executed in PKI - so we probably need to have another patch adding template there: https://review.opendev.org/c/openstack/openstack-ansible/+/948889/12/zuul.d/project-templates.yaml | 11:19 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-python_venv_build master: Allow to re-create wheels build venv https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/966162 | 12:10 |
| mgariepy | anyone runs 2024.1 ? any forks for the object store signed url issue ? | 12:31 |
| mgariepy | patch from 2024.2 does applies, beside some minor conflict on the style in unit-test. | 12:34 |
| noonedeadpunk | you're about swift and keystone patches from yesterday? | 12:34 |
| mgariepy | yes | 12:34 |
| noonedeadpunk | well, we don't run swift. and I haven't seen issues with rgw? | 12:35 |
| mgariepy | from what i read it's also affecting ceph object | 12:35 |
| noonedeadpunk | or are they present? | 12:35 |
| mgariepy | not 100% clear IMO. | 12:35 |
| noonedeadpunk | well. we're running 2025.1 already, but haven't seen any claims on broken rgw so far. And I did some testing of S3 in rgw with patches... Nothing too deep though | 12:36 |
| noonedeadpunk | but basic create/delete bucket/object did work as expected | 12:37 |
| mgariepy | i'll dig around and see. | 12:43 |
| mgariepy | canonical did apply the patch as-is for 2024.1 | 12:59 |
| opendevreview | Merged openstack/openstack-ansible-rabbitmq_server master: Add yum2 repo for EL distros https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/965198 | 13:10 |
| noonedeadpunk | https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966150/ <- this is fixing the rocky9 molecule issue | 13:14 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Bump global requirements https://review.opendev.org/c/openstack/openstack-ansible/+/951288 | 13:16 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2024.1: Bump SHAs for 2024.1 https://review.opendev.org/c/openstack/openstack-ansible/+/966174 | 13:28 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2025.1: Upgrade setuptools to version 75.8.2 https://review.opendev.org/c/openstack/openstack-ansible/+/966191 | 15:14 |
| opendevreview | Merged openstack/openstack-ansible-plugins stable/2025.1: Set Ansible config directory to /tmp for molecule https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966150 | 16:59 |
| noonedeadpunk | wtrf why https://zuul.opendev.org/t/openstack/build/eb61e925abf24fe9bae053b1ba31b365 is still failing... | 17:24 |
| opendevreview | Merged openstack/openstack-ansible-os_keystone master: Fix URI for Shibboleth on EL https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/965200 | 17:29 |
| opendevreview | Dmitriy Rabotyagov proposed 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 | 17:30 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Allow to reset peer during it's re-install https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966117 | 18:11 |
| opendevreview | Merged openstack/openstack-ansible master: Bump global requirements https://review.opendev.org/c/openstack/openstack-ansible/+/951288 | 18:14 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Document glusterfs_reset_peers https://review.opendev.org/c/openstack/openstack-ansible/+/966209 | 18:20 |
| noonedeadpunk | jrosser: so I come up with the following regarding gluster https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966117 It's super complex :( But it should be safe and idempotent. The only thing I didn't check is if can be run during initial setup | 18:21 |
| noonedeadpunk | thus - added a flag to include the code | 18:22 |
| jrosser | I did wonder if a filter would help | 18:22 |
| jrosser | but I admit I spent zero time to understand the jinja | 18:23 |
| noonedeadpunk | yeah, probably filter might help here | 18:24 |
| noonedeadpunk | The problem is that data is collected from all cluster hosts, and then it should be combined together... | 18:24 |
| noonedeadpunk | So filter indeed can simplify part of it | 18:24 |
| noonedeadpunk | Like specifically `_glusterd_peer_mapping` part | 18:25 |
| noonedeadpunk | But I was reluctant to make a filter until I understand what I need to do at all | 18:25 |
| jrosser | oh indeed | 18:26 |
| jrosser | it does worry me the maintainability currently | 18:26 |
| noonedeadpunk | of jinja or of filter ?:) | 18:26 |
| jrosser | I suspect that a python equivalent would be able to be written in a more obvious way | 18:26 |
| jrosser | but like I say I spend 0 time actually to try to understand yet :) | 18:27 |
| jrosser | other than 8-0 | 18:27 |
| noonedeadpunk | well. it won't be super easy in python either | 18:28 |
| noonedeadpunk | I can probably check on python filter. but also - do we want to add it to the collection overall? | 18:28 |
| noonedeadpunk | as it would be tailored to this specific case | 18:29 |
| noonedeadpunk | will try to check on it maybe during weekends.... | 18:29 |
| noonedeadpunk | would be also nice to cover it with tests... | 18:31 |
| jrosser | maybe there is an even more extreme approach | 18:32 |
| jrosser | to just take the result of map/select the whole of hostvars structure for the repo group and pass that into a filter | 18:33 |
| jrosser | and then just return if there is anything to clean up | 18:33 |
| jrosser | ie basically all python and no jinja | 18:34 |
| jrosser | I just mention this because when faced with only the jinja operators it forces you down a particular path to a solution | 18:35 |
| noonedeadpunk | I'm not sure about such huge amount of data passing around? | 18:35 |
| jrosser | it may not be so large tbh | 18:35 |
| jrosser | anyway I have a think about it | 18:35 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins stable/2025.1: [DNM] https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966211 | 18:37 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins stable/2025.1: [DNM] https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966211 | 18:50 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins stable/2025.1: [DNM] https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966211 | 19:00 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Allow to reset peer during it's re-install https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966117 | 19:48 |
| noonedeadpunk | hm, so seems that https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966150/3 has no effect and https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/966211? failing intermittently on rocky9 | 21:06 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!