13:01:00 <mnasiadka> #startmeeting kolla 13:01:00 <opendevmeet> Meeting started Wed May 8 13:01:00 2024 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. 13:01:00 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 13:01:00 <opendevmeet> The meeting name has been set to 'kolla' 13:01:04 <mnasiadka> #topic rollcall 13:01:05 <mnasiadka> o/ 13:01:16 <bbezak> o/ 13:01:16 <mattcrees> o/ 13:01:50 <gr4ytech> o/ 13:02:14 <SvenKieske> o/ 13:03:00 <frickler> \o 13:03:22 <mmalchuk> o/ 13:03:30 <mnasiadka> #topic agenda 13:03:30 <mnasiadka> * Announcements 13:03:30 <mnasiadka> * CI status 13:03:30 <mnasiadka> * Release tasks 13:03:30 <mnasiadka> * Additional agenda (from whiteboard) 13:03:31 <mnasiadka> * Open discussion 13:03:33 <mnasiadka> #topic Announcements 13:04:18 <mnasiadka> Kolla, Kolla-Ansible and ansible-collection-kolla 2024.1 RC1 is out - thank you to all for your contributions 13:04:27 <SvenKieske> \o/ 13:04:45 <mnasiadka> Kayobe will follow up probably next week, since Will is off 13:04:51 <mnasiadka> #topic CI status 13:05:08 <mnasiadka> I guess it's not bad, but I guess we should fix that Octavia CI failure 13:05:18 <mnasiadka> what is funny only cascade delete fails 13:05:31 <mnasiadka> Anybody has cycles to have a proper look? 13:06:05 <frickler> maybe next week 13:06:18 <mnasiadka> ok, I'll try to have a look as well 13:06:24 <mnasiadka> #topic Release tasks 13:06:55 <mnasiadka> So, rc1 is done - we have some patches to merge (https://review.opendev.org/c/openstack/kolla-ansible/+/918596 https://review.opendev.org/c/openstack/kolla-ansible/+/918597 and https://review.opendev.org/c/openstack/kolla/+/918599) 13:07:09 <mnasiadka> centos-release-openstack-caracal is missing in Rocky Linux - but I raised a bug in their Bug Tracker 13:07:16 <mnasiadka> hopefully NeilHanlon can help as usual :) 13:07:37 <mnasiadka> Once that's done and CI is stable - I assume we could post a final release 13:07:47 <mnasiadka> (And Kayobe will drag along a bit later as usual) 13:09:00 <mnasiadka> bbezak has raised the deploy guide patch, so we haven't forgotten this one :) 13:09:18 <frickler> approved already 13:09:45 <mnasiadka> I think let's freeze master for now, but I doubt we'll need any critical patches 13:10:14 <mnasiadka> #topic Additional agenda (from whiteboard) 13:10:43 <mnasiadka> (SvenKieske): can we quickly talk about the cells scenario failure in: https://review.opendev.org/c/openstack/kolla-ansible/+/909912 13:10:58 <mnasiadka> that one seems like not related to a feature for D ;-) 13:11:10 <mnasiadka> SvenKieske: so what is it? 13:12:13 <SvenKieske> well it was suggested that the proxysql testcase should be tested in cells scenario, but the error there does look - at first sight at least - totally unrelated, I was just hoping to get some eyes on it with more experience in the cells scenario. I will take myself a deeper look 13:12:52 <SvenKieske> apologies if I added it to the wrong meeting point. I thought this is not only strictly about the cycle planning? maybe I misunderstood. 13:14:15 <SvenKieske> so not necessary to discuss now, but I would appreciate if someone with more experience in this area can also have a look. I'll look into it myself, but the commit author has no real clue currently what needs fixing there. thanks 13:14:39 <mnasiadka> seems like some variable is undefined? 13:15:43 <mnasiadka> https://review.opendev.org/c/openstack/kolla-ansible/+/909912/19/tests/run.yml#560 13:15:44 <mnasiadka> here 13:16:14 <SvenKieske> ah! :D thanks! 13:16:34 <mnasiadka> which is funny 13:16:39 <SvenKieske> my bad, did overlook the missing "l" 13:16:45 <mnasiadka> ah 13:16:46 <mnasiadka> missing l 13:16:49 <mnasiadka> yeah, case solved 13:17:04 <mnasiadka> next one 13:17:07 <mnasiadka> (r-krcek): 13:17:07 <mnasiadka> What are your thoughts? I belive this can be a nice addition to quickly check health and state of containers. Should I continue? - https://review.opendev.org/c/openstack/kolla-ansible/+/599735 13:17:28 <mnasiadka> for me that makes sense and looks nice 13:17:31 <mnasiadka> any other comments? 13:18:24 <r-krcek> Hey guys. I woud like to get you opinion. The "kolla-ansible check" has been removed back in 2022 so if you think it is a good idea to re-add it and make it do something little different. 13:18:37 <SvenKieske> yeah looks reasonable, I suppose it doesn't add terrible runtime overhead adding it everywhere? 13:18:39 <gr4ytech> cephrgw is importing sahara vars: https://review.opendev.org/c/openstack/kolla-ansible/+/599735/19/ansible/roles/ceph-rgw/tasks/check.yml 13:19:55 <r-krcek> Thats the second question i might have . Do you think it is better to add it everywhere to "deploy" OR make it independent command that admins can run whnever they like. I personally am inclined to the second way. 13:20:45 <SvenKieske> I think the second option - so separate command - is better - if we want we can still also call this then during "deploy" phase, no? 13:23:01 <r-krcek> Okay, great. So ill re-add the "check" argument to kolla-ansible :) 13:23:45 <mnasiadka> thanks 13:24:05 <mnasiadka> ok then 13:24:11 <mnasiadka> #topic Open discussion 13:24:15 <mnasiadka> Anybody anything? 13:24:34 <mmalchuk> unmaintained reviews 13:24:43 <mmalchuk> https://review.opendev.org/q/(project:openstack/kolla+OR+project:openstack/kayobe)+branch:%5Eunmaintained/.*+status:open 13:26:14 <SvenKieske> what do you think about adding L7 healthchecks? was contributed by someone new yesterday: https://review.opendev.org/c/openstack/kolla-ansible/+/918437 13:27:41 <mmalchuk> need some tests for non-http services 13:29:27 <SvenKieske> at least the haproxy docs say it "should work" also for non http backends, but I don't know if that is sufficient for every service, yes. 13:29:52 <mnasiadka> yeah, might need a separate entry in each service to work properly 13:30:02 <mnasiadka> but we can't really work that out without testing in a local env 13:32:00 <mmalchuk> yep 13:32:01 <SvenKieske> okay, so better to split this up, ack. 13:32:30 <mmalchuk> lets split 13:36:08 <mnasiadka> Ok then 13:36:12 <mnasiadka> Thanks for coming! 13:36:19 <mnasiadka> #endmeeting