16:00:30 <gthiemonge> #startmeeting Octavia
16:00:30 <opendevmeet> Meeting started Wed Sep 27 16:00:30 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:30 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:30 <opendevmeet> The meeting name has been set to 'octavia'
16:00:31 <gthiemonge> o/
16:00:35 <tweining> o/
16:00:37 <johnsom> o/
16:01:26 <gthiemonge> #topic Announcements
16:01:35 <gthiemonge> * Project Team Lead Election Conclusion and Results
16:02:02 <gthiemonge> in case you missed the news, I'll be your PTL for the Caracal release cycle
16:02:13 <tweining> yay. congrats!
16:02:43 <gthiemonge> thanks ;-)
16:02:55 <gthiemonge> * 2023.2 Bobcat Release Schedule: R-1
16:03:12 <gthiemonge> this is the week of final RCs
16:03:19 <johnsom> Congratulations!
16:03:28 <gthiemonge> all good here, RC1 will be our Bobcat release (next week)
16:03:33 <gthiemonge> johnsom: thanks
16:04:16 <tweining> what? I thought we'll backport the mypy patch before the release?!
16:04:19 <tweining> just kidding
16:04:45 <gthiemonge> :D
16:04:57 <gthiemonge> * 2024.1 Caracal vPTG
16:05:13 <gthiemonge> I've scheduled the Octavia session
16:05:19 <gthiemonge> October 24th (Tuesday) - 14:00 UTC - 18:00 UTC
16:05:30 <gthiemonge> (note sure if we need 4 hours)
16:08:20 <gthiemonge> Reminder, the etherpad is
16:08:23 <gthiemonge> https://etherpad.opendev.org/p/caracal-ptg-octavia
16:08:26 <gthiemonge> you can add your topics there
16:08:58 <johnsom> gthiemonge FYI, that is not the etherpad linked from the PTG site: https://ptg.opendev.org/etherpads.html
16:09:07 <johnsom> You can update that link using the bot
16:09:09 <gthiemonge> oh, I have to fix that
16:09:12 <tweining> uh, active-active topology would be really nice I guess
16:09:16 <gthiemonge> yup
16:09:23 <johnsom> native BGP
16:09:29 <johnsom> SR-IOV
16:11:19 <tweining> also, don't forget to register (even though it's optional)
16:11:27 <gthiemonge> +1
16:11:40 <gthiemonge> let's skip the CI status section, I have no update
16:11:47 <gthiemonge> #topic Brief progress reports / bugs needing review
16:12:14 <gthiemonge> 896255: Set default value for HSTS options to None | https://review.opendev.org/c/openstack/python-octaviaclient/+/896255
16:12:28 <gthiemonge> could you review ^ guys
16:12:38 <gthiemonge> I've updated it because the release note was msising
16:13:21 <tweining> ok
16:13:25 <gthiemonge> thanks
16:14:01 <gthiemonge> FYI I'm working on a patch that retries (for a very long time) to set the provisioning_status of a LB if the DB is down
16:14:04 <gthiemonge> still wip
16:14:14 <gthiemonge> that should fix
16:14:16 <gthiemonge> https://bugs.launchpad.net/octavia/+bug/2036952
16:20:55 <gthiemonge> #topic Add option to limit recursion in to_data_model
16:20:59 <gthiemonge> special topic for
16:21:04 <gthiemonge> https://review.opendev.org/c/openstack/octavia/+/883063
16:21:21 <gthiemonge> https://bugs.launchpad.net/octavia/+bug/2019311
16:22:18 <gthiemonge> it's about an issue with GET /members requests that take more that 2sec
16:22:44 <gthiemonge> note that I also opened a launchpad for a similar issue in the driver-agent
16:22:48 <gthiemonge> https://bugs.launchpad.net/octavia/+bug/2033280
16:23:30 <gthiemonge> the idea is to limit the recursion of the to_data_model call
16:24:32 <gthiemonge> (for instance to get a pool, octavia may fetch all the children resources but also the parents resources - we observed 19 SQL queries in the driver-agent)
16:24:34 <johnsom> I had a look at this today. I think it looks ok to me. I have issues with the whole "to_data_model" thing in general, but in the members case, this is a reasonable fix IMO
16:24:43 <tweining> so how does it work currently? does that member GET respone only contain the direct child nodes information?
16:25:29 <gthiemonge> tweining: https://docs.openstack.org/api-ref/load-balancer/v2/index.html?expanded=show-member-details-detail,list-members-detail#list-members
16:26:03 <gthiemonge> AFAIK the only recursive call is for the tags, we need to check if the patch doesn't break it
16:26:16 <tweining> okay, so it should be fine then
16:26:37 <gthiemonge> I will test the possible corner cases in this patch
16:28:34 <gthiemonge> ok, anything else on this topic?
16:29:47 <tweining> I didn't test it but the unit test looks convincing, so I'll +2 it
16:29:57 <tweining> apart from that nothing
16:30:31 <gthiemonge> ack
16:30:36 <gthiemonge> #topic Open Discussion
16:31:47 <gthiemonge> any other topics folks?
16:33:50 <gthiemonge> I guess that's a no
16:33:59 <tweining> nothing that needs your attention now ;)
16:34:15 <tweining> I know you're busy with other stuff already
16:34:22 <gthiemonge> ok
16:34:34 <gthiemonge> then that's it for today!
16:34:40 <gthiemonge> thank you guys!
16:34:43 <gthiemonge> #endmeeting