| opendevreview | Takashi Kajinami proposed openstack/octavia master: dib: Remove remaining logic for CentOS/RHEL 8 https://review.opendev.org/c/openstack/octavia/+/912700 | 03:14 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/octavia master: dib: Remove remaining logic for CentOS/RHEL 8 https://review.opendev.org/c/openstack/octavia/+/912700 | 03:14 |
| opendevreview | Takashi Kajinami proposed openstack/octavia master: dib: Remove Ubuntu Forcal support https://review.opendev.org/c/openstack/octavia/+/912701 | 03:17 |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Fix fully-populated API with allowed_cidrs https://review.opendev.org/c/openstack/octavia/+/912721 | 08:53 |
| gthiemonge | #startmeeting Octavia | 16:00 |
| opendevmeet | Meeting started Wed Mar 13 16:00:33 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 |
| tweining | o/ | 16:00 |
| gthiemonge | hi | 16:00 |
| johnsom | o/ | 16:00 |
| oschwart | o/ | 16:01 |
| gthiemonge | #topic Announcements | 16:02 |
| gthiemonge | * 2024.1 Caracal Release Schedule: RC1 | 16:02 |
| gthiemonge | This is the RC1 milestone this week | 16:02 |
| gthiemonge | we don't expect to have another RC unless we find a critical bug | 16:03 |
| gthiemonge | so Octavia RC1 should be the version for Caracal | 16:03 |
| gthiemonge | right now we are only waiting for https://review.opendev.org/c/openstack/octavia/+/912599 to pass the CI (patch already approved) | 16:03 |
| gthiemonge | then I'll update the hash in the release patch | 16:03 |
| johnsom | Sounds good. Did your reload patch land? I saw that was holding up the above bug | 16:04 |
| gthiemonge | nop | 16:04 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia/+/909849 | 16:04 |
| johnsom | Ok, I will be looking at that | 16:05 |
| gthiemonge | thanks | 16:05 |
| gthiemonge | * Dalmatian PTG | 16:06 |
| gthiemonge | reminder: the PTG is next month | 16:06 |
| gthiemonge | I created an etherpad | 16:06 |
| gthiemonge | https://etherpad.opendev.org/p/apr2024-ptg-octavia | 16:07 |
| gthiemonge | you can add your topics there | 16:07 |
| gthiemonge | I haven't yet scheduled our session, IIRC the previous one was on Tuesday at 1400 UTC, maybe I'll book the same timeslot | 16:07 |
| gthiemonge | WDYT? | 16:08 |
| oschwart | sounds good to me | 16:08 |
| johnsom | Yeah, that works for me | 16:08 |
| tweining | for me too | 16:09 |
| gthiemonge | cool | 16:09 |
| gthiemonge | that's all I have for the announcements, did I miss something? | 16:10 |
| tweining | isn't now ptl election period? | 16:12 |
| johnsom | It is over: | 16:12 |
| johnsom | #link https://governance.openstack.org/election/ | 16:12 |
| johnsom | Congrats to gthiemonge! | 16:12 |
| tweining | ah, thanks. congrats btw | 16:13 |
| gthiemonge | \o/ | 16:13 |
| oschwart | congratulations gthiemonge! | 16:13 |
| johnsom | There were no contested projects, so the PTL election is over | 16:13 |
| gthiemonge | maybe in 6 months | 16:15 |
| johnsom | We could also mention there is going to be an OpenStack Summit Asia in South Korea in September. | 16:15 |
| johnsom | In case anyone is interested, the call for presentations is open | 16:16 |
| johnsom | #link https://openinfra.dev/summit/ | 16:16 |
| gthiemonge | OpenInfra Days in Europe too | 16:16 |
| gthiemonge | #link https://oideurope2024.openinfra.dev/#registration=1 | 16:16 |
| gthiemonge | #topic CI Status | 16:19 |
| gthiemonge | so, on the CI, there are still some issue with haproxy, but it looks a bit better than the past week | 16:19 |
| gthiemonge | as mentioned earlier, https://review.opendev.org/c/openstack/octavia/+/909849 is a workaround for this issue | 16:19 |
| johnsom | Yeah, it is still crashing for some reason | 16:19 |
| gthiemonge | if I can find some time, I'll investigate but it looks like an ubuntu-only bug | 16:20 |
| johnsom | Yeah, I am pretty convinced it is a bug in the ubuntu version of haproxy | 16:21 |
| gthiemonge | +1 | 16:23 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:25 |
| gthiemonge | I found and fixed a bug with the fully-populated LB API when using allowed_cidrs in the listener | 16:26 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia/+/912721 | 16:26 |
| johnsom | I fixed one open issue with the SR-IOV patch that gthiemonge noticed, where the failover flow was not honoring the Amphora api timeouts, so failover could be longer than desired. That is in the testing gates now | 16:26 |
| gthiemonge | https://bugs.launchpad.net/octavia/+bug/2057751 | 16:26 |
| johnsom | Otherwise, I am starting work on SR-IOV for members next | 16:27 |
| oschwart | I have this very WIP fix from the past: | 16:29 |
| oschwart | https://review.opendev.org/c/openstack/octavia/+/887593 | 16:29 |
| oschwart | I wanted to consult with you guys about the idea behind it | 16:30 |
| gthiemonge | yeah I looked at it | 16:30 |
| oschwart | (assuming that a little more effort will make the implementation much better) | 16:30 |
| gthiemonge | I think we need to split the _graph_create function of each controller into 2 functions: _graph_validate & _graph_creates | 16:30 |
| gthiemonge | and _graph_validate would be called before the creation of the port in neutron | 16:31 |
| oschwart | sounds good to me. (or any way of validating the parameters of the LB before starting to create resources) | 16:32 |
| oschwart | s/LB/listeners+pools | 16:32 |
| gthiemonge | we need to check that it's possible with the validation functions, it depednds on the type of the params they receive | 16:32 |
| johnsom | I wonder if that is related to: https://review.opendev.org/c/openstack/octavia/+/910449 | 16:32 |
| gthiemonge | no the issue happens in the API only, the VIP port is created before the subresources are validated | 16:33 |
| johnsom | Does the LB go to provisioning state of ERROR? | 16:34 |
| gthiemonge | so if the validation of a port/listener/etc... fails, an exception is raised, the API returns an error, but the port is still there | 16:34 |
| gthiemonge | nop, no LB in the DB, only a dangling port | 16:34 |
| johnsom | Ah, so it doesn't go into the flow yet. Got it | 16:34 |
| gthiemonge | yeah only in the API | 16:35 |
| gthiemonge | another solution would be to delete the port when an exception is raised, but there's no direct access to the neutron API, we need to update and pass through the provider driver, so it would be an unbackportable change | 16:35 |
| johnsom | Yeah, that is a bit nasty as it could lock up a requested fixed IP. | 16:36 |
| oschwart | I prefer calling the validation functions, we already have them implemented, at least almost fully | 16:37 |
| gthiemonge | +1 | 16:38 |
| johnsom | We should think about how to cleanup as well, i.e. if the DB commit fails due to the DB going down, would we end up in the same situation. | 16:40 |
| gthiemonge | right | 16:41 |
| tweining | I think we didn't discuss our policy about backporting to the new unmaintained/* branches in the future. Will we stop backporting fixes to those branches upstream? | 16:42 |
| tweining | (I just remember we did not discuss that yet, I think) | 16:42 |
| gthiemonge | maybe a good topic for the PTG, I'm a bit annoyed because all the backports I created for stable/* were abandonned when the branches were renamed | 16:43 |
| tweining | ok. i'll add it to the epad | 16:43 |
| johnsom | Yeah, that is a good PTG topic | 16:44 |
| gthiemonge | #topic Open Discussion | 16:45 |
| gthiemonge | I think we're already there | 16:45 |
| tweining | nothing else from me | 16:45 |
| johnsom | Yeah, I don't think I have anything else | 16:46 |
| oschwart | me neither | 16:46 |
| gthiemonge | okay | 16:47 |
| gthiemonge | thank you guys! good discussions! | 16:47 |
| gthiemonge | #endmeeting | 16:47 |
| opendevmeet | Meeting ended Wed Mar 13 16:47:17 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:47 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-03-13-16.00.html | 16:47 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-03-13-16.00.txt | 16:47 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-03-13-16.00.log.html | 16:47 |
| opendevreview | Pierre-Yves Jourel proposed openstack/octavia master: Add specs to resize a load balancer https://review.opendev.org/c/openstack/octavia/+/885490 | 17:21 |
| opendevreview | Pierre-Yves Jourel proposed openstack/octavia master: Add specs to resize a load balancer https://review.opendev.org/c/openstack/octavia/+/885490 | 17:41 |
| opendevreview | Merged openstack/octavia master: Check Amphora status on SR-IOV failover flows https://review.opendev.org/c/openstack/octavia/+/912599 | 18:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!