16:00:57 <gthiemonge> #startmeeting Octavia
16:00:57 <opendevmeet> Meeting started Wed Apr 27 16:00:57 2022 UTC and is due to finish in 60 minutes.  The chair is gthiemonge. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:57 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:57 <opendevmeet> The meeting name has been set to 'octavia'
16:01:21 <gthiemonge> Hi
16:01:24 <johnsom> o/
16:01:30 <tweining> Hi
16:01:54 <spencerharmon> Hello!
16:02:09 <gthiemonge> #topic Announcements
16:02:25 <gthiemonge> No announcement today... any announcements on your side?
16:02:57 <tweining> not really
16:03:57 <gthiemonge> ok
16:04:05 <gthiemonge> #topic Octavia CI status
16:04:22 <gthiemonge> The major CI outages are fixed, thank you Folks!
16:04:23 <johnsom> FYI, devstack is dropping centos-8-stream support
16:04:50 <gthiemonge> yeha I wanted to talk about c9s in this section ;-)
16:05:11 <gthiemonge> FYI I've worked on reproducing and analyzing our (not) "random" CI failures from
16:05:17 <johnsom> Ok, sorry for jumping ahead
16:05:19 <gthiemonge> #link https://etherpad.opendev.org/p/octavia-ci-failures
16:05:27 <gthiemonge> NP ;-)
16:05:46 <gthiemonge> I found that 2 of these issues are race conditions when updating a resource in amphorav2
16:05:55 <gthiemonge> I proposed a fix for it:
16:06:00 <gthiemonge> #link https://review.opendev.org/c/openstack/octavia/+/837195
16:06:09 <tweining> I'm currently reviewing it
16:06:15 <gthiemonge> I already got a +2 from johnsom ;-)
16:06:28 <tweining> I have at least one question I think.
16:06:50 <tweining> but you will see ;)
16:06:54 <gthiemonge> hehe
16:07:18 <gthiemonge> basically I fixed the v2 code by reusing the code from v1 ;-)
16:07:30 <gthiemonge> tweining: waiting for your review
16:07:38 <gthiemonge> still about CI:
16:07:48 <gthiemonge> we still have open patches for the CI configuration
16:07:58 <gthiemonge> (adding yoga jobs, removing old jobs)
16:08:03 <gthiemonge> #link https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/838060
16:08:06 <gthiemonge> #link https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/838096
16:08:24 <johnsom> Long term that should not be necessary as the v2 driver should migrate to using the native provider driver data, but this is an interim solution.
16:09:15 <johnsom> Oh, along that line, today Victoria is officially EM
16:09:16 <gthiemonge> yeah I agree, but this is a huge work
16:09:30 <johnsom> Yep, incremental
16:10:34 <gthiemonge> so... after the job cleanup, I think the next task will be to fix/update the centos-9-stream job
16:10:51 <gthiemonge> (c9s is part of the tested runtimes for Zed - and not c8s)
16:10:58 <gthiemonge> #link https://governance.openstack.org/tc/reference/runtimes/zed.html
16:11:37 <gthiemonge> we have 2 pending patches in o-t-p and octavia for c9s, I've rechecked them
16:12:05 <gthiemonge> still waiting to see if everything is alright
16:13:11 <johnsom> Question, here:
16:13:13 <johnsom> #link https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/838096/5/zuul.d/jobs.yaml#809
16:13:23 <johnsom> Why are we keeping spare pool job for wallaby?
16:14:05 <gthiemonge> If I remember correctly this is the only supported(non-EM) release that supports spare pool
16:14:23 <gthiemonge> but yeah, we could also remove it
16:14:30 <johnsom> Oh, it wasn't out of deprecation cycle for Wallaby, ok that makes sense
16:15:02 <gthiemonge> ok
16:16:21 <gthiemonge> #topic Brief progress reports / bugs needing review
16:16:44 <gthiemonge> well my activity was mostly related to the previous section ^^
16:17:00 <spencerharmon> It's the usual topic for me.
16:17:04 <spencerharmon> #link https://review.opendev.org/c/openstack/octavia/+/831051
16:17:21 <spencerharmon> Since last week, I removed all v1 changes, added a bit of documentation and release notes, added an option to disable notifications, and changed methods in rpc.py to use assert syntax.
16:18:33 <johnsom> I am doing reviews as time permits
16:18:34 <spencerharmon> I noticed Tom reviewed since then (thank you!), but let me know if there are any further changes needed.
16:18:41 <tweining> spencerharmon: I tested it and it seems to work well
16:19:07 <spencerharmon> Sure thing. No rush!
16:19:29 <gthiemonge> thanks spencerharmon I'll try to review it before our next meeting ;-)
16:19:59 <tweining> but I needed this patch to get it working: https://review.opendev.org/c/openstack/octavia/+/801860
16:20:12 <tweining> so maybe it makes sense to rebase the patch?
16:21:24 <gthiemonge> the CI merges the (branch of the) patch from gerrit with master, so it should work there
16:22:51 <spencerharmon> I see. Should I rebase or would that cause a conflict for CI?
16:23:24 <gthiemonge> it should not conflict on rebase, because zuul is already able to merge it
16:23:35 <gthiemonge> I have no opinion on that
16:24:02 <spencerharmon> Ok. That's a good call-out, thank you.
16:27:45 <tweining> oh, and I'm done with my scenario test for the new failover circuit breaker feature
16:28:24 <tweining> https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/837878 it would be nice to get some feedback on that and related patches (same topic)
16:28:41 <gthiemonge> I'll test it!
16:31:01 <gthiemonge> #topic Open Discussion
16:31:04 <gthiemonge> any other topics?
16:34:29 <tweining> no, but I commented on the change we discussed at the beginning
16:36:13 <gthiemonge> tweining: thanks
16:36:17 <gthiemonge> ok Folks, thank you!
16:36:24 <gthiemonge> #endmeeting