| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: mariadb: Create kolla user instead of using root https://review.opendev.org/c/openstack/kolla-ansible/+/982214 | 04:57 |
|---|---|---|
| mnasiadka | jangutter[m]: I did the same in my Nokia times, but then we used bash as cli :) | 06:42 |
| mnasiadka | jangutter[m]: and I was able to just import globals.yml - but maybe back then awx was more permissive | 06:46 |
| blanson[m] | I have this chain https://review.opendev.org/c/openstack/kolla-ansible/+/970861/16 that passes the gate with some rework from last time. If anyone has time to review it :) | 07:01 |
| blanson[m] | mnasiadka: before I start, I'll re-ask cause last time was a while ago. I'd like to add a wait for healthcheck on container restart if the container has a healthcheck. This may slow down the restart process quite a bit, but imo it's safer because you can do proper rolling restart for stuff like APIs ? what do you think ? | 07:15 |
| blanson[m] | bbezak frickler aswell | 07:15 |
| *** osmanlicilegi is now known as Guest6250 | 07:38 | |
| opendevreview | Merged openstack/kolla-ansible master: CI: fix prometheus-valkey-exporter inventory group https://review.opendev.org/c/openstack/kolla-ansible/+/982696 | 08:39 |
| Vii | mnasiadka: https://review.opendev.org/c/openstack/kolla-ansible/+/924575/comment/8e8bb123_2323116e/ I’ll remove it, but based on my initial observations, Tempest seems to fail less often. | 09:13 |
| Vii | I’d consider setting this globally. | 09:14 |
| mnasiadka | Vii: then it should be in a separate patch, please stop bundling things in one patch - this is not GitHub :) | 09:14 |
| Vii | :) | 09:14 |
| mnasiadka | https://docs.openstack.org/contributors/en_GB/code-and-documentation/patch-best-practices.html#structural-split-of-changes | 09:15 |
| Vii | It’s pretty hard to test this it takes a really long time, and sometimes some weird external components fail. Also, I don’t always understand what it’s complaining about, since with the exact same settings it works fine on my side :P | 09:16 |
| Vii | I’ll just remove it it was only meant as a “helper” anyway. | 09:17 |
| *** jhorstmann is now known as Guest6259 | 09:33 | |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2025.1: [stable-2025.1-only] valkey: Add ansible role https://review.opendev.org/c/openstack/kolla-ansible/+/973214 | 10:17 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: mariadb: Create kolla user instead of using root https://review.opendev.org/c/openstack/kolla-ansible/+/982214 | 10:28 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: CI: move aarch64 jobs to 16G ram nodes https://review.opendev.org/c/openstack/kolla-ansible/+/980691 | 10:33 |
| opendevreview | Verification of a change to openstack/kayobe master failed: Build Rocky based IPA images https://review.opendev.org/c/openstack/kayobe/+/982109 | 10:40 |
| opendevreview | Merged openstack/kayobe master: Add nmstate engine https://review.opendev.org/c/openstack/kayobe/+/974745 | 10:46 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2025.1: [stable-2025.1-only] valkey: Add ansible role https://review.opendev.org/c/openstack/kolla-ansible/+/973214 | 10:53 |
| opendevreview | Piotr Milewski proposed openstack/kolla master: Add monitoring for external component versions https://review.opendev.org/c/openstack/kolla/+/972434 | 10:56 |
| mnasiadka | tafkamax: how are you with https://review.opendev.org/c/openstack/kolla-ansible/+/975901 ? Soon to be ready? | 11:09 |
| mnasiadka | I see the patch is sort of WIP, so please mark it as work in progress | 11:10 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: glance: Add configuration for cinder backend https://review.opendev.org/c/openstack/kolla-ansible/+/978863 | 11:16 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: glance: Add configuration for cinder backend https://review.opendev.org/c/openstack/kolla-ansible/+/978863 | 11:17 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: glance: Add configuration for cinder backend https://review.opendev.org/c/openstack/kolla-ansible/+/978863 | 11:19 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: glance: Add configuration for cinder backend https://review.opendev.org/c/openstack/kolla-ansible/+/978863 | 11:20 |
| tafkamax | mnasiadka: sorry yes, I am busy with work, i haven't tested the final change in the apache2.conf for the errortemplate | 11:49 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: octavia: Support OVN in auto_configure https://review.opendev.org/c/openstack/kolla-ansible/+/970250 | 11:50 |
| tafkamax | tldr; I need to check what happens if I use errordocument instead of oidcerrortemplate. But if that error template part is disabled from rocky and trixie images then it works. On noble there is not a problem. | 11:50 |
| tafkamax | And I am on my final streak of family birthdays today, so I don't have time in the evening aswell :/ | 11:51 |
| stromgren | I hope that someone can point me in the rigth direction on this :) I'm upgrading from master (somewhere after 2024.1) to 2024.2. When runnning "kolla-ansible upgrade" it gets stuck on upgrading the Keystone database. In the Keystone log there's an error related to not being able verify the SSL certificate for MariaDB. It looks like the instance is using the self-signed certificate instead of | 11:55 |
| stromgren | the correct one. All other services have the correct certificate assigned. | 11:55 |
| opendevreview | Merged openstack/kolla master: Add OCI standard labels to container images https://review.opendev.org/c/openstack/kolla/+/971681 | 12:07 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Test for leaked passwords in syslog https://review.opendev.org/c/openstack/kolla-ansible/+/981068 | 12:08 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Test for leaked passwords in syslog https://review.opendev.org/c/openstack/kolla-ansible/+/981068 | 12:15 |
| mnasiadka | tafkamax: I’m fine with just removing support for oidcerrortemplate for now and finding a solution later on | 12:18 |
| tafkamax | mnasiadka: ok, I am just modifying the test cluster rn to test the changes | 12:22 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible stable/2025.1: [stable-2025.1-only] valkey: Add ansible role https://review.opendev.org/c/openstack/kolla-ansible/+/973214 | 12:30 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 12:34 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 12:36 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 12:40 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 12:43 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 12:44 |
| opendevreview | Merged openstack/kolla-ansible stable/2025.1: [2025.1 only] Support ProxySQL 3.0.x on 2025.1 https://review.opendev.org/c/openstack/kolla-ansible/+/974712 | 13:03 |
| opendevreview | Verification of a change to openstack/kolla master failed: Remove dsa from haproxy-ssh host key types https://review.opendev.org/c/openstack/kolla/+/980173 | 13:05 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 13:05 |
| opendevreview | Bartosz Bezak proposed openstack/kolla master: CI: publish aarch64 noble images https://review.opendev.org/c/openstack/kolla/+/982788 | 13:56 |
| opendevreview | Verification of a change to openstack/kolla master failed: Add Dockerfile for neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla/+/924302 | 15:14 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 15:25 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 15:39 |
| opendevreview | Merged openstack/kolla-ansible master: glance: Add configuration for cinder backend https://review.opendev.org/c/openstack/kolla-ansible/+/978863 | 16:10 |
| opendevreview | Merged openstack/kolla-ansible master: CI: move aarch64 jobs to 16G ram nodes https://review.opendev.org/c/openstack/kolla-ansible/+/980691 | 16:56 |
| opendevreview | Verification of a change to openstack/kayobe master failed: Build Rocky based IPA images https://review.opendev.org/c/openstack/kayobe/+/982109 | 17:01 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: release: valkey TODO https://review.opendev.org/c/openstack/kolla-ansible/+/982879 | 17:12 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: release: cinder TODO https://review.opendev.org/c/openstack/kolla-ansible/+/982880 | 17:13 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 17:20 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: release: ovn TODO https://review.opendev.org/c/openstack/kolla-ansible/+/982884 | 17:41 |
| opendevreview | Verification of a change to openstack/kolla master failed: Remove dsa from haproxy-ssh host key types https://review.opendev.org/c/openstack/kolla/+/980173 | 17:58 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: Implement neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla-ansible/+/924575 | 18:56 |
| tafkamax | mnasiadka: ok if this is ok for you I will leave it in for ubuntu noble, as it works there. | 19:03 |
| opendevreview | Taavi Ansper proposed openstack/kolla-ansible master: Fix keystone with IDP configured. https://review.opendev.org/c/openstack/kolla-ansible/+/975901 | 19:23 |
| tafkamax | OK bossman. I made it so the template can still be used with ubuntu. | 19:24 |
| tafkamax | Added reno aswell. | 19:24 |
| tafkamax | I hope my first reno is ok. | 19:24 |
| tafkamax | I didn't re-add the CI and normal test options yet. I want CI to fully run the apache codepath once more. | 19:24 |
| tafkamax | Hmm, Invalid input for field/attribute domain_id. Value: VALUE_SPECIFIED_IN_NO_LOG_PARAMETER. https://zuul.opendev.org/t/openstack/build/b2f9345d303f482da90aefba7823991e/log/primary/logs/ansible/deploy | 20:07 |
| tafkamax | mnasiadka Is this related to your CI changes to hide secrets? | 20:07 |
| opendevreview | Taavi Ansper proposed openstack/kolla-ansible master: Fix keystone with IDP configured. https://review.opendev.org/c/openstack/kolla-ansible/+/975901 | 20:13 |
| tafkamax | I rebased from master to test again. | 20:14 |
| tafkamax | Can I somehow specify that a specify variable be exempt? | 20:16 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Template disabled repositories https://review.opendev.org/c/openstack/kayobe/+/976615 | 20:31 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Adds dnf_repo_state_overrides https://review.opendev.org/c/openstack/kayobe/+/977028 | 20:31 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!