15:01:05 <hberaud> #startmeeting oslo
15:01:05 <opendevmeet> Meeting started Mon Jul  5 15:01:05 2021 UTC and is due to finish in 60 minutes.  The chair is hberaud. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:01:05 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:01:05 <opendevmeet> The meeting name has been set to 'oslo'
15:01:14 <hberaud> Courtesy ping for hberaud, bnemec, johnsom, redrobot, stephenfin, bcafarel, kgiusti, jungleboyj
15:01:14 <sboyron> o/
15:01:21 <ralonsoh> hi
15:01:23 <bcafarel> o/
15:01:25 <hberaud> #link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting
15:01:46 <hberaud> Will just wait a couple minutes for folks.
15:03:14 <hberaud> #topic Red flags for/from liaisons
15:04:15 <hberaud> AFAIK nothing from oslo
15:05:20 <ralonsoh> I had a proposal for oslo.privsep, but I don't know if this is the correct time
15:05:38 <hberaud> We will discuss it at the end of the meeting
15:05:44 <ralonsoh> thanks
15:05:50 <hberaud> :)
15:05:53 <hberaud> #topic Releases liaison
15:06:43 <hberaud> Next week will be the milestone 2 for xena
15:07:03 <hberaud> So I'll surely release a couple of new version for oslo*
15:07:13 <hberaud> s/version/versions/
15:07:27 <hberaud> #topic Security liaison
15:07:44 <hberaud> bnemec: o/ around?
15:08:38 <stephenfin> o/
15:09:33 <hberaud> I didn't noticed something about security
15:09:41 <hberaud> stephenfin: o/
15:09:50 <hberaud> #topic TaCT SIG liaison
15:09:57 <hberaud> sboyron: The floor is yours
15:10:10 <sboyron> I did not see anything on fire
15:10:17 <hberaud> cool thanks
15:11:05 <hberaud> #topic Action items from last meeting
15:11:09 <hberaud> Nothing here too
15:11:18 <hberaud> #topic Weekly Wayward Wallaby Review
15:11:55 <hberaud> https://review.opendev.org/c/openstack/oslo.metrics/+/799092
15:12:30 <hberaud> stephenfin: This one was related to one of your previous comment
15:12:32 <hberaud> ^
15:12:58 <stephenfin> ah, easy as
15:13:14 <stephenfin> Is that really wayward? It's only up 4 days
15:13:32 <hberaud> I take it randomly
15:13:52 <stephenfin> ah, okay :)
15:14:02 <hberaud> This one is more wayward :) => https://review.opendev.org/c/openstack/oslo.db/+/734744
15:15:07 <stephenfin> Looks like that's still held up pending bnemec's comments
15:15:08 <hberaud> I think that we could close it, any opinion?
15:15:14 <stephenfin> Yeah, close it
15:17:04 <hberaud> done
15:17:34 <hberaud> #topic Open discussion
15:17:46 <hberaud> ralonsoh: the floor is yours
15:17:50 <ralonsoh> thanks
15:17:58 <ralonsoh> this is related to https://review.opendev.org/c/openstack/oslo.privsep/+/794847
15:18:18 <ralonsoh> last week we merged https://review.opendev.org/c/openstack/oslo.privsep/+/794993
15:18:34 <ralonsoh> this patch created a privsep context with timeout
15:18:51 <ralonsoh> --> if the command is not executed in this time, we raise an exception
15:19:12 <ralonsoh> this has a problem in the daemon: the command being executed (in a thread) could never finish
15:19:17 <ralonsoh> and there is no way to stop it
15:19:32 <ralonsoh> we can run out of available threads in the daemon
15:19:45 <ralonsoh> my proposal uses fork instead of threads
15:20:02 <hberaud> I see
15:20:02 <ralonsoh> when a new command arrives, we fork the daemon (with the same permissions) and execute the command
15:20:22 <ralonsoh> advantages: we can kill the child and we can execute several commands in parallel
15:20:24 <ralonsoh> that's all
15:20:39 <ralonsoh> (questions?)
15:20:43 <hberaud> At first glance that LGTM
15:21:44 <hberaud> What's about eventlet?
15:21:56 <ralonsoh> we unmonkey-patch the daemon
15:22:01 <ralonsoh> (I submitted the patch)
15:22:13 <hberaud> Ah yes good point
15:22:22 <hberaud> I remember these changes
15:22:44 <ralonsoh> I would like to know a way to test it in, for example, Neutron CI
15:22:57 <ralonsoh> a way to force this patch installation and pass all neutron CI tests
15:23:02 <ralonsoh> (or Nova or anyone else)
15:23:25 <hberaud> That's a good question
15:23:25 <ralonsoh> soemthing like creating a devstack patch installing it
15:23:40 <ralonsoh> and then pushing a change in Neutron depending on this devstack change
15:23:48 <ralonsoh> could work?
15:23:52 <hberaud> stephenfin: do you have previous example of the same thing with nova?
15:23:55 <hberaud> ^
15:24:36 <hberaud> (An example to follow)
15:25:00 <stephenfin> hberaud: ralonsoh: typically we'd create a DNM patch against nova that 'Depends-On' the oslo patch. You'll need to modify 'required_projects' in .zuul.yaml so that it'll install the oslo.lib from git instead of pypi though
15:25:13 <ralonsoh> ah yes
15:25:18 <ralonsoh> required projects, that will work
15:25:23 <ralonsoh> thanks a lot
15:25:56 <opendevreview> Merged openstack/oslo.metrics master: Fix Heading Characters  https://review.opendev.org/c/openstack/oslo.metrics/+/799092
15:26:37 <ralonsoh> so once I have something "decent" (a patch passing al oslo.privsep tests and working in Neutron CI), I'll ping you again
15:27:26 <hberaud> Thanks ralonsoh
15:27:30 <ralonsoh> thank you
15:27:37 <hberaud> Anything else to discuss today?
15:27:53 <ajya> Hi, there are backports failing lower constraints jobs in victoria and ussuri
15:28:09 <ajya> Weren't they removed in other projects few months ago?
15:28:21 <stephenfin> not consistently
15:28:24 <ajya> the patches - https://review.opendev.org/c/openstack/oslo.db/+/798761 https://review.opendev.org/c/openstack/oslo.db/+/798762
15:28:30 <stephenfin> we haven't backported the removals everywhere
15:28:45 <stephenfin> (it's easier do it on demand)
15:29:22 <hberaud> ajya: do you want to handle the backport?
15:29:35 <stephenfin> so you might need to backport those removal patches first (don't forget to fold in follow-ups that addressed things missed like tox.ini files etc.) Luckily those should be trivial to approve
15:30:02 <ajya> hberaud, stephenfin, ok, will do it
15:31:25 <hberaud> ajya: you're welcome
15:31:32 <hberaud> Anything else?
15:37:34 <hberaud> OK, thanks everyone. Let's wrap up
15:37:43 <ralonsoh> bye
15:37:45 <hberaud> #endmeeting