16:00:17 <gthiemonge> #startmeeting Octavia
16:00:17 <opendevmeet> Meeting started Wed Aug 30 16:00:17 2023 UTC and is due to finish in 60 minutes.  The chair is gthiemonge. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:17 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:17 <opendevmeet> The meeting name has been set to 'octavia'
16:00:20 <gthiemonge> o/
16:00:24 <oschwart> o/
16:00:27 <tweining> o/
16:00:28 <johnsom> o/
16:01:06 <gthiemonge> #topic Announcements
16:01:18 <gthiemonge> * 2023.2 Bobcat Release Schedule: R-5
16:01:47 <gthiemonge> we are close to feature freeze and final release for client libraries
16:02:03 <gthiemonge> the release patch for python-octaviaclient was merged today
16:02:45 <gthiemonge> we have already merged a lot of bugfixes for Bobcat
16:02:49 <tweining> sounds good
16:03:03 <gthiemonge> https://etherpad.opendev.org/p/octavia-priority-reviews
16:03:34 <gthiemonge> I think I'll test again the new changes, check the CI, etc...
16:03:48 <gthiemonge> to ensure we didn't break anything ;-)
16:06:03 <gthiemonge> do you have any other announcements folks?
16:06:29 <tweining> that tempest test for HSTS doesn't have a deadline, right?
16:06:57 <gthiemonge> I think it's RC1
16:07:09 <johnsom> Yeah, we should get that in sooner than later
16:07:35 <gthiemonge> but we could have it in another milestone release if it doesn't make it
16:07:43 <tweining> okay, that might be a topic for later in the meeting then
16:08:07 <gthiemonge> hmm last thing for Bobcat:
16:08:30 <gthiemonge> the failover of IPv6 LBs is currently broken (neutron bug)
16:08:37 <gthiemonge> https://bugs.launchpad.net/octavia/+bug/2028524
16:08:40 <gthiemonge> https://bugs.launchpad.net/neutron/+bug/2028651
16:09:12 <gthiemonge> there's a patch proposed to neutron, we need to track it
16:09:19 <gthiemonge> https://review.opendev.org/c/openstack/neutron/+/892564
16:10:38 <gthiemonge> #topic CI Status
16:10:53 <gthiemonge> centos jobs are ok now
16:11:00 <gthiemonge> nothing else to report here
16:11:33 <gthiemonge> #topic Brief progress reports / bugs needing review
16:11:34 <johnsom> That was the global_venv thing right?
16:11:57 <gthiemonge> johnsom: yes, one change in devstack that doesn't work well in c9s
16:12:07 <oschwart> HSTS tempest tests are rebased on top of the TLS_TERMINATED listener API tests patch, it seems like some u/s jobs fail against that patch because they require the Barbican service as well
16:12:10 <oschwart> https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/893066
16:12:24 <gthiemonge> johnsom: devstack is disabling it for c9s, but as we don't use directly their job definition, we had to apply the same fix
16:12:59 <johnsom> Right, I was just trying to keep track of the issues with C9s
16:13:00 <gthiemonge> oschwart: interesting I think the api job should use a noop driver instead of barbican
16:13:39 <oschwart> yes they do, these are the u/s jobs that fail
16:13:41 <johnsom> Yeah, there is a fake driver for the certificates stuff.
16:14:19 <oschwart> so if we want to add API test that will test the TERMINATED_HTTPS protocol, I guess we will have to add barbican service (?)
16:14:46 <gthiemonge> oschwart: I will take a look at those CI results
16:15:19 <oschwart> johnsom: so I should have used that fake driver for the certificates instead?
16:15:25 <oschwart> gthiemonge: thanks
16:15:44 <johnsom> Yeah, the no-op jobs should probably use the no-op/local certificate driver instead of barbican
16:15:48 <tweining> I will try to finish my hsts api tempest test on top before I leave for vacation
16:16:25 <johnsom> #link https://github.com/openstack/octavia/blob/master/octavia/certificates/manager/local.py
16:16:36 <johnsom> I'm not 100% what shape that is in.
16:17:19 <johnsom> #link https://github.com/openstack/octavia/blob/master/setup.cfg#L92
16:17:31 <gthiemonge> maybe that doesn't work correctly because we haven't yet tested TLS_TERMINATED in the API jobs
16:17:54 <johnsom> Yeah, I don't know what state Adam left that in
16:18:36 <johnsom> #link https://github.com/openstack/octavia-tempest-plugin/blob/master/zuul.d/jobs.yaml#L489
16:18:40 <johnsom> In theory it's in use
16:19:32 <gthiemonge> oschwart: we can check that together tomorrow
16:19:39 <rm_work> I know I haven’t touched it in a LONG time, there may have been changes to how the manager interface works
16:19:39 <rm_work> It was mainly intended only for local testing
16:20:51 <rm_work> Ah that looks like what you want it for, so yeah it was working at least last time I checked 😅
16:21:07 <gthiemonge> rm_work: hi
16:21:26 <gthiemonge> rm_work: thanks, we will verify it
16:22:45 <oschwart> thanks rm_work gthiemonge johnsom
16:23:30 <oschwart> gthiemonge: sure, let's take a look at it tomorrow
16:24:35 <gthiemonge> #topic Open Discussion
16:25:34 <oschwart> how should we continue with
16:25:35 <oschwart> https://review.opendev.org/c/openstack/octavia/+/890814
16:27:08 <johnsom> Yeah, so I highlighted that this patch is an API behavior change. Where previously they would get a 404, now they will get a dict no mater what...
16:28:27 <gthiemonge> as a user, I would find it weird that the API returns a 404 when requesting the stats of an existing amp
16:29:09 <gthiemonge> but yeah this is what the API describes
16:29:10 <johnsom> Well, it may not be an existing amp
16:29:38 <gthiemonge> in case of non existing amp, line 220 would raise an exception, right?
16:29:45 <gthiemonge> maybe we need to make it explicitly
16:29:54 <johnsom> You could also consider the 404 to be "no stats exist"
16:30:44 <gthiemonge> johnsom: yeah the api-ref describes it like that
16:32:39 <gthiemonge> so if we want to fix this for the noop driver, we need to patch the noop drivers
16:33:09 <gthiemonge> (like creating dummy entries in the stats table)
16:33:10 <johnsom> The problem with this case is the drivers are never called, the request is handled purely at the API tier
16:33:43 <johnsom> Yeah, I guess if the amp is created somewhere, it could insert a record in the sqlite
16:35:19 <gthiemonge> yes
16:36:04 <oschwart> sounds like another topic for tomorrow gthiemonge :)
16:36:25 <gthiemonge> +1
16:37:53 <oschwart> thanks folks, nothing else from me
16:37:58 <opendevreview> Tom Weininger proposed openstack/octavia-tempest-plugin master: Test new HSTS feature  https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/881546
16:38:09 <gthiemonge> ok anything else folks?
16:38:15 <johnsom> Not from me
16:38:38 <tweining> no. the update I just pushed looks good, but I couldn't test it yet
16:38:47 <gthiemonge> tweining: thanks
16:39:03 <gthiemonge> tweining: enjoy your vacation
16:39:10 <gthiemonge> thank you guys!
16:39:13 <tweining> thanks
16:39:16 <gthiemonge> #endmeeting