Wednesday, 2023-04-12

mnasiadkafrickler: seems we’re good now with https://review.opendev.org/c/openstack/kolla-ansible/+/87970805:30
fricklermnasiadka: ack, let's get that in and do stable releases05:45
opendevreviewRafal Lewandowski proposed openstack/kolla stable/zed: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package  https://review.opendev.org/c/openstack/kolla/+/88009706:22
opendevreviewRafal Lewandowski proposed openstack/kolla stable/yoga: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package  https://review.opendev.org/c/openstack/kolla/+/88009806:22
opendevreviewRafal Lewandowski proposed openstack/kolla stable/xena: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package  https://review.opendev.org/c/openstack/kolla/+/88009906:24
opendevreviewRafal Lewandowski proposed openstack/kolla stable/wallaby: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package  https://review.opendev.org/c/openstack/kolla/+/88010006:25
mnasiadkafrickler: while we're at reviews - https://review.opendev.org/c/openstack/kolla-ansible/+/862287 maybe this should also get in Antelope07:33
opendevreviewMerged openstack/kolla-ansible stable/xena: Pin zun jobs to Docker 20  https://review.opendev.org/c/openstack/kolla-ansible/+/87970808:39
fricklermnasiadka: do we want to do stable releases for other branches now, too? that's still open for this month. we can also discuss in the meeting later08:42
mnasiadkafrickler: we'd need to merge at least https://review.opendev.org/c/openstack/kolla-ansible/+/87918508:50
frickleralways those weird issues with weird distros :-D08:55
opendevreviewMatt Crees proposed openstack/kolla-ansible master: RabbitMQ use maintenance mode on container restart  https://review.opendev.org/c/openstack/kolla-ansible/+/87724209:06
mnasiadkafrickler: who needs those distros anyway ;)09:41
SvenKieskeinteresting, I did not know scp was involved there..10:01
opendevreviewMatt Crees proposed openstack/kolla-ansible master: RabbitMQ use maintenance mode on container restart  https://review.opendev.org/c/openstack/kolla-ansible/+/87724210:09
opendevreviewMerged openstack/kolla-ansible stable/zed: nova: Fix live migration on RHEL9 derivatives  https://review.opendev.org/c/openstack/kolla-ansible/+/87918410:11
opendevreviewMerged openstack/kolla-ansible stable/yoga: nova: Fix live migration on RHEL9 derivatives  https://review.opendev.org/c/openstack/kolla-ansible/+/87918510:15
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: rabbitmq: Do not stop containers on upgrade  https://review.opendev.org/c/openstack/kolla-ansible/+/86228711:12
SvenKieskewell, didn't the meeting time switch https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-03-22-14.01.log.html ? I'm confused between timezones. reading the log I guess the agreed time is 3 p.m. CET, which should also be equivalent to 3pm CEST?11:39
SvenKieskebut the topic still says 14:00 UTC which is 16:00 CEST11:40
fricklerah, I should update the topic, thx for noticing11:40
*** ChanServ changes topic to "IRC meetings on Wednesdays @ 13:00 UTC - agenda @ https://goo.gl/OXB0DL | Whiteboard: https://bit.ly/2MM7mWF | IRC channel is *LOGGED* @ http://goo.gl/3mzZ7b"11:41
SvenKieskenow this also is accurate again: https://meetings.opendev.org/#Kolla_Team_Meeting11:43
fricklerSvenKieske: but this one isn't, do you want to submit a patch? https://docs.openstack.org/kolla/latest/contributor/meeting.html11:43
SvenKieskesure11:44
frickleror should we drop the time there and just have one location as source of truth?11:44
SvenKieskeI like one location, there's also the old wiki redirect..and I have trouble finding the source for this site, this is in the kolla repo, I guess?11:48
SvenKieskeI guess the wiki redirect at least doesn't hurt and it's good for old google links etc. to redirect to the actual source of truth.11:49
mnasiadkaI think let's link to https://meetings.opendev.org/#Kolla_Team_Meeting, less updating next time ;)12:47
mnasiadkageez12:54
mnasiadkais it just me, or somebody translated the etherpad to some asian language?12:55
mnasiadkahttps://etherpad.opendev.org/p/KollaWhiteBoard - this etherpad12:55
SvenKieskeyeah the beginning of it; a few hours ago it was just fine12:56
mnasiadkamgoddard mnasiadka hrw  bbezak frickler kevko SvenKieske - meeting in 3 minutes12:57
mnasiadka#startmeeting kolla13:00
opendevmeetMeeting started Wed Apr 12 13:00:01 2023 UTC and is due to finish in 60 minutes.  The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot.13:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.13:00
opendevmeetThe meeting name has been set to 'kolla'13:00
mnasiadka#topic rollcall13:00
mnasiadkao/13:00
SvenKieskeo/13:00
mhinero/13:00
mattcreeso/13:00
mnasiadka#topic agenda13:03
mnasiadka* CI status13:03
mnasiadka* Release tasks13:03
mnasiadka* Regular stable releases (first meeting in a month)13:03
mnasiadka* Current cycle planning13:03
mnasiadka* Additional agenda (from whiteboard)13:03
mnasiadka* Open discussion13:03
mnasiadka#topic CI status13:03
mnasiadkaseems all green (-ish)13:03
mnasiadka#topic Release tasks13:03
mnasiadkaWe need to go through the pre-branching tasks, I think some have been executed - some not13:03
mnasiadkaI'll have a look near the end of this week and raise what is needed13:04
ramona-beermann[m]o/13:04
mnasiadkaIt would be nice if core reviewers could go through RP+1 patches in the coming weeks13:05
mnasiadkaThe usual review dashboards are linked on the whiteboard13:05
mnasiadka#topic Regular stable releases13:06
mnasiadkafrickler: I think you asked about that earlier, from my perspective if we can merge the backports in stable branches soon-ish - then we could go for a stable release of all stable branches (instead of doing Xena only right now)13:06
fricklerthe only thing is the deadline for xena is next week13:07
fricklerso maybe I'll wait until tomorrow and then do either all or only xena13:07
fricklerdoesn't make much difference to have two release patches instead of one13:08
mnasiadkalet me have a look at xena13:08
mnasiadkaok, kolla last patch is merging13:10
mnasiadkagave a nudge to kayobians about two patches13:10
mnasiadkalet's check again tomorrow13:10
mnasiadka#topic Current cycle planning13:10
mnasiadkaI started organising Bobcat plans from the PTG, not yet done, should be done before the next meeting.13:11
mnasiadkaNot really anything to discuss, unless anybody wants to add something to the pile.13:11
mnasiadka#topic Additional agenda (from whiteboard)13:11
mnasiadka(mnasiadka): requirements-check and dependencies like podman13:12
mnasiadkaSo, docker-py is in u-c/requirements13:12
mnasiadkabut podman is not13:12
mnasiadkaI don't think we can add it to u-c for antelope (if we wanted to)13:12
mnasiadkaI think frickler mentioned we could just stop running requirements-check and manage requirements on our own (which wouldn't be a bad thing)13:13
frickleror we can make podman optional for now13:13
fricklerand install it in tox.ini explicitly for unit tests13:13
mnasiadkayes, that's what I proposed for mhiner/ihalomi13:14
mnasiadkaand propose it to be added in u-c for bobcat?13:14
mhinersounds good to me13:14
fricklermaybe it is even better in the long run not to have it as a hard requirement13:14
fricklerunless we want to drop docker support completely13:15
mnasiadkaI think currently it's good that we support both, but in the long run we might change the decision13:15
SvenKieskewell docker upstream get's more and more non floss imho  - they need to make money13:16
mnasiadkalet's see what headaches will we get with running podman ;-)13:16
mnasiadkaok, so for now not adding podman to requirements.txt and just adding it to tox.ini for testing13:17
fricklerI can still propose a patch to requirements repo and see what others think13:17
SvenKieskesounds good13:17
mnasiadkasounds nice frickler 13:17
mnasiadkaif docker is there, then podman should not be different I guess13:17
mnasiadkaOk, that's done13:18
mnasiadkasecond is13:18
fricklerthat seems to be a valid argument, yes. let's see how it turns out13:18
mnasiadka(kyarovoy): help needed with https://review.opendev.org/c/openstack/kolla/+/825786, if someone has time - please look at the problems that appear with kolla-build-ansible13:18
mnasiadkait seems this patch is failing on push_error13:19
frickleris that related to the podman work or independent?13:19
SvenKieskeI took a quick look, but I'm maybe missing context as there is no bugreport or story linked why this is done13:19
mnasiadka(which I mentioned in the review)13:19
mnasiadkaI think it's independent, but a nice addition (we don't need to build containers using podman/buildah, but it would be a bit non-optimal to enforce users to install docker for building images)13:20
SvenKieskethe topic is kolla-podman, so probably related to the podman work? just a wild guess though.13:20
mnasiadkaI don't have time to look into that, at least probably not in the next 2-3 weeks13:20
mnasiadkais there anybody that wants to help Konstantin?13:21
mnasiadkamhiner: he's from your team, right?13:21
mhineryes13:21
SvenKieskeI guess I'm not yet familiar enough with that part of the kolla codebase to be of much help there13:22
mhinersame for me13:23
fricklerI guess nobody is, opportunity to learn something ;)13:23
mnasiadkahrw is not here, but he probably would be a good candidate - if he has enough time (which he probably doesn't)13:23
mnasiadkaAnyway, I'll leave the topic for the next meeting, we'll see if there's a candidate for help next time :)13:24
SvenKieskedon't know if I got the time (looking at frickler): I can take a deeper look, I guess, if time permits.13:24
mnasiadkaWould be nice, let's revisit on next meeting13:25
mnasiadka#topic Open discussion13:25
opendevreviewMartin Hiner proposed openstack/kolla-ansible master: Add support of podman deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/79922913:25
fricklerthat was a good prelude for my topic today: introducing SvenKieske 13:26
SvenKieskehey there o/13:26
fricklerhe's working for OSISM now and as you may have already noticed quite interested in digging deeper into kolla, too13:27
mnasiadkanice, more the merrier (especially in these times) :-)13:27
hrwhola13:27
SvenKieskeyes, I was also around at my old employer, but really only for urgent bugfixes, so I already have some familiarity with kolla-ansible.13:29
SvenKieskeI hope I can extend that and help out where help is needed :)13:29
fricklersurely we won't run short of opportunities soon13:30
fricklerthe other topic I have is the octavia CI, not sure what the latest state was there mnasiadka?13:30
mnasiadkaGood question, I need to have a look again, but probably we need to hold the node and find out what is going on...13:31
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: docker: implement macros to alter upper-constraints.txt  https://review.opendev.org/c/openstack/kolla/+/87864913:31
fricklero.k., so I'll trigger a hold later and check13:32
fricklerI just deleted an older held node earlier because I assumed it was obsolete13:32
fricklerI guess that's all, then?13:36
mnasiadkafrickler: basically the vm is booting, but octavia can't connect to it13:36
mnasiadkaYes, we agreed on the PTG to add bug triaging/reviews time to our meetings, to focus on those - I'll add it to the next meeting13:36
frickleryeah, will check and give you access, too, if you want13:36
frickler(for the held node)13:37
mnasiadkaand will start the first week of regular new bugs triaging13:37
mnasiadkaand we'll try to find more assignees on the way13:37
frickler+113:38
mnasiadkafrickler: ok, let me know when I can have a look, I guess we need to fix it before the release ;)13:39
frickleryes and then also add the missing debian jobs before branching13:39
mnasiadkagood13:39
mnasiadkaok, let's finish for today13:39
mnasiadkathanks for coming13:39
mnasiadka#endmeeting13:40
opendevmeetMeeting ended Wed Apr 12 13:40:02 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)13:40
opendevmeetMinutes:        https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-04-12-13.00.html13:40
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-04-12-13.00.txt13:40
opendevmeetLog:            https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-04-12-13.00.log.html13:40
opendevreviewMartin Hiner proposed openstack/kolla-ansible master: Add support of podman deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/79922913:42
hrwSvenKieske, mhiner: so you need help with https://review.opendev.org/c/openstack/kolla/+/825786 and https://review.opendev.org/c/openstack/kolla/+/825791 ones?13:45
hrwdid I understood mnasiadka correctly?13:45
SvenKieskeiiuc: yes13:46
mnasiadkahrw: yes, mhiner's colleague needs the help (patch owner)13:46
mnasiadkahrw: seems there's a push_error, so probably that part of the logic got broken13:46
SvenKieskeI'll try to help where I already can13:46
mnasiadkahrw: other question - you've started some work with the github downloads on the PTG - do you have something working?13:47
hrwI commented one with 3.0.0 in title - since zed we have 'docker===6.0.x' in openstack/requirements so anything <3.0 should be imho print('update');sys.exit(-1)13:47
hrwmnasiadka: no, had some stuff to do and put it on shelf13:47
mnasiadkahrw: would be nice to get that, because now we stopped really checking for new versions ;)13:48
hrwSvenKieske: the https://review.opendev.org/c/openstack/kolla/+/825791 one needs update first as it is one year old now13:48
hrwmnasiadka: let me start by just addint script to review13:49
SvenKieskesure13:51
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: WIP: we want to track github releases easier  https://review.opendev.org/c/openstack/kolla/+/88015413:51
hrwok, script is there so I will not remove it by mistake13:52
hrwSvenKieske: I would also disable all CI jobs other than kolla-build ones and add set of kolla-build-podman ones.13:55
hrwSvenKieske: later we can rename kolla-build to kolla-build-docker but that needs to be in-sync with kolla-ansible so let postpone for now13:55
hrwSvenKieske: this way you have up-to-date kolla with podman support and we have CI running with it13:56
mmalchukmnasiadka there several forgotten backports without +w13:56
mmalchukhttps://review.opendev.org/q/I7d9c53297db1a9bd14e3e69b58867293014af69b13:56
hrwSvenKieske: once zuul is fine, adding k-a job which would use podman would be next13:56
mmalchukxena merged. but zed and yoga forgotten13:57
mmalchukhttps://review.opendev.org/q/I4cd9dd088cbcf38ff6f8d188ebcc56be7d9ea1c913:57
mmalchukhere zed forgotten13:57
mmalchukxena yoga merged13:58
hrwdone done13:58
mmalchukthx13:58
mmalchukthechdebt: https://review.opendev.org/q/topic:dib-yaml-width13:59
mmalchukplease review13:59
mmalchukvery usefull fix: https://review.opendev.org/q/topic:fix-merge-action-plugins14:00
mmalchukplease review14:00
hrwdib... argh... /me runs14:01
mmalchuklol14:01
hrwI wrote https://marcin.juszkiewicz.com.pl/2018/01/12/yadibp/ when I had to dig in dib.14:01
mnasiadkammalchuk: you haven't responded to previous review comments by Radek, there's no need of another review for dib-yaml-width14:02
mmalchukhm14:03
mnasiadkaand if two projects (kolla-ansible and kayobe) use the same plugin, maybe it's time to move it to ansible-collection-kolla?14:03
mmalchukimho first need merge and make backports, than move to collection in master14:05
mmalchukits hard to backport from collection14:05
mmalchukif its acceptable after backports can prepare move to collection14:06
mnasiadkamakes sense, not saying to do it right now :)14:07
mmalchukso review and merge please: https://review.opendev.org/q/topic:fix-merge-action-plugins14:10
mmalchukdib-yaml-width will be updated asap14:11
mmalchukmnasiadka https://review.opendev.org/c/openstack/kayobe/+/879539 - fixed bug you reported14:13
mmalchukanother one forgotten +W for yoga but xena merged almost: https://review.opendev.org/q/I666d7011bde0050ebc509b427c1d4f5a66b6231a14:17
mmalchukand one more (yoga again): https://review.opendev.org/q/I8e25fdc459102ca6e13bd3d38191554ba32c1f4814:18
mnasiadkadone and done14:18
mnasiadkawallaby is failing on first one14:18
* hrw out14:19
mmalchukthere many problems, don't know how do deal with them14:21
opendevreviewMerged openstack/kolla stable/xena: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package  https://review.opendev.org/c/openstack/kolla/+/88009914:42
opendevreviewMerged openstack/kolla stable/yoga: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package  https://review.opendev.org/c/openstack/kolla/+/88009814:45
opendevreviewMerged openstack/kolla stable/zed: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package  https://review.opendev.org/c/openstack/kolla/+/88009714:45
opendevreviewVerification of a change to openstack/kayobe stable/xena failed: Support configuring VLANs with systemd-networkd syntax  https://review.opendev.org/c/openstack/kayobe/+/87337415:14
opendevreviewMatt Crees proposed openstack/kolla-ansible master: RabbitMQ use maintenance mode on container restart  https://review.opendev.org/c/openstack/kolla-ansible/+/87724215:15
opendevreviewMerged openstack/kolla stable/yoga: Cinder backup fails due to absent multipath tools  https://review.opendev.org/c/openstack/kolla/+/87869915:33
opendevreviewMerged openstack/kolla stable/zed: Cinder backup fails due to absent multipath tools  https://review.opendev.org/c/openstack/kolla/+/87816715:33
opendevreviewMerged openstack/kolla-ansible stable/zed: Add LimitRequestBody configuration for Horizon  https://review.opendev.org/c/openstack/kolla-ansible/+/87932015:33
opendevreviewMaksim Malchuk proposed openstack/kayobe stable/wallaby: Support configuring VLANs with systemd-networkd syntax  https://review.opendev.org/c/openstack/kayobe/+/87776315:45
mnasiadkafrickler: https://review.opendev.org/c/openstack/kolla-ansible/+/862287 - looking for another +2 :)16:19
opendevreviewMichal Nasiadka proposed openstack/kolla stable/wallaby: Test for unsafe files in tarfile.extractall  https://review.opendev.org/c/openstack/kolla/+/87776016:31
opendevreviewMerged openstack/kayobe stable/xena: ironic: switch to kernel_append_params  https://review.opendev.org/c/openstack/kayobe/+/87763717:19
opendevreviewMerged openstack/kayobe stable/yoga: Support configuring VLANs with systemd-networkd syntax  https://review.opendev.org/c/openstack/kayobe/+/87337317:19
opendevreviewMerged openstack/kayobe stable/yoga: Ubuntu: Fix networkd config for multiple VLANs without MTU  https://review.opendev.org/c/openstack/kayobe/+/87411817:19
opendevreviewMaksim Malchuk proposed openstack/kolla stable/wallaby: Test for unsafe files in tarfile.extractall  https://review.opendev.org/c/openstack/kolla/+/87776018:34
opendevreviewMaksim Malchuk proposed openstack/kolla stable/wallaby: Add multipath to cinder-volume  https://review.opendev.org/c/openstack/kolla/+/87918220:11
opendevreviewMaksim Malchuk proposed openstack/kolla stable/wallaby: Add multipath to cinder-volume  https://review.opendev.org/c/openstack/kolla/+/87918220:12
opendevreviewMaksim Malchuk proposed openstack/kolla stable/wallaby: Add multipath to cinder-volume  https://review.opendev.org/c/openstack/kolla/+/87918220:13
opendevreviewMerged openstack/kolla-ansible master: rabbitmq: Do not stop containers on upgrade  https://review.opendev.org/c/openstack/kolla-ansible/+/86228720:26
opendevreviewMaksim Malchuk proposed openstack/kolla stable/wallaby: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package  https://review.opendev.org/c/openstack/kolla/+/88010021:49
opendevreviewMerged openstack/kayobe stable/xena: Support configuring VLANs with systemd-networkd syntax  https://review.opendev.org/c/openstack/kayobe/+/87337422:27

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