| *** gregraka is now known as gregraka-afk | 02:54 | |
| *** ysandeep|out is now known as ysandeep | 04:26 | |
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/xena: Switch centos8 jobs to centos-8-stream https://review.opendev.org/c/openstack/octavia/+/835779 | 06:38 |
|---|---|---|
| opendevreview | Gregory Thiemonge proposed openstack/octavia stable/wallaby: Switch centos8 jobs to centos-8-stream https://review.opendev.org/c/openstack/octavia/+/835780 | 06:38 |
| *** ysandeep is now known as ysandeep|lunch | 07:59 | |
| *** ysandeep|lunch is now known as ysandeep | 08:58 | |
| opendevreview | Tom Weininger proposed openstack/octavia master: Reject invalid whitespace in HM url_path value https://review.opendev.org/c/openstack/octavia/+/834636 | 09:33 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Validate L7Rule value and cookie name https://review.opendev.org/c/openstack/octavia/+/835240 | 09:34 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Reject invalid whitespace in HM url_path value https://review.opendev.org/c/openstack/octavia/+/834636 | 09:42 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Validate L7Rule value and cookie name https://review.opendev.org/c/openstack/octavia/+/835240 | 09:42 |
| *** ysandeep is now known as ysandeep|afk | 10:09 | |
| *** ysandeep|afk is now known as ysandeep | 11:18 | |
| opendevreview | Tom Weininger proposed openstack/octavia master: Failover stop threshold https://review.opendev.org/c/openstack/octavia/+/656811 | 11:18 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Failover stop threshold https://review.opendev.org/c/openstack/octavia/+/656811 | 11:33 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Failover stop threshold https://review.opendev.org/c/openstack/octavia/+/656811 | 12:04 |
| *** ysandeep is now known as ysandeep|afk | 12:20 | |
| *** ysandeep|afk is now known as ysandeep | 12:58 | |
| *** gregraka-afk is now known as gregraka | 13:00 | |
| *** ysandeep is now known as ysandeep|out | 14:37 | |
| opendevreview | Tom Weininger proposed openstack/octavia master: Failover stop threshold https://review.opendev.org/c/openstack/octavia/+/656811 | 15:49 |
| gthiemonge | #startmeeting Octavia | 16:00 |
| opendevmeet | Meeting started Wed Mar 30 16:00:55 2022 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:01 |
| spencerh` | Hello! | 16:01 |
| tweining_ | hi | 16:01 |
| johnsom | o/ | 16:01 |
| gthiemonge | #topic Announcements | 16:02 |
| gthiemonge | * Yoga is released! | 16:02 |
| gthiemonge | #link http://lists.openstack.org/pipermail/openstack-announce/2022-March/002059.html | 16:02 |
| gthiemonge | Congratulations! and Thank you! | 16:03 |
| tweining_ | nice job everyone! | 16:03 |
| spencerh` | Awesome! | 16:03 |
| gthiemonge | * PTG Next week! | 16:04 |
| johnsom | Wahoo | 16:05 |
| gthiemonge | Again, an announcement for the upcoming PTG | 16:05 |
| gthiemonge | I hope I will see you there! | 16:05 |
| gthiemonge | #link https://etherpad.opendev.org/p/zed-ptg-octavia | 16:05 |
| gthiemonge | * Issues with nested virt pools for Centos | 16:08 |
| gthiemonge | I saw a lot of NODE_FAILURE yesterday with our centos-*-stream and *-fips (also centos-based) jobs | 16:08 |
| gthiemonge | it seems that a pool of hosts used for nested virt on centos is no longer available | 16:08 |
| gthiemonge | this morning some jobs passed, but I'll keep an eye on it | 16:09 |
| johnsom | There must be some still around as some jobs are still passing. | 16:09 |
| gthiemonge | yeah, maybe not so many hosts | 16:10 |
| johnsom | Yeah, and more projects are using that pool now | 16:11 |
| gthiemonge | Any other announcements? | 16:13 |
| tweining_ | only a request for feedback | 16:14 |
| tweining_ | I proposed a new amphora state for the failover circuit breaker change | 16:15 |
| johnsom | Octavia is in the release announce and Greg will be presenting during the OpenInfra Live tomorrow | 16:15 |
| johnsom | #link https://www.openstack.org/software/yoga | 16:15 |
| tweining_ | https://review.opendev.org/c/openstack/octavia/+/656811 and https://review.opendev.org/c/openstack/octavia-lib/+/835049 | 16:15 |
| gthiemonge | johnsom: I didn't see it | 16:15 |
| gthiemonge | cool | 16:15 |
| tweining_ | if there are any objections about that change please let me know ASAP | 16:16 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:17 |
| gthiemonge | I'm in favor of adding a new status for this feature | 16:18 |
| gthiemonge | but we need to be really careful with all the uses of the amphora.status variable | 16:18 |
| johnsom | Yeah, state machine changes need to be well thought out | 16:19 |
| gthiemonge | but this is also part of our job as reviewers ;-) | 16:19 |
| johnsom | Yep | 16:19 |
| gthiemonge | for instance | 16:22 |
| gthiemonge | in https://opendev.org/openstack/octavia/src/branch/master/octavia/db/repositories.py#L1507 | 16:22 |
| gthiemonge | what should we do with AMPHORA_FAILOVER_STOPPED amphora? | 16:22 |
| gthiemonge | so yeah, we need to review it carefully, but I don't have any concerns with a new status in the amp | 16:23 |
| tweining_ | I also described in the doc how error recovery could work | 16:24 |
| tweining_ | or does work right now | 16:24 |
| gthiemonge | adding a doc is really great, thanks | 16:25 |
| gthiemonge | FYI I proposed backports to switch our centos-8 jobs for centos-8-stream on stable/xena and stable/wallaby | 16:26 |
| gthiemonge | (centos-8 nodes were removed) | 16:26 |
| gthiemonge | It's not ready yet because I have to backport a fix for c8s in devstack | 16:27 |
| gthiemonge | and I would like to get some reviews on 2 patches: | 16:28 |
| gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/740478 | 16:28 |
| gthiemonge | this is an old patch that improves the validation in the API | 16:28 |
| gthiemonge | and | 16:28 |
| gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/829887 | 16:28 |
| gthiemonge | we were not using the correct dnf repo for haproxy on centos-8-stream | 16:29 |
| gthiemonge | #topic Open Discussion | 16:31 |
| gthiemonge | any other topics? | 16:32 |
| spencerh` | I'm just here in case anyone has any questions about the notifications patch. | 16:32 |
| spencerh` | #link https://review.opendev.org/c/openstack/octavia/+/831051 | 16:32 |
| gthiemonge | I haven't looked at it yet :/ | 16:33 |
| spencerh` | Also, I tried reaching out to the owner of the other notifications patch review via email. Does anyone have contact with them? I'd like to get them involved in the review process if possible so we can settle on a solution. | 16:33 |
| spencerh` | #link https://review.opendev.org/c/openstack/octavia/+/784628 | 16:33 |
| spencerh` | No worries! | 16:33 |
| gthiemonge | nop sorry, I don't have contact with them, I sent some questions in a comment I didn't get any response | 16:35 |
| johnsom | Yeah, the original patch was mnaser, but I don't know if the recent contributor to it is on IRC | 16:35 |
| spencerh` | Rather, I reached out to the uploader. Perhaps I'll try the owner as well. | 16:36 |
| mnaser | spencerh`: they probably will not be responding | 16:36 |
| mnaser | they were working with us but they've moved on and i dont know if i have the bw to look into that patch right now | 16:36 |
| spencerh` | Oh hey! That's good to know. | 16:36 |
| spencerh` | Ok well thanks for speaking up, that answers all of my questions for now. | 16:37 |
| spencerh` | If anyone does get the chance to review, reach out any time! :) | 16:37 |
| gthiemonge | spencerh`: ok! thanks! | 16:37 |
| johnsom | I think this topic is on the PTG agenda. I plan to re-read through both of these before that topic and raise any comments | 16:37 |
| spencerh` | Oh cool. I didn't notice that. Can I attend? I've never joined one before. | 16:39 |
| gthiemonge | spencerh`: https://etherpad.opendev.org/p/zed-ptg-octavia | 16:40 |
| johnsom | absolutely, it is free and virtual | 16:40 |
| johnsom | #link https://www.openstack.org/ptg/ | 16:40 |
| johnsom | An the above etherpad link | 16:40 |
| spencerh` | Yeah. I see. I just join the diablo meeting room at the scheduled time? | 16:40 |
| gthiemonge | I haven't prepared an agenda yet, but if you're available on April 7th, we can discuss the notification patch | 16:41 |
| johnsom | Check the etherpad for the meeting link, they usually get generated the week of. | 16:41 |
| spencerh` | Ok great! I'll be available for sure. If I have any trouble getting in, I'll look for help in here! :D | 16:42 |
| johnsom | +1 | 16:43 |
| gthiemonge | cool! | 16:43 |
| gthiemonge | is that all for today? | 16:44 |
| spencerh` | That's it from me. Thanks for the info, everyone! | 16:45 |
| gthiemonge | Ok! Thanks everyone! | 16:47 |
| gthiemonge | #endmeeting | 16:47 |
| opendevmeet | Meeting ended Wed Mar 30 16:47:04 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:47 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-03-30-16.00.html | 16:47 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-03-30-16.00.txt | 16:47 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-03-30-16.00.log.html | 16:47 |
| rm_work | oh, still working on landing Failover Threshold eh? | 16:57 |
| rm_work | didn't I whip out most of that like... during a summit like 3 years ago? XD | 16:58 |
| tweining_ | yes, we still have a customer request for it. | 16:59 |
| johnsom | One of many summit started patches still in WIP. grin | 17:00 |
| gthiemonge | our goal for Zed is to complete rm_work's patches :D | 17:13 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Failover stop threshold https://review.opendev.org/c/openstack/octavia/+/656811 | 17:21 |
| rm_work | lol | 17:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!