| opendevreview | Omer Schwartz proposed openstack/octavia master: Fix HealthMonitorToErrorOnRevertTask revert method https://review.opendev.org/c/openstack/octavia/+/847977 | 09:53 |
|---|---|---|
| *** beagles is now known as beagles|afk | 12:17 | |
| *** beagles|afk is now known as beagles | 12:17 | |
| *** beagles is now known as beagles_afk | 12:17 | |
| opendevreview | Tom Weininger proposed openstack/octavia master: Proposal for Amphora vertical scaling https://review.opendev.org/c/openstack/octavia/+/848105 | 13:07 |
| tweining | gthiemonge, johnsom: ^ I started this to organize my thoughts and discuss ideas | 13:09 |
| gthiemonge | tweining: great! | 13:15 |
| johnsom | tweining Cool. Also note, I added some thoughts to the BZ for this | 13:53 |
| tweining | thanks. I'll have a look. | 13:53 |
| johnsom | tweining Opps, I meant Jira. Greg asked me to add them to the Jira ticket | 13:54 |
| gthiemonge | #startmeeting Octavia | 16:00 |
| opendevmeet | Meeting started Wed Jun 29 16:00:31 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 Folks! | 16:00 |
| tweining | o/ | 16:00 |
| johnsom | o/ | 16:01 |
| fkr | o/ | 16:01 |
| matfechner | o/ | 16:02 |
| gthiemonge | #topic Announcements | 16:02 |
| gthiemonge | a quick announcement | 16:02 |
| gthiemonge | I'm proposing to add tweining to the Octavia core reviewers | 16:02 |
| gthiemonge | see "[Octavia] Proposing Tom Weininger as core reviewer" on openstack-discuss | 16:02 |
| gthiemonge | if no one raises any concerns about it, I'm going to add him next Tuesday | 16:03 |
| johnsom | Yes! congratulations. I think we have a core quorum on the email chain | 16:03 |
| gthiemonge | then we will celebrate during the next weekly meeting ;-) | 16:03 |
| tweining | thank you :) | 16:03 |
| gthiemonge | well all the active core reviwes have replied | 16:03 |
| fkr | congrats tweining | 16:03 |
| gthiemonge | folks, prepare your list of patches you want to get merged! | 16:04 |
| gthiemonge | ;-) | 16:05 |
| gthiemonge | and another small announcement: | 16:05 |
| gthiemonge | I'm going to restore the Octavia priority reviews etherpad | 16:06 |
| gthiemonge | (the one linked in the topic of this channel) | 16:06 |
| gthiemonge | so we will be able to prioritize the reviews of feature patches | 16:06 |
| gthiemonge | we have 9 weeks until the Feature Freeze for Zed | 16:06 |
| tweining | mmh, it seems hard to keep the list updated | 16:07 |
| johnsom | We have been pretty successful with a priority list in the past | 16:08 |
| gthiemonge | tweining: we use at the very end of the development release, when we are really close to the deadlines | 16:08 |
| johnsom | It just takes work to maintain it.... | 16:08 |
| gthiemonge | it could give us a look of what Octava zed will be | 16:08 |
| gthiemonge | I may have a script that would help me | 16:08 |
| tweining | it's good that it exists, but more automation would certainly help | 16:09 |
| gthiemonge | yeah I agree | 16:09 |
| tweining | for instance there could be a tag field in Gerrit for it | 16:09 |
| tweining | than we could filter there | 16:10 |
| gthiemonge | I think some project have a priority review vote | 16:10 |
| gthiemonge | but it is limited | 16:10 |
| gthiemonge | yeah devstack utilizes it: https://review.opendev.org/c/openstack/devstack/+/848083 | 16:10 |
| *** beagles_afk is now known as beagles | 16:11 | |
| gthiemonge | I don't know if people pay attention to it, but I think a full list in the etherpad is also a good way to split the reviews into multiple categories and to give more visibility than a single field | 16:12 |
| gthiemonge | anyways, I will send a message on this channel when the list for Zed is ready ;-) | 16:12 |
| gthiemonge | tweining: you're going to have a lot of work | 16:13 |
| tweining | hehe. we'll see :) | 16:13 |
| gthiemonge | #topic CI Status | 16:14 |
| gthiemonge | so far so good! | 16:14 |
| gthiemonge | I'm planning to focus on centos 9 straem support | 16:14 |
| gthiemonge | but... | 16:14 |
| gthiemonge | AFAIK there's a bug in libvirt in c9s that prevents an interface from being plugged into a VM (like the VIP port in an amphora :/) | 16:14 |
| gthiemonge | downgrading libvirt (to 8.3.0) is a workaround but I don't know if we can pin RPM packages in devstack | 16:15 |
| gthiemonge | maybe we should wait for a fix... | 16:15 |
| johnsom | Last I heard, it might make it into centos 9 stream the second week in July | 16:15 |
| tweining | Regarding CI: tripleo gate was blocked which cause a bit frustration on my side: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/842073 I hope it works now | 16:16 |
| tweining | otherwise I'll probably need to open a bugreport for it | 16:16 |
| gthiemonge | tweining: for these types of issues you can ping people on #tripleo or on #oooq | 16:17 |
| tweining | yes, I did this morning | 16:18 |
| tweining | I discussed it briefly with Takashi | 16:18 |
| gthiemonge | ok | 16:19 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:19 |
| opendevreview | Merged openstack/octavia stable/ussuri: Fix compile_amphora_details when using UDP listeners https://review.opendev.org/c/openstack/octavia/+/839852 | 16:20 |
| gthiemonge | just FYI, we have a few opened backports: | 16:20 |
| gthiemonge | #link https://review.opendev.org/q/project:openstack/octavia+status:open+branch:%255Estable/.* | 16:20 |
| gthiemonge | I also need to review them | 16:20 |
| johnsom | Just a few, lol | 16:21 |
| gthiemonge | only 2 pages | 16:22 |
| gthiemonge | the CI for octavia-dashboard is broken (new angularJS release), I have a patch that fixes most of the issues | 16:24 |
| gthiemonge | #link https://review.opendev.org/c/openstack/octavia-dashboard/+/846176 | 16:24 |
| johnsom | That was discussed in the horizon meeting today. | 16:24 |
| gthiemonge | but it is failing because, I think, we need to update/bump the testing tools | 16:24 |
| johnsom | It seems there is a ` instead of ' in horizon somewhere that breaks with phantomjs. | 16:24 |
| gthiemonge | oh | 16:25 |
| gthiemonge | https://zuul.opendev.org/t/openstack/build/680a0fe17bcb468db43aa17d4e774c7f | 16:25 |
| johnsom | Someone was going to propose a fix | 16:25 |
| gthiemonge | phantomjs is not happy with the horizon code | 16:25 |
| gthiemonge | I'll take a look at the horizon meeting notes | 16:25 |
| johnsom | I saw the same with designate-dashboard, but stuck with firefox instead of phantomjs to work around it | 16:25 |
| johnsom | Yeah, it's impacting a bunch of horizon plugins | 16:25 |
| johnsom | Most moved to phantomjs to get around some breaking changes that were happening in firefox | 16:26 |
| gthiemonge | ok | 16:27 |
| gthiemonge | #topic Open Discussion | 16:32 |
| johnsom | I don't have anything this week. Just a bunch of backports to review. lol | 16:33 |
| fkr | Regarding the OIF summit forum session: I've been reaching out to the various operators in order to get feedback on items to be able to provide stories / feedback / bugreports | 16:33 |
| fkr | I've started to further comment in the etherpad - since johnsonm was so kind to add various feedback there | 16:33 |
| fkr | (https://etherpad.opendev.org/p/oif-summit-berlin-2022-forum-lbaas) | 16:34 |
| fkr | to have it in the log directly | 16:34 |
| johnsom | Oh cool, I will go have a look again | 16:34 |
| gthiemonge | thank you fkr! | 16:34 |
| fkr | johnsom: don't rush it, here and there it will take a while, but am on it :) | 16:34 |
| johnsom | Ok, feel free to ping me | 16:35 |
| fkr | last week it was also concluded that certain statements are very broad, am also trying to get them to become more specific | 16:35 |
| johnsom | Oh, I do have one more topic. I will be taking some vacation next week, so limited availability. | 16:35 |
| tweining | I may take one or two days off next week as well. not sure yet. | 16:36 |
| johnsom | Yeah, the error field thing. So operators that participated in the Octavia design had SLA concerns about exposing too much error information directly to the users. I.e. they didn't want us to say "Nova is out of compute resources". | 16:37 |
| fkr | johnsom: ack. I did ping damien from OVH about it to see wether he can provide some more insights into what they have in mind. | 16:38 |
| johnsom | Yeah, would be good to get feedback/story to help address that. | 16:38 |
| johnsom | Maybe we add an API that queries the root error information that can be managed with RBAC for who can see the info. | 16:38 |
| johnsom | Definitely options, just need guidence | 16:39 |
| gthiemonge | johnsom: yes I agree | 16:43 |
| gthiemonge | ok folks, anything else? | 16:44 |
| tweining | maybe a note that I started looking into amp vertical scaling | 16:45 |
| tweining | https://review.opendev.org/c/openstack/octavia/+/848105 is anyone is interested | 16:46 |
| gthiemonge | ack, thanks tweining | 16:46 |
| tweining | apart from that I think there's nothing. do more reviews. | 16:46 |
| johnsom | It's really "optimizing" vertical scaling as we already have vertical scaling with the latest LB engines | 16:46 |
| tweining | right | 16:47 |
| gthiemonge | ok folks, thank you! | 16:50 |
| gthiemonge | #endmeeting | 16:50 |
| opendevmeet | Meeting ended Wed Jun 29 16:50:14 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:50 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-06-29-16.00.html | 16:50 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-06-29-16.00.txt | 16:50 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-06-29-16.00.log.html | 16:50 |
| fkr | o/ | 16:50 |
| matfechner | o/ | 16:51 |
| johnsom | Ok, I have done a pass over those stable branch backports. Pretty easy reviews really. | 17:07 |
| opendevreview | Tom Weininger proposed openstack/octavia master: Proposal for Amphora vertical scaling https://review.opendev.org/c/openstack/octavia/+/848105 | 17:13 |
| opendevreview | Merged openstack/octavia stable/train: Fix compile_amphora_details when using UDP listeners https://review.opendev.org/c/openstack/octavia/+/839853 | 17:21 |
| *** beagles_ is now known as beagles | 18:24 | |
| opendevreview | Merged openstack/octavia stable/xena: Validate L7Rule value and cookie name https://review.opendev.org/c/openstack/octavia/+/846548 | 19:17 |
| opendevreview | Merged openstack/octavia stable/wallaby: Validate L7Rule value and cookie name https://review.opendev.org/c/openstack/octavia/+/846549 | 19:17 |
| opendevreview | Merged openstack/octavia stable/victoria: Validate L7Rule value and cookie name https://review.opendev.org/c/openstack/octavia/+/846570 | 19:17 |
| opendevreview | Merged openstack/octavia stable/yoga: Validate L7Rule value and cookie name https://review.opendev.org/c/openstack/octavia/+/846547 | 19:18 |
| opendevreview | Merged openstack/octavia stable/train: Validate L7Rule value and cookie name https://review.opendev.org/c/openstack/octavia/+/846572 | 19:18 |
| opendevreview | Merged openstack/octavia stable/ussuri: Validate L7Rule value and cookie name https://review.opendev.org/c/openstack/octavia/+/846571 | 19:18 |
| opendevreview | Merged openstack/octavia stable/ussuri: Fix disabled UDP pools https://review.opendev.org/c/openstack/octavia/+/846585 | 19:18 |
| opendevreview | Merged openstack/octavia stable/train: Fix disabled UDP pools https://review.opendev.org/c/openstack/octavia/+/846586 | 19:18 |
| opendevreview | Merged openstack/octavia stable/yoga: Fix duplicate object error messages https://review.opendev.org/c/openstack/octavia/+/847555 | 19:18 |
| opendevreview | Merged openstack/octavia stable/xena: Fix potential race conditions on update requests in the v2 worker https://review.opendev.org/c/openstack/octavia/+/839855 | 20:26 |
| opendevreview | Merged openstack/octavia stable/wallaby: Fix potential race conditions on update requests in the v2 worker https://review.opendev.org/c/openstack/octavia/+/839856 | 20:26 |
| opendevreview | Merged openstack/octavia stable/ussuri: Fix potential race conditions on update requests in the v2 worker https://review.opendev.org/c/openstack/octavia/+/839858 | 20:26 |
| opendevreview | Merged openstack/octavia stable/yoga: Fix potential race conditions on update requests in the v2 worker https://review.opendev.org/c/openstack/octavia/+/839854 | 20:27 |
| opendevreview | Merged openstack/octavia stable/wallaby: Fix disabled UDP pools https://review.opendev.org/c/openstack/octavia/+/846583 | 20:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!