*** lifeless has quit IRC | 01:06 | |
*** lifeless has joined #openstack-oslo | 01:11 | |
*** edmondsw has quit IRC | 01:47 | |
*** lifeless has quit IRC | 01:47 | |
*** lifeless has joined #openstack-oslo | 01:49 | |
*** bobh has quit IRC | 02:14 | |
*** lifeless has quit IRC | 02:19 | |
*** lifeless has joined #openstack-oslo | 02:20 | |
*** bobh has joined #openstack-oslo | 02:29 | |
*** yassine has joined #openstack-oslo | 02:30 | |
*** yassine is now known as Guest63393 | 02:31 | |
*** bobh has quit IRC | 02:38 | |
*** bobh has joined #openstack-oslo | 02:51 | |
*** Guest63393 has quit IRC | 02:53 | |
*** edmondsw has joined #openstack-oslo | 02:54 | |
*** bobh has quit IRC | 02:56 | |
*** edmondsw has quit IRC | 02:59 | |
*** bobh has joined #openstack-oslo | 03:00 | |
*** bobh has quit IRC | 03:05 | |
*** Guest63393 has joined #openstack-oslo | 03:07 | |
*** bobh has joined #openstack-oslo | 03:10 | |
*** bobh has quit IRC | 03:16 | |
*** bobh has joined #openstack-oslo | 03:20 | |
*** bobh has quit IRC | 03:25 | |
*** bobh has joined #openstack-oslo | 03:32 | |
*** bobh has quit IRC | 03:36 | |
*** lifeless_ has joined #openstack-oslo | 03:46 | |
*** lifeless has quit IRC | 03:46 | |
*** lifeless has joined #openstack-oslo | 03:52 | |
*** bobh has joined #openstack-oslo | 03:52 | |
*** lifeless_ has quit IRC | 03:52 | |
*** bobh has quit IRC | 03:57 | |
*** Guest63393 has quit IRC | 04:05 | |
*** Guest63393 has joined #openstack-oslo | 04:10 | |
*** links has joined #openstack-oslo | 04:14 | |
*** bobh has joined #openstack-oslo | 04:19 | |
*** jbadiapa_ has joined #openstack-oslo | 04:20 | |
*** bobh has quit IRC | 04:24 | |
*** yamamoto has joined #openstack-oslo | 04:28 | |
*** bobh has joined #openstack-oslo | 04:30 | |
*** bobh has quit IRC | 04:35 | |
*** bobh has joined #openstack-oslo | 04:41 | |
*** Guest63393 has quit IRC | 04:43 | |
*** bobh has quit IRC | 04:46 | |
*** Guest63393 has joined #openstack-oslo | 04:48 | |
*** bobh has joined #openstack-oslo | 04:56 | |
*** bobh has quit IRC | 05:01 | |
*** bobh has joined #openstack-oslo | 05:01 | |
*** bobh has quit IRC | 05:06 | |
*** bobh has joined #openstack-oslo | 05:16 | |
*** bobh has quit IRC | 05:20 | |
*** bobh has joined #openstack-oslo | 05:22 | |
*** bobh has quit IRC | 05:27 | |
*** e0ne has joined #openstack-oslo | 05:28 | |
*** e0ne has quit IRC | 05:33 | |
*** bobh has joined #openstack-oslo | 05:34 | |
*** bobh has quit IRC | 05:39 | |
*** bobh has joined #openstack-oslo | 05:43 | |
*** bobh has quit IRC | 05:48 | |
*** bobh has joined #openstack-oslo | 05:55 | |
*** bobh has quit IRC | 06:00 | |
*** AlexeyAbashkin has joined #openstack-oslo | 06:04 | |
*** bobh has joined #openstack-oslo | 06:05 | |
*** bobh has quit IRC | 06:10 | |
*** bobh has joined #openstack-oslo | 06:13 | |
*** bobh has quit IRC | 06:18 | |
*** bobh has joined #openstack-oslo | 06:22 | |
*** pcaruana has joined #openstack-oslo | 06:26 | |
*** bobh has quit IRC | 06:27 | |
*** bobh has joined #openstack-oslo | 06:32 | |
*** yamamoto has quit IRC | 06:35 | |
*** Guest63393 has quit IRC | 06:36 | |
*** bobh has quit IRC | 06:36 | |
*** yamamoto has joined #openstack-oslo | 06:41 | |
*** Guest63393 has joined #openstack-oslo | 06:51 | |
*** bobh has joined #openstack-oslo | 06:54 | |
*** bobh has quit IRC | 06:59 | |
*** bobh has joined #openstack-oslo | 07:04 | |
*** bobh has quit IRC | 07:09 | |
*** jaosorior has joined #openstack-oslo | 07:10 | |
*** rcernin has quit IRC | 07:10 | |
*** bobh has joined #openstack-oslo | 07:21 | |
*** pooja_jadhav has joined #openstack-oslo | 07:25 | |
*** bobh has quit IRC | 07:26 | |
*** bobh has joined #openstack-oslo | 07:34 | |
*** bobh has quit IRC | 07:39 | |
*** tosky has joined #openstack-oslo | 07:40 | |
*** bobh has joined #openstack-oslo | 07:49 | |
*** yamamoto has quit IRC | 07:50 | |
*** links has quit IRC | 07:50 | |
*** yamamoto has joined #openstack-oslo | 07:51 | |
*** yamamoto has quit IRC | 07:54 | |
*** bobh has quit IRC | 07:54 | |
*** yamamoto has joined #openstack-oslo | 07:54 | |
*** yamamoto has quit IRC | 07:54 | |
*** pblaho has joined #openstack-oslo | 07:55 | |
*** yamamoto has joined #openstack-oslo | 07:56 | |
*** bobh has joined #openstack-oslo | 08:05 | |
*** lifeless_ has joined #openstack-oslo | 08:06 | |
*** lifeless has quit IRC | 08:07 | |
*** namnh has joined #openstack-oslo | 08:08 | |
*** bobh has quit IRC | 08:10 | |
*** shardy has joined #openstack-oslo | 08:11 | |
*** bobh has joined #openstack-oslo | 08:13 | |
openstackgerrit | Nam Nguyen Hoai proposed openstack/oslo.config master: Handle config mapping changes https://review.openstack.org/526314 | 08:16 |
---|---|---|
*** bobh has quit IRC | 08:17 | |
*** bobh has joined #openstack-oslo | 08:23 | |
*** bobh has quit IRC | 08:28 | |
*** d0ugal has joined #openstack-oslo | 08:28 | |
*** bobh has joined #openstack-oslo | 08:34 | |
*** bobh has quit IRC | 08:39 | |
*** lifeless has joined #openstack-oslo | 08:46 | |
*** lifeless_ has quit IRC | 08:47 | |
*** bobh has joined #openstack-oslo | 09:02 | |
*** salv-orlando has joined #openstack-oslo | 09:05 | |
*** bobh has quit IRC | 09:07 | |
*** e0ne has joined #openstack-oslo | 09:12 | |
*** salv-orlando has quit IRC | 09:18 | |
*** bobh has joined #openstack-oslo | 09:22 | |
*** bobh has quit IRC | 09:27 | |
*** giblet is now known as gibi | 09:31 | |
*** bobh has joined #openstack-oslo | 09:35 | |
*** bobh has quit IRC | 09:40 | |
*** bobh has joined #openstack-oslo | 09:42 | |
*** pbourke has quit IRC | 09:43 | |
*** pbourke has joined #openstack-oslo | 09:43 | |
*** finucannot is now known as stephenfin | 09:45 | |
*** bobh has quit IRC | 09:47 | |
*** bobh has joined #openstack-oslo | 09:59 | |
*** bobh has quit IRC | 10:05 | |
*** bobh has joined #openstack-oslo | 10:12 | |
*** bobh has quit IRC | 10:17 | |
*** salv-orlando has joined #openstack-oslo | 10:18 | |
*** bobh has joined #openstack-oslo | 10:24 | |
*** salv-orlando has quit IRC | 10:24 | |
*** bobh has quit IRC | 10:28 | |
openstackgerrit | Alessandro Nesta proposed openstack/oslo.limit master: fix tox python3 overrides https://review.openstack.org/573954 | 10:29 |
*** namnh has quit IRC | 10:34 | |
*** bobh has joined #openstack-oslo | 10:35 | |
*** nicolasbock has joined #openstack-oslo | 10:38 | |
*** bobh has quit IRC | 10:40 | |
*** bobh has joined #openstack-oslo | 10:43 | |
*** jrollen is now known as jroll | 10:44 | |
*** bobh has quit IRC | 10:48 | |
openstackgerrit | Duc Nguyen proposed openstack/osprofiler master: Put 'db' parameter back and provide a deprecation warning before remove https://review.openstack.org/574207 | 10:48 |
*** bobh has joined #openstack-oslo | 10:51 | |
*** bobh has quit IRC | 10:56 | |
*** bobh has joined #openstack-oslo | 11:00 | |
*** zzzeek_ has quit IRC | 11:01 | |
*** zzzeek has joined #openstack-oslo | 11:03 | |
*** bobh has quit IRC | 11:05 | |
*** sileht has left #openstack-oslo | 11:07 | |
*** bobh has joined #openstack-oslo | 11:10 | |
openstackgerrit | Moisés Guimarães de Medeiros proposed openstack/oslo.config master: Add config_source option https://review.openstack.org/554316 | 11:11 |
*** lucasagomes is now known as lucas-hungry | 11:15 | |
*** bobh has quit IRC | 11:15 | |
*** lucas-hungry is now known as lucasagomes | 11:15 | |
*** jroll has quit IRC | 11:18 | |
*** jroll has joined #openstack-oslo | 11:18 | |
*** salv-orlando has joined #openstack-oslo | 11:20 | |
*** bobh has joined #openstack-oslo | 11:20 | |
*** salv-orlando has quit IRC | 11:24 | |
*** bobh has quit IRC | 11:25 | |
*** bobh has joined #openstack-oslo | 11:31 | |
*** bobh has quit IRC | 11:36 | |
*** shardy is now known as shardy_lunch | 11:37 | |
*** e0ne has quit IRC | 11:38 | |
*** bobh has joined #openstack-oslo | 11:40 | |
*** bobh has quit IRC | 11:45 | |
*** lifeless has quit IRC | 11:45 | |
*** lifeless_ has joined #openstack-oslo | 11:45 | |
*** e0ne has joined #openstack-oslo | 11:48 | |
*** bobh has joined #openstack-oslo | 12:00 | |
*** bobh has quit IRC | 12:05 | |
*** edmondsw has joined #openstack-oslo | 12:12 | |
*** bobh has joined #openstack-oslo | 12:14 | |
*** shardy_lunch is now known as shardy | 12:16 | |
*** bobh has quit IRC | 12:19 | |
*** salv-orlando has joined #openstack-oslo | 12:20 | |
*** bobh has joined #openstack-oslo | 12:22 | |
*** salv-orlando has quit IRC | 12:26 | |
*** bobh has quit IRC | 12:27 | |
*** raildo has joined #openstack-oslo | 12:34 | |
*** bobh has joined #openstack-oslo | 12:34 | |
*** pcaruana has quit IRC | 12:39 | |
*** bobh has quit IRC | 12:39 | |
*** ansmith has joined #openstack-oslo | 12:41 | |
*** e0ne has quit IRC | 12:42 | |
*** AlexeyAbashkin has quit IRC | 12:42 | |
*** e0ne has joined #openstack-oslo | 12:47 | |
*** kgiusti has joined #openstack-oslo | 12:53 | |
*** mago_ has joined #openstack-oslo | 12:58 | |
*** salv-orlando has joined #openstack-oslo | 13:22 | |
*** salv-orlando has quit IRC | 13:27 | |
*** pcaruana has joined #openstack-oslo | 13:32 | |
*** janzian has joined #openstack-oslo | 13:52 | |
*** shardy has quit IRC | 13:53 | |
*** shardy has joined #openstack-oslo | 13:53 | |
*** bobh has joined #openstack-oslo | 14:00 | |
*** ianychoi has quit IRC | 14:02 | |
*** AlexeyAbashkin has joined #openstack-oslo | 14:04 | |
*** bobh has quit IRC | 14:04 | |
*** Alexey_Abashkin has joined #openstack-oslo | 14:11 | |
*** yamamoto has quit IRC | 14:11 | |
*** yamamoto has joined #openstack-oslo | 14:12 | |
*** AlexeyAbashkin has quit IRC | 14:13 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 14:13 | |
*** bobh has joined #openstack-oslo | 14:15 | |
*** yamamoto has quit IRC | 14:19 | |
*** chhagarw has joined #openstack-oslo | 14:20 | |
*** salv-orlando has joined #openstack-oslo | 14:23 | |
*** salv-orlando has quit IRC | 14:28 | |
*** yamamoto has joined #openstack-oslo | 14:32 | |
beekneemech | #startmeeting oslo | 15:00 |
openstack | Meeting started Mon Jun 11 15:00:29 2018 UTC and is due to finish in 60 minutes. The chair is beekneemech. 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 | |
*** beekneemech is now known as bnemec | 15:00 | |
openstack | The meeting name has been set to 'oslo' | 15:00 |
bnemec | courtesy ping for amotoki, amrith, ansmith, bnemec, dansmith, dhellmann, dims | 15:00 |
bnemec | courtesy ping for dougwig, e0ne, electrocucaracha, flaper87, garyk, gcb, haypo | 15:00 |
bnemec | courtesy ping for jd__, johnsom, jungleboyj, kgiusti, kragniz, lhx_, raildo | 15:00 |
bnemec | courtesy ping for redrobot, sileht, spamaps, sreshetnyak, stephenfin, stevemar, therve | 15:00 |
bnemec | courtesy ping for thinrichs, toabctl, zhiyan, zxy, zzzeek | 15:00 |
kgiusti | o/ | 15:01 |
redrobot | o/ | 15:01 |
ansmith | o/ | 15:01 |
dhellmann | o/ | 15:01 |
jungleboyj | o/ | 15:01 |
bnemec | #topic Red flags for/from liaisons | 15:02 |
jungleboyj | Nothing from Cinder. | 15:03 |
bnemec | The bot didn't update the topic. Must be a case of the Mondays. :-) | 15:04 |
bnemec | I can't think of anything from the Oslo side. | 15:04 |
jungleboyj | That makes two of us. | 15:04 |
bnemec | Actually, there was one thing. | 15:04 |
bnemec | It probably doesn't affect anyone here, but it was a kind of significant change that may have broken something. | 15:05 |
dhellmann | bnemec : you're not the chair, beekneemech is | 15:05 |
bnemec | We reverted a Windows-related patch in pbr. | 15:05 |
bnemec | Oh, shoot. | 15:05 |
*** bnemec is now known as beekneemech | 15:06 | |
beekneemech | Guess that means it's Friday again. :-) | 15:06 |
dhellmann | you can #chair your other nick if you want to switch back | 15:06 |
jungleboyj | Yay! | 15:06 |
smcginnis | You can - #chair bnemec and switch back I think. | 15:06 |
dhellmann | although I could use a short week, so... | 15:06 |
smcginnis | ;) | 15:06 |
beekneemech | Okay, thanks. | 15:06 |
beekneemech | #chair bnemec | 15:06 |
openstack | Warning: Nick not in channel: bnemec | 15:06 |
openstack | Current chairs: beekneemech bnemec | 15:06 |
*** beekneemech is now known as bnemec | 15:06 | |
bnemec | #topic Red flags for/from liaisons | 15:07 |
*** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)" | 15:07 | |
jungleboyj | That was weird. | 15:07 |
bnemec | So, on that note, this was the red flag I was talking about: | 15:07 |
bnemec | #link #topic Red flags for/from liaisons | 15:07 |
* bnemec is not having a great start to the week | 15:07 | |
bnemec | #link https://review.openstack.org/#/c/571503/ | 15:07 |
bnemec | I'm not sure exactly what that breaks, but it probably breaks something. | 15:08 |
bnemec | I don't know that anyone in this meeting is running pbr on Windows though, so I doubt there's much more to discuss. | 15:08 |
bnemec | #topic Releases | 15:09 |
*** openstack changes topic to "Releases (Meeting topic: oslo)" | 15:09 | |
bnemec | Business as usual. | 15:09 |
bnemec | We did release that pbr change so it's out in the wild. | 15:10 |
bnemec | #topic Action items from last meeting | 15:10 |
*** openstack changes topic to "Action items from last meeting (Meeting topic: oslo)" | 15:10 | |
bnemec | "kgiusti to sync up with Heat team on heat-tempest-plugin incompatibility with stable gabbi" | 15:10 |
bnemec | I saw a mailing list thread. | 15:10 |
kgiusti | bnemec: yah | 15:11 |
bnemec | #link http://lists.openstack.org/pipermail/openstack-dev/2018-June/131170.html | 15:11 |
kgiusti | bnemec: I've tried to run heat through the gate but: https://review.openstack.org/#/c/573437/ | 15:11 |
kgiusti | bnemec: so if I did that correctly, heat doesn't attempt to run those plugins either | 15:12 |
bnemec | kgiusti: You may need to change code to trigger the jobs. | 15:12 |
kgiusti | bnemec: I'll try that. | 15:12 |
bnemec | Some projects don't run all jobs on doc-only changes. | 15:13 |
kgiusti | bnemec: Is ceilometer still a thing - I seem to recall it was being phased out | 15:13 |
bnemec | kgiusti: That was just to verify Heat has the same problem, right? | 15:13 |
kgiusti | bnemec: right - see if it worked on queens | 15:14 |
kgiusti | bnemec: to be honest these telemetry tests in oslo.messaging have been a PITA for awhile now | 15:14 |
bnemec | Yeah, ceilometer has been deprecated for a while. | 15:14 |
bnemec | Off the top of my head I don't know what the support status is for it. | 15:14 |
kgiusti | bnemec: Ideally oslo.messaging would just leverage a "telemetry" test defined in another project. | 15:15 |
kgiusti | bnemec: the current tests were mooched from heat I think, with mooched === cut and pasted | 15:15 |
bnemec | Huh, there's still quite a bit going on in ceilometer: https://docs.openstack.org/releasenotes/ceilometer/unreleased.html | 15:16 |
bnemec | Maybe only the parts that overlapped with gnocchi were deprecated. | 15:17 |
kgiusti | bnemec: the purpose of the tests are to ensure we don't break anything "telemetry"-ish, but I'm not certain we're actually testing against the proper projects | 15:18 |
kgiusti | bnemec: In other words - what does "telemetry" mean in the context of Openstack? Heat, aodh? | 15:18 |
bnemec | kgiusti: Do we know why these were added? Has telemetry historically been something we break a lot? | 15:18 |
* kgiusti confused | 15:19 | |
kgiusti | bnemec: I'm not certain why they were added. It hasn't worked for hybrid for awhile now | 15:20 |
kgiusti | with hybrid == dual rabbitmq's (1 for notifications, one for rpc) | 15:20 |
kgiusti | and I've spend far, far too much time trying to root cause that | 15:21 |
bnemec | Telemetry according to https://docs.openstack.org/queens/projects.html is ceilometer, but stuff like gnocchi lives outside OpenStack now. | 15:21 |
dhellmann | ceilometer was a heavy user of messaging and added some APIs, so maybe we added the tests around then? | 15:21 |
dhellmann | sileht and jd used to be contributors to messaging, too | 15:21 |
kgiusti | dhellmann: that makes the most sense. | 15:22 |
dhellmann | if the tests are failing and we can't work out why and they don't seem related to messaging and we don't have support from the telemetry team maybe we should drop the tests | 15:22 |
kgiusti | dhellmann: I'll reach out directly to them | 15:23 |
bnemec | Maybe ask for someone to look at them and if no one does we start the process of removing the jobs. | 15:23 |
kgiusti | bnemec: +1 | 15:23 |
dhellmann | ++ | 15:23 |
bnemec | #action kgiusti to reach out to the telemetry team about their oslo.messaging test jobs | 15:23 |
bnemec | Okay, we have a way forward on that. | 15:24 |
bnemec | "bnemec to remove cliff from Oslo review list" | 15:24 |
bnemec | Done. | 15:24 |
*** salv-orlando has joined #openstack-oslo | 15:24 | |
bnemec | And that was it for action items. | 15:24 |
bnemec | #topic networkx uncap in taskflow | 15:25 |
*** openstack changes topic to "networkx uncap in taskflow (Meeting topic: oslo)" | 15:25 | |
bnemec | We need to be able to move forward with networkx, but unfortunately their 2.0 is very incompatible with the previous versions. | 15:25 |
bnemec | Unfortunately we merged the requirements change without fixing the code. | 15:26 |
bnemec | I've proposed a revert since currently the taskflow requirements are wrong, but I've gotten pushback for reasons I don't fully understand. | 15:27 |
bnemec | #link https://review.openstack.org/#/c/573752/ | 15:27 |
stephenfin | I thought upper-constraints managed that for us? | 15:28 |
stephenfin | which is currently constraining to 1.11 https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L82 | 15:28 |
*** salv-orlando has quit IRC | 15:29 | |
bnemec | It does, and I think the uncap patch was related to moving that up. | 15:29 |
dhellmann | bnemec : we need to start a mailing list thread explaining that we need someone to do that work | 15:29 |
stephenfin | bnemec: So if that's the case, the patch _is_ correct but we've to make sure the global requirements change doesn't go in | 15:29 |
*** prometheanfire has joined #openstack-oslo | 15:29 | |
prometheanfire | hi | 15:29 |
stephenfin | I don't really know how that would work | 15:30 |
bnemec | dhellmann: Yeah, that was actually why I initially added this topic to the meeting, and then the requirements patch merged which kind of changed the focus. | 15:30 |
dhellmann | prometheanfire : we have some concerns about the fact that someone is trying to move networkx to a new version, but taskflow doesn't support it | 15:30 |
dhellmann | and we have no one to do that work in taskflow | 15:30 |
dhellmann | is taskflow in the list of projects that are tested against upper-constraints changes? | 15:31 |
dhellmann | maybe we can add that, at least until the networkx change is done? | 15:31 |
bnemec | I think it's used by cinder, so it's probably covered. | 15:31 |
dhellmann | that would prevent someone from updating the constraint until taskflow also worked with it | 15:31 |
dhellmann | bnemec : maybe? I wonder if any of those tests are mocking networkx. | 15:33 |
prometheanfire | dhellmann: a bunch of projects don't support the new version iirc | 15:34 |
bnemec | dhellmann: Don't we run a devstack integration job? | 15:34 |
dhellmann | bnemec : ah, yes, right | 15:34 |
prometheanfire | requirements is uncapped so that projects can start working on it | 15:34 |
prometheanfire | but no one is | 15:34 |
prometheanfire | if they are not willing to update they should stop using it imo | 15:34 |
bnemec | I don't think we're saying we won't update, just that we're not there yet. | 15:35 |
dhellmann | well, and we don't have anyone to work on it | 15:35 |
dhellmann | and I'm not sure I buy the argument that we must always update past every API-breaking change of every dependency, either. | 15:35 |
smcginnis | Is there a patch out there showing the failure(s) caused by the newer networkx? | 15:35 |
smcginnis | dhellmann: ++ | 15:35 |
bnemec | smcginnis: No, because upper-constraints prevents networkx 2.0 from being used in the gate. | 15:35 |
smcginnis | Unless there are other necessary fixes included in the newer API-breaking change. | 15:36 |
bnemec | To figure out it was broken I ran it locally without u-c. | 15:36 |
smcginnis | bnemec: Ah, OK. | 15:36 |
dhellmann | bnemec : we can propose a constraint update and then propose a change to taskflow that depends on the uc update to see what would break | 15:36 |
prometheanfire | in that case we'd be back to capping things and becoming out of date | 15:36 |
dhellmann | prometheanfire : who is pushing to have the requirements opened? | 15:36 |
bnemec | dhellmann: Can we? tox.ini directly references the latest git version. | 15:37 |
prometheanfire | me | 15:37 |
dhellmann | bnemec : that value is overridden to the local filesystem in the gate | 15:37 |
dhellmann | prometheanfire : so you're working on updating the projects, then? | 15:37 |
bnemec | Ah, okay. | 15:37 |
prometheanfire | as I can | 15:37 |
dhellmann | ok | 15:37 |
prometheanfire | but only because distros are going to drop the old version | 15:37 |
dhellmann | maybe when they do that, they'll come help with the upgrade, then | 15:38 |
prometheanfire | lol, I think you know that | 15:38 |
smcginnis | We don't want to be in the state where a critical security issue is resolved in 2.0+ but we are still on 1.11 with no one having looked at what it would take to move forward. | 15:38 |
*** mago_ has quit IRC | 15:38 | |
smcginnis | But yeah, the work can also be done at the time it's needed. | 15:39 |
dhellmann | networkx builds graphs of data relationships in memory, so it doesn't strike me as high on the security list, but I do get your point | 15:39 |
smcginnis | Yeah, maybe not for this lib, but in general. | 15:39 |
dhellmann | was there a mailing list thread about this already? | 15:39 |
prometheanfire | I did send a message a while ago | 15:39 |
dhellmann | ok | 15:39 |
dhellmann | maybe it's time to follow-up and indicate that now that the requirements are uncapped it would be good to have some help updating? | 15:40 |
dhellmann | and then maybe we can find one or two people to volunteer | 15:40 |
bnemec | How hard is it to become 2.0 compatible? | 15:40 |
bnemec | Is this something where we could try to leverage the typo-fixing community? | 15:41 |
prometheanfire | from what I've seen it's not bad, but not trivial either. | 15:41 |
prometheanfire | clarkb: hi | 15:42 |
prometheanfire | clarkb: we are talking about it here too | 15:42 |
clarkb | ok | 15:42 |
prometheanfire | dhellmann: correct me if I'm wrong, but projects need to support 1.x and 2.x at the same time in order for us to update right? | 15:43 |
dhellmann | prometheanfire : yes, because we can't land 1 patch to update everything at once, we need support for both versions | 15:43 |
prometheanfire | dhellmann: that's been the sticking point from what I've seen | 15:43 |
dhellmann | bnemec : that's exactly what I was thinking. | 15:44 |
dhellmann | the APIs are different enough that it's hard to support both versions? | 15:44 |
clarkb | in the case of dib if its tests aren't blocking anyone but dib itself I'd say just merge it and dib will "catch up" but sounds like dib is at least part of the test blockign? | 15:44 |
bnemec | clarkb: For Oslo we have a problem with taskflow too. | 15:45 |
clarkb | dhellmann: https://review.openstack.org/#/c/506524/7/diskimage_builder/block_device/config.py looking at that its not too terrible just have to check if dg hasattr node and nodes and then use the right name | 15:45 |
dhellmann | that doesn't sound too bad | 15:46 |
bnemec | That is a really obnoxious api change. | 15:46 |
dhellmann | maybe we could convince the networkx folks to take a compatibility fix upstream? | 15:47 |
bnemec | They couldn't just leave node as an alias of nodes? | 15:47 |
dhellmann | even if it emits a warning | 15:47 |
dhellmann | right | 15:47 |
dhellmann | prometheanfire : do you want to try that? ^^ | 15:47 |
bnemec | That would make it much less painful to support both versions. | 15:48 |
dhellmann | right, and the warnings would motivate folks to update to the new API | 15:48 |
dhellmann | and we could require 2.0.1 or whatever as a minimum | 15:48 |
prometheanfire | that would be nice | 15:49 |
bnemec | The other change seems to be that a function call now returns a generator instead of a list, which isn't so bad. | 15:49 |
dhellmann | yeah, those we can deal with | 15:49 |
prometheanfire | so, just make an issue upstream for the node=nodes thing? | 15:52 |
clarkb | it is possible other projects like taskflow have other api incompatibilities. Not sure how representative dib is | 15:53 |
dhellmann | maybe even a patch? | 15:53 |
dhellmann | clarkb : good point | 15:53 |
dhellmann | we should keep investigating | 15:53 |
prometheanfire | clarkb: that's what I was wondering | 15:53 |
dhellmann | but this one seems like it would be easy enough to shim, and we're going to want to make separate patches anyway | 15:54 |
bnemec | That's true, I don't think nodes was the missing attribute I saw in taskflow. | 15:54 |
* bnemec doorbell | 15:55 | |
bnemec | AttributeError: 'OrderedDiGraph' object has no attribute 'nodes_iter' | 15:56 |
bnemec | I wonder if nodes is the iterator now in 2.0. | 15:56 |
dhellmann | probably | 15:57 |
prometheanfire | depend on https://review.openstack.org/531902 | 15:57 |
dhellmann | prometheanfire : thanks | 15:58 |
bnemec | Okay, so we're almost out of time for the meeting. It seems like we need to start collecting this information in the mailing list thread. | 15:58 |
dhellmann | ++ | 15:59 |
*** lifeless_ has quit IRC | 15:59 | |
*** lifeless has joined #openstack-oslo | 16:00 | |
bnemec | Okay, it looks like this is the thread: | 16:00 |
bnemec | #link http://lists.openstack.org/pipermail/openstack-dev/2017-December/125791.html | 16:00 |
bnemec | And Josh actually commented on the iterator thing in http://lists.openstack.org/pipermail/openstack-dev/2017-December/125826.html | 16:01 |
bnemec | #action collect details about uncapping networkx in the mailing list thread | 16:01 |
bnemec | And we're out of time. Thanks for joining everyone. | 16:02 |
bnemec | #endmeeting | 16:02 |
*** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo" | 16:02 | |
openstack | Meeting ended Mon Jun 11 16:02:04 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:02 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/oslo/2018/oslo.2018-06-11-15.00.html | 16:02 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/oslo/2018/oslo.2018-06-11-15.00.txt | 16:02 |
openstack | Log: http://eavesdrop.openstack.org/meetings/oslo/2018/oslo.2018-06-11-15.00.log.html | 16:02 |
jungleboyj | Thanks! | 16:02 |
smcginnis | Just wanted to point out https://bugs.launchpad.net/oslo.db/+bug/1775866 | 16:02 |
openstack | Launchpad bug 1775866 in oslo.db "enginefacade is not compatible with python 3.7" [Undecided,New] | 16:02 |
smcginnis | It's early, but would be good to get that addressed before it actually becomes an issue. | 16:03 |
*** dtruong_ has quit IRC | 16:05 | |
*** e0ne has quit IRC | 16:09 | |
*** AlexeyAbashkin has quit IRC | 16:11 | |
*** AlexeyAbashkin has joined #openstack-oslo | 16:13 | |
*** salv-orlando has joined #openstack-oslo | 16:25 | |
dhellmann | oh, that's going to require a breaking API change, I think | 16:30 |
*** salv-orlando has quit IRC | 16:30 | |
dhellmann | zzzeek : do you have any thoughts on ^^ | 16:30 |
zzzeek | not compatible w py37, geesh | 16:30 |
dhellmann | there's a method called "async" | 16:30 |
* zzzeek shakes fist at guido | 16:31 | |
zzzeek | dhellmann: welp. we gotta deprecate it, and then do a monkeypatch trick so that there is a method called "async" on less than python 3.7 | 16:31 |
zzzeek | dhellmann: like "setattr(the_class, "async", getattr(the_class, "async_")) | 16:32 |
zzzeek | dhellmann: if sys.version_info < (3, 7) | 16:32 |
dhellmann | zzzeek : how widely used is that method? | 16:33 |
zzzeek | dhellmann: it's likely out there a bit | 16:33 |
zzzeek | dhellmann: but all those applciation are also not py3.7 compatible :) | 16:33 |
*** shardy has quit IRC | 16:33 | |
dhellmann | http://codesearch.openstack.org/?q=%5C.async%5C(&i=nope&files=&repos= | 16:33 |
zzzeek | dhellmann: so..we make the async_ upgraed path availble in oslo | 16:33 |
dhellmann | that's probably not the right search? | 16:33 |
dhellmann | because that's not returning any callers at all | 16:34 |
zzzeek | dhellmann: it is possible nobody is using it | 16:34 |
dhellmann | if no one is using it we can just change the name and make it a breaking version release | 16:34 |
*** Alexey_Abashkin has joined #openstack-oslo | 16:35 | |
zzzeek | dhellmann: yes | 16:35 |
*** AlexeyAbashkin has quit IRC | 16:36 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 16:36 | |
dhellmann | a broader search on async\( isn't showing any users either | 16:38 |
zzzeek | dhellmann: im doing a more tedios search | 16:38 |
zzzeek | dhellmann: mmm that's a @property | 16:38 |
zzzeek | no \| | 16:38 |
zzzeek | \( | 16:38 |
dhellmann | oh, it is? | 16:38 |
zzzeek | dhellmann: boom http://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/api.py#n204 | 16:39 |
dhellmann | oh, glance has an async module. nice. | 16:40 |
dhellmann | http://codesearch.openstack.org/?q=%5C.async&i=nope&files=.*%5C.py&repos= | 16:40 |
zzzeek | dhellmann: the nova hit might be the only one | 16:40 |
dhellmann | I haven't gotten that far | 16:40 |
zzzeek | dhellmann: yeah that's the same search I have :) | 16:41 |
dhellmann | so yeah, a new method, an alias, a release, fix nova, drop the alias | 16:41 |
dhellmann | http://codesearch.openstack.org/?q=%5C.async(%20%7C%24)&i=nope&files=.*%5C.py&repos= is a bit narrowoer | 16:42 |
dhellmann | narrower even | 16:42 |
*** Alexey_Abashkin has joined #openstack-oslo | 16:42 | |
dhellmann | looks like we have a similar issue in oslo.messaging, kgiusti | 16:42 |
zzzeek | dhellmann: can't we just get python to unilateraly remove asyncio and admit it is horrible instead | 16:42 |
dhellmann | although that's an attribute of the zmq driver, which we keep saying we're going to delete | 16:43 |
dhellmann | zzzeek: if that's the hill you want to storm, I'll wait down here in the valley | 16:43 |
zzzeek | haha | 16:43 |
kgiusti | dhellmann: right - zmq has been deprecated | 16:43 |
dhellmann | zzzeek : the fix doesn't look too bad, we could probably get it done this week if you have time | 16:44 |
dhellmann | kgiusti : ok, we were looking at the use of "async" as a symbol name since in 3.7 it's a reserved keyword | 16:44 |
zzzeek | dhellmann: I am amazed by your deft mutations of pronouns in that sentence :) | 16:44 |
*** AlexeyAbashkin has quit IRC | 16:44 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 16:44 | |
dhellmann | zzzeek : I can't do it by myself (need more reviewers). I have time to help, though. | 16:45 |
zzzeek | dhellmann: ill see if i can throw up the basic idea | 16:45 |
dhellmann | zzzeek : re-reading, I left off the "to help" at the end of that original sentence :-) | 16:45 |
zzzeek | dhellmann: ah yes of course I can help :) | 16:46 |
dhellmann | kgiusti : if we're going to finally delete the driver, it's not a big deal. when did we agree to do that? this cycle? next? | 16:46 |
kgiusti | dhellmann: those zmq APIs are internal... at least that's the indent | 16:46 |
dhellmann | yeah, we could fix it, or just delete them | 16:46 |
kgiusti | "intent" | 16:47 |
*** bobh has quit IRC | 16:52 | |
*** e0ne has joined #openstack-oslo | 17:21 | |
*** e0ne has quit IRC | 17:25 | |
*** salv-orlando has joined #openstack-oslo | 17:25 | |
*** bobh has joined #openstack-oslo | 17:26 | |
*** salv-orlando has quit IRC | 17:31 | |
*** bobh has quit IRC | 17:32 | |
*** AlexeyAbashkin has quit IRC | 17:35 | |
*** e0ne has joined #openstack-oslo | 17:43 | |
openstackgerrit | qingszhao proposed openstack-dev/pbr master: fix tox python3 overrides https://review.openstack.org/574364 | 17:47 |
*** bobh has joined #openstack-oslo | 18:04 | |
*** bobh has quit IRC | 18:09 | |
openstackgerrit | Monty Taylor proposed openstack/futurist master: Restrict contextlib2 to only python2 https://review.openstack.org/574371 | 18:11 |
openstackgerrit | Monty Taylor proposed openstack/futurist master: Only install monotonic on python2 https://review.openstack.org/574372 | 18:13 |
*** dave-mcc_ has quit IRC | 18:25 | |
*** salv-orlando has joined #openstack-oslo | 18:26 | |
*** dave-mccowan has joined #openstack-oslo | 18:28 | |
*** salv-orlando has quit IRC | 18:31 | |
*** dtruong has joined #openstack-oslo | 18:31 | |
*** e0ne has quit IRC | 18:33 | |
*** chhagarw has quit IRC | 18:50 | |
*** e0ne has joined #openstack-oslo | 19:07 | |
*** e0ne has quit IRC | 19:11 | |
openstackgerrit | Ken Giusti proposed openstack/oslo.messaging master: Mark the ZeroMQ driver deprecated https://review.openstack.org/574387 | 19:11 |
*** mmethot has quit IRC | 19:18 | |
*** salv-orlando has joined #openstack-oslo | 19:27 | |
*** salv-orlando has quit IRC | 19:32 | |
*** mmethot has joined #openstack-oslo | 19:34 | |
*** e0ne has joined #openstack-oslo | 19:49 | |
*** yamamoto has quit IRC | 19:51 | |
-openstackstatus- NOTICE: Zuul was restarted for a software upgrade; changes uploaded or approved between 19:30 and 19:50 will need to be rechecked | 19:57 | |
*** hoonetorg has joined #openstack-oslo | 20:05 | |
*** rmascena has joined #openstack-oslo | 20:22 | |
*** raildo has quit IRC | 20:23 | |
*** kgiusti has left #openstack-oslo | 20:27 | |
*** salv-orlando has joined #openstack-oslo | 20:28 | |
*** rmascena has quit IRC | 20:31 | |
*** salv-orlando has quit IRC | 20:32 | |
*** AlexeyAbashkin has joined #openstack-oslo | 20:32 | |
*** Alexey_Abashkin has joined #openstack-oslo | 20:46 | |
*** AlexeyAbashkin has quit IRC | 20:47 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 20:47 | |
*** ansmith has quit IRC | 20:50 | |
*** Alexey_Abashkin has joined #openstack-oslo | 20:51 | |
*** yamamoto has joined #openstack-oslo | 20:51 | |
*** AlexeyAbashkin has quit IRC | 20:51 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 20:51 | |
*** AlexeyAbashkin has quit IRC | 20:56 | |
*** yamamoto has quit IRC | 20:58 | |
*** AlexeyAbashkin has joined #openstack-oslo | 21:08 | |
*** lifeless_ has joined #openstack-oslo | 21:28 | |
*** lifeless has quit IRC | 21:28 | |
*** lifeless_ has quit IRC | 21:38 | |
*** lifeless has joined #openstack-oslo | 21:44 | |
*** ansmith has joined #openstack-oslo | 21:46 | |
*** AlexeyAbashkin has quit IRC | 21:54 | |
*** e0ne has quit IRC | 21:59 | |
*** nicolasbock has quit IRC | 22:05 | |
*** rcernin has joined #openstack-oslo | 22:16 | |
*** EmilienM|PTO is now known as EmilienM | 22:41 | |
*** snapiri has quit IRC | 22:47 | |
*** edmondsw has quit IRC | 22:47 | |
*** snapiri has joined #openstack-oslo | 22:47 | |
*** hoonetorg has quit IRC | 23:24 | |
*** tosky has quit IRC | 23:32 | |
*** hoonetorg has joined #openstack-oslo | 23:36 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!