mnasiadka | kevko, frickler: can we move forward with https://review.opendev.org/c/openstack/ansible-collection-kolla/+/926044 and https://review.opendev.org/c/openstack/ansible-collection-kolla/+/925994? | 05:43 |
---|---|---|
mnasiadka | morning | 05:43 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: external/internal vip: use proper ip comparison https://review.opendev.org/c/openstack/kolla-ansible/+/926175 | 07:58 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: external/internal vip: use proper ip comparison https://review.opendev.org/c/openstack/kolla-ansible/+/926175 | 08:00 |
SvenKieske | kevko: could you review https://review.opendev.org/c/openstack/kolla-ansible/+/925670 ? my local linter does not work without it, which is a mess if I try to run tests locally on different changesets. | 08:11 |
opendevreview | Merged openstack/kolla-ansible master: CI: fix lint requirements for python 3.12 https://review.opendev.org/c/openstack/kolla-ansible/+/925670 | 08:41 |
SvenKieske | mnasiadka: frickler is on PTO until and including tomorrow :) thanks for the merge btw | 08:44 |
SvenKieske | I'll be on PTO starting tomorrow, but only until sunday. | 08:44 |
mnasiadka | good to know | 08:44 |
mnasiadka | I'll be off Thu/Fri | 08:45 |
SvenKieske | regarding the python3-docker/podman stuff afaik I fear kevko is right about that, at least there was a patch floating around installing it form osbpo and afaik it was even backported? | 08:46 |
SvenKieske | I guess I found the commit, linked it in 926044 in ack review | 08:48 |
SvenKieske | mhm, not sure about jammy though, maybe that's still not the case..maybe I should go look at a live install instead :D | 08:50 |
SvenKieske | well on my prod install with jammy python3-docker comes from the ubuntu archive, but that's also still version 5.x | 08:53 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2024.1: CI: fix lint requirements for python 3.12 https://review.opendev.org/c/openstack/kolla-ansible/+/926181 | 08:55 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: rabbitmq: Add support for using stream queues for fanout https://review.opendev.org/c/openstack/kolla-ansible/+/916911 | 09:13 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: bump py38 to py312 in tox.ini https://review.opendev.org/c/openstack/kolla-ansible/+/926184 | 09:13 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: bump py38 to py312 in tox.ini https://review.opendev.org/c/openstack/kolla-ansible/+/926184 | 09:19 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: bump py38 to py312 in tox.ini https://review.opendev.org/c/openstack/kolla-ansible/+/926184 | 09:20 |
SvenKieske | mhm, we probably want https://review.opendev.org/c/openstack/kolla-ansible/+/898307 instead | 09:46 |
opendevreview | Maksim Malchuk proposed openstack/kolla-ansible master: Fix issue with Swift Recon middleware https://review.opendev.org/c/openstack/kolla-ansible/+/926190 | 10:00 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: remove kolla_python_version https://review.opendev.org/c/openstack/kolla-ansible/+/926192 | 10:19 |
opendevreview | Alex Welsh proposed openstack/kolla-ansible master: Fix prometheus.yml templating https://review.opendev.org/c/openstack/kolla-ansible/+/926198 | 11:48 |
opendevreview | Alex Welsh proposed openstack/kolla-ansible master: Fix prometheus.yml templating https://review.opendev.org/c/openstack/kolla-ansible/+/926198 | 11:49 |
opendevreview | Alex Welsh proposed openstack/kolla-ansible master: Fix prometheus.yml templating https://review.opendev.org/c/openstack/kolla-ansible/+/926198 | 12:01 |
SvenKieske | mnasiadka: this seems to be stuck somehow and didn't merge? possibly I added Backport Candidate Voting too late? https://review.opendev.org/c/openstack/kolla-ansible/+/922840 | 12:48 |
mnasiadka | seems it's gating now | 12:52 |
SvenKieske | I'm in the process of building the same for the prometheus yml stuff, that's why I wondered | 13:01 |
SvenKieske | mnasiadka: prometheus linter incoming, once I please ansible-lint.. | 13:22 |
SvenKieske | ah, it actually catched an indentation bug.. | 13:24 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs https://review.opendev.org/c/openstack/kolla-ansible/+/926209 | 13:29 |
SvenKieske | it still might need some work | 13:29 |
opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default. https://review.opendev.org/c/openstack/kolla-ansible/+/904959 | 13:31 |
SvenKieske | anyone has any idea why specifically templating variable names is bad? I had to disable the check: https://github.com/ansible/ansible-lint/discussions/3794 | 13:31 |
mnasiadka | as you can see maintainers don't think it's a good idea to reply :) | 13:33 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Enable internal/backend TLS in LE scenario https://review.opendev.org/c/openstack/kolla-ansible/+/926210 | 13:51 |
opendevreview | Merged openstack/kolla-ansible master: add a config validation precheck for haproxy https://review.opendev.org/c/openstack/kolla-ansible/+/922840 | 14:19 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Enable internal/backend TLS in LE scenario https://review.opendev.org/c/openstack/kolla-ansible/+/926210 | 14:19 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs https://review.opendev.org/c/openstack/kolla-ansible/+/926209 | 14:26 |
opendevreview | Matúš Jenča proposed openstack/kolla-ansible master: Add support for RabbitMQ internode tls https://review.opendev.org/c/openstack/kolla-ansible/+/921381 | 14:29 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Enable internal/backend TLS in LE scenario https://review.opendev.org/c/openstack/kolla-ansible/+/926210 | 15:02 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Enable internal/backend TLS in LE scenario https://review.opendev.org/c/openstack/kolla-ansible/+/926210 | 15:06 |
*** jhorstmann is now known as Guest353 | 15:29 | |
chembervint | Hi, I'm again about 2 our patches - what I have to proceed it through Workflow? .. https://review.opendev.org/c/openstack/kolla-ansible/+/924322 https://review.opendev.org/c/openstack/kolla-ansible/+/920377 | 15:43 |
chembervint | it's already has a lot of +'s .. :) | 15:43 |
mnasiadka | chembervint: they should be merging now | 15:45 |
chembervint | mnasiadka: thank you! ...here also https://review.opendev.org/c/openstack/kolla-ansible/+/920377 ? should wait some time? | 15:56 |
mnasiadka | commented, I'm not a fan of this solution - if you can add some more background in the doc entry from what wuchunyang wrote in the Gerrit comment - I'd be happier. | 16:07 |
chembervint | thank you, I'll try to improve the doc | 16:09 |
opendevreview | Merged openstack/kolla-ansible stable/2023.1: Add ca_path for module uri in opensearch role https://review.opendev.org/c/openstack/kolla-ansible/+/924322 | 16:56 |
opendevreview | Merged openstack/kolla-ansible stable/2024.1: Add ca_path for module uri in opensearch role https://review.opendev.org/c/openstack/kolla-ansible/+/924320 | 17:18 |
opendevreview | Merged openstack/kolla-ansible stable/2023.2: Add ca_path for module uri in opensearch role https://review.opendev.org/c/openstack/kolla-ansible/+/924321 | 17:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!