| opendevreview | Ghanshyam proposed openstack/octavia-tempest-plugin master: DNM: Testing tempest pin on stable/ussuri https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/843089 | 02:34 |
|---|---|---|
| opendevreview | Ghanshyam proposed openstack/octavia stable/ussuri: DNM: Testing tempest pin on stable/ussuri https://review.opendev.org/c/openstack/octavia/+/843090 | 02:34 |
| *** ysandeep|out is now known as ysandeep|rover | 04:35 | |
| *** ysandeep|rover is now known as ysandeep|rover|brb | 05:25 | |
| *** ysandeep|rover|brb is now known as ysandeep|rover | 05:32 | |
| *** ysandeep|rover is now known as ysandeep|rover|lunch | 07:44 | |
| *** ysandeep|rover|lunch is now known as ysandeep|rover | 08:33 | |
| opendevreview | Gregory Thiemonge proposed openstack/octavia master: Fix driver-agent cleanup https://review.opendev.org/c/openstack/octavia/+/843248 | 08:40 |
| tweining | https://bholley.net/blog/2015/must-be-this-tall-to-write-multi-threaded-code.html ;D | 08:43 |
| gthiemonge | :D | 08:45 |
| *** ysandeep|rover is now known as ysandeep|rover|afk | 12:41 | |
| johnsom | Hey now, I am that tall. | 13:21 |
| johnsom | Hmm, that code used to work just fine. What changed? | 13:25 |
| tweining | stopping DA takes a very long time. I observed that as well in devstack. | 13:25 |
| johnsom | Neutron doesn’t shutdown anymore either, it drives me nuts with devstack | 13:26 |
| johnsom | What changed to cause that though, that is what I am curious about, because that did work properly before | 13:27 |
| johnsom | I wonder if this is some issue with puthon3 | 13:28 |
| *** ysandeep|rover|afk is now known as ysandeep|rover | 13:31 | |
| gthiemonge | johnsom: I've observed this behavior since I switched to c9s | 13:44 |
| gthiemonge | johnsom: so maybe py39 | 13:44 |
| tweining | maybe that: "Changed in version 3.7: socketserver.ForkingMixIn.server_close() and socketserver.ThreadingMixIn.server_close() now waits until all child processes and non-daemonic threads complete." | 14:47 |
| tweining | https://docs.python.org/3/library/socketserver.html#socketserver.ThreadingMixIn | 14:47 |
| tweining | "Add a new socketserver.ForkingMixIn.block_on_close class attribute to opt-in for the pre-3.7 behaviour." | 14:48 |
| opendevreview | Ghanshyam proposed openstack/octavia stable/victoria: DNM: testing Tempest pin in stable/victoria https://review.opendev.org/c/openstack/octavia/+/843296 | 15:35 |
| opendevreview | Ghanshyam proposed openstack/octavia master: DNM: testing Tempest pin in stable/victoria https://review.opendev.org/c/openstack/octavia/+/843306 | 15:54 |
| gthiemonge | #startmeeting Octavia | 16:00 |
| opendevmeet | Meeting started Wed May 25 16:00:10 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:00 |
| johnsom | o/ | 16:00 |
| spencerharmon | hello! | 16:00 |
| tweining | hi | 16:00 |
| gthiemonge | #topic Announcements | 16:00 |
| gthiemonge | * PTG October 2022 | 16:01 |
| gthiemonge | it was announced this week, the next PTG will be in-person! | 16:01 |
| gthiemonge | October 17-20th in Columbus, Ohio | 16:01 |
| gthiemonge | #link http://lists.openstack.org/pipermail/openstack-discuss/2022-May/028601.html | 16:02 |
| gthiemonge | any other announcemnts? | 16:03 |
| gthiemonge | also: | 16:03 |
| gthiemonge | #link https://openinfra.dev/ptg/ | 16:03 |
| gthiemonge | #topic Brief progress reports / bugs needing review | 16:04 |
| gthiemonge | I was focused on downstream tasks, but I have proposed a fix for the graceful shutdown of the driver-agent: | 16:05 |
| gthiemonge | #link https://review.opendev.org/c/openstack/octavia/+/843248 | 16:05 |
| tweining | it seems we've been lazy wrt. to reviews lately | 16:05 |
| gthiemonge | there were 2 issues with the driver-agent: systemctl stop killed the driver-agent after a 5min timeout, and there were a lot of leaked processes after each restart of the service | 16:06 |
| gthiemonge | #topic Open Discussion | 16:10 |
| gthiemonge | anything else folkss? | 16:10 |
| tweining | maybe a stupid question about the PTG: does in-person mean that only people who a physically there can participate in discussions? (ie. no virtual meetings anymore) | 16:10 |
| spencerharmon | No changes on my end, but if there are any questions/comments on notifications patch, please lmk. :) | 16:11 |
| spencerharmon | #link https://review.opendev.org/c/openstack/octavia/+/831051 | 16:11 |
| gthiemonge | I think they will provide a solution for people in remote | 16:11 |
| gthiemonge | they did it in the past | 16:11 |
| gthiemonge | spencerharmon: yeah I still need to review it :/ | 16:12 |
| gthiemonge | I'm off for a few days but I'm going to take a look when I'm back | 16:12 |
| gthiemonge | spencerharmon: maybe a quick question | 16:13 |
| spencerharmon | Not a rush. Just hoping to get it in for next release. | 16:13 |
| gthiemonge | spencerharmon: IIRC at the PTG we discuss adding ".begin" events (that would match the .end events) | 16:13 |
| gthiemonge | "discussed" | 16:13 |
| gthiemonge | do you plan to add it? | 16:14 |
| spencerharmon | That's right. I don't need them in my use case. I'd be happy to add them, but I think it could be a long process of deciding exactly which notifications should exist. My hope is that this pattern is simple enough that folks wanting new notification types can add them easily in future patches. If you'd like those before merging, I'll be happy to create them. | 16:15 |
| gthiemonge | ok, that's a fair point | 16:17 |
| spencerharmon | I'd like these ones at a minimum for triggering a service we use to make recordsets in designate. I don't object to any other notification types, though. :) | 16:17 |
| gthiemonge | yeah I see | 16:18 |
| gthiemonge | I don't know if other folks have an opinion on it | 16:18 |
| gthiemonge | anywys, I will review it | 16:18 |
| tweining | #action gthiemonge review notifications change ;) | 16:18 |
| spencerharmon | :D | 16:18 |
| gthiemonge | is it on my TODO lsit now? | 16:19 |
| gthiemonge | Ok, anything else? | 16:21 |
| gthiemonge | ack | 16:22 |
| gthiemonge | Thanks everyone! | 16:22 |
| tweining | it would be cool if the bot would send a reminder in the next meeting | 16:22 |
| gthiemonge | #endmeeting | 16:22 |
| opendevmeet | Meeting ended Wed May 25 16:22:17 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:22 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-05-25-16.00.html | 16:22 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-05-25-16.00.txt | 16:22 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/octavia/2022/octavia.2022-05-25-16.00.log.html | 16:22 |
| gthiemonge | lol I hope no | 16:22 |
| spencerharmon | Thanks yall! | 16:22 |
| gthiemonge | but we could review the action items from the previous meeting | 16:22 |
| tweining | at least it's in the meeting summary | 16:22 |
| *** ysandeep|rover is now known as ysandeep|out | 16:59 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!