| *** yamamoto has joined #openstack-oslo | 00:13 | |
| *** yamamoto has quit IRC | 00:18 | |
| *** bobh has joined #openstack-oslo | 00:44 | |
| *** tovin07_ has joined #openstack-oslo | 00:45 | |
| *** yamamoto has joined #openstack-oslo | 01:13 | |
| *** yamamoto has quit IRC | 01:19 | |
| *** yamamoto has joined #openstack-oslo | 01:41 | |
| *** salv-orl_ has joined #openstack-oslo | 02:51 | |
| *** salv-orlando has quit IRC | 02:53 | |
| *** namnh has joined #openstack-oslo | 03:21 | |
| *** rcernin_ has joined #openstack-oslo | 03:23 | |
| *** rcernin has quit IRC | 03:25 | |
| *** rcernin_ has quit IRC | 03:29 | |
| *** rcernin has joined #openstack-oslo | 03:29 | |
| *** rcernin has quit IRC | 03:33 | |
| *** pooja_jadhav has joined #openstack-oslo | 03:43 | |
| *** rcernin has joined #openstack-oslo | 03:49 | |
| *** bobh has quit IRC | 04:11 | |
| *** bobh has joined #openstack-oslo | 04:20 | |
| *** bobh has quit IRC | 04:25 | |
| *** bhagyashri_s is now known as bhagyashris | 04:33 | |
| *** bhagyashris is now known as bhagyashri_s | 04:33 | |
| *** bhagyashri_s is now known as bhagyashris | 04:33 | |
| *** bobh has joined #openstack-oslo | 04:34 | |
| *** bobh has quit IRC | 04:39 | |
| *** bobh has joined #openstack-oslo | 04:43 | |
| *** bobh has quit IRC | 04:48 | |
| *** bobh has joined #openstack-oslo | 04:51 | |
| *** bobh has quit IRC | 04:57 | |
| *** bobh has joined #openstack-oslo | 05:00 | |
| *** bobh has quit IRC | 05:04 | |
| *** bobh has joined #openstack-oslo | 05:10 | |
| *** bobh has quit IRC | 05:10 | |
| *** WUAM1Vandreagus has joined #openstack-oslo | 05:11 | |
| *** bobh has joined #openstack-oslo | 05:21 | |
| *** bobh has quit IRC | 05:25 | |
| openstackgerrit | Merged openstack/oslo.messaging master: Modify grammatical errors https://review.openstack.org/541963 | 05:30 |
|---|---|---|
| *** openstackgerrit has quit IRC | 05:49 | |
| *** bobh has joined #openstack-oslo | 05:53 | |
| *** bobh has quit IRC | 05:58 | |
| *** bobh has joined #openstack-oslo | 06:00 | |
| *** bobh has quit IRC | 06:05 | |
| *** bobh has joined #openstack-oslo | 06:12 | |
| *** bobh has quit IRC | 06:17 | |
| *** bobh has joined #openstack-oslo | 06:20 | |
| *** msimonin has joined #openstack-oslo | 06:22 | |
| *** bobh has quit IRC | 06:25 | |
| *** ianychoi__ has joined #openstack-oslo | 06:26 | |
| *** ianychoi_ has quit IRC | 06:29 | |
| *** bobh has joined #openstack-oslo | 06:31 | |
| *** bobh has quit IRC | 06:37 | |
| *** bobh has joined #openstack-oslo | 06:41 | |
| *** bobh has quit IRC | 06:46 | |
| *** sileht has quit IRC | 06:57 | |
| *** sileht has joined #openstack-oslo | 06:59 | |
| *** d0ugal has joined #openstack-oslo | 07:00 | |
| *** d0ugal has quit IRC | 07:00 | |
| *** d0ugal has joined #openstack-oslo | 07:00 | |
| *** bobh has joined #openstack-oslo | 07:03 | |
| *** rcernin has quit IRC | 07:07 | |
| *** bobh has quit IRC | 07:08 | |
| *** bobh has joined #openstack-oslo | 07:11 | |
| *** bobh has quit IRC | 07:16 | |
| *** bobh has joined #openstack-oslo | 07:23 | |
| *** bobh has quit IRC | 07:27 | |
| *** pcaruana has joined #openstack-oslo | 07:39 | |
| *** bobh has joined #openstack-oslo | 07:42 | |
| *** bobh has quit IRC | 07:47 | |
| *** e0ne has joined #openstack-oslo | 07:52 | |
| *** AlexeyAbashkin has joined #openstack-oslo | 07:54 | |
| *** bobh has joined #openstack-oslo | 07:55 | |
| *** bobh has quit IRC | 07:59 | |
| *** tesseract has joined #openstack-oslo | 08:17 | |
| *** jaosorior has joined #openstack-oslo | 08:40 | |
| *** bobh has joined #openstack-oslo | 08:51 | |
| *** bobh has quit IRC | 08:56 | |
| *** shardy has joined #openstack-oslo | 08:59 | |
| *** salv-orl_ has quit IRC | 09:09 | |
| *** salv-orlando has joined #openstack-oslo | 09:09 | |
| *** bobh has joined #openstack-oslo | 09:14 | |
| *** salv-orlando has quit IRC | 09:14 | |
| *** bobh has quit IRC | 09:19 | |
| *** hoonetorg has quit IRC | 09:19 | |
| *** bobh has joined #openstack-oslo | 09:21 | |
| *** chhagarw has joined #openstack-oslo | 09:23 | |
| *** bobh has quit IRC | 09:26 | |
| *** gibi_ is now known as gibi | 09:30 | |
| *** bobh has joined #openstack-oslo | 09:30 | |
| *** hoonetorg has joined #openstack-oslo | 09:32 | |
| *** bobh has quit IRC | 09:35 | |
| *** openstackgerrit has joined #openstack-oslo | 09:42 | |
| openstackgerrit | Thomas Bechtold proposed openstack/oslo.log master: Increase sleep time in testsuite to make it more robust https://review.openstack.org/554180 | 09:42 |
| *** gus has quit IRC | 09:43 | |
| openstackgerrit | Thomas Bechtold proposed openstack/oslo.log master: Increase sleep time in testsuite to make it more robust https://review.openstack.org/554180 | 09:43 |
| *** gus has joined #openstack-oslo | 09:44 | |
| *** bobh has joined #openstack-oslo | 09:57 | |
| *** tovin07_ has quit IRC | 10:00 | |
| *** bobh has quit IRC | 10:01 | |
| *** bobh has joined #openstack-oslo | 10:05 | |
| *** namnh has quit IRC | 10:05 | |
| *** sambetts_ is now known as sambetts | 10:08 | |
| *** bobh has quit IRC | 10:10 | |
| *** bobh has joined #openstack-oslo | 10:12 | |
| *** yamamoto has quit IRC | 10:16 | |
| *** bobh has quit IRC | 10:16 | |
| *** e0ne has quit IRC | 10:19 | |
| *** bobh has joined #openstack-oslo | 10:22 | |
| *** e0ne has joined #openstack-oslo | 10:25 | |
| *** bobh has quit IRC | 10:28 | |
| *** bobh has joined #openstack-oslo | 10:45 | |
| *** bobh has quit IRC | 10:51 | |
| *** e0ne has quit IRC | 10:51 | |
| *** bobh has joined #openstack-oslo | 10:51 | |
| *** bobh has quit IRC | 10:56 | |
| *** ansmith has quit IRC | 10:58 | |
| *** bobh has joined #openstack-oslo | 11:01 | |
| *** yamamoto has joined #openstack-oslo | 11:02 | |
| *** salv-orlando has joined #openstack-oslo | 11:05 | |
| *** bobh has quit IRC | 11:06 | |
| *** bobh has joined #openstack-oslo | 11:13 | |
| *** edmondsw has joined #openstack-oslo | 11:14 | |
| *** edmondsw has quit IRC | 11:18 | |
| *** bobh has quit IRC | 11:18 | |
| *** bobh has joined #openstack-oslo | 11:20 | |
| *** pbourke_ has quit IRC | 11:22 | |
| *** pbourke_ has joined #openstack-oslo | 11:22 | |
| *** e0ne has joined #openstack-oslo | 11:23 | |
| *** bobh has quit IRC | 11:25 | |
| *** bobh has joined #openstack-oslo | 11:30 | |
| *** pblaho has joined #openstack-oslo | 11:33 | |
| *** bobh has quit IRC | 11:36 | |
| *** e0ne has quit IRC | 11:40 | |
| *** bobh has joined #openstack-oslo | 11:40 | |
| *** e0ne has joined #openstack-oslo | 11:44 | |
| *** bobh has quit IRC | 11:45 | |
| *** jrklein744 has joined #openstack-oslo | 11:47 | |
| *** bobh has joined #openstack-oslo | 11:53 | |
| *** raildo has joined #openstack-oslo | 12:04 | |
| *** edmondsw has joined #openstack-oslo | 12:15 | |
| *** gyankum has joined #openstack-oslo | 12:23 | |
| *** bobh has quit IRC | 12:23 | |
| *** dmellado has quit IRC | 12:24 | |
| *** dmellado has joined #openstack-oslo | 12:27 | |
| *** ansmith has joined #openstack-oslo | 12:45 | |
| *** kgiusti has joined #openstack-oslo | 12:54 | |
| *** yamamoto has quit IRC | 13:08 | |
| *** eck`` is now known as eck`gone | 13:26 | |
| *** eck`gone is now known as eck` | 13:27 | |
| *** shardy is now known as shardy_mtg | 13:28 | |
| *** jroll has quit IRC | 13:42 | |
| *** jroll has joined #openstack-oslo | 13:43 | |
| *** slaweq has joined #openstack-oslo | 13:44 | |
| *** yamamoto has joined #openstack-oslo | 13:45 | |
| *** yamamoto_ has joined #openstack-oslo | 13:48 | |
| *** janzian has joined #openstack-oslo | 13:50 | |
| *** yamamoto has quit IRC | 13:52 | |
| *** beekneemech is now known as bnemec | 14:02 | |
| *** bobh has joined #openstack-oslo | 14:13 | |
| *** namnh has joined #openstack-oslo | 14:22 | |
| *** spilla has joined #openstack-oslo | 14:29 | |
| *** yamamoto_ has quit IRC | 14:30 | |
| *** crushil has joined #openstack-oslo | 14:31 | |
| *** salv-orlando has quit IRC | 14:32 | |
| *** salv-orlando has joined #openstack-oslo | 14:33 | |
| *** gyankum has quit IRC | 14:36 | |
| *** takedakn has joined #openstack-oslo | 14:37 | |
| *** yamamoto has joined #openstack-oslo | 14:46 | |
| *** salv-orlando has quit IRC | 14:46 | |
| *** salv-orlando has joined #openstack-oslo | 14:47 | |
| *** salv-orlando has quit IRC | 14:51 | |
| *** yamamoto has quit IRC | 14:51 | |
| *** kiennt26 has joined #openstack-oslo | 14:52 | |
| *** daidv has joined #openstack-oslo | 14:54 | |
| bnemec | #startmeeting oslo | 15:00 |
| openstack | Meeting started Mon Mar 19 15:00:29 2018 UTC and is due to finish in 60 minutes. The chair is bnemec. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| *** openstack changes topic to " (Meeting topic: oslo)" | 15:00 | |
| openstack | The meeting name has been set to 'oslo' | 15:00 |
| bnemec | courtesy ping for amotoki, amrith, bknudson, bnemec, crushil, dansmith, dhellmann | 15:00 |
| bnemec | courtesy ping for dims, dougwig, e0ne, electrocucaracha, flaper87, garyk, gcb | 15:00 |
| bnemec | courtesy ping for GheRivero, haypo, jd__, jecarey, johnsom, jungleboyj, kgiusti | 15:00 |
| bnemec | courtesy ping for kragniz, lhx_, lifeless, lxsli, Nakato, ozamiatin, raildo | 15:00 |
| bnemec | courtesy ping for rbradfor, redrobot, rpodolyaka, sergmelikyan, sileht, spamaps, sreshetnyak | 15:00 |
| bnemec | courtesy ping for stevemar, therve, thinrichs, toabctl, viktors, zhiyan, zxy | 15:00 |
| bnemec | courtesy ping for zzzeek | 15:00 |
| e0ne | hi | 15:00 |
| crushil | \o | 15:00 |
| namnh | ]o | 15:01 |
| namnh | \o | 15:01 |
| daidv | o/ | 15:01 |
| kgiusti | o/ | 15:01 |
| ansmith | o/ | 15:01 |
| *** takedakn has quit IRC | 15:01 | |
| jungleboyj | o/ | 15:02 |
| bnemec | #topic Red flags for/from liaisons | 15:03 |
| *** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)" | 15:03 | |
| bnemec | We had an issue with the latest oslo.db release. | 15:03 |
| bnemec | #link https://bugs.launchpad.net/oslo.db/+bug/1756352 | 15:03 |
| openstack | Launchpad bug 1756352 in oslo.db "removal of testresources from setup.cfg has broken this dependency for oslo.db 4.34" [Undecided,Fix released] - Assigned to Mike Bayer (zzzeek) | 15:03 |
| zzzeek | it's going well ! | 15:03 |
| bnemec | It affected a number of projects that consumed the oslo.db test fixtures. | 15:03 |
| bnemec | Yeah, it should be fixed now. The new release with the fix was approved. | 15:03 |
| zzzeek | yep | 15:04 |
| jungleboyj | I addressed the config concern from last week. | 15:04 |
| *** yamamoto has joined #openstack-oslo | 15:04 | |
| bnemec | jungleboyj: Yeah, I saw that. Thanks! | 15:04 |
| jungleboyj | bnemec: No problem. | 15:04 |
| bnemec | So a few issues this week, but I think they've all been addressed. | 15:05 |
| bnemec | Anything else? | 15:05 |
| *** takedakn has joined #openstack-oslo | 15:05 | |
| bnemec | #topic Releases for Rocky | 15:06 |
| *** openstack changes topic to "Releases for Rocky (Meeting topic: oslo)" | 15:06 | |
| bnemec | Initial releases for anything with changes are done. | 15:06 |
| bnemec | #link https://review.openstack.org/#/c/552943/ | 15:06 |
| bnemec | Then the followup oslo.db release, as mentioned earlier. | 15:07 |
| bnemec | I held off on releasing oslo.config for now pending dhellmann's patch series, which we'll discuss in the topics. | 15:07 |
| bnemec | Once that finishes merging I'll go ahead with that release. | 15:07 |
| bnemec | There was also a request to release sphinx-feature-classification. | 15:08 |
| bnemec | It's a 0.1.0 so we're not committing to anything forever and always yet, but I thought I'd mention it in case anyone has concerns. | 15:08 |
| bnemec | If not I'll ack the request today. | 15:09 |
| bnemec | That should be it for releases. | 15:09 |
| bnemec | #topic Action items from last meeting | 15:10 |
| *** openstack changes topic to "Action items from last meeting (Meeting topic: oslo)" | 15:10 | |
| bnemec | Looks like everything was done. | 15:10 |
| jungleboyj | Woot woot | 15:10 |
| bnemec | I got up to speed on releases, scheduled the onboarding session, and lbragstad sent the email about oslo.limit. | 15:10 |
| bnemec | Nothing more on that topic then. | 15:11 |
| bnemec | #topic There are 2 more changes in the series of oslo.config changes to let an app detect whether a user has changed a config value (dhellmann) | 15:11 |
| *** openstack changes topic to "There are 2 more changes in the series of oslo.config changes to let an app detect whether a user has changed a config value (dhellmann) (Meeting topic: oslo)" | 15:11 | |
| bnemec | #link https://review.openstack.org/#/c/537400/ | 15:11 |
| bnemec | #link https://review.openstack.org/#/c/537401/ | 15:11 |
| bnemec | I believe I've +2'd both already. | 15:11 |
| bnemec | Anyone with core on oslo.config please take a look. Those are the patches I was waiting the Rocky oslo.config release on, since that series has partially merged already. | 15:12 |
| bnemec | #topic oslo.limit addition followup | 15:13 |
| *** openstack changes topic to "oslo.limit addition followup (Meeting topic: oslo)" | 15:13 | |
| bnemec | As noted, lbragstad sent an email and created a spec review for this. | 15:14 |
| * lbragstad lingers | 15:14 | |
| bnemec | #link https://review.openstack.org/552907 | 15:14 |
| bnemec | Seems like there is consensus that we want to do this. | 15:14 |
| lbragstad | wxy mentioned he wants to contribute - should i respin the patch to add him to the list of contributors? | 15:15 |
| lbragstad | (it'll clear all the +2s) | 15:15 |
| lbragstad | which is why i was hesitant | 15:15 |
| bnemec | lbragstad: Sure, for an addition like that we don't need to wait for everyone to vote again. | 15:15 |
| lbragstad | ack | 15:15 |
| *** kiennt26 has quit IRC | 15:15 | |
| bnemec | If you want to push a new PS I can go ahead and merge it. I don't anticipate any further feedback beyond what you've already gotten on the ML and spec. | 15:15 |
| *** takedakn has quit IRC | 15:16 | |
| openstackgerrit | Lance Bragstad proposed openstack/oslo-specs master: Propose specification for oslo.limit library https://review.openstack.org/552907 | 15:17 |
| lbragstad | ^ done | 15:17 |
| bnemec | So if anyone has strong objections to the new oslo.limit library contact me ASAP. | 15:17 |
| bnemec | Otherwise I'll plan to merge that today. | 15:17 |
| lbragstad | sweet | 15:17 |
| lbragstad | i appreciate all the help with the process | 15:17 |
| lbragstad | i assume once that is done, i can move forward with the whole project creation bits and getting the repository created | 15:18 |
| bnemec | I'm excited to finally have common quota code for OpenStack. :-) | 15:18 |
| lbragstad | hah you and me both bnemec | 15:18 |
| bnemec | Yeah, I've actually got a todo item for myself to follow up on whether our new library process needs some updating. | 15:18 |
| bnemec | AFAIK, the only place we document it is the graduation wiki page, which isn't entirely relevant now that incubator is gone. | 15:18 |
| openstackgerrit | Merged openstack/oslo.log master: Increase sleep time in testsuite to make it more robust https://review.openstack.org/554180 | 15:18 |
| lbragstad | sure - i did notice that | 15:19 |
| lbragstad | as i was reading the doc | 15:19 |
| bnemec | So unless there is a document I don't know about we need to make some updates here. | 15:19 |
| bnemec | This is probably a good opportunity to clear some of the graduation cruft. | 15:20 |
| bnemec | So I guess I have a couple of followup items from this. | 15:20 |
| bnemec | #action bnemec to approve oslo.limit spec | 15:20 |
| bnemec | #action bnemec to investigate/update new library process docs | 15:20 |
| bnemec | #topic Configuration change handling over releases (namnh) | 15:21 |
| *** openstack changes topic to "Configuration change handling over releases (namnh) (Meeting topic: oslo)" | 15:21 | |
| bnemec | I'm afraid I haven't revisited this recently. | 15:22 |
| namnh | :) | 15:22 |
| bnemec | namnh: The floor is yours | 15:22 |
| bnemec | #link https://review.openstack.org/#/c/520043 | 15:22 |
| bnemec | #link https://review.openstack.org/#/c/526314 | 15:22 |
| namnh | yes, I and daidv are doing this feature | 15:22 |
| namnh | could you please review the spec and coding for me. | 15:23 |
| bnemec | Okay, so the main thing at this point is just reviews? | 15:24 |
| namnh | yead, and hope you can give us some advices because we have a problem with a option called "transport_url" | 15:25 |
| bnemec | I think at the PTG we agreed that the initial version of this tooling might not handle complex deprecations like transport_url. | 15:26 |
| namnh | basically, the feature can update from old-config to new-config. | 15:26 |
| namnh | oh, really? | 15:26 |
| bnemec | Basically we would raise a warning when we hit something like that and tell the user they need to handle it manually. | 15:26 |
| namnh | i did not join the PTG | 15:26 |
| bnemec | As a followup improvement, we might be able to provide some sort of templating feature or in-project hooks to handle complex migrations. | 15:27 |
| bnemec | But in the interest of getting to a usable tool quickly, we decided to focus on the simpler ones that will cover 95% of the deprecated opt cases first. | 15:28 |
| bnemec | Basically create the tool, prove that it is useful and works, then iterate on it to handle the harder stuff. | 15:28 |
| bnemec | Does that work for you? | 15:29 |
| namnh | yead, as i wrote the spec, we have four cases which need to solve. For now, the tool can solve 3 cases. | 15:29 |
| namnh | yes, it does | 15:30 |
| bnemec | For the initial version of the tool that might be enough. | 15:30 |
| bnemec | I did mention this in my (long) PTG summary email. | 15:30 |
| bnemec | #link http://lists.openstack.org/pipermail/openstack-dev/2018-March/128055.html | 15:30 |
| bnemec | Under the "automatic configuration migration on upgrade" heading. | 15:30 |
| namnh | thanks, | 15:30 |
| bnemec | I'll try to leave some further comments on the spec. | 15:31 |
| namnh | here is a patch set, I tried testing with cinder: https://review.openstack.org/#/c/549672/ | 15:31 |
| namnh | bnemec: thanks in advange. | 15:31 |
| bnemec | #action review the config migration spec: https://review.openstack.org/#/c/520043/ | 15:32 |
| namnh | great :) | 15:32 |
| daidv | Nice, so we will start with a simple tool first and solve the transport_url and dynamic section case after | 15:32 |
| bnemec | Exactly | 15:33 |
| bnemec | It was kind of a theme of the PTG for Oslo. | 15:33 |
| namnh | :)) | 15:33 |
| bnemec | Write a simpler version of a tool that handles most of the cases, and then solve the harder ones later. | 15:33 |
| namnh | i will update a doc how to setup an environment for testing | 15:34 |
| bnemec | That would be helpful, thanks. | 15:34 |
| bnemec | #action namnh to update config migration testing doc | 15:34 |
| bnemec | Okay, anything else on that topic? | 15:35 |
| namnh | ok, it is enough for me | 15:36 |
| bnemec | Cool, moving on then. | 15:36 |
| bnemec | #topic MultiConfigParser removal | 15:36 |
| *** openstack changes topic to "MultiConfigParser removal (Meeting topic: oslo)" | 15:36 | |
| bnemec | This came up when I was reviewing dhellmann's oslo.config changes. | 15:36 |
| bnemec | They affected MultiConfigParser, but it's a deprecated class that was supposed to have been removed several releases ago. | 15:37 |
| bnemec | I took a look at what it would take to remove it, and found only two projects still using it in OpenStack. | 15:37 |
| dhellmann | it looks like networking-cisco still uses it :-( | 15:37 |
| dhellmann | oh, which is the other? | 15:37 |
| bnemec | The other I already fixed. | 15:37 |
| dhellmann | ah :-) | 15:37 |
| bnemec | They had deprecated the feature that was using it and it was marked for removal in Pike. | 15:38 |
| * dhellmann doesn't even remember what MultiConfigParser does | 15:38 | |
| bnemec | The thing I'm struggling with is that the replacement functionality in the docstring is all private to oslo.config. | 15:38 |
| bnemec | https://github.com/openstack/oslo.config/blob/master/oslo_config/cfg.py#L2017 | 15:39 |
| bnemec | So it's not really suitable for external consumers. :-/ | 15:39 |
| dhellmann | is that saying the class reads multiple config files? or that it handles options with multiple values? | 15:39 |
| dhellmann | because I think ConfigOpts does both of those now | 15:39 |
| dhellmann | I think they can just replace their use of this class with ConfigOpts? | 15:40 |
| bnemec | Possibly? I'm also not terribly familiar with this which is why I added it to the agenda. :-) | 15:40 |
| dhellmann | yeah, I'm not either | 15:41 |
| dhellmann | ah, they're using the parser but not ConfigOpts | 15:42 |
| dhellmann | so they aren't registering options, but they're using the same code to parse a bunch of files | 15:43 |
| dhellmann | they could probably just use the built-in module configparser | 15:43 |
| bnemec | Yeah, and the patch related to http://git.openstack.org/cgit/openstack/networking-cisco/tree/releasenotes/notes/elim_MultiConfigParser_fr_nexus-6a50c543949d1ca4.yaml was quite extensive. | 15:44 |
| bnemec | I'm _hoping_ there's a simpler way for the other two uses in that project. | 15:44 |
| dhellmann | well, MultiConfigParser doesn't use any private implementation details, so worst case we could just move that class | 15:45 |
| bnemec | Ah, that's a thought. | 15:45 |
| dhellmann | looking at how they're using it, that might be "best case" rather than "worst" | 15:46 |
| bnemec | Yeah, that might be easiest for everyone. | 15:46 |
| bnemec | Obviously that code isn't changing much. | 15:46 |
| dhellmann | and if they end up not wanting it, they know best what that code is doing and we can work with them to figure out how to do it using the classes we do support | 15:47 |
| dhellmann | was this marked for removal because we didn't want anyone to use it? | 15:47 |
| dhellmann | or because we were simplifying the config library api? | 15:47 |
| bnemec | I have no idea. It's been deprecated for years. | 15:47 |
| dhellmann | too bad we didn't leave a note about the reason :-/ | 15:48 |
| dhellmann | well, I think the simplest thing to do is going to be to propose a patch adding that class to the networking-cisco repo | 15:50 |
| bnemec | Sounds like the functionality was moved into the _Namespace class, which made the separate parser redundant. | 15:50 |
| bnemec | https://github.com/openstack/oslo.config/commit/c663acebc697e92f086684b2ee87546fba57cb7d | 15:50 |
| bnemec | Yeah, that sounds good | 15:50 |
| dhellmann | aha | 15:50 |
| dhellmann | this stuff is likely to be refactored again with the driver work | 15:50 |
| bnemec | #action move MultiConfigParser to networking-cisco | 15:50 |
| bnemec | Yeah, with all the config work scheduled I figured it would be good to get rid of it now so we don't end up duplicating work. | 15:51 |
| dhellmann | ++ | 15:51 |
| dhellmann | are you going to do that? or are you looking for a volunteer? | 15:52 |
| bnemec | I can do it, although if anyone else has a burning desire to I won't stop them. :-) | 15:52 |
| bnemec | Hopefully moving the class won't be too much work though. | 15:53 |
| dhellmann | yeah, it should be pretty straightforward to make a new module to hold it | 15:53 |
| bnemec | Okay, we can come back to this next week if it turns out to be harder than we think. | 15:55 |
| bnemec | #topic Open Discussion | 15:55 |
| *** openstack changes topic to "Open Discussion (Meeting topic: oslo)" | 15:55 | |
| bnemec | That was it for the agenda. Anything else in the last 5 minutes? | 15:55 |
| dhellmann | I have that requirements sync work coming up soon | 15:56 |
| dhellmann | let me find that email link | 15:56 |
| dhellmann | #link http://lists.openstack.org/pipermail/openstack-dev/2018-March/128352.html | 15:56 |
| bnemec | Yeah, I did read that whole thing. I think there were a couple more replies since then though that I haven't had a chance to look at. | 15:56 |
| dhellmann | I just want to make sure that folks are aware, since it will have an effect on oslo | 15:57 |
| dhellmann | I'm going to give the requirements team another day or two to respond before I start writing any patches | 15:57 |
| bnemec | It should be mostly good for Oslo though, right? | 15:57 |
| dhellmann | oh, yes, in a good way | 15:57 |
| dhellmann | fewer releases just for requirements updates | 15:58 |
| bnemec | Projects that want to consume a new Oslo feature don't have to wait for a g-r minimum bump. | 15:58 |
| dhellmann | and that, too | 15:58 |
| dhellmann | they may still have to wait for the constraint update | 15:58 |
| dhellmann | but in general I expect it to make things go more smoothly | 15:58 |
| bnemec | Yeah, that's true. | 15:58 |
| bnemec | It sounds good to me, but my experience with requirements stuff has been "here be dragons" :-) | 15:59 |
| *** gyankum has joined #openstack-oslo | 15:59 | |
| dhellmann | that's pretty true, although the constraints feature in pip is a dragon slayer | 16:00 |
| *** gyankum has quit IRC | 16:00 | |
| bnemec | Yeah, it has been a lot better since we started using that. | 16:00 |
| bnemec | Okay, sounds good and we're at time so I'll let everyone go. | 16:01 |
| bnemec | Always available for further discussions in this channel though. | 16:01 |
| bnemec | Thanks! | 16:01 |
| bnemec | #endmeeting | 16:01 |
| *** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo" | 16:01 | |
| openstack | Meeting ended Mon Mar 19 16:01:40 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/oslo/2018/oslo.2018-03-19-15.00.html | 16:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/oslo/2018/oslo.2018-03-19-15.00.txt | 16:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/oslo/2018/oslo.2018-03-19-15.00.log.html | 16:01 |
| *** daidv has quit IRC | 16:01 | |
| *** namnh has quit IRC | 16:02 | |
| *** salv-orlando has joined #openstack-oslo | 16:05 | |
| *** pcaruana has quit IRC | 16:07 | |
| *** salv-orlando has quit IRC | 16:09 | |
| *** salv-orlando has joined #openstack-oslo | 16:18 | |
| *** crushil has quit IRC | 16:20 | |
| *** yamamoto has quit IRC | 16:23 | |
| openstackgerrit | Merged openstack/oslo-specs master: Propose specification for oslo.limit library https://review.openstack.org/552907 | 16:30 |
| *** msimonin has quit IRC | 16:30 | |
| *** e0ne has quit IRC | 16:48 | |
| *** AlexeyAbashkin has quit IRC | 16:51 | |
| *** shardy_mtg has quit IRC | 16:52 | |
| *** msimonin has joined #openstack-oslo | 16:59 | |
| *** crushil has joined #openstack-oslo | 17:07 | |
| *** shardy_mtg has joined #openstack-oslo | 17:08 | |
| *** dikonoor has joined #openstack-oslo | 17:09 | |
| dikonoor | dhellmann:Hi | 17:13 |
| dikonoor | I am looking for some information on oslo privsep on whether it's tightly coupled to sudo for providing escalted privileges | 17:14 |
| dikonoor | fungi and I were having some discussions in the openstack-security channel and we thought this might be the right channel to bring this up in. | 17:15 |
| dikonoor | the documentation of https://docs.openstack.org/oslo.privsep/latest/ is sparse and doesn't really give enough information on this | 17:16 |
| openstackgerrit | Merged openstack/oslo.config master: Update links in README https://review.openstack.org/551114 | 17:16 |
| bnemec | dikonoor: It is not tightly coupled to sudo. There are a couple of options for starting the privsep daemon: https://docs.openstack.org/oslo.privsep/latest/reference/api/oslo_privsep.daemon.html | 17:18 |
| bnemec | (one of them is sudo, however) | 17:18 |
| fungi | do we have any good design/model documentation for privsep's implementation, by any chance? | 17:19 |
| dikonoor | bnemec: Thanks for pointing me to this link. I am trying to figure our if OpenStack can be used with these newer privilege mgmt softwares like PowerBroker and BOKS (from a company named FOX) etc | 17:20 |
| *** msimonin has quit IRC | 17:20 | |
| *** msimonin has joined #openstack-oslo | 17:20 | |
| fungi | i'm skimming the code and it's not immediately apparent whether the sudo integration in the daemon and context modules are just for initialization or part of the backend operation | 17:20 |
| bnemec | fungi: Probably just http://specs.openstack.org/openstack/oslo-specs/specs/liberty/privsep.html | 17:21 |
| fungi | thanks! | 17:21 |
| *** d0ugal has quit IRC | 17:21 | |
| dikonoor | I have not used these softwares myself but from what I understand these softwares have more advanced features compared to sudo | 17:22 |
| bnemec | Unfortunately gus is no longer working on OpenStack so we've lost most of our expertise on this. :-( | 17:22 |
| bnemec | dikonoor: It may be possible to use those for providing the necessary privileges to the daemon, but I'm not familiar enough with them to say for sure. | 17:22 |
| bnemec | The basic requirement for privsep is that you need _some_ method of starting the daemon that will give it the necessary capabilities to do the privileged operations it needs to. | 17:23 |
| bnemec | I wouldn't expect it to particularly care how you do that though. | 17:23 |
| dikonoor | bnmec: Ok..meaning it doesn't necessarily need to be sudo | 17:24 |
| fungi | yeah, looking at this, oslo.privsep is itself an alternative to running shell commands under sudo | 17:24 |
| bnemec | Right | 17:25 |
| fungi | so oslo.privsep's daemon is basically a configurable command broker running as another/root user and taking requests to run things and evaluating them against policy before executing them | 17:25 |
| fungi | but it's not using sudo to execute them | 17:26 |
| fungi | (necessarily) | 17:26 |
| *** shardy_mtg has quit IRC | 17:26 | |
| bnemec | Ideally I don't think it should use sudo. | 17:26 |
| bnemec | Eliminating the sudo calls and complicated rootwrap configuration was the main point of the library. | 17:26 |
| dikonoor | bnemec: ok that sounds promising | 17:27 |
| bnemec | The daemon should already have the necessary capabilities so it doesn't need sudo. | 17:27 |
| bnemec | dikonoor: Cool | 17:27 |
| dikonoor | thanks fungi bnemec. Let me explore this further | 17:30 |
| *** jbadiapa_ has joined #openstack-oslo | 17:37 | |
| *** jbadiapa has quit IRC | 17:40 | |
| *** chhagarw has quit IRC | 17:45 | |
| *** dikonoor has quit IRC | 17:45 | |
| lbragstad | bnemec: i should have mentioned this earlier - but your approval is going to be needed here, too https://review.openstack.org/#/c/550491/ | 18:05 |
| *** jbadiapa_ is now known as jbadiapa | 18:05 | |
| bnemec | lbragstad: Ah, and I need to change my vote on https://review.openstack.org/#/c/550496/ too. Thanks for the reminder. | 18:06 |
| lbragstad | no problem | 18:07 |
| *** tesseract has quit IRC | 18:07 | |
| bnemec | lbragstad: Left a comment on https://review.openstack.org/#/c/550496 | 18:11 |
| *** sambetts is now known as sambetts|afk | 18:12 | |
| lbragstad | bnemec: responded | 18:13 |
| lbragstad | good question | 18:13 |
| bnemec | dhellmann: Maybe you have more experience with this. Do we generally mark pre-1.0 libraries as stable:follows-policy? | 18:17 |
| bnemec | Related to the discussion in https://review.openstack.org/#/c/550496/1/reference/projects.yaml | 18:17 |
| dhellmann | bnemec : well, by definition a version number that starts with 0 is already declared as unstable | 18:17 |
| dhellmann | meaning the API is subject to change | 18:17 |
| dhellmann | that's why we encourage project teams to move to 1.0 as quickly as is reasonable | 18:17 |
| bnemec | Yeah, it sounds like the plan is to have a 1.0 by the end of Rocky. | 18:18 |
| bnemec | Assuming that happens, we would need to follow the backport policy for the Rocky branch. | 18:19 |
| dhellmann | you can backport without claiming the tag | 18:20 |
| *** AlexeyAbashkin has joined #openstack-oslo | 18:20 | |
| dhellmann | I would wait and add the tag after hitting 1.0 | 18:20 |
| dhellmann | that gives us time to fiddle with the API, and it also gives some track-record for the stable team to look at to verify that the tag rules are being honored | 18:20 |
| bnemec | Okay, WFM. | 18:20 |
| dhellmann | though it's not the end of the world if people want to go ahead and put the tag on to start | 18:21 |
| bnemec | I was the only one asking and I'm not that hung up on it. | 18:21 |
| *** vdrok has joined #openstack-oslo | 18:22 | |
| dhellmann | ok :-) | 18:22 |
| vdrok | hello folks, now seeing the following on pike in nova http://logs.openstack.org/69/543569/4/check/openstack-tox-py35/c706b82/job-output.txt#_2018-03-19_16_23_53_920564, and suspect that there is a problem with setuptools 39 | 18:24 |
| *** AlexeyAbashkin has quit IRC | 18:24 | |
| vdrok | at least downgrading to 38 helps | 18:24 |
| *** dave-mccowan has joined #openstack-oslo | 18:28 | |
| dhellmann | vdrok : I think I remember someone mentioning a similar issue but just to make sure you probably want to raise that in #openstack-infra | 18:30 |
| bnemec | vdrok: Hmm, seeing the same thing on the oslo.utils pike branch. | 18:30 |
| bnemec | Oddly enough it's not happening on master. | 18:31 |
| vdrok | dhellmann: I did write it there as well :) | 18:31 |
| dhellmann | vdrok : ah, maybe that was you :-) | 18:31 |
| dhellmann | bnemec : I think they got the constraint update into master but not the stable branches? Oh, no, it was smcginnis who I saw mention it on the ML | 18:31 |
| smcginnis | Yep, working on a patch dance right now to sort out pike. | 18:33 |
| openstackgerrit | Doug Hellmann proposed openstack/oslo.config master: WIP: add config_source option https://review.openstack.org/554316 | 18:33 |
| bnemec | Oh, I see. https://review.openstack.org/#/c/554053/ | 18:34 |
| smcginnis | Basically, pip made a backwards incompatible change in a bugfix release that impacts the ryu package. Oslo.utils uses that for some neutron stuff I think. So anything touching that code is blowing up until we get to a newer ryu. And a backport of version changes. | 18:34 |
| smcginnis | bnemec: Yep, that's the one. | 18:34 |
| smcginnis | But I need to disable some tests to get everything merged right. | 18:35 |
| smcginnis | So working on that now. | 18:35 |
| *** e0ne has joined #openstack-oslo | 18:36 | |
| bnemec | smcginnis: Okay, but oslo.utils passed ci, so there's nothing blocking that fix? | 18:45 |
| smcginnis | I have a depends on there to test the requirements bump. | 18:45 |
| smcginnis | So https://review.openstack.org/#/c/554318/ makes that job non-voting, get in 554053, then get in 554270, then revert the non-voting change. | 18:45 |
| smcginnis | "That job" being legacy-tempest-dsvm-neutron-src which fails without the requirements bump. | 18:46 |
| bnemec | smcginnis: Ah, got it. I skimmed over the depends-on. :-) | 18:47 |
| smcginnis | bnemec: I always miss those too. ;) | 18:48 |
| smcginnis | OK, all patches redone to have the appropriate dependency chain. | 18:48 |
| *** openstackgerrit has quit IRC | 18:48 | |
| smcginnis | https://review.openstack.org/#/q/status:open+topic:pip/5081 | 18:51 |
| *** openstackgerrit has joined #openstack-oslo | 19:05 | |
| openstackgerrit | Merged openstack/automaton master: Updated from global requirements https://review.openstack.org/551451 | 19:05 |
| * bnemec needs to step out for a bit to eat | 19:07 | |
| *** AlexeyAbashkin has joined #openstack-oslo | 19:20 | |
| *** AlexeyAbashkin has quit IRC | 19:24 | |
| *** crushil has quit IRC | 19:32 | |
| *** e0ne has quit IRC | 19:34 | |
| smcginnis | Non-voting job change went through. This should (knock on wood) pass recheck - https://review.openstack.org/#/c/554053/ | 19:37 |
| *** msimonin1 has joined #openstack-oslo | 19:46 | |
| *** openstackgerrit has quit IRC | 19:48 | |
| *** msimonin has quit IRC | 19:48 | |
| *** AlexeyAbashkin has joined #openstack-oslo | 20:21 | |
| *** AlexeyAbashkin has quit IRC | 20:26 | |
| *** openstackgerrit has joined #openstack-oslo | 20:35 | |
| openstackgerrit | Doug Hellmann proposed openstack/oslotest master: make the CaptureOutput fixture easier to control https://review.openstack.org/534447 | 20:35 |
| smcginnis | dhellmann or others - https://review.openstack.org/554053 should pass since the remaining job is non-voting. | 20:38 |
| smcginnis | I realized we also need to get this into stable/ocata before we make the job voting again, so I've already proposed that backport with https://review.openstack.org/554338 | 20:38 |
| smcginnis | And we'll need to get official releases for oslo.utils for these stable branches too. | 20:39 |
| dhellmann | good thing we know a release manager or two | 20:40 |
| smcginnis | ;) | 20:41 |
| *** ihrachys has joined #openstack-oslo | 20:45 | |
| *** kgiusti has left #openstack-oslo | 20:46 | |
| *** ansmith has quit IRC | 21:05 | |
| *** e0ne has joined #openstack-oslo | 21:09 | |
| *** raildo has quit IRC | 21:30 | |
| smcginnis | bnemec: Would you like to submit the stable/pike release patch for oslo.util, or would you rather I do that? | 21:54 |
| bnemec | smcginnis: If I send it you can +2, right? | 21:55 |
| smcginnis | bnemec: Yep! | 21:55 |
| bnemec | Okay, I'll do it then. | 21:55 |
| smcginnis | bnemec: Great, I will watch for that. Hopefully we can get the rest of this moving. | 21:55 |
| bnemec | I need to go afk shortly though, so if ocata isn't in by then you might have to do that one. | 21:55 |
| smcginnis | bnemec: Sounds good. | 21:56 |
| *** spilla has quit IRC | 21:56 | |
| bnemec | Changes between 3.36.0 and 36afeff | 21:58 |
| bnemec | * 36afeff 2018-03-15 07:51:35 +0000 Updated from global requirements | 21:58 |
| bnemec | Hmm | 21:58 |
| smcginnis | I don't see 36afeff in the commits. | 21:59 |
| * bnemec doesn't see it yet | 21:59 | |
| bnemec | http://git.openstack.org/cgit/openstack/oslo.utils/log/ | 21:59 |
| smcginnis | bnemec: Oh, make sure you're comparing from last stable/pike release tag. | 22:00 |
| smcginnis | bnemec: https://github.com/openstack/oslo.utils/compare/3.36.0...stable/pike | 22:00 |
| bnemec | smcginnis: Oh, PEBKAC | 22:00 |
| bnemec | s|pike|stable/pike| | 22:00 |
| *** ansmith has joined #openstack-oslo | 22:00 | |
| bnemec | Now I see it. | 22:00 |
| bnemec | smcginnis: https://review.openstack.org/#/c/554364 | 22:02 |
| smcginnis | bnemec: Great, looks right to me. Should pass the validation tests. | 22:03 |
| bnemec | ocata should merge any minute. I'll try to get that proposed as well. | 22:03 |
| bnemec | smcginnis: https://review.openstack.org/#/c/554366 | 22:06 |
| bnemec | smcginnis: I need to run now. That should unblock everything from an oslo perspective, right? | 22:07 |
| smcginnis | bnemec: Yep, we should be good. Thanks! | 22:07 |
| bnemec | Cool, catch you later | 22:08 |
| smcginnis | bnemec: Have a good one. | 22:08 |
| *** e0ne has quit IRC | 22:12 | |
| *** bobh has quit IRC | 22:19 | |
| *** AlexeyAbashkin has joined #openstack-oslo | 22:20 | |
| *** AlexeyAbashkin has quit IRC | 22:25 | |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.utils master: Add bindep.txt file to prevent fallback to generic list https://review.openstack.org/554371 | 22:28 |
| *** rcernin has joined #openstack-oslo | 22:34 | |
| *** edmondsw has quit IRC | 22:49 | |
| *** msimonin1 has quit IRC | 23:18 | |
| *** msimonin has joined #openstack-oslo | 23:18 | |
| *** msimonin has quit IRC | 23:18 | |
| *** AlexeyAbashkin has joined #openstack-oslo | 23:20 | |
| *** janzian has quit IRC | 23:21 | |
| *** AlexeyAbashkin has quit IRC | 23:24 | |
| *** yamamoto has joined #openstack-oslo | 23:27 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!