13:00:24 #startmeeting kolla 13:00:24 Meeting started Wed Sep 25 13:00:24 2024 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. 13:00:24 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 13:00:24 The meeting name has been set to 'kolla' 13:00:29 #topic rollcall 13:00:30 o/ 13:00:32 o/ 13:00:33 \o 13:00:34 o/ 13:00:37 \o/ 13:00:43 o/ 13:00:47 0/ 13:01:35 #topic agenda... (full message at ) 13:01:38 #topic CI status 13:01:49 Green? Red? Amber? Watermelon? 13:02:37 o/ 13:02:50 that agenda topic was mangled in the matrix 13:02:53 what about honeydew melon? :D 13:03:14 Yeah, switched to Element, probably it's mangling ;-) 13:03:23 I'll try better next time 13:03:25 yeah, you need to post agenda line by line for poor old IRC 13:03:28 CI looks not bad, except aarch64 13:04:32 frickler: do you know if there's any outlook on aarch64 improvements? 13:04:59 mnasiadka: none that I'm aware of 13:05:08 ok, it is what it is 13:05:16 #topic Current cycle planning 13:06:17 So, Ubuntu 24.04 seems to be closer, some small CI failures to revisit and I guess I'll ask for reviews next week 13:06:21 anything else major that we should be thinking of? 13:06:27 There's RMQ 4.0 13:06:37 As in no patches, but it got released 13:07:00 Since 4.0 release - 3.13 community support has ended 13:07:12 yes, we need to shift everything over from the old queues afaik for that to work, so afaik we need to finish the fanout/reply queue stuff 13:07:23 Sounds fantastic 13:07:31 for that to work in turn, we need the queue manager which kevko is working on 13:07:40 Michal Arbet proposed openstack/kolla-ansible master: Drop support for py38, py39 https://review.opendev.org/c/openstack/kolla-ansible/+/929408 13:07:40 Michal Arbet proposed openstack/kolla-ansible master: Add ansible-core as a dependency https://review.opendev.org/c/openstack/kolla-ansible/+/922369 13:07:53 but he seems busy :P ;) 13:08:13 with queue manager it's little bit hard ..we need to bind /dev/shm to containers using oslo.messaging 13:08:28 is that secure? 13:08:45 just..bind mount it? :) /dev/shm is actually just a fancy way to say "tmpfs". 13:09:07 well, i was thinking about the option to create tmpfs and mount as /dev/shm into container 13:09:14 I hope oslo has guarded it and there are no race conditions with concurrent access :D 13:09:20 but then I realized we have already some containers which has /dev/shm already mounted 13:09:42 Isn't --ipc=host for that? 13:09:44 and ...moreover ...in classic openstack installations from packages ..there is shared memory also ... 13:09:56 no, ipc turning more than /dev/shm 13:10:28 but yes ..if you set ipc_mode ...it's effectively turn on bind /dev/shm into container 13:10:40 but we don't need it ... 13:10:44 there's the alternative of shared ipc for containers 13:11:06 but that was a bit... tricky, because you start a container pointing to other running containers ipc 13:11:09 maybe we should stop doing containers if we share everything again anyway in the end? (almost, but not completely non-serious) 13:11:10 I think the short term conclusion is we need some time before we can truly merge rmq 4? 13:11:18 yeah, that's another point 13:11:39 if we need shared mem for all containers, why even containers? 13:12:37 so, is that bloody queue manager completely required for RMQ 4? 13:13:30 well if we wanna split hairs it's afaik not required for rmq, but for how oslo/openstack uses rmq :) 13:13:44 frickler: It is still an effective way to deploy OpenStack services and, for example, run two different versions on one system... it's containerized. Anyway... we use also like net=host namespace...it's similar 13:14:00 the oslo.messaging patch says it's optional 13:14:01 frickler: But yes, I understand your comment.. :) 13:14:19 what it does is essentially keeping track of queue names, so they always stay the same "per host" (that's the tricky part with containers). so consumers can just always bind to the same queue 13:14:58 mnasiadka: it's optional yes ... but we also provides ability to override configuration per user ...so if user choose to override and turn on the queue_manager ...he brokes his cloud because we are kolla ..right ? 13:15:08 xD 13:15:12 and it's also very nice feature :) 13:15:26 ok, but it doesn't sound like required 13:15:43 well for first release we could put big fat warning that queue manager is not yet supported? if it works? 13:15:46 so should we add precheck if user override includes queue_manager = true and fail the reconfigure ? :D 13:16:05 user overrides mean user knows what he's doing 13:16:06 but agree that it's not important issue for now 13:16:20 we never checked what is in user overrides 13:16:28 don't forget it's also requirement for streams for fanout 13:16:34 so let's not create a drama 13:16:56 agree ... 13:17:23 it's mainly my downstream project which i am (as always) tracking in gerrit ... and I like that feature 13:18:35 Well, if we can create a tmpfs volume and mount that as shm, that would probably be a bit better 13:19:56 That's most clean i think ...but i need to test if - when you turn on the ipc if it's not automatically mounted host /dev/shm and overrides your tmpfs mount 13:20:19 well, we only run libvirt with ipc=host IIRC 13:20:26 that doesn't interact with RMQ 13:20:43 hacluster ? 13:20:53 cinder ? 13:21:09 masakari ? 13:21:12 ah, and nova-compute 13:21:16 manila ? 13:21:36 well then let's drop containers 13:21:37 :) 13:21:41 we already has bunch of containers host's /dev/shm accessible 13:22:16 just saying .. :D 13:22:24 various neutron stuff also has quite some host access (necessary for SRIOV etc) afaik 13:22:39 Sounds like a lot of work 13:22:49 Maybe it's easier to write a container-friendly queue_manager? :) 13:22:52 it's just inherent complexity, the k8s deployment tools are basically doing the same thing 13:24:44 so then it probably means we need to identify which containers need to have "/dev/shm" mounted 13:24:52 Containerization is still great for delivering code; we don't necessarily have to use everything. Moreover, we also copy config files into the container instead of using binds as it should be... for example. 13:24:52 and that's it 13:25:38 I just want to say that we, as Kolla, definitely don't do everything the way it should be done... and with the complexity of OpenStack, I don't think it's even possible. 13:25:45 well, binds have the problem of permissions 13:26:10 so probably that's why the project started with this 13:26:38 anyway, we have probably other priorities this cycle 13:27:22 Anyway, I will work on it and track it in Gerrit. It's probably not worth discussing it much. I’d rather ask about ProxySQL. I’ve fixed everything, including IPv6, and I think it’s OK to switch. 13:27:22 MariaDB bump is not happening, unless they release a new version of 11.4 where they fixed SST between old MariaDB and new MariaDB 13:28:52 Well, I think I commented we need to remove the unused mariadb-clustercheck running container in upgrade tasks 13:29:11 if we're changing the default 13:29:48 mnasiadka: yeah, I replied that it can be used by external scripts maybe ... and in reno it's specified that it can be removed manually 13:30:12 mnasiadka: but I agree ... i will write something in followup ..so i will add on the top of relation chain .. 13:30:20 ok then 13:30:28 +1 13:30:57 I've added my +2 13:31:11 https://review.opendev.org/c/openstack/kolla-ansible/+/913724 13:31:25 Do we need to backport any of the fixes you had to do? 13:32:03 yep ..we should one 13:32:35 this one https://review.opendev.org/c/openstack/kolla-ansible/+/929848 only 13:32:40 it's just ipv6 ... 13:33:30 nothing more ..because that dance with --initial and removal of db file << ..this was only in master ...lower versions still working fine ... 13:33:55 and maybe this ... https://review.opendev.org/c/openstack/kolla-ansible/+/930077/4 << but this sounds like a feature .... 13:34:06 mnasiadka: thanks 13:34:36 Merged openstack/kolla-ansible stable/2024.1: Fix links in docs for unmaintained releases https://review.opendev.org/c/openstack/kolla-ansible/+/928579 13:34:38 Merged openstack/kolla-ansible stable/2023.2: Fix links in docs for unmaintained releases https://review.opendev.org/c/openstack/kolla-ansible/+/928581 13:35:14 Merged openstack/kolla-ansible stable/2023.1: Fix links in docs for unmaintained releases https://review.opendev.org/c/openstack/kolla-ansible/+/928587 13:35:14 frickler, SvenKieske : around backporting 930077 - it would be useful for my $downstream - are you fine with doing that? 13:35:33 doesn't include any inventory changes, so won't include any problems hopefully 13:36:22 nice to hear that you also using ProxySQL ;-) 13:36:29 Not yet, but planning to. 13:36:35 ah, ok 13:36:36 we need an exception for those, no? via ML? but looks okayish I think 13:36:38 And monitoring is an important aspect 13:37:15 kevko: propose the backports please for the ipv6 and the prometheus endpoint via Gerrit and we'll sort it then 13:37:16 ML ? what is the shortcut for ? :D 13:37:21 mailing list 13:37:25 ah ok 13:37:26 but no, never did it that way 13:37:28 mnasiadka: ack 13:37:52 #topic Additional agenda (from whiteboard) 13:38:13 mhiner - please review: Move to high level client in DockerWorker https://review.opendev.org/c/openstack/kolla-ansible/+/908295 13:38:19 Grzegorz Koper proposed openstack/kolla-ansible stable/2023.1: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/930466 13:39:25 mhiner: commented, will also do a proper review 13:39:34 thanks 13:39:40 Grzegorz Koper proposed openstack/kolla-ansible stable/2023.2: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/930467 13:39:52 r-krcek: https://review.opendev.org/c/openstack/kolla-ansible/+/923110 13:40:04 Grzegorz Koper proposed openstack/kolla-ansible stable/2024.1: Fixing typo in etc/kolla/globals.yml https://review.opendev.org/c/openstack/kolla-ansible/+/930468 13:41:16 ok, commented - would be nice to get a python based CLI this cycle I guess 13:42:12 SvenKieske: I've seen you did some reviews - can you help r-krcek to get those over the line? 13:42:40 Another one from r-krcek - https://review.opendev.org/c/openstack/kolla-ansible/+/599735 13:43:38 reviewed 13:43:56 yeah, probably over the next days, still stuck in upgrading stuff, sadly have considerable less time currently during working hours for reviews :-/ 13:44:15 will make some room for reviews in my free time I guess 13:44:53 Last one is a question if anybody is interested in Swift and would like to help defining tests in https://review.opendev.org/c/openstack/kolla-ansible/+/797498 13:46:23 If yes, wuchunyang and r-krcek would be probably grateful 13:46:25 #topic Open discussion 13:46:30 anything else to cover today? 13:47:16 Thank you guys for the reviews and comments :) 13:48:31 i would like to ask again for some reviews ...but you know which one right :D 13:49:04 r-krcek: if you are interested in being a more permanent part of Kolla community - given your not small contributions - you could review other people patches and become a core some day 13:49:25 kevko: yeah, will have a look on your patches tomorrow 13:50:15 * frickler will be on holiday for the coming three weeks, just fyi. will be back in time for the ptg 13:50:39 Remember PTG for Kolla starts on Tue - I already booked the slots 13:50:46 ack 13:50:48 #link https://etherpad.opendev.org/p/kolla-epoxy-ptg 13:50:48 ack 13:50:54 Please try to populate the topics 13:50:59 we have 2x3 hours 13:51:08 (and there's kayobe as well) 13:51:10 jovial: ^^ 13:51:38 Ok then, that's all for me - see you next week (have fun frickler !) 13:51:55 awesome - I'll populate a few topics. Would definitely appreciate any input there. 13:51:59 see you :) 13:52:15 o/ 13:52:20 #endmeeting