| opendevreview | Merged openstack/octavia stable/victoria: Fix duplicate SG creation for listener peer port https://review.opendev.org/c/openstack/octavia/+/808987 | 07:30 |
|---|---|---|
| gthiemonge | #startmeeting Octavia | 16:00 |
| opendevmeet | Meeting started Wed Sep 22 16:00:23 2021 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:00 |
| johnsom | o/ | 16:00 |
| gthiemonge | #topic Announcements | 16:02 |
| gthiemonge | I don't have announcement for this week | 16:02 |
| gthiemonge | RC1 for Xena was last week, we don't expect to have another RC | 16:03 |
| johnsom | Cool, I don't think I have anything this week either. | 16:03 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:04 |
| gthiemonge | I'm still working on the fix for member networks with multiple subnets that rm_work started | 16:05 |
| gthiemonge | I'm trying to get more control on the creation of the port for members | 16:05 |
| johnsom | +1 | 16:06 |
| gthiemonge | (we delegate this work to nova/neutron, and the result seems inconsitent - some unnecessary subnets are plugged) | 16:06 |
| johnsom | I have been doing reviews mostly. | 16:06 |
| gthiemonge | but creating ports requires changes in the flow (we need to explicity delete them, when deleting members or when triggering failover) | 16:07 |
| gthiemonge | so I think the patch will be bigger :D | 16:07 |
| johnsom | Yep, it's a big one I'm sure. But an important one | 16:07 |
| gthiemonge | I may try to split it into multiple patches, but I don't know if that makes sense | 16:08 |
| johnsom | Yeah, sometimes you can, sometimes you can't. | 16:09 |
| gthiemonge | or you don't want | 16:09 |
| gthiemonge | #topic Open Discussion | 16:11 |
| gthiemonge | any other topics? | 16:11 |
| johnsom | Nothing from me. Thanks for taking on that beast of a task for member subnets | 16:12 |
| gthiemonge | ;-) | 16:12 |
| gthiemonge | maybe one last thing: | 16:12 |
| gthiemonge | I proposed to backport the "generic interface management" patch to wallaby | 16:12 |
| gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/807310 | 16:13 |
| johnsom | Ah yeah. I will take a look. That is a big one too | 16:13 |
| gthiemonge | I consider that it's a fix for "modern" distributions that dropped old fashion way to configure network | 16:13 |
| gthiemonge | backport went smoothly | 16:13 |
| gthiemonge | my concern is that people may think that it's a new feature | 16:14 |
| johnsom | Well, it fixes some issues we have had with the distro network management tools not behaving correctly. | 16:15 |
| johnsom | Like "already exists" issues when doing the ifup | 16:15 |
| gthiemonge | my other concern is that this commit fixes an issue that someone fixed in an open review: | 16:15 |
| gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/794602 | 16:15 |
| gthiemonge | so we may have to add this last patch on stable branches that will not have the "generic iface mgmt" stuff | 16:16 |
| johnsom | Hmm, yeah. You could also give co-author on the wallaby for the author of the other patch. | 16:17 |
| gthiemonge | so I don't know if we want to backport the huge patch to all stable branches | 16:17 |
| gthiemonge | johnsom: oh yes | 16:18 |
| johnsom | Yeah, I guess the question is of need. Does anyone need it on the other stable branches. For the most part, the older distros work. I guess it's also a question of will it make other backports harder, etc. | 16:19 |
| gthiemonge | good point | 16:19 |
| johnsom | I know we need it for wallaby and newer for sure. It's going farther back I'm not sure about. | 16:20 |
| gthiemonge | the fix for member networks with multiple subnets needs that patch | 16:20 |
| johnsom | Yeah, so that may push us to backport farther. | 16:20 |
| gthiemonge | Ok, I'll propose the backports (and will add the co-author line) and core reviewers will give us their feedback if they think it could be an issue | 16:22 |
| johnsom | Sounds like a plan | 16:22 |
| johnsom | I will have a look at the wallaby patch after the meeting | 16:22 |
| gthiemonge | thanks johnsom | 16:23 |
| gthiemonge | anything else? | 16:24 |
| johnsom | Not from me | 16:24 |
| gthiemonge | Ok! Thank you Folks! | 16:25 |
| gthiemonge | #endmeeting | 16:25 |
| opendevmeet | Meeting ended Wed Sep 22 16:25:13 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:25 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2021/octavia.2021-09-22-16.00.html | 16:25 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2021/octavia.2021-09-22-16.00.txt | 16:25 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2021/octavia.2021-09-22-16.00.log.html | 16:25 |
| johnsom | Thanks Greg | 16:25 |
| -opendevstatus- NOTICE: Zuul has been restarted in order to address a performance regression related to event processing; any changes pushed or approved between roughly 17:00 and 18:30 UTC should be rechecked if they're not already enqueued according to the Zuul status page | 18:36 | |
| rm_work | hmmm i wonder if that network management patch will fix the issue I had in centos where I had to force garps | 20:26 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!