| *** rpittau|afk is now known as rpittau | 07:45 | |
| *** iurygregory is now known as iury|holiday | 09:18 | |
| *** ralonsoh_ is now known as ralonsoh | 09:20 | |
| hberaud | #startmeeting oslo | 15:01 |
|---|---|---|
| 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| opendevmeet | The meeting name has been set to 'oslo' | 15:01 |
| hberaud | Courtesy ping for hberaud, bnemec, johnsom, redrobot, stephenfin, bcafarel, kgiusti, jungleboyj | 15:01 |
| sboyron | o/ | 15:01 |
| ralonsoh | hi | 15:01 |
| bcafarel | o/ | 15:01 |
| hberaud | #link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting | 15:01 |
| hberaud | Will just wait a couple minutes for folks. | 15:01 |
| hberaud | #topic Red flags for/from liaisons | 15:03 |
| hberaud | AFAIK nothing from oslo | 15:04 |
| ralonsoh | I had a proposal for oslo.privsep, but I don't know if this is the correct time | 15:05 |
| hberaud | We will discuss it at the end of the meeting | 15:05 |
| ralonsoh | thanks | 15:05 |
| hberaud | :) | 15:05 |
| hberaud | #topic Releases liaison | 15:05 |
| hberaud | Next week will be the milestone 2 for xena | 15:06 |
| hberaud | So I'll surely release a couple of new version for oslo* | 15:07 |
| hberaud | s/version/versions/ | 15:07 |
| hberaud | #topic Security liaison | 15:07 |
| hberaud | bnemec: o/ around? | 15:07 |
| stephenfin | o/ | 15:08 |
| hberaud | I didn't noticed something about security | 15:09 |
| hberaud | stephenfin: o/ | 15:09 |
| hberaud | #topic TaCT SIG liaison | 15:09 |
| hberaud | sboyron: The floor is yours | 15:09 |
| sboyron | I did not see anything on fire | 15:10 |
| hberaud | cool thanks | 15:10 |
| hberaud | #topic Action items from last meeting | 15:11 |
| hberaud | Nothing here too | 15:11 |
| hberaud | #topic Weekly Wayward Wallaby Review | 15:11 |
| hberaud | https://review.opendev.org/c/openstack/oslo.metrics/+/799092 | 15:11 |
| hberaud | stephenfin: This one was related to one of your previous comment | 15:12 |
| hberaud | ^ | 15:12 |
| stephenfin | ah, easy as | 15:12 |
| stephenfin | Is that really wayward? It's only up 4 days | 15:13 |
| hberaud | I take it randomly | 15:13 |
| stephenfin | ah, okay :) | 15:13 |
| hberaud | This one is more wayward :) => https://review.opendev.org/c/openstack/oslo.db/+/734744 | 15:14 |
| stephenfin | Looks like that's still held up pending bnemec's comments | 15:15 |
| hberaud | I think that we could close it, any opinion? | 15:15 |
| stephenfin | Yeah, close it | 15:15 |
| hberaud | done | 15:17 |
| hberaud | #topic Open discussion | 15:17 |
| hberaud | ralonsoh: the floor is yours | 15:17 |
| ralonsoh | thanks | 15:17 |
| ralonsoh | this is related to https://review.opendev.org/c/openstack/oslo.privsep/+/794847 | 15:17 |
| ralonsoh | last week we merged https://review.opendev.org/c/openstack/oslo.privsep/+/794993 | 15:18 |
| ralonsoh | this patch created a privsep context with timeout | 15:18 |
| ralonsoh | --> if the command is not executed in this time, we raise an exception | 15:18 |
| ralonsoh | this has a problem in the daemon: the command being executed (in a thread) could never finish | 15:19 |
| ralonsoh | and there is no way to stop it | 15:19 |
| ralonsoh | we can run out of available threads in the daemon | 15:19 |
| ralonsoh | my proposal uses fork instead of threads | 15:19 |
| hberaud | I see | 15:20 |
| ralonsoh | when a new command arrives, we fork the daemon (with the same permissions) and execute the command | 15:20 |
| ralonsoh | advantages: we can kill the child and we can execute several commands in parallel | 15:20 |
| ralonsoh | that's all | 15:20 |
| ralonsoh | (questions?) | 15:20 |
| hberaud | At first glance that LGTM | 15:20 |
| hberaud | What's about eventlet? | 15:21 |
| ralonsoh | we unmonkey-patch the daemon | 15:21 |
| ralonsoh | (I submitted the patch) | 15:22 |
| hberaud | Ah yes good point | 15:22 |
| hberaud | I remember these changes | 15:22 |
| ralonsoh | I would like to know a way to test it in, for example, Neutron CI | 15:22 |
| ralonsoh | a way to force this patch installation and pass all neutron CI tests | 15:22 |
| ralonsoh | (or Nova or anyone else) | 15:23 |
| hberaud | That's a good question | 15:23 |
| ralonsoh | soemthing like creating a devstack patch installing it | 15:23 |
| ralonsoh | and then pushing a change in Neutron depending on this devstack change | 15:23 |
| ralonsoh | could work? | 15:23 |
| hberaud | stephenfin: do you have previous example of the same thing with nova? | 15:23 |
| hberaud | ^ | 15:23 |
| hberaud | (An example to follow) | 15:24 |
| 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 |
| ralonsoh | ah yes | 15:25 |
| ralonsoh | required projects, that will work | 15:25 |
| ralonsoh | thanks a lot | 15:25 |
| opendevreview | Merged openstack/oslo.metrics master: Fix Heading Characters https://review.opendev.org/c/openstack/oslo.metrics/+/799092 | 15:25 |
| 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:26 |
| hberaud | Thanks ralonsoh | 15:27 |
| ralonsoh | thank you | 15:27 |
| hberaud | Anything else to discuss today? | 15:27 |
| ajya | Hi, there are backports failing lower constraints jobs in victoria and ussuri | 15:27 |
| ajya | Weren't they removed in other projects few months ago? | 15:28 |
| stephenfin | not consistently | 15:28 |
| ajya | the patches - https://review.opendev.org/c/openstack/oslo.db/+/798761 https://review.opendev.org/c/openstack/oslo.db/+/798762 | 15:28 |
| stephenfin | we haven't backported the removals everywhere | 15:28 |
| stephenfin | (it's easier do it on demand) | 15:28 |
| hberaud | ajya: do you want to handle the backport? | 15:29 |
| 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:29 |
| ajya | hberaud, stephenfin, ok, will do it | 15:30 |
| hberaud | ajya: you're welcome | 15:31 |
| hberaud | Anything else? | 15:31 |
| hberaud | OK, thanks everyone. Let's wrap up | 15:37 |
| ralonsoh | bye | 15:37 |
| hberaud | #endmeeting | 15:37 |
| opendevmeet | Meeting ended Mon Jul 5 15:37:45 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:37 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/oslo/2021/oslo.2021-07-05-15.01.html | 15:37 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/oslo/2021/oslo.2021-07-05-15.01.txt | 15:37 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/oslo/2021/oslo.2021-07-05-15.01.log.html | 15:37 |
| *** rpittau is now known as rpittau|afk | 16:20 | |
| opendevreview | Ghanshyam proposed openstack/oslo.policy master: Make policy rule default change warning configurable https://review.opendev.org/c/openstack/oslo.policy/+/799539 | 19:16 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!