mnasiadka | frickler: seems we’re good now with https://review.opendev.org/c/openstack/kolla-ansible/+/879708 | 05:30 |
---|---|---|
frickler | mnasiadka: ack, let's get that in and do stable releases | 05:45 |
opendevreview | Rafal Lewandowski proposed openstack/kolla stable/zed: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package https://review.opendev.org/c/openstack/kolla/+/880097 | 06:22 |
opendevreview | Rafal Lewandowski proposed openstack/kolla stable/yoga: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package https://review.opendev.org/c/openstack/kolla/+/880098 | 06:22 |
opendevreview | Rafal Lewandowski proposed openstack/kolla stable/xena: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package https://review.opendev.org/c/openstack/kolla/+/880099 | 06:24 |
opendevreview | Rafal Lewandowski proposed openstack/kolla stable/wallaby: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package https://review.opendev.org/c/openstack/kolla/+/880100 | 06:25 |
mnasiadka | frickler: while we're at reviews - https://review.opendev.org/c/openstack/kolla-ansible/+/862287 maybe this should also get in Antelope | 07:33 |
opendevreview | Merged openstack/kolla-ansible stable/xena: Pin zun jobs to Docker 20 https://review.opendev.org/c/openstack/kolla-ansible/+/879708 | 08:39 |
frickler | mnasiadka: 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 later | 08:42 |
mnasiadka | frickler: we'd need to merge at least https://review.opendev.org/c/openstack/kolla-ansible/+/879185 | 08:50 |
frickler | always those weird issues with weird distros :-D | 08:55 |
opendevreview | Matt Crees proposed openstack/kolla-ansible master: RabbitMQ use maintenance mode on container restart https://review.opendev.org/c/openstack/kolla-ansible/+/877242 | 09:06 |
mnasiadka | frickler: who needs those distros anyway ;) | 09:41 |
SvenKieske | interesting, I did not know scp was involved there.. | 10:01 |
opendevreview | Matt Crees proposed openstack/kolla-ansible master: RabbitMQ use maintenance mode on container restart https://review.opendev.org/c/openstack/kolla-ansible/+/877242 | 10:09 |
opendevreview | Merged openstack/kolla-ansible stable/zed: nova: Fix live migration on RHEL9 derivatives https://review.opendev.org/c/openstack/kolla-ansible/+/879184 | 10:11 |
opendevreview | Merged openstack/kolla-ansible stable/yoga: nova: Fix live migration on RHEL9 derivatives https://review.opendev.org/c/openstack/kolla-ansible/+/879185 | 10:15 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: rabbitmq: Do not stop containers on upgrade https://review.opendev.org/c/openstack/kolla-ansible/+/862287 | 11:12 |
SvenKieske | well, 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 |
SvenKieske | but the topic still says 14:00 UTC which is 16:00 CEST | 11:40 |
frickler | ah, I should update the topic, thx for noticing | 11: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 | |
SvenKieske | now this also is accurate again: https://meetings.opendev.org/#Kolla_Team_Meeting | 11:43 |
frickler | SvenKieske: but this one isn't, do you want to submit a patch? https://docs.openstack.org/kolla/latest/contributor/meeting.html | 11:43 |
SvenKieske | sure | 11:44 |
frickler | or should we drop the time there and just have one location as source of truth? | 11:44 |
SvenKieske | I 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 |
SvenKieske | I 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 |
mnasiadka | I think let's link to https://meetings.opendev.org/#Kolla_Team_Meeting, less updating next time ;) | 12:47 |
mnasiadka | geez | 12:54 |
mnasiadka | is it just me, or somebody translated the etherpad to some asian language? | 12:55 |
mnasiadka | https://etherpad.opendev.org/p/KollaWhiteBoard - this etherpad | 12:55 |
SvenKieske | yeah the beginning of it; a few hours ago it was just fine | 12:56 |
mnasiadka | mgoddard mnasiadka hrw bbezak frickler kevko SvenKieske - meeting in 3 minutes | 12:57 |
mnasiadka | #startmeeting kolla | 13:00 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
opendevmeet | The meeting name has been set to 'kolla' | 13:00 |
mnasiadka | #topic rollcall | 13:00 |
mnasiadka | o/ | 13:00 |
SvenKieske | o/ | 13:00 |
mhiner | o/ | 13:00 |
mattcrees | o/ | 13:00 |
mnasiadka | #topic agenda | 13:03 |
mnasiadka | * CI status | 13:03 |
mnasiadka | * Release tasks | 13:03 |
mnasiadka | * Regular stable releases (first meeting in a month) | 13:03 |
mnasiadka | * Current cycle planning | 13:03 |
mnasiadka | * Additional agenda (from whiteboard) | 13:03 |
mnasiadka | * Open discussion | 13:03 |
mnasiadka | #topic CI status | 13:03 |
mnasiadka | seems all green (-ish) | 13:03 |
mnasiadka | #topic Release tasks | 13:03 |
mnasiadka | We need to go through the pre-branching tasks, I think some have been executed - some not | 13:03 |
mnasiadka | I'll have a look near the end of this week and raise what is needed | 13:04 |
ramona-beermann[m] | o/ | 13:04 |
mnasiadka | It would be nice if core reviewers could go through RP+1 patches in the coming weeks | 13:05 |
mnasiadka | The usual review dashboards are linked on the whiteboard | 13:05 |
mnasiadka | #topic Regular stable releases | 13:06 |
mnasiadka | frickler: 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 |
frickler | the only thing is the deadline for xena is next week | 13:07 |
frickler | so maybe I'll wait until tomorrow and then do either all or only xena | 13:07 |
frickler | doesn't make much difference to have two release patches instead of one | 13:08 |
mnasiadka | let me have a look at xena | 13:08 |
mnasiadka | ok, kolla last patch is merging | 13:10 |
mnasiadka | gave a nudge to kayobians about two patches | 13:10 |
mnasiadka | let's check again tomorrow | 13:10 |
mnasiadka | #topic Current cycle planning | 13:10 |
mnasiadka | I started organising Bobcat plans from the PTG, not yet done, should be done before the next meeting. | 13:11 |
mnasiadka | Not 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 podman | 13:12 |
mnasiadka | So, docker-py is in u-c/requirements | 13:12 |
mnasiadka | but podman is not | 13:12 |
mnasiadka | I don't think we can add it to u-c for antelope (if we wanted to) | 13:12 |
mnasiadka | I 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 |
frickler | or we can make podman optional for now | 13:13 |
frickler | and install it in tox.ini explicitly for unit tests | 13:13 |
mnasiadka | yes, that's what I proposed for mhiner/ihalomi | 13:14 |
mnasiadka | and propose it to be added in u-c for bobcat? | 13:14 |
mhiner | sounds good to me | 13:14 |
frickler | maybe it is even better in the long run not to have it as a hard requirement | 13:14 |
frickler | unless we want to drop docker support completely | 13:15 |
mnasiadka | I think currently it's good that we support both, but in the long run we might change the decision | 13:15 |
SvenKieske | well docker upstream get's more and more non floss imho - they need to make money | 13:16 |
mnasiadka | let's see what headaches will we get with running podman ;-) | 13:16 |
mnasiadka | ok, so for now not adding podman to requirements.txt and just adding it to tox.ini for testing | 13:17 |
frickler | I can still propose a patch to requirements repo and see what others think | 13:17 |
SvenKieske | sounds good | 13:17 |
mnasiadka | sounds nice frickler | 13:17 |
mnasiadka | if docker is there, then podman should not be different I guess | 13:17 |
mnasiadka | Ok, that's done | 13:18 |
mnasiadka | second is | 13:18 |
frickler | that seems to be a valid argument, yes. let's see how it turns out | 13: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-ansible | 13:18 |
mnasiadka | it seems this patch is failing on push_error | 13:19 |
frickler | is that related to the podman work or independent? | 13:19 |
SvenKieske | I took a quick look, but I'm maybe missing context as there is no bugreport or story linked why this is done | 13:19 |
mnasiadka | (which I mentioned in the review) | 13:19 |
mnasiadka | I 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 |
SvenKieske | the topic is kolla-podman, so probably related to the podman work? just a wild guess though. | 13:20 |
mnasiadka | I don't have time to look into that, at least probably not in the next 2-3 weeks | 13:20 |
mnasiadka | is there anybody that wants to help Konstantin? | 13:21 |
mnasiadka | mhiner: he's from your team, right? | 13:21 |
mhiner | yes | 13:21 |
SvenKieske | I guess I'm not yet familiar enough with that part of the kolla codebase to be of much help there | 13:22 |
mhiner | same for me | 13:23 |
frickler | I guess nobody is, opportunity to learn something ;) | 13:23 |
mnasiadka | hrw is not here, but he probably would be a good candidate - if he has enough time (which he probably doesn't) | 13:23 |
mnasiadka | Anyway, I'll leave the topic for the next meeting, we'll see if there's a candidate for help next time :) | 13:24 |
SvenKieske | don't know if I got the time (looking at frickler): I can take a deeper look, I guess, if time permits. | 13:24 |
mnasiadka | Would be nice, let's revisit on next meeting | 13:25 |
mnasiadka | #topic Open discussion | 13:25 |
opendevreview | Martin Hiner proposed openstack/kolla-ansible master: Add support of podman deployment https://review.opendev.org/c/openstack/kolla-ansible/+/799229 | 13:25 |
frickler | that was a good prelude for my topic today: introducing SvenKieske | 13:26 |
SvenKieske | hey there o/ | 13:26 |
frickler | he's working for OSISM now and as you may have already noticed quite interested in digging deeper into kolla, too | 13:27 |
mnasiadka | nice, more the merrier (especially in these times) :-) | 13:27 |
hrw | hola | 13:27 |
SvenKieske | yes, 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 |
SvenKieske | I hope I can extend that and help out where help is needed :) | 13:29 |
frickler | surely we won't run short of opportunities soon | 13:30 |
frickler | the other topic I have is the octavia CI, not sure what the latest state was there mnasiadka? | 13:30 |
mnasiadka | Good 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 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: docker: implement macros to alter upper-constraints.txt https://review.opendev.org/c/openstack/kolla/+/878649 | 13:31 |
frickler | o.k., so I'll trigger a hold later and check | 13:32 |
frickler | I just deleted an older held node earlier because I assumed it was obsolete | 13:32 |
frickler | I guess that's all, then? | 13:36 |
mnasiadka | frickler: basically the vm is booting, but octavia can't connect to it | 13:36 |
mnasiadka | Yes, 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 meeting | 13:36 |
frickler | yeah, will check and give you access, too, if you want | 13:36 |
frickler | (for the held node) | 13:37 |
mnasiadka | and will start the first week of regular new bugs triaging | 13:37 |
mnasiadka | and we'll try to find more assignees on the way | 13:37 |
frickler | +1 | 13:38 |
mnasiadka | frickler: ok, let me know when I can have a look, I guess we need to fix it before the release ;) | 13:39 |
frickler | yes and then also add the missing debian jobs before branching | 13:39 |
mnasiadka | good | 13:39 |
mnasiadka | ok, let's finish for today | 13:39 |
mnasiadka | thanks for coming | 13:39 |
mnasiadka | #endmeeting | 13:40 |
opendevmeet | Meeting ended Wed Apr 12 13:40:02 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:40 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-04-12-13.00.html | 13:40 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-04-12-13.00.txt | 13:40 |
opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-04-12-13.00.log.html | 13:40 |
opendevreview | Martin Hiner proposed openstack/kolla-ansible master: Add support of podman deployment https://review.opendev.org/c/openstack/kolla-ansible/+/799229 | 13:42 |
hrw | SvenKieske, 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 |
hrw | did I understood mnasiadka correctly? | 13:45 |
SvenKieske | iiuc: yes | 13:46 |
mnasiadka | hrw: yes, mhiner's colleague needs the help (patch owner) | 13:46 |
mnasiadka | hrw: seems there's a push_error, so probably that part of the logic got broken | 13:46 |
SvenKieske | I'll try to help where I already can | 13:46 |
mnasiadka | hrw: other question - you've started some work with the github downloads on the PTG - do you have something working? | 13:47 |
hrw | I 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 |
hrw | mnasiadka: no, had some stuff to do and put it on shelf | 13:47 |
mnasiadka | hrw: would be nice to get that, because now we stopped really checking for new versions ;) | 13:48 |
hrw | SvenKieske: the https://review.opendev.org/c/openstack/kolla/+/825791 one needs update first as it is one year old now | 13:48 |
hrw | mnasiadka: let me start by just addint script to review | 13:49 |
SvenKieske | sure | 13:51 |
opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: WIP: we want to track github releases easier https://review.opendev.org/c/openstack/kolla/+/880154 | 13:51 |
hrw | ok, script is there so I will not remove it by mistake | 13:52 |
hrw | SvenKieske: I would also disable all CI jobs other than kolla-build ones and add set of kolla-build-podman ones. | 13:55 |
hrw | SvenKieske: later we can rename kolla-build to kolla-build-docker but that needs to be in-sync with kolla-ansible so let postpone for now | 13:55 |
hrw | SvenKieske: this way you have up-to-date kolla with podman support and we have CI running with it | 13:56 |
mmalchuk | mnasiadka there several forgotten backports without +w | 13:56 |
mmalchuk | https://review.opendev.org/q/I7d9c53297db1a9bd14e3e69b58867293014af69b | 13:56 |
hrw | SvenKieske: once zuul is fine, adding k-a job which would use podman would be next | 13:56 |
mmalchuk | xena merged. but zed and yoga forgotten | 13:57 |
mmalchuk | https://review.opendev.org/q/I4cd9dd088cbcf38ff6f8d188ebcc56be7d9ea1c9 | 13:57 |
mmalchuk | here zed forgotten | 13:57 |
mmalchuk | xena yoga merged | 13:58 |
hrw | done done | 13:58 |
mmalchuk | thx | 13:58 |
mmalchuk | thechdebt: https://review.opendev.org/q/topic:dib-yaml-width | 13:59 |
mmalchuk | please review | 13:59 |
mmalchuk | very usefull fix: https://review.opendev.org/q/topic:fix-merge-action-plugins | 14:00 |
mmalchuk | please review | 14:00 |
hrw | dib... argh... /me runs | 14:01 |
mmalchuk | lol | 14:01 |
hrw | I wrote https://marcin.juszkiewicz.com.pl/2018/01/12/yadibp/ when I had to dig in dib. | 14:01 |
mnasiadka | mmalchuk: you haven't responded to previous review comments by Radek, there's no need of another review for dib-yaml-width | 14:02 |
mmalchuk | hm | 14:03 |
mnasiadka | and if two projects (kolla-ansible and kayobe) use the same plugin, maybe it's time to move it to ansible-collection-kolla? | 14:03 |
mmalchuk | imho first need merge and make backports, than move to collection in master | 14:05 |
mmalchuk | its hard to backport from collection | 14:05 |
mmalchuk | if its acceptable after backports can prepare move to collection | 14:06 |
mnasiadka | makes sense, not saying to do it right now :) | 14:07 |
mmalchuk | so review and merge please: https://review.opendev.org/q/topic:fix-merge-action-plugins | 14:10 |
mmalchuk | dib-yaml-width will be updated asap | 14:11 |
mmalchuk | mnasiadka https://review.opendev.org/c/openstack/kayobe/+/879539 - fixed bug you reported | 14:13 |
mmalchuk | another one forgotten +W for yoga but xena merged almost: https://review.opendev.org/q/I666d7011bde0050ebc509b427c1d4f5a66b6231a | 14:17 |
mmalchuk | and one more (yoga again): https://review.opendev.org/q/I8e25fdc459102ca6e13bd3d38191554ba32c1f48 | 14:18 |
mnasiadka | done and done | 14:18 |
mnasiadka | wallaby is failing on first one | 14:18 |
* hrw out | 14:19 | |
mmalchuk | there many problems, don't know how do deal with them | 14:21 |
opendevreview | Merged openstack/kolla stable/xena: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package https://review.opendev.org/c/openstack/kolla/+/880099 | 14:42 |
opendevreview | Merged openstack/kolla stable/yoga: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package https://review.opendev.org/c/openstack/kolla/+/880098 | 14:45 |
opendevreview | Merged openstack/kolla stable/zed: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package https://review.opendev.org/c/openstack/kolla/+/880097 | 14:45 |
opendevreview | Verification of a change to openstack/kayobe stable/xena failed: Support configuring VLANs with systemd-networkd syntax https://review.opendev.org/c/openstack/kayobe/+/873374 | 15:14 |
opendevreview | Matt Crees proposed openstack/kolla-ansible master: RabbitMQ use maintenance mode on container restart https://review.opendev.org/c/openstack/kolla-ansible/+/877242 | 15:15 |
opendevreview | Merged openstack/kolla stable/yoga: Cinder backup fails due to absent multipath tools https://review.opendev.org/c/openstack/kolla/+/878699 | 15:33 |
opendevreview | Merged openstack/kolla stable/zed: Cinder backup fails due to absent multipath tools https://review.opendev.org/c/openstack/kolla/+/878167 | 15:33 |
opendevreview | Merged openstack/kolla-ansible stable/zed: Add LimitRequestBody configuration for Horizon https://review.opendev.org/c/openstack/kolla-ansible/+/879320 | 15:33 |
opendevreview | Maksim Malchuk proposed openstack/kayobe stable/wallaby: Support configuring VLANs with systemd-networkd syntax https://review.opendev.org/c/openstack/kayobe/+/877763 | 15:45 |
mnasiadka | frickler: https://review.opendev.org/c/openstack/kolla-ansible/+/862287 - looking for another +2 :) | 16:19 |
opendevreview | Michal Nasiadka proposed openstack/kolla stable/wallaby: Test for unsafe files in tarfile.extractall https://review.opendev.org/c/openstack/kolla/+/877760 | 16:31 |
opendevreview | Merged openstack/kayobe stable/xena: ironic: switch to kernel_append_params https://review.opendev.org/c/openstack/kayobe/+/877637 | 17:19 |
opendevreview | Merged openstack/kayobe stable/yoga: Support configuring VLANs with systemd-networkd syntax https://review.opendev.org/c/openstack/kayobe/+/873373 | 17:19 |
opendevreview | Merged openstack/kayobe stable/yoga: Ubuntu: Fix networkd config for multiple VLANs without MTU https://review.opendev.org/c/openstack/kayobe/+/874118 | 17:19 |
opendevreview | Maksim Malchuk proposed openstack/kolla stable/wallaby: Test for unsafe files in tarfile.extractall https://review.opendev.org/c/openstack/kolla/+/877760 | 18:34 |
opendevreview | Maksim Malchuk proposed openstack/kolla stable/wallaby: Add multipath to cinder-volume https://review.opendev.org/c/openstack/kolla/+/879182 | 20:11 |
opendevreview | Maksim Malchuk proposed openstack/kolla stable/wallaby: Add multipath to cinder-volume https://review.opendev.org/c/openstack/kolla/+/879182 | 20:12 |
opendevreview | Maksim Malchuk proposed openstack/kolla stable/wallaby: Add multipath to cinder-volume https://review.opendev.org/c/openstack/kolla/+/879182 | 20:13 |
opendevreview | Merged openstack/kolla-ansible master: rabbitmq: Do not stop containers on upgrade https://review.opendev.org/c/openstack/kolla-ansible/+/862287 | 20:26 |
opendevreview | Maksim Malchuk proposed openstack/kolla stable/wallaby: nova-libvirt: Fix for missing libvirt-daemon-driver-nodedev package https://review.opendev.org/c/openstack/kolla/+/880100 | 21:49 |
opendevreview | Merged openstack/kayobe stable/xena: Support configuring VLANs with systemd-networkd syntax https://review.opendev.org/c/openstack/kayobe/+/873374 | 22:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!