| opendevreview | Verification of a change to openstack/kolla-ansible stable/2025.2 failed: rabbitmq: fix precheck missing backend TLS cert fallback https://review.opendev.org/c/openstack/kolla-ansible/+/1004642 | 00:29 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/kolla master: Updated from check-sources https://review.opendev.org/c/openstack/kolla/+/1004951 | 02:23 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2026.1: kolla-toolbox: Retry ansible --version check https://review.opendev.org/c/openstack/kolla-ansible/+/1004916 | 05:35 |
| opendevreview | Pierre Riteau proposed openstack/kayobe stable/2026.1: Fix duplicate word https://review.opendev.org/c/openstack/kayobe/+/1004966 | 07:05 |
| opendevreview | Rowan Johns proposed openstack/tenks master: Add support for persistent networking https://review.opendev.org/c/openstack/tenks/+/998806 | 07:46 |
| opendevreview | Rowan Johns proposed openstack/tenks master: Add support for persistent networking https://review.opendev.org/c/openstack/tenks/+/998806 | 08:35 |
| opendevreview | Tikhon Masalov proposed openstack/kolla-ansible stable/2026.1: Fix trailing comma bug in DockerWorker.generate_tls() https://review.opendev.org/c/openstack/kolla-ansible/+/1004932 | 09:17 |
| opendevreview | Merged openstack/kolla master: Updated from check-sources https://review.opendev.org/c/openstack/kolla/+/1004951 | 10:06 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Fix configuration dump with ansible-core 2.19+ https://review.opendev.org/c/openstack/kayobe/+/1004990 | 10:33 |
| opendevreview | Jed Preist proposed openstack/kayobe master: Refactor baremetal node registration https://review.opendev.org/c/openstack/kayobe/+/952979 | 10:35 |
| blanson[m] | morning! | 10:35 |
| blanson[m] | https://review.opendev.org/c/openstack/kolla-ansible/+/965471 if someone can take a quick look, this has been sitting in RP for ages I feel like | 10:36 |
| blanson[m] | yh close to a year now lol | 10:36 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Fix configuration dump with ansible-core 2.19+ https://review.opendev.org/c/openstack/kayobe/+/1004990 | 11:18 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: Drop support for CentOS Stream 10 https://review.opendev.org/c/openstack/kolla/+/997944 | 11:20 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: base: Drop redhat binary versionlock block https://review.opendev.org/c/openstack/kolla/+/1004817 | 11:20 |
| mnasiadka | blanson[m]:only a year ;) | 11:20 |
| opendevreview | Tikhon Masalov proposed openstack/kolla-ansible stable/2026.1: Fix trailing comma bug in DockerWorker.generate_tls() https://review.opendev.org/c/openstack/kolla-ansible/+/1004932 | 11:23 |
| fprzewozn | blanson[m] hmm is it really still needed? I'm configuring .keyring with: key = {{ cinder_key }} where cinder_key is variable in passwords.yml with success on envs deployed in 2025.1 | 11:37 |
| blanson[m] | mnasiadka: standard waiting time, nothing to see here :D | 11:38 |
| tafkamax | Is it needed if you umm have a direct file e.g. config/nova/ceph.client.cinder-volumes.keyring and you want to reference it inside there? | 11:39 |
| tafkamax | currently we encrypt the whole .keyring file aswell | 11:39 |
| fprzewozn | I'm talking about f.e. cinder/cinder-volume/ceph-1.cinder.keyring file unencrypted, with standard ceph format inside and key itself being referenced with {{ cinder_key}} | 11:43 |
| fprzewozn | and it's working fine without such change implemented | 11:44 |
| tafkamax | Maybe kevko can explain his reasoning :-) | 11:45 |
| opendevreview | Verification of a change to openstack/kayobe stable/2026.1 failed: CI: Download cirros images instead of using cache https://review.opendev.org/c/openstack/kayobe/+/1004926 | 11:46 |
| blanson[m] | fprzewozn: well, I think so ? I assume this avoid unecessary templating more it makes the thing work. I don't have strong opinions on it, I'm fine merging it or dropping it but we have to make a choice :D | 11:48 |
| blanson[m] | more than it make the thing work | 11:49 |
| opendevreview | Jed Preist proposed openstack/kayobe master: Refactor baremetal node registration https://review.opendev.org/c/openstack/kayobe/+/952979 | 12:05 |
| opendevreview | Rowan Johns proposed openstack/kayobe master: Install extra kernel modules on redhat hosts https://review.opendev.org/c/openstack/kayobe/+/1005000 | 12:10 |
| opendevreview | Rowan Johns proposed openstack/kayobe master: Install extra kernel modules on redhat hosts https://review.opendev.org/c/openstack/kayobe/+/1005000 | 12:14 |
| kevko | tafkamax: normally we store /etc/kolla in git ..... and there is a keyring ..which have secret ...and secrets should be in passwords which is ecnrypted as ansible vault | 12:32 |
| kevko | blanson[m], fprzewozn: I am sure that templating was not working ... | 12:37 |
| tafkamax | kevko: got it. | 12:48 |
| tafkamax | we store that directly in the keyring file, I guess it would be nice to move it out of there to the passwords.yaml file | 12:49 |
| fprzewozn | kevko I've just removed keyrings: # rm /etc/kolla/cinder-volume/ceph/*.keyring and running reconfigure -t cinder | 12:50 |
| fprzewozn | and it's back: key = AQCY... | 12:50 |
| fprzewozn | https://paste.opendev.org/show/bKX8KdBEybtuGI303Cca/ | 12:52 |
| opendevreview | Aleksey proposed openstack/kolla-ansible stable/2025.2: fix problem parsing json https://review.opendev.org/c/openstack/kolla-ansible/+/1005009 | 12:53 |
| kevko | fprzewozn: yeah, you are right ..i also tried a moment ago ...but at the time i reported the bug it was not working ... i don't remember why it was not working ... | 13:04 |
| blanson[m] | is it possible that ansible pre 2.19 wasn't working ? | 13:05 |
| blanson[m] | The templating engine changed quite a bit from 2.19 onward | 13:05 |
| fprzewozn | I had it working on 2025.1 | 13:05 |
| blanson[m] | hum | 13:06 |
| fprzewozn | configured on January | 13:06 |
| blanson[m] | that's the end of my idea pool | 13:06 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/2025.2: [stable/2025.* only] fix problem parsing json https://review.opendev.org/c/openstack/kolla-ansible/+/1005009 | 13:08 |
| fprzewozn | also this PR got backport +1, but you added comment that it will break existing configs | 13:08 |
| kevko | fprzewozn: i will try it on older release and will comment ... | 13:12 |
| fprzewozn | please do! thanks | 13:12 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Make Ceph keyring extraction compatible with variable substitution https://review.opendev.org/c/openstack/kolla-ansible/+/965471 | 13:13 |
| kevko | fprzewozn, blanson[m]: you can check this trivial patch https://review.opendev.org/c/openstack/kolla/+/1001716 | 13:14 |
| fprzewozn | and as we already speak, anyone experienced issues with "Enable all stable feature flags" when upgrading from 2025.1 to 2025.2? Upgraded like 7 clusters and on 8th I've hit "khepri_dbdisabled". It appears disabled on all of the upgraded clusters, but it was just passing with OK with khepri_db disabled | 13:16 |
| kevko | fprzewozn: oooh, maybe i remember already :) ... {{ cinder_key }} where cinder_key: "string" is working ... but {{ cinder_key }} if cinder_key: "{{ cinder_key_original }}" where cinder_key_original: "string" was not working .... | 13:20 |
| kevko | now tryint ... | 13:20 |
| kevko | but it should ... | 13:21 |
| tafkamax | you have some crazy logic | 13:21 |
| fprzewozn | hahaha | 13:21 |
| tafkamax | should puth concrete example into patch, so you can rememember more easily maybe and also for clarification for others? but cool that it got merged | 13:22 |
| fprzewozn | kevko soo it was nested variable thingy ? | 13:23 |
| fprzewozn | also regarding 1001716 I'm unable to test it now | 13:23 |
| kevko | i will | 13:24 |
| fprzewozn | but I see that .cache is being saved to host: # ls /var/lib/docker/volumes/glance/_data/.cache/python-entrypoints/ contains some data | 13:24 |
| kevko | and it should not be | 13:25 |
| fprzewozn | back to "Enable all stable feature flags" - it failed on upgrade on same env twice, but passed with OK on reconfigure, where nothing changed there. interesting ... | 13:32 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: Fix Let's Encrypt when enable_letsencrypt is true https://review.opendev.org/c/openstack/kolla-ansible/+/1005013 | 13:36 |
| kkaptanoglu1 | fprzewozn: This error also occurs during the upgrade from 2025.1 to 2026.1. As a workaround, I first manually enable all currently disabled plugins before starting the upgrade. | 13:47 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Fix configuration dump with ansible-core 2.19+ https://review.opendev.org/c/openstack/kayobe/+/1004990 | 13:47 |
| opendevreview | Michal Nasiadka proposed openstack/ansible-collection-kolla master: Add stats callback plugin - take 2 https://review.opendev.org/c/openstack/ansible-collection-kolla/+/1005021 | 14:34 |
| opendevreview | Michal Nasiadka proposed openstack/kayobe master: Allow to continue when some hosts are unreachable - take 2 https://review.opendev.org/c/openstack/kayobe/+/910509 | 14:34 |
| opendevreview | Merged openstack/kayobe stable/2025.2: CI: Download cirros images instead of using cache https://review.opendev.org/c/openstack/kayobe/+/1004925 | 14:39 |
| opendevreview | Merged openstack/kayobe stable/2025.1: CI: Download cirros images instead of using cache https://review.opendev.org/c/openstack/kayobe/+/1004928 | 14:39 |
| opendevreview | Merged openstack/kolla-ansible stable/2025.2: rabbitmq: fix precheck missing backend TLS cert fallback https://review.opendev.org/c/openstack/kolla-ansible/+/1004642 | 14:39 |
| opendevreview | Jed Preist proposed openstack/kayobe master: Refactor baremetal node registration https://review.opendev.org/c/openstack/kayobe/+/952979 | 14:52 |
| opendevreview | Michal Nasiadka proposed openstack/kayobe master: DNM: Test --continue-on-unreachable https://review.opendev.org/c/openstack/kayobe/+/910511 | 14:52 |
| opendevreview | Michal Nasiadka proposed openstack/kayobe master: DNM: Test --continue-on-unreachable https://review.opendev.org/c/openstack/kayobe/+/910511 | 14:52 |
| opendevreview | Rowan Johns proposed openstack/kayobe master: Install extra kernel modules on redhat hosts Fixes bug: https://bugs.launchpad.net/kayobe/+bug/2166461 Change-Id: I8711e85c3f2439e3d7aa5bcd28530bed8d8dd00a Signed-off-by: RowanJohns <rowan@stackhpc.com> https://review.opendev.org/c/openstack/kayobe/+/1005063 | 15:09 |
| opendevreview | Merged openstack/kayobe master: Never reboot without explicit permission https://review.opendev.org/c/openstack/kayobe/+/991575 | 15:21 |
| opendevreview | Verification of a change to openstack/kayobe master failed: Enable systemd-resolved configuration for Debian based distros https://review.opendev.org/c/openstack/kayobe/+/997692 | 15:21 |
| opendevreview | Verification of a change to openstack/kolla master failed: Drop support for CentOS Stream 10 https://review.opendev.org/c/openstack/kolla/+/997944 | 15:46 |
| opendevreview | Verification of a change to openstack/kayobe stable/2026.1 failed: CI: Download cirros images instead of using cache https://review.opendev.org/c/openstack/kayobe/+/1004926 | 18:04 |
| fprzewozn | @kkaptanoglu1 Yeah but enabling all, means we ensure that khepri_db should also be enabled, but that requires opt-in (one way operation) | 21:37 |
| fprzewozn | IMO we should handle that in some smart way | 21:38 |
| fprzewozn | and looking for ppl with expertise here on what should be the correct approach for us | 21:38 |
| opendevreview | Franciszek Przewoźny proposed openstack/kolla-ansible master: Fix ProxySQL sending all reads to writer node https://review.opendev.org/c/openstack/kolla-ansible/+/1005138 | 22:14 |
| fprzewozn | I'm curious about your opinion on that one ^ | 22:15 |
| fprzewozn | https://bugs.launchpad.net/kolla-ansible/+bug/2167037 | 22:15 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!