Wednesday, 2022-02-16

opendevreviewMerged openstack/kolla-ansible master: [haproxy] optionally set socket to allow admin commands  https://review.opendev.org/c/openstack/kolla-ansible/+/82810700:33
opendevreviewconna proposed openstack/kolla master: Skyline: Add skyline dockerfile  https://review.opendev.org/c/openstack/kolla/+/82694802:12
opendevreviewMichal Nasiadka proposed openstack/kolla stable/victoria: erlang: use packages from Erlang Solutions on AArch64  https://review.opendev.org/c/openstack/kolla/+/82926906:35
opendevreviewwushiming proposed openstack/kolla-ansible master: Updating python testing classifier as per Yoga testing runtime  https://review.opendev.org/c/openstack/kolla-ansible/+/82948407:03
opendevreviewRadosÅ‚aw Piliszek proposed openstack/kolla-ansible stable/victoria: Fix remove libvirt apparmor disabled profile  https://review.opendev.org/c/openstack/kolla-ansible/+/77050907:43
mnasiadkayoctozepto: see my reply in https://review.opendev.org/c/openstack/kolla/+/829267?tab=comments - I don't know how long will this CentOS drama persist...08:22
opendevreviewwushiming proposed openstack/kolla master: Updating python testing classifier as per Yoga testing runtime  https://review.opendev.org/c/openstack/kolla/+/82949008:26
yoctozeptomnasiadka: ack, sad08:32
mnasiadkayoctozepto: here's an example - https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_67b/829263/1/check-arm64/kolla-build-centos8s-source-aarch64/67b38db/kolla/build/000_FAILED_rabbitmq.log08:34
mnasiadkathanks to aarch64 non-voting job, we don't need to merge two patches in one (collectd-pcie and rabbitmq) ;)08:35
mnasiadkaSkyline and Venus PTL-less? geez08:35
yoctozeptoyeah, worked well, huh?08:36
opendevreviewwangxiyuan proposed openstack/kolla master: [WIP]Add openEuler Distro support  https://review.opendev.org/c/openstack/kolla/+/82949509:07
opendevreviewMerged openstack/kolla stable/xena: collectd: pcie-errors is x86-64 only now  https://review.opendev.org/c/openstack/kolla/+/82926310:08
opendevreviewMerged openstack/kolla stable/wallaby: collectd: pcie-errors is x86-64 only now  https://review.opendev.org/c/openstack/kolla/+/82926410:08
opendevreviewMerged openstack/kolla stable/victoria: collectd: pcie-errors is x86-64 only now  https://review.opendev.org/c/openstack/kolla/+/82926510:21
opendevreviewMerged openstack/kolla stable/ussuri: collectd: pcie-errors is x86-64 only now  https://review.opendev.org/c/openstack/kolla/+/82926210:21
opendevreviewMerged openstack/kolla stable/wallaby: erlang: use packages from Erlang Solutions on AArch64  https://review.opendev.org/c/openstack/kolla/+/82926810:21
opendevreviewMerged openstack/kolla-ansible stable/victoria: Fix remove libvirt apparmor disabled profile  https://review.opendev.org/c/openstack/kolla-ansible/+/77050910:22
opendevreviewMerged openstack/kolla stable/xena: Unpin td-agent and cap elasticsearch gem  https://review.opendev.org/c/openstack/kolla/+/82856010:32
opendevreviewMerged openstack/kolla stable/wallaby: Unpin td-agent and cap elasticsearch gem  https://review.opendev.org/c/openstack/kolla/+/82856310:40
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: haproxy: support different internal/external ports  https://review.opendev.org/c/openstack/kolla-ansible/+/82339511:04
opendevreviewMerged openstack/kolla-ansible master: Updating python testing classifier as per Yoga testing runtime  https://review.opendev.org/c/openstack/kolla-ansible/+/82948411:13
opendevreviewMerged openstack/kolla master: Updating python testing classifier as per Yoga testing runtime  https://review.opendev.org/c/openstack/kolla/+/82949011:51
opendevreviewMerged openstack/kolla stable/xena: erlang: use packages from Erlang Solutions on AArch64  https://review.opendev.org/c/openstack/kolla/+/82926712:13
opendevreviewMichal Nasiadka proposed openstack/kolla stable/ussuri: erlang: use packages from Erlang Solutions on AArch64  https://review.opendev.org/c/openstack/kolla/+/82926612:15
opendevreviewalecorps proposed openstack/kolla-ansible master: Add support for VMware First Class Disk (FCD)  https://review.opendev.org/c/openstack/kolla-ansible/+/80876012:24
opendevreviewChristian Berendt proposed openstack/kolla-ansible master: Make the ironic notification level configurable  https://review.opendev.org/c/openstack/kolla-ansible/+/82896212:49
opendevreviewMerged openstack/kolla stable/victoria: erlang: use packages from Erlang Solutions on AArch64  https://review.opendev.org/c/openstack/kolla/+/82926912:56
opendevreviewMichal Nasiadka proposed openstack/kolla stable/ussuri: Build CentOS Stream 8 images  https://review.opendev.org/c/openstack/kolla/+/82905413:44
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: baremetal: some small tidy ups  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82100314:33
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: baremetal: refactor kolla user & group creation into a separate role  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82101514:33
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: baremetal: refactor docker-sdk installation to a separate role  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82101614:33
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: docker-sdk: use become_user=kolla to create virtualenv  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82101714:33
mnasiadkamgoddard mnasiadka hrw egonzalez yoctozepto rafaelweingartne cosmicsound osmanlicilegi bbezak parallax Fl1nt frickler adrian-a - meting in 1014:50
mnasiadka#startmeeting Kolla15:00
opendevmeetMeeting started Wed Feb 16 15:00:20 2022 UTC and is due to finish in 60 minutes.  The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'kolla'15:00
mnasiadka#topic rollcall15:00
mnasiadkao/15:00
yoctozeptoo/15:01
mgoddard|O15:02
mgoddard0115:02
yoctozepto!!!!1111oneoneoneeleven15:04
opendevmeetyoctozepto: Error: "!!!1111oneoneoneeleven" is not a valid command.15:04
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: haproxy: support different internal/external ports  https://review.opendev.org/c/openstack/kolla-ansible/+/82339515:04
mnasiadkalol15:04
mnasiadka#topic agenda15:04
mnasiadka* Announcements15:05
mnasiadka* Review action items from the last meeting15:05
mnasiadka* CI status15:05
mnasiadka* Release tasks15:05
mnasiadka* Current cycle planning15:05
mnasiadka* Additional agenda (from whiteboard)15:05
mnasiadka* Open discussion15:05
mnasiadka#topic Announcements15:05
mnasiadkaIt seems I'm the PTL for Zed cycle - in case anyone wonders ;-)15:05
mnasiadka#topic Review action items from the last meeting15:05
mnasiadkamnasiadka post a patch for docs - standard topics that should be discussed over PTG and then revisited in mid-cycle15:06
mnasiadkamnasiadka to triage security bugs and update them with resolution plan (if needed)15:06
mnasiadkahrw to discuss with pynacl upstream to release binary wheel of 1.4.0 for aarch6415:06
mnasiadkaI haven't progressed on mine, hrw is not here15:06
mnasiadka#action mnasiadka post a patch for docs - standard topics that should be discussed over PTG and then revisited in mid-cycle15:06
mnasiadka#action mnasiadka to triage security bugs and update them with resolution plan (if needed)15:06
mnasiadka#action hrw to discuss with pynacl upstream to release binary wheel of 1.4.0 for aarch6415:06
mnasiadka#topic CI Status15:06
mgoddardcongrats mnasiadka on winning the election15:07
mgoddardlong may your PTLship last!15:07
mnasiadkaCI looks fairly ok, not counting the ansible-lint blowout and centos-rabbitmq-38 repo missing on centos8s :)15:07
mnasiadkathanks mgoddard 15:07
mnasiadkaCan we update Kolla-Ansible to GREEN? Any objections yoctozepto or mgoddard?15:08
mgoddardgo ahead15:08
* frickler sneaks in late and joins in congratulations15:08
yoctozeptoyeah, congrats, mnasiadka15:08
yoctozeptoand yeah, no objections15:09
mnasiadkaupdated15:09
mnasiadka#topic Release tasks15:09
mnasiadkaThis week is R-615:09
mnasiadkaR-2 is Feature Freeze15:10
mnasiadka#topic Current cycle planning15:10
mnasiadkaAnyone has anything on this topic? Changes needing reviews? Something to discuss?15:11
mgoddardShould we look at the priorities list?15:11
yoctozeptobetter not15:11
yoctozeptolet's live happier lives15:11
mgoddardsome are complete15:12
mgoddardand still 4 weeks to go!15:12
mnasiadkaDrop binary images seems to be missing something, hmm15:12
yoctozeptomissing enthusiasm15:12
mnasiadkaTODO(): add release note in Xena informing that since Yoga binary images will be deprecated15:12
mnasiadkaDo we really need that?15:12
yoctozeptonope15:13
mnasiadkaso let's cross that out15:13
mnasiadkaTODO(frickler): Look into "What do we need to make sure that users like kevko are not left on ice with their issues with source images?"15:13
mnasiadkaBut kevko proposed something by himself IIRC15:13
mgoddardkevko proposed some patches for patching15:13
mgoddardI left some comments15:14
mgoddardtl;dr: I would like a way to get arbitrary files into images15:14
mgoddardthey could be patches15:14
mgoddardwhich could be applied in overrides15:14
mgoddardI think that would be a good starting point, and maybe we build on it if there is interest15:15
mnasiadkaSure, let's see how this evolves15:15
mgoddardwell, how do you want it to evolve?15:16
mnasiadkaI mean where the discussion in the patch evolves, we need to try to make it user friendly and good documented, doesn't really need to be state of the art :)15:16
mnasiadkaAnother one is Elasticsearch to OpenSearch (AWS fork of ES) - but I don't think I've seen any progress with it15:17
mnasiadkaparallax has been busy with $work15:17
mgoddardis it likely to make yoga?15:18
mnasiadkaI don't think so, unless there's somebody else that wants to help him15:18
yoctozeptoah, too bad15:18
mnasiadkaI'll try to find a candidate internally at SHPC, let's see15:19
yoctozeptoI think we will backport whatever we achieve at any point in time15:19
yoctozeptowe can't run outdated ES forever xD15:19
mgoddardhow about systemd15:20
mgoddardyoctozepto: I saw you put some time into it15:20
yoctozeptomgoddard: yeah, I directed the unit testing15:22
mnasiadkaunit testing director15:22
mgoddardhow close is it?15:22
yoctozeptoyeah15:23
mgoddard(IYHO)15:23
yoctozeptohard to say, I did not dig deep enough, just to get rid of failing units tests and any tricker to achieve that15:23
yoctozeptotrickery*15:23
mgoddardsecure RBAC got a bit stalled15:25
mnasiadkaRocky Linux host support is waiting for nodepool patches to merge, so we can add proper CI with that15:26
mnasiadkaI saw similar patches for AlmaLinux and OpenEuler15:26
mgoddardopeneuler was for the container images, right?15:26
mnasiadkaah yes, that one won't go in surely15:27
mnasiadkaALma Linux is for Host OS only15:27
yoctozeptoyeah, but openeuler for kolla-ansible would be quick I guess15:27
mnasiadkait's a RHEL/CentOS/Rocky clone, but I don't know if we should allow anything that's not tested in CI - althought the difference probably is none15:27
frickleropeneuler has a devstack job, so that image should be available already15:27
mgoddardwe do need to draw the line somewhere 15:28
yoctozeptomnasiadka: yeah, like between 8.2, 8.3, 8.4 and stream15:28
yoctozeptoreally "no difference" ;-)15:28
yoctozeptoand we can't test all the clones15:29
yoctozeptoit probably does not make sense15:29
yoctozeptoand just wastes resources15:29
mgoddardindeed15:29
mgoddardwe probably need some policy on it15:29
mgoddardwe could be more relaxed about the host IMO15:29
mnasiadkaeither we let it in, but claim in support matrix it's not tested15:30
mnasiadkaor we don't let it in at all15:30
mgoddardand have some untested supported distros e.g. alma15:30
mnasiadkafor the sake of community, we could do the first15:30
mgoddardbut container images we should be more strict about15:30
mnasiadkacorrect15:31
yoctozeptowe could test rocky as it's to be different from stream by design15:32
yoctozeptothen alma should work the same15:32
yoctozeptoso we say it's supported but not tested15:33
yoctozeptoand annotation: "please try on rocky before reporting the issue"15:33
mnasiadkamakes sense15:33
yoctozeptothen we have popular Chinese OS - openeuler15:33
mnasiadkabut first let's merge Rocky :)15:33
yoctozeptowhich we can test independently15:33
yoctozeptomnasiadka: exactly15:33
mnasiadkaopeneuler patch proposal is only in Kolla I think - we would need to convince the author to only pursue Host OS support in kolla-ansible15:34
mnasiadkaok, I think we all agree on the topic15:34
yoctozeptoseems so15:34
mnasiadkamgoddard: how is the Let's Encrypt one?15:34
mgoddardthe image merged15:34
mgoddardI reviewed the k-a side, left lots of comments15:35
mnasiadkaI remember James notified us that there's a bug in haproxy 2.2 that complicates things again15:35
mgoddardI'm still not 100% how it works15:35
mgoddardor whether it makes sense to use LE for the internal VIP15:35
mnasiadkanot really, since you would need to do a DNS based auth15:36
mnasiadkaand as I mentioned in the image review - certbot and it's DNS plugins are marriage made in hell15:36
mgoddardwhich it doesn't support15:37
mgoddardcould be close, could be far, depending on a few things15:38
mnasiadkaI've used lego in my recent deployment, been more than happy with it - and since it's written in Go, it's just one small binary - but anyways15:39
mnasiadkaI'll try to look at the K-A patch15:39
fricklerlego works fine with designate, yes15:40
mgoddardVMware nsxp has had some reviews15:41
mgoddardI thought it was ok, radek disagreed15:42
mgoddardI think my standards have slipped over the years :)15:42
yoctozeptomgoddard: yeah, you even mention my name all in lowercase15:42
mgoddardtoo many stalled patches15:42
mgoddardINDEED15:42
mnasiadkahaha15:42
mnasiadkaok, seems we know what needs to be reviewed15:43
yoctozeptovmware patches are close to merge15:43
yoctozeptoand they are simple enough to understand that they are fine enough15:43
mgoddardopenstack.kolla collection has lots of patches15:43
yoctozeptoobviously no idea how appropriate this is for actual vmware deployment15:43
mgoddardwould be nice to get https://review.opendev.org/c/openstack/kolla-ansible/+/819430 merged, to stop people changing the baremetal role15:44
yoctozeptomgoddard: and Radek has lots of unrelated work to do :-(15:44
mgoddarddon't we all15:44
yoctozeptomgoddard: whoa, the "merge conflicts" list is.... long15:44
mgoddardmaybe if you spent less time correcting grammar you'd have more free time? :p15:44
mgoddardyoctozepto: well, any patch that touches the baremetal role gets hit15:45
yoctozeptomgoddard: maybe!15:45
yoctozeptomgoddard: yeah15:45
wuchunyangIs there any documentation on  openstack.kolla collection?15:45
mgoddardgood question wuchunyang 15:46
mgoddardhttps://docs.openstack.org/ansible-collection-kolla/latest/15:46
mgoddardit exists15:46
mgoddardbut there is not much content15:46
mnasiadkathere's contributor guide!15:46
mgoddardI added info about how to install it in https://review.opendev.org/c/openstack/kolla-ansible/+/81943015:47
yoctozeptomnasiadka: to encourage people to contribute the docs!15:47
wuchunyangjust a contributor guide  lol..15:47
mgoddardunsure what docs we would want on it15:47
mgoddardbut I could probably come up with something15:47
mnasiadkaprobably variables from defaults/ would be a start ;-)15:47
mgoddardmaybe a list of included content15:47
wuchunyangi will try to deploy it. thanks. 15:48
mgoddardwe don't provide that for kolla-ansible15:48
mnasiadkamgoddard: I was thinking of doing that for per role variables - but we can discuss that over PTG ;-)15:48
mnasiadkalet's move to next topic15:48
mnasiadka#topic Additional agenda (from whiteboard)15:48
mnasiadka(yoctozepto) Singlenode cephadm job, multinode ipv6 job voting and gating15:49
mnasiadkaideal plan: https://etherpad.opendev.org/p/KollaAnsibleScenarios-202215:49
mnasiadkasee also our previous notes: https://etherpad.opendev.org/p/KollaAnsibleScenarios15:49
yoctozeptoindeed15:49
yoctozeptothis idea happened after a short comment by mnasiadka that we are able to deploy ceph on a single node15:50
yoctozepto(with cephadm)15:50
mgoddardlol yoctozepto proposes switching all CI to debian15:51
mnasiadkawell, that was expected :)15:52
yoctozeptonot all15:52
yoctozeptobut the things that don't need to be tested across the boad15:52
yoctozeptoboard*15:52
mgoddardthe list of current gating jobs is wrong - no ceph jobs gate15:52
mnasiadkayes, they still are a bit flaky sometimes, but hopefully it gets better with upgrade to Pacific15:53
yoctozeptoso gate on singlenode ceph?15:53
yoctozeptoI would accept that amendment15:53
mgoddardwhen was this list written? debian is gating too15:53
mnasiadkawell, I think ceph or not ceph is irrelevant, we wanted to have HA multinode jobs to check if it works15:54
mnasiadkaCeph is used, because that's the obvious choice for multinode15:54
yoctozeptomgoddard: my mistake15:54
mnasiadkaand that's what most users choose to use as storage backend15:54
yoctozeptomnasiadka: yeah, my point precisely15:55
yoctozeptowe are unlikely to break ceph with our changes but ceph likes to break us with their changes15:55
mnasiadkawell, that was a bit of my fault, to rely on /etc/hosts, now it should be a lot better :)15:56
yoctozeptomnasiadka: :-)15:56
mnasiadkaanyway, time is running up - let's maybe rethink if there are any fundamental changes we'd like to do for CI - and discuss next week or on PTG?15:57
yoctozepto++15:58
mnasiadkagreat15:58
mnasiadkaThanks for the meeting :)15:59
mnasiadka#endmeeting15:59
opendevmeetMeeting ended Wed Feb 16 15:59:13 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:59
opendevmeetMinutes:        https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-02-16-15.00.html15:59
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-02-16-15.00.txt15:59
opendevmeetLog:            https://meetings.opendev.org/meetings/kolla/2022/kolla.2022-02-16-15.00.log.html15:59
priteauWhile people are still around, I wanted to ask for a review15:59
priteauI've submitted this a couple of weeks ago: https://review.opendev.org/c/openstack/kolla-ansible/+/82686815:59
yoctozeptothanks mnasiadka16:00
yoctozeptopriteau: +216:01
priteauThanks :)16:01
wuchunyang+1 now, will +2 after  a careful review tomorrow.16:04
* yoctozepto off16:04
priteauThere is a comment from mgoddard, so there may be a revised patch16:04
mnasiadkaah, just FYI - our boss has asked me to invest time in getting CentOS Stream source builds working on Ussuri and Train to enable customers still on C7/Train to move forwards - I'll chase for reviews when those patches are ready.16:05
halomivahey, could you check systemd new patchset? it passed all tests. So we can start fully working on the podman version if its okay :) https://review.opendev.org/c/openstack/kolla-ansible/+/816724/1416:05
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: designate: transform designate_ns_record into list  https://review.opendev.org/c/openstack/kolla-ansible/+/80230416:19
mnasiadkamgoddard, yoctozepto: https://review.opendev.org/c/openstack/kolla-ansible/+/745111 - can we try to merge this one? the buffer syntax patch relies on it (https://review.opendev.org/c/openstack/kolla-ansible/+/780629)16:21
fricklermnasiadka: jinx, I was just thinking the other day that something like https://review.opendev.org/c/openstack/kolla-ansible/+/802304 was needed, let me know if I can help with that16:40
opendevreviewMerged openstack/kolla stable/ussuri: erlang: use packages from Erlang Solutions on AArch64  https://review.opendev.org/c/openstack/kolla/+/82926616:42
opendevreviewMark Goddard proposed openstack/kayobe master: kolla-target-venv: remove permissions workaround  https://review.opendev.org/c/openstack/kayobe/+/82958017:08
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: baremetal: refactor docker_sdk installation to a separate role  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82101617:45
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/+/82120417:45
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: docker: restart docker and containerd in handlers  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82120517:45
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: docker: add registry CA configuration  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82120617:45
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/+/82958617:45
opendevreviewMark Goddard proposed openstack/ansible-collection-kolla master: baremetal: refactor libvirt apparmor configuration  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82958717:45
opendevreviewMark Goddard proposed openstack/kayobe master: Stop using kolla-ansible bootstrap-servers  https://review.opendev.org/c/openstack/kayobe/+/82499817:47
opendevreviewMark Goddard proposed openstack/kayobe master: Stop using kolla-ansible bootstrap-servers  https://review.opendev.org/c/openstack/kayobe/+/82499817:48
opendevreviewMark Goddard proposed openstack/kayobe master: Stop using kolla-ansible bootstrap-servers  https://review.opendev.org/c/openstack/kayobe/+/82499817:49
opendevreviewMark Goddard proposed openstack/kayobe master: Stop using kolla-ansible bootstrap-servers  https://review.opendev.org/c/openstack/kayobe/+/82499817:51
opendevreviewMerged openstack/kayobe stable/xena: Use net_mask filter instead of ansible's ipaddr  https://review.opendev.org/c/openstack/kayobe/+/82389418:39
opendevreviewMerged openstack/kayobe stable/xena: Set correct gateway for the bifrost provision network  https://review.opendev.org/c/openstack/kayobe/+/82689018:39
opendevreviewMerged openstack/kolla-ansible master: Refactor fluentd syslog logging  https://review.opendev.org/c/openstack/kolla-ansible/+/74511120:55
opendevreviewMerged openstack/kolla-ansible master: Fix fluentd v1 buffer syntax issue  https://review.opendev.org/c/openstack/kolla-ansible/+/78062920:55
opendevreviewPierre Riteau proposed openstack/kolla-ansible master: Allow to define extra parameters for Prometheus exporters  https://review.opendev.org/c/openstack/kolla-ansible/+/82686821:28

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