opendevreview | Rico Lin proposed openstack/octavia master: fix: specify endpoint info. for neutron client https://review.opendev.org/c/openstack/octavia/+/905794 | 08:17 |
---|---|---|
opendevreview | Gregory Thiemonge proposed openstack/octavia-tempest-plugin master: Add missing cleanups after octavia resources creation https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/861743 | 11:04 |
gthiemonge | #startmeeting Octavia | 16:00 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
opendevmeet | The meeting name has been set to 'octavia' | 16:00 |
gthiemonge | hi | 16:00 |
tweining | o/ | 16:01 |
johnsom | o/ | 16:01 |
gthiemonge | #topic Announcements | 16:02 |
gthiemonge | * EOL/Unmaintained releases | 16:02 |
gthiemonge | FYI we are transitioning Yoga to unmaintained, a final release of octavia for yoga will be released soon (10.1.1) | 16:03 |
gthiemonge | and Train and Ussuri releases are now EOL (End of Life) | 16:03 |
gthiemonge | * 2024.1 Caracal Release Schedule | 16:04 |
gthiemonge | a reminder, the final release for non-client libraries (octavia-lib) is in 2 weeks | 16:05 |
gthiemonge | and Caracal-3 milestone (Feature freeze, final release for client libs) is in 3 weeks! | 16:05 |
gthiemonge | anything to add guys? | 16:06 |
johnsom | Nope, just gasping for air given the short window we have | 16:07 |
gthiemonge | #topic CI Status | 16:08 |
gthiemonge | Now that Yoga is unmaintained for many projects, we need to remove the stable-yoga jobs from octaiva-tempest-plugin gates | 16:09 |
gthiemonge | tkajinam has proposed a change | 16:10 |
gthiemonge | https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/907959 | 16:10 |
gthiemonge | but it's currently blocked because of random failures in the CI | 16:10 |
gthiemonge | it seems that haproxy consumes too much memory during some tests and kills itself | 16:10 |
gthiemonge | we are working on a fix in | 16:10 |
gthiemonge | https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/861743 | 16:10 |
gthiemonge | #topic Brief progress reports / bugs needing review | 16:12 |
gthiemonge | Just a reminder we have 2 patches for important bugs/patches for the neutron auth settings | 16:12 |
gthiemonge | https://review.opendev.org/c/openstack/octavia/+/905794 | 16:12 |
gthiemonge | https://review.opendev.org/c/openstack/octavia/+/907426 | 16:12 |
johnsom | I have posted another patch in the SR-IOV chain enabling nftables on the VIP. | 16:13 |
tweining | https://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 issues | 16:13 |
pyjou | Hello guys sorry I´m late. I respond to you @johnsom about the resize. | 16:14 |
johnsom | This patch is semi-related and could use reviews for merge now: https://review.opendev.org/c/openstack/octavia/+/898863/1 | 16:14 |
johnsom | It removes a bunch of dead code | 16:14 |
johnsom | I posted it in October and it hasn't had any reviews | 16:14 |
johnsom | pyjou Thank you, I think it was mostly about documentation and some guard rails on the resize flavor | 16:16 |
pyjou | I´m agree with you. I think we can add a check before resize that prevents resizing from one topology to another. | 16:18 |
johnsom | Yes, I think that is at least a good short term solution | 16:18 |
pyjou | Perfect, so I'll integrate it into my code | 16:19 |
tweining | because 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 patches | 16:23 |
tweining | https://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 |
gthiemonge | ack | 16:24 |
gthiemonge | #topic Open Discussion | 16:25 |
QG | o/ | 16:27 |
QG | do you want to talk about https://bugs.launchpad.net/octavia/+bug/2052455 ? | 16:27 |
gthiemonge | QG yep | 16:28 |
gthiemonge | my 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 |
gthiemonge | if 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 error | 16:30 |
gthiemonge | like this | 16:30 |
gthiemonge | https://opendev.org/openstack/octavia/src/commit/bbb4713464eee61add6ce32427621e36fa4086e9/octavia/api/v2/controllers/load_balancer.py#L224-L226 | 16:30 |
gthiemonge | IMHO the only thing that is missing is that the multivip feature is not validated when a port is passed to POST /loadbalancer | 16:31 |
QG | Totally agree with this | 16:33 |
gthiemonge | I can work on a fix | 16:34 |
gthiemonge | and I think the multivip feature is poorly (or not) documented, we need to improve it (with use cases, limitations, etc..) | 16:35 |
johnsom | I would agree with that | 16:39 |
gthiemonge | cool | 16:41 |
gthiemonge | QG: thanks for the bug report BTW! | 16:41 |
johnsom | +1 | 16:41 |
gthiemonge | any other topics? | 16:43 |
johnsom | Nothing from me this week | 16:45 |
gthiemonge | ok | 16:46 |
gthiemonge | thanks for the discussion guys | 16:46 |
gthiemonge | bye | 16:46 |
gthiemonge | #endmeeting | 16:46 |
opendevmeet | Meeting ended Wed Feb 7 16:46:24 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:46 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-02-07-16.00.html | 16:46 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-02-07-16.00.txt | 16:46 |
opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-02-07-16.00.log.html | 16:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!