Tuesday, 2022-05-03

opendevreviewJames Kirsch proposed openstack/kolla master: Replace Certbot with Lego for Let's Encrypt container  https://review.opendev.org/c/openstack/kolla/+/83421100:22
opendevreviewTim Beermann proposed openstack/kolla-ansible master: Remove sanity checks  https://review.opendev.org/c/openstack/kolla-ansible/+/83971607:18
SvenKieskejamesbenson: taking a look. I'm not sure I like the opening sentence, though ("Ceph may encounter issues if SSL certificates are used."). while true, this is only the case if it is not setup correctly, no?07:53
SvenKieskejamesbenson: on the plus side, this leads me to investigate my own setup :D07:53
kevkohi \o/08:13
kevkoit looks like kolla-ansible requirements are broken currently 08:13
priteauGood morning.08:32
kevkopriteau: morning 08:37
fricklerkevko: where do you see broken reqs?08:53
kevkofrickler: hmm, i hoped no one would notice what i've written :D 08:53
kevkofrickler: i have my branches downstream ...and had problem with jinja2 ...but i already found that stable branches has fix for this :) 08:54
kevkoso i rebased 08:54
fricklerkevko: ah, ok, np, fixed issues are good issues ;)08:57
kevkoyeah :) 08:57
kevkofrickler: do you have a time for review ? :) 08:57
kevkofrickler: and maybe help to check some logs because i am blind probably :( 08:58
fricklerkevko: depends on the size, but giving a link would help to find out08:59
kevkofrickler: Radoslaw changed my original patch in a chain and CI job started to fail ...but i don't know why ..because he changed thins in place which i think cannot be reflected in a logs :( 09:01
kevkofrickler: https://review.opendev.org/c/openstack/kolla-ansible/+/838916/509:01
kevkofrickler: he changed this https://review.opendev.org/c/openstack/kolla-ansible/+/838930/2..309:02
kevkofrickler: in CI job before was passing ..now failing .. don't know why ..it looks like mariadb role is somehow broken 09:02
fricklerkevko: ah, I looked at that patch earlier and thought that you both are very brave to touch this setup, my hope was that I could stay away from it ;) but I'll need more time in order to look at it close enough. and probably a local setup to replicate. or maybe I'll hold a build so we can all take a look09:05
kevkofrickler: haha, yeah I am brave :) 09:34
kevkofrickler: we are using in production from stein and it's working like a charm 09:34
fricklerkevko: I'm mostly worried about the automation for the migration09:42
kevkomigration ? 09:43
kevkoyou mean switch haproxy-> proxysql ? 09:43
fricklerkevko: yes. too many weird corner cases09:43
fricklerkevko: does it actually make sense to debug the cells job with c8s which we are going to dump anyway? can we use debuntu instead?09:44
kevkofrickler: yeah, we can 09:45
* frickler would be much more comfortable with looking at things there09:45
kevkofrickler: btw, if you are curious how it works ...i can deploy test deployment and give you access :) 09:45
fricklerkevko: thanks for the offer, but I do have test capacity at hand, it's more a matter of having enough time. I'll take a deeper look later today, gotta do some meetings first09:47
kevkofrickler: ok 09:51
opendevreviewMerged openstack/kolla-ansible stable/ussuri: fix missing lang env in curator crontab  https://review.opendev.org/c/openstack/kolla-ansible/+/83991209:51
opendevreviewDr. Jens Harbott proposed openstack/kolla-ansible master: [DNM] Trigger cells jobs  https://review.opendev.org/c/openstack/kolla-ansible/+/83891610:15
fricklerkevko: ^^ I hope I did this right, will be holding the ubuntu job (assuming it will fail like c8s)10:16
kevkofrickler: thanks 10:16
opendevreviewMerged openstack/kayobe stable/wallaby: CI: separate image builds into a non-voting job  https://review.opendev.org/c/openstack/kayobe/+/83747710:31
opendevreviewTim Beermann proposed openstack/kolla-ansible master: Remove sanity checks  https://review.opendev.org/c/openstack/kolla-ansible/+/83971611:04
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Cleanup orphan interfaces  https://review.opendev.org/c/openstack/kayobe/+/84003311:20
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: baremetal: refactor docker deployment into a separate role  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82120411:32
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: docker: restart docker and containerd in handlers  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82120511:32
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: docker: add registry CA configuration  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82120611:32
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: baremetal: refactor package installation into a separate role  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82958611:32
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: baremetal: refactor libvirt apparmor configuration  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82958711:32
kevkoyoctozepto: ok, i found a bug in that cells job ...but not my bug ..but old mariadb bug :D 11:42
kevkoyoctozepto: here ? 12:08
*** bsanjeewa[m] is now known as bsanjeewa12:17
fricklerlooks like https://review.opendev.org/c/openstack/kayobe/+/827653 is needed to unblock kayobe now that the k-a dep has merged?13:30
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Cleanup orphan interfaces  https://review.opendev.org/c/openstack/kayobe/+/84003314:30
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Cleanup orphan interfaces  https://review.opendev.org/c/openstack/kayobe/+/84003314:46
fricklerkevko: how does that bug look like? will it resolve the issue with your patchset?14:58
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Cleanup orphan interfaces  https://review.opendev.org/c/openstack/kayobe/+/84003315:20
mmalchukplease merge https://review.opendev.org/c/openstack/kayobe/+/82765315:26
mmalchukall the jobs failed with: "ironic_dnsmasq_dhcp_ranges must be a list\n"15:26
mmalchukbecause k-a fix merged15:26
opendevreviewMerged openstack/kayobe master: Multiple DHCP ranges for Ironic Inspector  https://review.opendev.org/c/openstack/kayobe/+/82765317:40
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Cleanup orphan interfaces  https://review.opendev.org/c/openstack/kayobe/+/84003317:57
opendevreviewJames Kirsch proposed openstack/kolla master: Replace Certbot with Lego for Let's Encrypt container  https://review.opendev.org/c/openstack/kolla/+/83421118:54
opendevreviewJuan Pablo Suazo proposed openstack/kolla-ansible master: Modifies openvswitch tasks to exclude nodes with no openvswitch services  https://review.opendev.org/c/openstack/kolla-ansible/+/84038220:53
opendevreviewJames Kirsch proposed openstack/kolla-ansible master: Add support for LetsEncrypt-managed certs  https://review.opendev.org/c/openstack/kolla-ansible/+/74134021:45

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