16:00:20 <gthiemonge> #startmeeting Octavia
16:00:20 <opendevmeet> Meeting started Wed Aug 11 16:00:20 2021 UTC and is due to finish in 60 minutes.  The chair is gthiemonge. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:20 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:20 <opendevmeet> The meeting name has been set to 'octavia'
16:00:29 <gthiemonge> Hi!
16:00:37 <johnsom> o/
16:02:49 <gthiemonge> #topic Announcements
16:02:55 <gthiemonge> Gates are/were broken after a pip-check-reqs update
16:02:59 <gthiemonge> master is now fixed
16:03:10 <gthiemonge> but wallaby and victoria are still affect
16:03:18 <johnsom> I just reviewed some of the stable branch patches this morning
16:03:20 <gthiemonge> there's a fix to cap pip and pip-check-reqs:
16:03:27 <gthiemonge> #link https://review.opendev.org/q/I6ae2740557f0e62b6805e5dadd025e7bf44ec57d
16:04:12 <gthiemonge> perhaps another core reviewer can take a look
16:04:20 <gthiemonge> cgoncalves: rm_work: ^
16:05:52 <gthiemonge> Next week is the deadline for the final release for non-client libraries for Xena
16:06:16 <gthiemonge> johnsom: i guess this concerns octavia-lib, right?
16:06:24 <johnsom> Correct
16:07:13 <gthiemonge> ok, there are 3 reviews that are waiting for a CR+2, I'll review them
16:07:49 <gthiemonge> Any other announcements?
16:08:08 <johnsom> There was an e-mail about elections coming up
16:08:35 <rm_work> Kk
16:08:36 <gthiemonge> elections?? oh
16:08:38 <johnsom> #link http://lists.openstack.org/pipermail/openstack-discuss/2021-August/024093.html
16:10:10 <gthiemonge> thanks johnsom
16:11:13 <gthiemonge> #topic Brief progress reports / bugs needing review
16:11:32 <gthiemonge> I was on PTO but I'd like to highlight some reviews that are still open:
16:11:38 <gthiemonge> Add generic network interface management in the amphora
16:11:42 <gthiemonge> #link https://review.opendev.org/c/openstack/octavia/+/761195
16:12:09 <gthiemonge> Switch centos-8 jobs to centos-8-stream
16:12:13 <gthiemonge> #link https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/698450
16:12:36 <gthiemonge> ^ it fixes centos-8 that has been broken for ... years?
16:12:43 <gthiemonge> Alias change amphorav2 -> amphora
16:12:48 <gthiemonge> #link https://review.opendev.org/c/openstack/octavia/+/740432
16:13:08 <gthiemonge> this one is important, we discussed it 3 weeks ago
16:13:34 <gthiemonge> we agreed that amphorav2 is still a goal for octavia in xena
16:14:15 <johnsom> Yes, this needs to happen.
16:14:24 <johnsom> Looks like I lost track of that one
16:14:51 <gthiemonge> this is amphorav2 _without_ persistence
16:19:09 <gthiemonge> #topic Open Discussion
16:19:35 <gthiemonge> tkajinam has been working on the octavia.conf generation (instead of maintaining it manually)
16:19:44 <gthiemonge> #link https://review.opendev.org/c/openstack/octavia/+/804175
16:20:30 <johnsom> Yeah, I have concerns about that. We didn't do this on purpose as the generated conf file was inaccurate and didn't allow for better comments, etc.
16:21:09 <gthiemonge> I'm not against this change if we manage to not lose information with this commit
16:22:48 <gthiemonge> so I think we need to carefully review the patch to find if anything is missing
16:23:06 <gthiemonge> Michael already pointed an error in a generated field
16:24:37 <gthiemonge> any other opinions?
16:24:46 <johnsom> Yeah, I have not yet gone through it line-by-line, but that will be what is required.
16:26:30 <gthiemonge> ok
16:27:32 <gthiemonge> of course I can help if we need to fix/extend some strings
16:28:55 <johnsom> It's changes like this that... concern me
16:28:56 <johnsom> https://review.opendev.org/c/openstack/octavia/+/804175/3/octavia/common/config.py#531
16:30:04 <johnsom> It will just take time to walk through.
16:33:29 <gthiemonge> ack
16:33:36 <gthiemonge> Any other topics today?
16:35:24 <gthiemonge> Ok guys! Thanks everyone!
16:35:31 <gthiemonge> #endmeeting