Wednesday, 2024-08-21

opendevreviewTom Weininger proposed openstack/octavia master: Add spec for rate limiting  https://review.opendev.org/c/openstack/octavia/+/92331812:13
gthiemon1e#startmeeting Octavia16:00
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
opendevmeetThe meeting name has been set to 'octavia'16:00
gthiemon1eo/16:00
tweiningo/16:00
johnsomo/16:00
gthiemon1e#topic Announcements16:01
gthiemon1e* 2024.2 Dalmatian Release Schedule: R-616:01
gthiemon1e6 weeks!16:01
gthiemon1ethis week: it's Final release for non-client libraries16:01
gthiemon1eI think we're good with octavia-lib16:01
gthiemon1ewe published an intermediate release (3.6.0) to unblock johnsom's work16:02
gthiemon1enext week is Dalmatian-3 milestone: feature freeze, final release for clients, etc..16:02
johnsom+1 I think octavia-lib is good as well16:02
gthiemon1eI think johnsom you need reviews for a few patches, right?16:02
johnsomYes, the SR-IOV patches are in need of review16:03
johnsom#link https://review.opendev.org/c/openstack/python-octaviaclient/+/91477416:03
johnsom#link https://review.opendev.org/c/openstack/octavia/+/926565?usp=search16:03
johnsomThat one fixes the tips gate16:03
johnsomThough, hmm, it seems to have an issue. I will look at that this morning16:04
johnsom#link https://review.opendev.org/c/openstack/octavia/+/920156?usp=search16:05
johnsom#link https://review.opendev.org/c/openstack/octavia/+/922673?usp=search16:05
gthiemon1eI think we need https://review.opendev.org/c/openstack/requirements/+/92639916:05
johnsom#link https://review.opendev.org/c/openstack/octavia/+/919974?usp=search16:05
johnsomYeah, probably16:05
johnsom#link https://review.opendev.org/c/openstack/octavia/+/919776?usp=search16:06
johnsomI think that is the list16:06
gthiemon1elol16:06
gthiemon1eyeah16:06
* johnsom thinks I really should have tagged those16:07
gthiemon1eI'll review them16:07
johnsomI will bug the reqs folks16:07
gthiemon1e+116:08
gthiemon1edon't forget to review other possible feature/spec patches folks!16:09
johnsomI think I have done at least one pass on all of them now16:09
gthiemon1eI don't have any other announcements, do you have anything guys?16:09
johnsomPTL/TC nominations are open16:09
gthiemon1e+116:10
johnsom#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/6YK5DJRVICRAHVYBUUJLTCMJ4LMHWNMZ/16:11
johnsomThe deadline is the 28th16:11
gthiemon1e#topic CI Status16:13
gthiemon1eon this topic, I think we need to take a look at the oslo.policy related patches16:14
gthiemon1ehttps://review.opendev.org/c/openstack/octavia/+/92553216:14
gthiemon1eI notice that the child patch doesn't pass the CI (failure in the tempest tests) we may have a major issue there16:15
johnsomYeah, it's also likely that when that lands we may have tempest issues.16:15
gthiemon1eok that looks like issues with DIB, so not related, I'm going to recheck it16:15
gthiemon1e#topic Brief progress reports / bugs needing review16:17
tweiningyou have mentioned https://review.opendev.org/c/openstack/octavia/+/919776 before. that json is autogenerated somehow I assume? it is huge.16:17
gthiemon1eI 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 ready16:18
johnsomI am in review mode post those oslo policy patches. I'm looking at the circuit breaker patch tom asked me about.16:18
tweiningthanks16:18
johnsomIt's not completely auto generated, I have to create the dashboard in grafana and that is a json export of it.16:18
johnsomI would not read through it all IMO. If you want to test it or confirm the fix, take that json and import it into grafana16:19
gthiemon1ewe trust you16:22
tweininghttps://review.opendev.org/c/openstack/octavia/+/923318 rate limiting spec16:22
tweiningI am happy with the progess so far. The document has changes a lot since the beginning. Thanks for your input.16:22
gthiemon1ecool, I'll look at it16:22
tweiningThe 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
tweiningIt may be a good idea to define some realistic use cases and sketch out how they would translate into a HAProxy configuration.16:23
johnsomYeah, take a look at the macros we have already. There are similar cases that can provide some examples of how to simplify it16:23
johnsomFYI, that DIB issue is related to the UC change for oslo policy 4.4, so it will probably fail again until the UC change lands16:26
johnsomhttps://zuul.opendev.org/t/openstack/build/cf6319a1005341758b66eb5c43a5f9de/log/controller/logs/dib-build/amphora-x64-haproxy.qcow2_log.txt#479216:26
gthiemon1ehmm ok16:27
johnsomDIB isn't honoring the depends-on16:27
tweiningwell, there are much more patches that are waiting for review, but I'd better leave it at that ;)16:30
gthiemon1e#topic Open Discussion16:30
gthiemon1eanyother topics16:30
gthiemon1e?16:30
tweiningno, only that I will be out for the next 2 weeks16:31
gthiemon1eenjoy!16:32
johnsomYeah, enjoy!16:33
tweiningthanks. I have no idea what to do, but I will find something to do. ;)16:33
gthiemon1eok I guess that's it!16:34
gthiemon1ethank you guys!16:34
gthiemon1e#endmeeting16:34
opendevmeetMeeting ended Wed Aug 21 16:34:12 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:34
opendevmeetMinutes:        https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-08-21-16.00.html16:34
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-08-21-16.00.txt16:34
opendevmeetLog:            https://meetings.opendev.org/meetings/octavia/2024/octavia.2024-08-21-16.00.log.html16:34
opendevreviewMerged openstack/python-octaviaclient master: Add SR-IOV support for members  https://review.opendev.org/c/openstack/python-octaviaclient/+/91477417:50
opendevreviewBrian 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/+/78021518:45

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