| blanson[m] | Hello good morning everyone ! https://review.opendev.org/c/openstack/kolla-ansible/+/973803 I'm kind of stuck on this one if anyone has motivation to take a quick look. essentially the issue is that the container check finds a diff in the ovn-relay because the loop iterator is not visible (?) when the redenering of https://opendev.org/openstack/kolla-ansible/src/branch/master/ansible/roles/ovn-db/defaults/main.yml#L84-L87 happens, | 09:51 |
|---|---|---|
| blanson[m] | so the new volume is detected without it, but the current volume has the iterator at the end. I've tried a couple of things but I can't get it to render properly :/ | 09:51 |
| opendevreview | Matt Crees proposed openstack/kayobe master: Increase stackhpc.linux collection to v1.5.1 https://review.opendev.org/c/openstack/kayobe/+/975379 | 09:57 |
| opendevreview | Matt Crees proposed openstack/kayobe master: Bump stackhpc.linux collection to v1.5.1 https://review.opendev.org/c/openstack/kayobe/+/975379 | 10:10 |
| opendevreview | Matt Crees proposed openstack/kayobe master: Bump stackhpc.linux collection to v1.5.1 https://review.opendev.org/c/openstack/kayobe/+/975379 | 10:14 |
| *** jhorstmann is now known as Guest1228 | 10:29 | |
| bbezak | blanson[m]: will take a look | 10:34 |
| bbezak | (Morning) | 10:34 |
| bbezak | blanson[m]: I’ll test sth if you don’t mind | 11:23 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: DNM: Making iterated container-check idempotent https://review.opendev.org/c/openstack/kolla-ansible/+/973803 | 11:30 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Fix issue with systemd-networkd-wait-online when no IP https://review.opendev.org/c/openstack/kayobe/+/975386 | 12:08 |
| fprzewozn | Good morning! Anyone here free to review: https://review.opendev.org/c/openstack/kolla-ansible/+/974328 ? Thanks! | 12:31 |
| opendevreview | Matt Crees proposed openstack/kayobe master: Deprecate kolla-tags and kolla-limit https://review.opendev.org/c/openstack/kayobe/+/935669 | 13:25 |
| bbezak | fprzewo | 13:32 |
| bbezak | fprzewozn: will do! | 13:32 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: DNM: Making iterated container-check idempotent https://review.opendev.org/c/openstack/kolla-ansible/+/973803 | 13:35 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: DNM: Making iterated container-check idempotent https://review.opendev.org/c/openstack/kolla-ansible/+/973803 | 14:15 |
| blanson[m] | bbezak: go ahead :) I've tried enough I'll take another stab at it later this week but it's burning me out | 14:41 |
| bbezak | Cool, thx | 14:41 |
| blanson[m] | Also, quick question, during a POC of mine, I noticed that if you only set a single vip (same for internal and external), you can't really use the single haproxy frontend because the external_backend never get written in the haproxy config ? | 14:42 |
| blanson[m] | it that normal ? I assumed I would be able to get keystone internal on https://keystone.domain.tld:5000 and keystone external on https://keystone.domain.tld ? | 14:42 |
| blanson[m] | my poc is in 2025.1 but I believe it's still the case on master | 14:43 |
| bbezak | I recall that kind of bug | 14:44 |
| blanson[m] | I found out the why | 14:44 |
| blanson[m] | I just weonder if that would warrant a fix | 14:44 |
| blanson[m] | or if it's byu design ? | 14:44 |
| bbezak | I think initial design was that those would be separate | 14:46 |
| blanson[m] | yh I think I get the initial design, but from my understanding, nothing would prevent you/stop working if I only had a single vip and single_external_frontend ? internal and external backends would be the same, but other than that, everything would work ? | 14:52 |
| bbezak | Well, we’ve got a kolla_same_external_internal_vip | 15:04 |
| bbezak | So I think single frontend could potentially support that as well | 15:04 |
| bbezak | but | 15:06 |
| bbezak | ./ansible/group_vars/all/haproxy.yml:8:haproxy_enable_external_vip: "{{ 'no' if kolla_same_external_internal_vip | bool else 'yes' }}" | 15:06 |
| bbezak | hehe | 15:06 |
| blanson[m] | yh ! I was wondering if I could simply make a change in https://github.com/openstack/kolla-ansible/blob/2157483456a1c0b2a9d67f600c08e47b807b4edd/ansible/roles/haproxy-config/templates/haproxy_single_service_split.cfg.j2#L125 | 15:09 |
| blanson[m] | {% if haproxy_service.enabled | bool and (not external or haproxy_enable_external_vip | bool or haproxy_single_external_frontend | bool) %} | 15:10 |
| blanson[m] | this way external backends get rendered if you enable single external frontend | 15:10 |
| blanson[m] | the rest stays the same | 15:10 |
| bbezak | Hmm, I think it would work | 15:17 |
| opendevreview | Merged openstack/kayobe master: Bump stackhpc.linux collection to v1.5.1 https://review.opendev.org/c/openstack/kayobe/+/975379 | 15:28 |
| opendevreview | Matt Crees proposed openstack/kayobe stable/2025.2: Bump stackhpc.linux collection to v1.5.1 https://review.opendev.org/c/openstack/kayobe/+/975409 | 15:29 |
| opendevreview | Matt Crees proposed openstack/kayobe stable/2025.1: Bump stackhpc.linux collection to v1.5.1 https://review.opendev.org/c/openstack/kayobe/+/975410 | 15:29 |
| opendevreview | Matt Crees proposed openstack/kayobe stable/2024.2: Bump stackhpc.linux collection to v1.5.1 https://review.opendev.org/c/openstack/kayobe/+/975411 | 15:29 |
| opendevreview | Bertrand Lanson proposed openstack/kolla-ansible master: Allow SNI frontend when using a single haproxy VIP https://review.opendev.org/c/openstack/kolla-ansible/+/975421 | 16:38 |
| opendevreview | Pablo Colson proposed openstack/kolla master: neutron-ovn-vpn-agent: Add kolla_patch_sources macro https://review.opendev.org/c/openstack/kolla/+/975423 | 16:41 |
| opendevreview | Pablo Colson proposed openstack/kolla master: Add Dockerfile for neutron-ovn-vpn-agent https://review.opendev.org/c/openstack/kolla/+/924302 | 16:44 |
| blanson[m] | bbezak: https://review.opendev.org/c/openstack/kolla-ansible/+/975421 should pass I believe ? | 16:52 |
| bbezak | Will take a look tomorrow | 17:04 |
| opendevreview | Bartosz Bezak proposed openstack/kolla-ansible master: DNM: Making iterated container-check idempotent https://review.opendev.org/c/openstack/kolla-ansible/+/973803 | 17:07 |
| opendevreview | Doug Szumski proposed openstack/kolla-ansible master: Support multiple Nova Compute Ironic instances https://review.opendev.org/c/openstack/kolla-ansible/+/973881 | 17:41 |
| opendevreview | Doug Szumski proposed openstack/kolla-ansible master: Support multiple Nova Compute Ironic instances https://review.opendev.org/c/openstack/kolla-ansible/+/973881 | 17:42 |
| opendevreview | Doug Szumski proposed openstack/kolla-ansible master: Support multiple Nova Compute Ironic instances https://review.opendev.org/c/openstack/kolla-ansible/+/973881 | 17:45 |
| opendevreview | Merged openstack/kayobe stable/2025.2: Bump stackhpc.linux collection to v1.5.1 https://review.opendev.org/c/openstack/kayobe/+/975409 | 18:23 |
| fprzewozn | bbezak thank you! | 18:35 |
| fprzewozn | mnasiadka frickler could you also have a look at https://review.opendev.org/c/openstack/kolla-ansible/+/974328 ? | 18:35 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: kolla_container: Add support for user and security_opt parameters https://review.opendev.org/c/openstack/kolla-ansible/+/975065 | 18:39 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: loadbalancer: set privileged=false for keepalived container https://review.opendev.org/c/openstack/kolla-ansible/+/975452 | 18:56 |
| bbezak | blanson[m]: CI was ok with fixes of mine. Tomorrow I’ll check on some vm with more atomic changes - which specifically fixes the problem | 19:03 |
| bbezak | I recon permissions, As I checked on one of deployment of mine kolla_set_configs —check command in the relay container and it complained about permissions | 19:04 |
| bbezak | Something for tomorrow | 19:04 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Fix issue with systemd-networkd-wait-online when no IP https://review.opendev.org/c/openstack/kayobe/+/975386 | 19:24 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Fix issue with systemd-networkd-wait-online when no IP https://review.opendev.org/c/openstack/kayobe/+/975386 | 19:27 |
| opendevreview | Piotr Milewski proposed openstack/kolla-ansible master: kolla_container: Add support for user and security_opt parameters https://review.opendev.org/c/openstack/kolla-ansible/+/975065 | 19:33 |
| blanson[m] | bbezak: I'll take a look at your approach ! thanks again for looking into it | 19:38 |
| opendevreview | Verification of a change to openstack/kayobe stable/2024.2 failed: Bump stackhpc.linux collection to v1.5.1 https://review.opendev.org/c/openstack/kayobe/+/975411 | 19:55 |
| opendevreview | Merged openstack/kayobe stable/2025.1: Bump stackhpc.linux collection to v1.5.1 https://review.opendev.org/c/openstack/kayobe/+/975410 | 20:48 |
| opendevreview | Merged openstack/kolla-ansible master: keystone: Manage OIDC metadata files as templates https://review.opendev.org/c/openstack/kolla-ansible/+/974328 | 20:48 |
| opendevreview | Franciszek Przewoźny proposed openstack/kolla-ansible stable/2025.2: keystone: Manage OIDC metadata files as templates https://review.opendev.org/c/openstack/kolla-ansible/+/975472 | 20:51 |
| opendevreview | Franciszek Przewoźny proposed openstack/kolla-ansible stable/2025.1: keystone: Manage OIDC metadata files as templates https://review.opendev.org/c/openstack/kolla-ansible/+/975473 | 20:58 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!