13:00:20 <bbezak> #startmeeting kolla
13:00:20 <opendevmeet> Meeting started Wed Jun  5 13:00:20 2024 UTC and is due to finish in 60 minutes.  The chair is bbezak. Information about MeetBot at http://wiki.debian.org/MeetBot.
13:00:20 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
13:00:20 <opendevmeet> The meeting name has been set to 'kolla'
13:00:24 <bbezak> #topic rollcall
13:00:28 <SvenKieske> o/
13:00:37 <jovial[m]> o/
13:00:38 <mmalchuk> o/
13:00:42 <frickler> \o
13:00:44 <mhiner> o/
13:01:12 <jhorstmann> o/
13:01:32 <r-krcek> 0/
13:02:57 <bbezak> #topic agenda
13:02:57 <bbezak> * Roll-call
13:02:57 <bbezak> * Agenda
13:02:57 <bbezak> * Announcements
13:02:57 <bbezak> * Review action items from the last meeting
13:02:58 <bbezak> * CI status
13:02:58 <bbezak> * Release tasks
13:03:00 <bbezak> * Regular stable releases (first meeting in a month)
13:03:00 <bbezak> * Current cycle planning
13:03:02 <bbezak> * Additional agenda (from whiteboard)
13:03:02 <bbezak> * Open discussion
13:03:47 <bbezak> #topic CI status
13:04:06 <bbezak> periodics jobs are failing
13:04:25 <bbezak> possible fix - https://review.opendev.org/c/openstack/kolla/+/921032. I'm working on bumping community.docker in CI
13:04:49 <bbezak> test change of periodics jobs will follow after the meeting
13:05:27 <jovial[m]> I think we are all green in kayobe land
13:05:31 <bbezak> today we fixed bobcat and antelope for kolla
13:06:12 <bbezak> great
13:06:34 <bbezak> I think except periodics we're OK
13:06:36 <SvenKieske> why do the periodic jobs need another fix? I don't understand the comment in your commit message
13:07:08 <SvenKieske> https://opendev.org/openstack/kolla/src/commit/968711579fbe73316227bd2b1993c5656adad9f4/tests/playbooks/publish.yml#L8 this explictly tells that it's using docker sdk (I guess they mean docker-py?) from kolla venv
13:07:20 <bbezak> publish.yml is the only place where we're using community.docker collection
13:07:32 <bbezak> and apparently it also needs to be bumped
13:07:44 <bbezak> and publish.yml is being used only in periodics
13:08:04 <bbezak> docker_host_info:
13:08:05 <bbezak> for example
13:08:51 <SvenKieske> okay, unfortunate
13:09:13 <frickler> maybe creating a dedicated venv for the publish job might be a better solution though?
13:09:39 <bbezak> maybe
13:10:00 <bbezak> as ansible version (and default community.docker) is coming from opendev base jobs
13:10:38 <bbezak> I'll look into it
13:10:51 <SvenKieske> great
13:11:18 <bbezak> #topic Release tasks
13:11:34 <bbezak> how we're doing on Kayobe release jovial[m] ?
13:12:19 <jovial[m]> Just trying to merge the last few patches and then I was going to propose a release candidate
13:13:30 <bbezak> ok, let me know if you need to push some changes forward
13:13:37 <jovial[m]> Any further patches people would like to see in the release?
13:13:48 <jovial[m]> bbezak, will do - thanks
13:15:11 <bbezak> I think we should just merge what we can and do a release at this point
13:15:32 <jovial[m]> Right, we are cutting it fine already :)
13:15:40 <bbezak> :)
13:15:51 <bbezak> #topic Regular stable releases (first meeting in a month)
13:16:02 <bbezak> I'd say let's move stable release when we have kayobe as well
13:16:21 <bbezak> as we recently released kolla/k-a
13:17:10 <bbezak> #topic Current cycle planning
13:17:19 <opendevreview> Will Szumski proposed openstack/kayobe master: Add proxy support to Vagrantfile  https://review.opendev.org/c/openstack/kayobe/+/703666
13:17:59 <bbezak> I think we should add Dalmatian release and move the priorities there
13:18:02 <bbezak> I'll look into that
13:18:13 <bbezak> (on the whiteboard)
13:20:42 <bbezak> #topic Additional agenda (from whiteboard)
13:22:10 <bbezak> speaking of federation bug - I somehow don't have this problem in antelope deployment, however I didn't dig into that why is that
13:22:21 <bbezak> https://etherpad.opendev.org/p/KollaWhiteBoard#L72
13:22:34 <bbezak> I added a comment in the change
13:24:24 <SvenKieske> okay, thanks, I just wanted to get some reviews, I didn't really reproduce it locally (no federation available locally on my laptop), but it seemed like an important bugfix :)
13:24:55 <SvenKieske> and as far as I could tell the fixed syntax looks more correct to me, I think I even consulted the haproxy docs for that
13:25:52 <bbezak> yeah it maybe makes sense to update the syntax too
13:26:14 <SvenKieske> maybe some ansible/jinja shenanigans
13:27:19 <bbezak> there are two comments of r-krcek
13:27:28 <bbezak> https://etherpad.opendev.org/p/KollaWhiteBoard#L74
13:28:22 <SvenKieske> I guess those didn't merge because backportvoting was not done when workflow was set to +1
13:28:33 <SvenKieske> I hoped me doing backport +1 would suffice
13:28:42 <frickler> the first one only needed another W+1 after backport flag was set
13:28:45 <SvenKieske> maybe toggle that w+1 switch again
13:29:49 <SvenKieske> that trove thing, does this need a backport? I really like the new switch, honestly.
13:31:38 <frickler> https://review.opendev.org/c/openstack/kolla-ansible/+/920714 is a big change, might need some more discussion. but I only took a quick look just now
13:32:24 <r-krcek> Definitely, I'd be glad for any feedback/discussion .)
13:33:40 <SvenKieske> maybe I can squeeze that in my ever growing list, I'm on the reviewers list already anyway.
13:33:56 <bbezak> I added some more reviewers
13:34:48 <opendevreview> Matúš Jenča proposed openstack/kolla-ansible master: Add certificates for RabbitMQ internode TLS  https://review.opendev.org/c/openstack/kolla-ansible/+/921380
13:35:44 <SvenKieske> the bug description and the commit message don't really seem to well connected and might need an update as well, but I just took a quick look
13:35:58 <SvenKieske> and yes, I read both bug reports.
13:37:05 <SvenKieske> my initial reaction is, this should be split into at least two parts: the refactoring (converting scripts into playbooks) and the bugfixing (removing all datadirs) just going by the commit message and the size of the patch
13:37:27 <SvenKieske> unless there is a very good reason why both must be done at the same time, of course.
13:38:03 <opendevreview> Matúš Jenča proposed openstack/kolla-ansible master: Add certificates for RabbitMQ internode TLS  https://review.opendev.org/c/openstack/kolla-ansible/+/921380
13:38:09 <r-krcek> SvenKieske: Since the list of containers that will be destroyed is taken from the config, the playbooks wont remove any containers that were not deployed directly by kolla-anisble.
13:38:28 <SvenKieske> mhm, is that a bug, or a feature? ;)
13:38:37 <r-krcek> I see, okay, I will split it into multiple. I dont belive there is a direct need to do it at the same time
13:39:12 <r-krcek> Huh, did not think of if was a feature :O
13:39:33 <SvenKieske> I at least know of some deployments who run totally separate playbooks (with distinct inventory files) for kolla and "other stuff". I know that's not good, but that's what people seem to do..
13:39:57 <SvenKieske> so there might very well be containers not deployed by k-a that you surely don't want k-a to delete :)
13:40:10 <jovial[m]> I like the move to use playbooks - finally we will be able to use --tags :)
13:40:14 <SvenKieske> as long as there is an option to turn that feature/bug off, I guess it's fine
13:40:52 <SvenKieske> ftr: I also like the general idea and the idea to "own" all containers on kolla deployed hosts, but I guess the history of deployments needs some escape switches :)
13:41:34 <bbezak> we've also got some ihalomi changes on whiteboard
13:41:35 <SvenKieske> I guess we should write the rest directly into gerrit for preservation of review history :)
13:41:44 <bbezak> https://etherpad.opendev.org/p/KollaWhiteBoard#L78
13:44:25 <ihalomi> the most important is to merge the first patchset https://review.opendev.org/c/openstack/ansible-collection-kolla/+/916258
13:44:50 <ihalomi> so debian tests will pass on the docker worker refactor patchset
13:46:41 <bbezak> ok I'll try to look into that
13:47:34 <bbezak> and some changes of mhiner as well
13:48:05 <mhiner> yes, both ready for review
13:49:01 <bbezak> ok, let's try to review them in following week
13:49:05 <bbezak> commas one is pretty simple :)
13:50:14 <bbezak> #topic Open discussion
13:51:04 <bbezak> anything to discuss?
13:51:15 <frickler> next PTG dates announced: 2024-10-21 - -25 https://openinfra.dev/ptg/
13:51:22 <frickler> just if ppl like to plan early
13:51:35 <bbezak> thx!
13:51:48 <mmalchuk> thanks for info
13:52:06 <SvenKieske> only 4 months to go
13:52:29 <opendevreview> Matúš Jenča proposed openstack/kolla-ansible master: Add support for RabbitMQ internode tls  https://review.opendev.org/c/openstack/kolla-ansible/+/921381
13:52:31 <bbezak> :)
13:52:52 <bbezak> ok let's finish for today then
13:52:53 <bbezak> #endmeeting