opendevreview | Tom Weininger proposed openstack/octavia master: Add spec for rate limiting https://review.opendev.org/c/openstack/octavia/+/923318 | 12:13 |
---|---|---|
gthiemon1e | #startmeeting Octavia | 16:00 |
opendevmeet | Meeting started Wed Aug 21 16:00:18 2024 UTC and is due to finish in 60 minutes. The chair is gthiemon1e. 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 |
gthiemon1e | o/ | 16:00 |
tweining | o/ | 16:00 |
johnsom | o/ | 16:00 |
gthiemon1e | #topic Announcements | 16:01 |
gthiemon1e | * 2024.2 Dalmatian Release Schedule: R-6 | 16:01 |
gthiemon1e | 6 weeks! | 16:01 |
gthiemon1e | this week: it's Final release for non-client libraries | 16:01 |
gthiemon1e | I think we're good with octavia-lib | 16:01 |
gthiemon1e | we published an intermediate release (3.6.0) to unblock johnsom's work | 16:02 |
gthiemon1e | next week is Dalmatian-3 milestone: feature freeze, final release for clients, etc.. | 16:02 |
johnsom | +1 I think octavia-lib is good as well | 16:02 |
gthiemon1e | I think johnsom you need reviews for a few patches, right? | 16:02 |
johnsom | Yes, the SR-IOV patches are in need of review | 16:03 |
johnsom | #link https://review.opendev.org/c/openstack/python-octaviaclient/+/914774 | 16:03 |
johnsom | #link https://review.opendev.org/c/openstack/octavia/+/926565?usp=search | 16:03 |
johnsom | That one fixes the tips gate | 16:03 |
johnsom | Though, hmm, it seems to have an issue. I will look at that this morning | 16:04 |
johnsom | #link https://review.opendev.org/c/openstack/octavia/+/920156?usp=search | 16:05 |
johnsom | #link https://review.opendev.org/c/openstack/octavia/+/922673?usp=search | 16:05 |
gthiemon1e | I think we need https://review.opendev.org/c/openstack/requirements/+/926399 | 16:05 |
johnsom | #link https://review.opendev.org/c/openstack/octavia/+/919974?usp=search | 16:05 |
johnsom | Yeah, probably | 16:05 |
johnsom | #link https://review.opendev.org/c/openstack/octavia/+/919776?usp=search | 16:06 |
johnsom | I think that is the list | 16:06 |
gthiemon1e | lol | 16:06 |
gthiemon1e | yeah | 16:06 |
* johnsom thinks I really should have tagged those | 16:07 | |
gthiemon1e | I'll review them | 16:07 |
johnsom | I will bug the reqs folks | 16:07 |
gthiemon1e | +1 | 16:08 |
gthiemon1e | don't forget to review other possible feature/spec patches folks! | 16:09 |
johnsom | I think I have done at least one pass on all of them now | 16:09 |
gthiemon1e | I don't have any other announcements, do you have anything guys? | 16:09 |
johnsom | PTL/TC nominations are open | 16:09 |
gthiemon1e | +1 | 16:10 |
johnsom | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/6YK5DJRVICRAHVYBUUJLTCMJ4LMHWNMZ/ | 16:11 |
johnsom | The deadline is the 28th | 16:11 |
gthiemon1e | #topic CI Status | 16:13 |
gthiemon1e | on this topic, I think we need to take a look at the oslo.policy related patches | 16:14 |
gthiemon1e | https://review.opendev.org/c/openstack/octavia/+/925532 | 16:14 |
gthiemon1e | I notice that the child patch doesn't pass the CI (failure in the tempest tests) we may have a major issue there | 16:15 |
johnsom | Yeah, it's also likely that when that lands we may have tempest issues. | 16:15 |
gthiemon1e | ok that looks like issues with DIB, so not related, I'm going to recheck it | 16:15 |
gthiemon1e | #topic Brief progress reports / bugs needing review | 16:17 |
tweining | you have mentioned https://review.opendev.org/c/openstack/octavia/+/919776 before. that json is autogenerated somehow I assume? it is huge. | 16:17 |
gthiemon1e | I see that the etcd-jobboard patch has 2 CR+2 but not approved yet, johnsom left a comment there, I'll update the patch and ping you when it's ready | 16:18 |
johnsom | I am in review mode post those oslo policy patches. I'm looking at the circuit breaker patch tom asked me about. | 16:18 |
tweining | thanks | 16:18 |
johnsom | It's not completely auto generated, I have to create the dashboard in grafana and that is a json export of it. | 16:18 |
johnsom | I would not read through it all IMO. If you want to test it or confirm the fix, take that json and import it into grafana | 16:19 |
gthiemon1e | we trust you | 16:22 |
tweining | https://review.opendev.org/c/openstack/octavia/+/923318 rate limiting spec | 16:22 |
tweining | I am happy with the progess so far. The document has changes a lot since the beginning. Thanks for your input. | 16:22 |
gthiemon1e | cool, I'll look at it | 16:22 |
tweining | The way the feature is specified currently makes it very powerful and flexible. I am only a bit worried about the complexity in our Jinja templating that this may introduce. | 16:23 |
tweining | It may be a good idea to define some realistic use cases and sketch out how they would translate into a HAProxy configuration. | 16:23 |
johnsom | Yeah, take a look at the macros we have already. There are similar cases that can provide some examples of how to simplify it | 16:23 |
johnsom | FYI, that DIB issue is related to the UC change for oslo policy 4.4, so it will probably fail again until the UC change lands | 16:26 |
johnsom | https://zuul.opendev.org/t/openstack/build/cf6319a1005341758b66eb5c43a5f9de/log/controller/logs/dib-build/amphora-x64-haproxy.qcow2_log.txt#4792 | 16:26 |
gthiemon1e | hmm ok | 16:27 |
johnsom | DIB isn't honoring the depends-on | 16:27 |
tweining | well, there are much more patches that are waiting for review, but I'd better leave it at that ;) | 16:30 |
gthiemon1e | #topic Open Discussion | 16:30 |
gthiemon1e | anyother topics | 16:30 |
gthiemon1e | ? | 16:30 |
tweining | no, only that I will be out for the next 2 weeks | 16:31 |
gthiemon1e | enjoy! | 16:32 |
johnsom | Yeah, enjoy! | 16:33 |
tweining | thanks. I have no idea what to do, but I will find something to do. ;) | 16:33 |
gthiemon1e | ok I guess that's it! | 16:34 |
gthiemon1e | thank you guys! | 16:34 |
gthiemon1e | #endmeeting | 16:34 |
opendevmeet | Meeting ended Wed Aug 21 16:34:12 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-08-21-16.00.html | 16:34 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-08-21-16.00.txt | 16:34 |
opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-08-21-16.00.log.html | 16:34 |
opendevreview | Merged openstack/python-octaviaclient master: Add SR-IOV support for members https://review.opendev.org/c/openstack/python-octaviaclient/+/914774 | 17:50 |
opendevreview | Brian Haley proposed openstack/octavia master: Don't fail if a provider driver cannot be loaded in Octavia API https://review.opendev.org/c/openstack/octavia/+/780215 | 18:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!