Wednesday, 2024-02-07

opendevreviewRico Lin proposed openstack/octavia master: fix: specify endpoint info. for neutron client  https://review.opendev.org/c/openstack/octavia/+/90579408:17
opendevreviewGregory Thiemonge proposed openstack/octavia-tempest-plugin master: Add missing cleanups after octavia resources creation  https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/86174311:04
gthiemonge#startmeeting Octavia16:00
opendevmeetMeeting started Wed Feb  7 16:00:23 2024 UTC and is due to finish in 60 minutes.  The chair is gthiemonge. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
opendevmeetThe meeting name has been set to 'octavia'16:00
gthiemongehi16:00
tweiningo/16:01
johnsomo/16:01
gthiemonge#topic Announcements16:02
gthiemonge* EOL/Unmaintained releases16:02
gthiemongeFYI we are transitioning Yoga to unmaintained, a final release of octavia for yoga will be released soon (10.1.1)16:03
gthiemongeand Train and Ussuri releases are now EOL (End of Life)16:03
gthiemonge* 2024.1 Caracal Release Schedule16:04
gthiemongea reminder, the final release for non-client libraries (octavia-lib) is in 2 weeks16:05
gthiemongeand Caracal-3 milestone (Feature freeze, final release for client libs) is in 3 weeks!16:05
gthiemongeanything to add guys?16:06
johnsomNope, just gasping for air given the short window we have16:07
gthiemonge#topic CI Status16:08
gthiemongeNow that Yoga is unmaintained for many projects, we need to remove the stable-yoga jobs from octaiva-tempest-plugin gates16:09
gthiemongetkajinam has proposed a change16:10
gthiemongehttps://review.opendev.org/c/openstack/octavia-tempest-plugin/+/90795916:10
gthiemongebut it's currently blocked because of random failures in the CI16:10
gthiemongeit seems that haproxy consumes too much memory during some tests and kills itself16:10
gthiemongewe are working on a fix in16:10
gthiemongehttps://review.opendev.org/c/openstack/octavia-tempest-plugin/+/86174316:10
gthiemonge#topic Brief progress reports / bugs needing review16:12
gthiemongeJust a reminder we have 2 patches for important bugs/patches for the neutron auth settings16:12
gthiemongehttps://review.opendev.org/c/openstack/octavia/+/90579416:12
gthiemongehttps://review.opendev.org/c/openstack/octavia/+/90742616:12
johnsomI have posted another patch in the SR-IOV chain enabling nftables on the VIP.16:13
tweininghttps://review.opendev.org/q/topic:%22add_tls_terminated_api_tests-patch16%22+-status:abandoned I've worked on THTTPS and HSTS tempest test patches and I think I fixed most if not all open issues16:13
pyjouHello guys sorry I´m late. I respond to you @johnsom about the resize.16:14
johnsomThis patch is semi-related and could use reviews for merge now: https://review.opendev.org/c/openstack/octavia/+/898863/116:14
johnsomIt removes a bunch of dead code16:14
johnsomI posted it in October and it hasn't had any reviews16:14
johnsompyjou Thank you, I think it was mostly about documentation and some guard rails on the resize flavor16:16
pyjouI´m agree with you. I think we can add a check before resize that prevents resizing from one topology to another.16:18
johnsomYes, I think that is at least a good short term solution16:18
pyjouPerfect, so I'll integrate it into my code16:19
tweiningbecause I haven't brought that topic up for some time:16:22
tweining https://review.opendev.org/q/topic:%22mypy%22+repo:openstack/octavia please don't forget about the typing/mypy patches16:23
tweininghttps://review.opendev.org/q/topic:%22pyupgrade%22+repo:openstack/octavia ... and also the pyupgrade patches. I've split them into small parts so it's easier to review.16:23
gthiemongeack16:24
gthiemonge#topic Open Discussion16:25
QGo/16:27
QGdo you want to talk about https://bugs.launchpad.net/octavia/+bug/2052455 ?16:27
gthiemongeQG yep16:28
gthiemongemy opinion on it: in case of multivip, the user has to explicitely pass all the subnets they want to plug (with vip_subnet_id or additional_vips[].subnet_id)16:29
gthiemongeif a port is passed with vip_port_id, if it has only one subnet, the subnet is the vip_subnet, if it has multiple subnets and the user didn't specify them with the API, it's a validation error16:30
gthiemongelike this16:30
gthiemongehttps://opendev.org/openstack/octavia/src/commit/bbb4713464eee61add6ce32427621e36fa4086e9/octavia/api/v2/controllers/load_balancer.py#L224-L22616:30
gthiemongeIMHO the only thing that is missing is that the multivip feature is not validated when a port is passed to POST /loadbalancer16:31
QGTotally agree with this 16:33
gthiemongeI can work on a fix16:34
gthiemongeand I think the multivip feature is poorly (or not) documented, we need to improve it (with use cases, limitations, etc..)16:35
johnsomI would agree with that16:39
gthiemongecool16:41
gthiemongeQG: thanks for the bug report BTW!16:41
johnsom+116:41
gthiemongeany other topics?16:43
johnsomNothing from me this week16:45
gthiemongeok16:46
gthiemongethanks for the discussion guys16:46
gthiemongebye16:46
gthiemonge#endmeeting16:46
opendevmeetMeeting ended Wed Feb  7 16:46:24 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:46
opendevmeetMinutes:        https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-02-07-16.00.html16:46
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-02-07-16.00.txt16:46
opendevmeetLog:            https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-02-07-16.00.log.html16:46

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!