| opendevreview | Takashi Kajinami proposed openstack/octavia-lib master: Add pre-commit config https://review.opendev.org/c/openstack/octavia-lib/+/882127 | 04:24 |
|---|---|---|
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: DNM: Testing memory https://review.opendev.org/c/openstack/octavia/+/909227 | 08:43 |
| gthiemonge | #startmeeting Octavia | 16:00 |
| opendevmeet | Meeting started Wed Feb 21 16:00:12 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 | hello | 16:00 |
| johnsom | o/ | 16:00 |
| oschwart | o/ | 16:00 |
| tweining | o/ | 16:00 |
| gthiemonge | #topic Announcements | 16:01 |
| johnsom | The final caracal release of octavia-lib is due this week. | 16:01 |
| gthiemonge | * 2024.1 Caracal Release Schedule: Caracal-3 milestone | 16:01 |
| gthiemonge | right | 16:01 |
| gthiemonge | I haven't yet approved the release | 16:02 |
| tweining | can we get https://review.opendev.org/c/openstack/octavia-lib/+/882127 in before? | 16:02 |
| tweining | Takashi updated it today | 16:02 |
| johnsom | That is for dev, so not really critical to get into the release | 16:03 |
| tweining | right | 16:03 |
| johnsom | What about this repr patch? | 16:03 |
| johnsom | #link https://review.opendev.org/c/openstack/octavia-lib/+/881701/ | 16:04 |
| gthiemonge | let's target that for the next release | 16:04 |
| gthiemonge | it's not a big requirement | 16:04 |
| johnsom | Is that important for debugging that we should try to slip it in? or again mostly dev focus? | 16:04 |
| gthiemonge | and it's backportable | 16:05 |
| tweining | well, I think nobody except me really misses that either | 16:05 |
| gthiemonge | I already gave a +2, I can update the hash of the release if we merge it today or tomorrow | 16:06 |
| johnsom | I will fully admit that I have not looked at lib in a while. | 16:06 |
| johnsom | Ok, I will take a look at repr after the meeting | 16:06 |
| gthiemonge | so yeah octavia-lib this week | 16:07 |
| gthiemonge | and next week is the caracal-3 milestone | 16:07 |
| gthiemonge | feature freeze & final release for client libraries | 16:07 |
| johnsom | Which means feature freeze. | 16:07 |
| gthiemonge | right | 16:07 |
| johnsom | As you know SR-IOV support for the VIPs is a major feature we are trying to get into caracal. | 16:07 |
| johnsom | Sadly, due to a bunch of other issues, it's coming in close to the wire. | 16:08 |
| gthiemonge | right | 16:08 |
| gthiemonge | there's an attempt to fix those issues | 16:08 |
| johnsom | If you can please have a look at the first two patches and give initial feedback (thank you gthiemonge for already doing this) I would appreciate it. | 16:08 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/861743 | 16:08 |
| gthiemonge | but the code doesn't look right | 16:09 |
| gthiemonge | I'll update it tomorrow | 16:09 |
| gthiemonge | yeah if we fix the issues with the gates, I think SR-IOV could be merged soon | 16:09 |
| tweining | does that SRIOV stuff need a special setup for testing? | 16:09 |
| johnsom | I hope to have an initial patch posted for the last functional part today. I am working through bugs | 16:09 |
| johnsom | Yes, you have to have a NIC that supports SR-IOV until April when the next LTS will have a qemu driver we can use for testing. | 16:10 |
| johnsom | So, at this point it's code review and unit/functional tests | 16:11 |
| gthiemonge | and check that it doesn't break non-SRIOV stuff :D | 16:11 |
| johnsom | +1 to that | 16:11 |
| johnsom | Everything should be behind a flavor profile flag | 16:13 |
| gthiemonge | I have another announcement | 16:15 |
| gthiemonge | * PTL 2024.2 Election Season | 16:15 |
| gthiemonge | it's already there | 16:15 |
| gthiemonge | PTL election for the next cycle | 16:15 |
| johnsom | Ah yes, nominations are open | 16:15 |
| gthiemonge | and that's it | 16:17 |
| gthiemonge | any other announcements? | 16:17 |
| johnsom | That is all I have | 16:17 |
| tweining | nothing from me | 16:17 |
| gthiemonge | #topic CI Status | 16:18 |
| gthiemonge | we already talked about that | 16:18 |
| gthiemonge | there are some issues with haproxy in the amphora in the CI | 16:18 |
| gthiemonge | I'll spend some time on it tomorrow, on the octavia-tempest-plugin patch | 16:18 |
| gthiemonge | we need to fix it for next week | 16:18 |
| johnsom | I saw you ran the DNM patch last night, any insights? | 16:18 |
| gthiemonge | I wanted to catch the moment when haproxy recreates a worker, but it's difficult to synchronize /bin/free and haproxy :D | 16:19 |
| johnsom | Yeah | 16:19 |
| gthiemonge | I may also try a DNM patch with a smaller value for haproxy maxconn, just to check if it's really a mem issue | 16:20 |
| gthiemonge | we could also change the connection-limit in the test classes that are imapcted | 16:21 |
| gthiemonge | I'll keep you updated | 16:21 |
| johnsom | +1 | 16:21 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:22 |
| johnsom | I am making good progress on wrapping up the SR-IOV work. I have rules being pushed down to the amphora now on LB create (for VRRP protocol for example). | 16:23 |
| gthiemonge | the patches for the bugs with the neutron endpoint settings are still open: | 16:23 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia/+/905794 | 16:23 |
| gthiemonge | https://review.opendev.org/c/openstack/octavia/+/907426 | 16:23 |
| gthiemonge | I need to re-review the first one | 16:23 |
| johnsom | I just need to finish debugging it, put in the tasks in the other flows to also update the rules as needed and finish up test coverage. | 16:24 |
| johnsom | #link https://review.opendev.org/q/project:openstack/octavia+topic:%22sriov%22 | 16:24 |
| johnsom | Those are the two that have already been posted. The -1 zuul vote is the potential memory issue we talked about above. | 16:25 |
| tweining | I had a look at the 905794 one today, but I didn't fully understand it | 16:25 |
| gthiemonge | johnsom: question, do we need a python-octaviaclient patch? | 16:25 |
| gthiemonge | for the type of the vnic? | 16:26 |
| johnsom | No, when I pivoted I no longer need lib/client/sdk changes. It's all keyed off of the flavor profile now | 16:26 |
| gthiemonge | or maybe it's not mandatory | 16:26 |
| gthiemonge | ack | 16:26 |
| johnsom | Oh, for the field I added to the LB VIP. Hmm, | 16:26 |
| johnsom | The client just ignores it, but yeah, I guess we should update that so it shows up | 16:27 |
| gthiemonge | ok | 16:27 |
| johnsom | I can probably get that hammered out today | 16:27 |
| johnsom | I keep forgetting I added that field | 16:28 |
| gthiemonge | #topic Open Discussion | 16:32 |
| gthiemonge | any other topics before we close the meeting? | 16:33 |
| tweining | no | 16:33 |
| johnsom | I can't think of anything. | 16:33 |
| oschwart | nothing from me | 16:34 |
| gthiemonge | ok! | 16:34 |
| gthiemonge | thank you guys | 16:34 |
| gthiemonge | have a good week | 16:34 |
| gthiemonge | #endmeeting | 16:34 |
| opendevmeet | Meeting ended Wed Feb 21 16:34:24 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:34 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-02-21-16.00.html | 16:34 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-02-21-16.00.txt | 16:34 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-02-21-16.00.log.html | 16:34 |
| opendevreview | Merged openstack/octavia-lib master: Better __repr__() for data model https://review.opendev.org/c/openstack/octavia-lib/+/881701 | 18:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!