Tuesday, 2024-08-13

mnasiadkakevko, 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
mnasiadkamorning05:43
opendevreviewSven Kieske proposed openstack/kolla-ansible master: external/internal vip: use proper ip comparison  https://review.opendev.org/c/openstack/kolla-ansible/+/92617507:58
opendevreviewSven Kieske proposed openstack/kolla-ansible master: external/internal vip: use proper ip comparison  https://review.opendev.org/c/openstack/kolla-ansible/+/92617508:00
SvenKieskekevko: 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
opendevreviewMerged openstack/kolla-ansible master: CI: fix lint requirements for python 3.12  https://review.opendev.org/c/openstack/kolla-ansible/+/92567008:41
SvenKieskemnasiadka: frickler is on PTO until and including tomorrow :) thanks for the merge btw08:44
SvenKieskeI'll be on PTO starting tomorrow, but only until sunday.08:44
mnasiadkagood to know08:44
mnasiadkaI'll be off Thu/Fri08:45
SvenKieskeregarding 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
SvenKieskeI guess I found the commit, linked it in 926044 in ack review08:48
SvenKieskemhm, not sure about jammy though, maybe that's still not the case..maybe I should go look at a live install instead :D08:50
SvenKieskewell on my prod install with jammy python3-docker comes from the ubuntu archive, but that's also still version 5.x08:53
opendevreviewSven Kieske proposed openstack/kolla-ansible stable/2024.1: CI: fix lint requirements for python 3.12  https://review.opendev.org/c/openstack/kolla-ansible/+/92618108:55
opendevreviewSven Kieske proposed openstack/kolla-ansible master: rabbitmq: Add support for using stream queues for fanout  https://review.opendev.org/c/openstack/kolla-ansible/+/91691109:13
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: bump py38 to py312 in tox.ini  https://review.opendev.org/c/openstack/kolla-ansible/+/92618409:13
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: bump py38 to py312 in tox.ini  https://review.opendev.org/c/openstack/kolla-ansible/+/92618409:19
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: bump py38 to py312 in tox.ini  https://review.opendev.org/c/openstack/kolla-ansible/+/92618409:20
SvenKieskemhm, we probably want https://review.opendev.org/c/openstack/kolla-ansible/+/898307 instead09:46
opendevreviewMaksim Malchuk proposed openstack/kolla-ansible master: Fix issue with Swift Recon middleware  https://review.opendev.org/c/openstack/kolla-ansible/+/92619010:00
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: remove kolla_python_version  https://review.opendev.org/c/openstack/kolla-ansible/+/92619210:19
opendevreviewAlex Welsh proposed openstack/kolla-ansible master: Fix prometheus.yml templating  https://review.opendev.org/c/openstack/kolla-ansible/+/92619811:48
opendevreviewAlex Welsh proposed openstack/kolla-ansible master: Fix prometheus.yml templating  https://review.opendev.org/c/openstack/kolla-ansible/+/92619811:49
opendevreviewAlex Welsh proposed openstack/kolla-ansible master: Fix prometheus.yml templating  https://review.opendev.org/c/openstack/kolla-ansible/+/92619812:01
SvenKieskemnasiadka: 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/+/92284012:48
mnasiadkaseems it's gating now12:52
SvenKieskeI'm in the process of building the same for the prometheus yml stuff, that's why I wondered13:01
SvenKieskemnasiadka: prometheus linter incoming, once I please ansible-lint..13:22
SvenKieskeah, it actually catched an indentation bug..13:24
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs  https://review.opendev.org/c/openstack/kolla-ansible/+/92620913:29
SvenKieskeit still might need some work13:29
opendevreviewRafal Lewandowski proposed openstack/kolla-ansible master: [WIP] Enable ML2/OVN and distributed FIP by default.  https://review.opendev.org/c/openstack/kolla-ansible/+/90495913:31
SvenKieskeanyone has any idea why specifically templating variable names is bad? I had to disable the check: https://github.com/ansible/ansible-lint/discussions/379413:31
mnasiadkaas you can see maintainers don't think it's a good idea to reply :)13:33
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Enable internal/backend TLS in LE scenario  https://review.opendev.org/c/openstack/kolla-ansible/+/92621013:51
opendevreviewMerged openstack/kolla-ansible master: add a config validation precheck for haproxy  https://review.opendev.org/c/openstack/kolla-ansible/+/92284014:19
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Enable internal/backend TLS in LE scenario  https://review.opendev.org/c/openstack/kolla-ansible/+/92621014:19
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs  https://review.opendev.org/c/openstack/kolla-ansible/+/92620914:26
opendevreviewMatúš Jenča proposed openstack/kolla-ansible master: Add support for RabbitMQ internode tls  https://review.opendev.org/c/openstack/kolla-ansible/+/92138114:29
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Enable internal/backend TLS in LE scenario  https://review.opendev.org/c/openstack/kolla-ansible/+/92621015:02
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: CI: Enable internal/backend TLS in LE scenario  https://review.opendev.org/c/openstack/kolla-ansible/+/92621015:06
*** jhorstmann is now known as Guest35315:29
chembervintHi, 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/+/92037715:43
chembervintit's already has a lot of +'s .. :) 15:43
mnasiadkachembervint: they should be merging now15:45
chembervintmnasiadka: thank you! ...here also https://review.opendev.org/c/openstack/kolla-ansible/+/920377 ? should wait some time?15:56
mnasiadkacommented, 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
chembervintthank you, I'll try to improve the doc16:09
opendevreviewMerged openstack/kolla-ansible stable/2023.1: Add ca_path for module uri in opensearch role  https://review.opendev.org/c/openstack/kolla-ansible/+/92432216:56
opendevreviewMerged openstack/kolla-ansible stable/2024.1: Add ca_path for module uri in opensearch role  https://review.opendev.org/c/openstack/kolla-ansible/+/92432017:18
opendevreviewMerged openstack/kolla-ansible stable/2023.2: Add ca_path for module uri in opensearch role  https://review.opendev.org/c/openstack/kolla-ansible/+/92432117:18

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!