| openstackgerrit | Merged openstack/taskflow: Rename memory backend filesystem -> fake filesystem https://review.openstack.org/165662 | 00:13 |
|---|---|---|
| *** achanda has quit IRC | 00:22 | |
| *** liusheng has quit IRC | 00:26 | |
| *** liusheng has joined #openstack-oslo | 00:26 | |
| *** achanda has joined #openstack-oslo | 00:53 | |
| *** amrith is now known as _amrith_ | 00:58 | |
| *** sputnik13 has quit IRC | 01:00 | |
| *** liusheng has quit IRC | 01:00 | |
| *** dims has joined #openstack-oslo | 01:01 | |
| *** dims has quit IRC | 01:01 | |
| *** _amrith_ is now known as amrith | 01:15 | |
| *** dims__ has joined #openstack-oslo | 01:15 | |
| *** stevemar has quit IRC | 01:34 | |
| *** sputnik13 has joined #openstack-oslo | 01:40 | |
| *** sputnik13 has quit IRC | 01:51 | |
| *** sputnik13 has joined #openstack-oslo | 01:57 | |
| *** achanda has quit IRC | 01:57 | |
| *** mtanino has joined #openstack-oslo | 02:08 | |
| *** ccrouch has joined #openstack-oslo | 02:25 | |
| *** takedakn has joined #openstack-oslo | 02:44 | |
| *** amrith is now known as _amrith_ | 02:52 | |
| *** achanda has joined #openstack-oslo | 02:58 | |
| *** dims__ has quit IRC | 03:01 | |
| *** ccrouch is now known as ccrouch-afk | 03:02 | |
| *** achanda has quit IRC | 03:03 | |
| *** dims__ has joined #openstack-oslo | 04:06 | |
| *** achanda has joined #openstack-oslo | 04:08 | |
| *** stevemar has joined #openstack-oslo | 04:15 | |
| *** achanda has quit IRC | 04:34 | |
| *** dims__ has quit IRC | 04:36 | |
| *** yamahata has joined #openstack-oslo | 04:54 | |
| *** kiseok7_ has joined #openstack-oslo | 05:06 | |
| *** pcaruana has quit IRC | 05:07 | |
| *** achanda has joined #openstack-oslo | 05:08 | |
| *** rushiagr_away is now known as rushiagr | 05:19 | |
| *** achanda has quit IRC | 05:24 | |
| *** achanda has joined #openstack-oslo | 05:26 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Allow using shallow copy instead of deep copy https://review.openstack.org/165693 | 05:33 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: By default use a in memory backend (when none is provided) https://review.openstack.org/165654 | 05:33 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Just unify having a single requirements.txt file https://review.openstack.org/164523 | 05:35 |
| *** yamahata has quit IRC | 05:41 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.utils: Imported Translations from Transifex https://review.openstack.org/166700 | 06:11 |
| *** mtanino has quit IRC | 06:22 | |
| *** dims__ has joined #openstack-oslo | 06:22 | |
| *** ajo_ has joined #openstack-oslo | 06:28 | |
| *** jamielennox is now known as jamielennox|away | 06:28 | |
| *** stevemar has quit IRC | 06:48 | |
| *** inc0 has joined #openstack-oslo | 06:52 | |
| *** dims__ has quit IRC | 06:54 | |
| *** pcaruana has joined #openstack-oslo | 06:58 | |
| *** rushiagr is now known as rushiagr_away | 07:03 | |
| *** saikrishna has joined #openstack-oslo | 07:07 | |
| *** ajo_ is now known as ajo | 07:10 | |
| *** achanda has quit IRC | 07:29 | |
| *** yamahata has joined #openstack-oslo | 07:36 | |
| *** ajo has quit IRC | 07:42 | |
| *** gtt116 has joined #openstack-oslo | 07:51 | |
| gtt116 | good morning? | 07:52 |
| gtt116 | https://blueprints.launchpad.net/oslo.messaging/+spec/consistent-exception-notifications does somebody still work on this blueprint? | 07:52 |
| *** yamahata has quit IRC | 07:53 | |
| *** YorikSar has quit IRC | 07:58 | |
| *** saikrishna_ has joined #openstack-oslo | 07:58 | |
| *** saikrishna has quit IRC | 08:01 | |
| *** saikrishna__ has joined #openstack-oslo | 08:14 | |
| *** saikrishna_ has quit IRC | 08:16 | |
| *** ndipanov has joined #openstack-oslo | 08:21 | |
| *** ihrachyshka has joined #openstack-oslo | 08:25 | |
| *** YorikSar has joined #openstack-oslo | 08:34 | |
| *** gtt116 has quit IRC | 08:37 | |
| *** gtt116 has joined #openstack-oslo | 08:37 | |
| *** dims__ has joined #openstack-oslo | 08:39 | |
| *** andreykurilin_ has joined #openstack-oslo | 08:40 | |
| *** ihrachyshka has quit IRC | 08:48 | |
| *** inc0 has quit IRC | 08:51 | |
| *** ajo has joined #openstack-oslo | 08:52 | |
| *** e0ne has joined #openstack-oslo | 09:02 | |
| *** inc0 has joined #openstack-oslo | 09:04 | |
| *** e0ne has quit IRC | 09:08 | |
| *** inc0 has quit IRC | 09:11 | |
| *** inc0 has joined #openstack-oslo | 09:11 | |
| *** BrianShang has quit IRC | 09:12 | |
| *** dims__ has quit IRC | 09:12 | |
| *** BrianShang has joined #openstack-oslo | 09:13 | |
| openstackgerrit | Merged openstack/tooz: Use a sentinel connection pool to manage failover https://review.openstack.org/166291 | 09:17 |
| openstackgerrit | Merged openstack/tooz: Updated from global requirements https://review.openstack.org/166479 | 09:17 |
| *** andreykurilin_ has quit IRC | 09:26 | |
| haypo | hi. did someone saw a test_save_and_reraise_exception_dropped() failure with python 3.4 in Oslo Incubator? | 09:26 |
| haypo | oh, it's in oslo.utils sorry, not incubator | 09:29 |
| *** inc0 has quit IRC | 09:32 | |
| *** inc0 has joined #openstack-oslo | 09:34 | |
| *** kbyrne has joined #openstack-oslo | 09:35 | |
| *** viktors has joined #openstack-oslo | 09:36 | |
| *** liusheng has joined #openstack-oslo | 09:41 | |
| *** dims__ has joined #openstack-oslo | 09:51 | |
| *** gtt116_ has joined #openstack-oslo | 09:53 | |
| *** e0ne has joined #openstack-oslo | 09:54 | |
| *** gtt116 has quit IRC | 09:56 | |
| *** e0ne is now known as e0ne_ | 10:00 | |
| *** inc0 has quit IRC | 10:01 | |
| *** inc0 has joined #openstack-oslo | 10:02 | |
| *** e0ne_ has quit IRC | 10:05 | |
| *** subscope has joined #openstack-oslo | 10:06 | |
| *** inc0 has quit IRC | 10:10 | |
| *** enikanorov_ has quit IRC | 10:11 | |
| *** enikanorov_ has joined #openstack-oslo | 10:12 | |
| *** e0ne has joined #openstack-oslo | 10:14 | |
| *** cdent has joined #openstack-oslo | 10:18 | |
| openstackgerrit | Merged openstack/oslo.db: Updated from global requirements https://review.openstack.org/166452 | 10:20 |
| *** ihrachyshka has joined #openstack-oslo | 10:23 | |
| *** takedakn has quit IRC | 10:25 | |
| *** vigneshvar has joined #openstack-oslo | 10:41 | |
| *** dulek has joined #openstack-oslo | 10:46 | |
| *** ihrachyshka has quit IRC | 10:47 | |
| *** dulek has quit IRC | 10:47 | |
| *** dulek has joined #openstack-oslo | 10:47 | |
| *** saikrishna_ has joined #openstack-oslo | 10:52 | |
| *** dulek_ has joined #openstack-oslo | 10:53 | |
| *** jaosorior has joined #openstack-oslo | 10:53 | |
| *** saikrishna__ has quit IRC | 10:54 | |
| *** dulek has quit IRC | 10:55 | |
| dims__ | sigmavirus24_awa: so going through some really old stuff, does this still happen? https://bugs.launchpad.net/nova/+bug/1101404 | 10:58 |
| openstack | Launchpad bug 1101404 in oslo.log "nova syslog logging to /dev/log race condition in python 2.6 and 2.7" [Medium,Confirmed] | 10:58 |
| *** saikrishna__ has joined #openstack-oslo | 10:59 | |
| *** saikrishna_ has quit IRC | 11:02 | |
| *** saikrishna__ has quit IRC | 11:09 | |
| *** e0ne is now known as e0ne_ | 11:10 | |
| *** _amrith_ is now known as amrith | 11:13 | |
| *** e0ne_ is now known as e0ne | 11:17 | |
| *** saikrishna has joined #openstack-oslo | 11:26 | |
| *** zul has joined #openstack-oslo | 11:32 | |
| *** saikrishna has quit IRC | 11:33 | |
| sileht | dims__, yes it still happens | 11:35 |
| sileht | dims__, I have added a link to new eventlet bug about that | 11:36 |
| *** shardy has joined #openstack-oslo | 11:42 | |
| *** BrianShang has quit IRC | 11:42 | |
| mdbooth | Any interest in this: https://review.openstack.org/#/c/146228/ | 11:42 |
| mdbooth | I'd like to use it in Nova | 11:42 |
| *** BrianShang has joined #openstack-oslo | 11:43 | |
| dims__ | sileht: i see the patch for logging/handlers.py in my python 2.7 installation - http://bugs.python.org/issue15179 - does it happen even with that patch? | 11:44 |
| dims__ | viktors: rpodolyaka: please see ping from mdbooth when you get a chance - https://review.openstack.org/#/c/146228/ | 11:46 |
| sileht | dims__, yes | 11:54 |
| *** pblaho has joined #openstack-oslo | 11:57 | |
| *** kgiusti has joined #openstack-oslo | 12:01 | |
| *** e0ne is now known as e0ne_ | 12:05 | |
| *** stpierre has joined #openstack-oslo | 12:07 | |
| *** sreshetnyak has quit IRC | 12:18 | |
| haypo | i don't understand how i'm supposed to use the subunit module :-( | 12:20 |
| haypo | python -m subunit.run discover --list > test-list; python -m subunit.run discover --load-list test-list => UnicodeDecodeError ?? | 12:20 |
| haypo | --load-list expects a text file? | 12:21 |
| *** sreshetnyak has joined #openstack-oslo | 12:22 | |
| haypo | i'm investigating the bug https://bugs.launchpad.net/oslo.utils/+bug/1433800 | 12:22 |
| openstack | Launchpad bug 1433800 in oslo.utils "Intermittent "expected string or buffer" exception in py3 jobs" [Medium,Confirmed] | 12:22 |
| *** gordc has joined #openstack-oslo | 12:27 | |
| *** e0ne_ has quit IRC | 12:29 | |
| *** yassine has joined #openstack-oslo | 12:31 | |
| *** ihrachyshka has joined #openstack-oslo | 12:31 | |
| *** dims__ has quit IRC | 12:34 | |
| *** dims__ has joined #openstack-oslo | 12:34 | |
| *** amrith is now known as _amrith_ | 12:35 | |
| *** jecarey has quit IRC | 12:52 | |
| *** dims__ is now known as dims | 12:52 | |
| dhellmann | _amrith_, bnemec, dims, haypo : I randomly got this more detailed traceback for the utils issue: http://paste.openstack.org/show/195344/ | 12:55 |
| haypo | dhellmann: yes. see https://bugs.launchpad.net/oslo.utils/+bug/1433800 | 12:56 |
| openstack | Launchpad bug 1433800 in oslo.utils "Intermittent "expected string or buffer" exception in py3 jobs" [Medium,Confirmed] | 12:56 |
| haypo | dhellmann: i wrote a shell script to reproduce it in 100% cases | 12:56 |
| haypo | i'm still trying to understand how to reproduce it with less commands | 12:56 |
| dhellmann | haypo: at least you have it reproducable! | 12:57 |
| haypo | dhellmann: do you know how to load a list of tests using subunit? | 12:57 |
| haypo | 13:20 < haypo> python -m subunit.run discover --list > test-list; python -m subunit.run discover --load-list test-list => UnicodeDecodeError ?? | 12:57 |
| dhellmann | haypo: no :-( | 12:57 |
| haypo | i don't understand all these subunit tools | 12:58 |
| dims | haypo: folks on -qa may know | 13:00 |
| dhellmann | haypo: it always takes me a while to find the docs because they aren't linked from the readme page: http://testrepository.readthedocs.org/en/latest/ | 13:00 |
| *** amotoki has quit IRC | 13:01 | |
| haypo | dhellmann: i wrote my own list of docs :-) http://haypo-notes.readthedocs.org/openstack.html#tests | 13:04 |
| haypo | dhellmann: with simple reminders like "how to run a test"... it's not so simple in practice :) | 13:04 |
| dhellmann | haypo: heh | 13:04 |
| *** amotoki has joined #openstack-oslo | 13:06 | |
| *** vigneshvar has quit IRC | 13:09 | |
| *** leakypipes has quit IRC | 13:13 | |
| dhellmann | haypo: your script doesn't cause the failure to happen for me :-/ | 13:14 |
| haypo | dhellmann: too bad | 13:16 |
| eezhova | dims, ping | 13:19 |
| dims | eezhova: pong | 13:19 |
| eezhova | dims, could you please check out my reply to your comment to https://bugs.launchpad.net/oslo-incubator/+bug/1433142 ? | 13:20 |
| openstack | Launchpad bug 1433142 in oslo-incubator "ProcessLauncher should support reloading config file for parent process on receiving SIGHUP" [Undecided,Fix committed] - Assigned to Elena Ezhova (eezhova) | 13:20 |
| dims | eezhova: ah. i tested nova-compute and verified that it does reload config files. i missed testing nova-api | 13:22 |
| dims | eezhova: so nova-api is still busted? | 13:22 |
| eezhova | dims, does nova-compute use ProcessLauncher? | 13:22 |
| dims | eezhova: nova/service.py seems to indicate so | 13:23 |
| haypo | dhellmann: cool, i reduced the list of tests from 12,000+ tests to 2 tests :) i'm still able to reprodue the issue | 13:25 |
| dhellmann | haypo: that's good progress | 13:26 |
| dhellmann | haypo: which 2 tests? | 13:27 |
| *** mriedem1 has joined #openstack-oslo | 13:27 | |
| *** mriedem has quit IRC | 13:27 | |
| *** alexpilotti has joined #openstack-oslo | 13:31 | |
| eezhova | dims, it seems to that nova-compute uses ServiceLauncher | 13:31 |
| haypo | dhellmann: (py34)haypo@smithers$ python -m testtools.run oslo_utils.tests.test_netutils.TestIsIPv6Enabled.test_disabled tests.test_excutils.SaveAndReraiseTest.test_save_and_reraise_exception_dropped | 13:31 |
| haypo | dhellmann: ok, the problem is in oslo_utils.tests.test_netutils.TestIsIPv6Enabled.test_disabled, it never stops a mock | 13:31 |
| eezhova | dims, https://github.com/openstack/nova/blob/97d63d8745cd9b3b391ce96b94b4da263b3a053d/nova/service.py#L440 | 13:31 |
| eezhova | dims, serve accepts workers as a parameter | 13:32 |
| eezhova | dims, with default value - None | 13:32 |
| eezhova | and here https://github.com/openstack/nova/blob/97d63d8745cd9b3b391ce96b94b4da263b3a053d/nova/cmd/compute.py#L73 no workers parameter is passed | 13:33 |
| dhellmann | haypo: ah! it looks like the test author assumed that the base class set up the mock fixture to stop mocks | 13:33 |
| eezhova | dims, so, ServiceLauncher would be used: https://github.com/openstack/oslo-incubator/blob/master/openstack/common/service.py#L488 | 13:34 |
| *** zzzeek has joined #openstack-oslo | 13:36 | |
| eezhova | dims, it is not nova's bug, it is ProcessLauncher that is missing the required functionality | 13:37 |
| *** superdan is now known as dansmith | 13:39 | |
| *** dulek_ has quit IRC | 13:43 | |
| *** exploreshaifali has joined #openstack-oslo | 13:45 | |
| *** jecarey has joined #openstack-oslo | 13:48 | |
| *** salv-orlando has joined #openstack-oslo | 13:50 | |
| openstackgerrit | Victor Stinner proposed openstack/oslo.utils: Fix test_netutils: stop patches https://review.openstack.org/166835 | 13:50 |
| *** exploreshaifali has quit IRC | 13:51 | |
| *** e0ne has joined #openstack-oslo | 13:52 | |
| dims | eezhova: easy to fix? | 13:56 |
| eezhova | dims, yes, we just have to reload config and call reset() in wait method after receiving SIGHUP | 13:57 |
| eezhova | dims, https://review.openstack.org/#/c/165104/ | 13:57 |
| eezhova | dims, that's almost the same that is done for ServiceLauncher | 13:59 |
| *** hogepodge has joined #openstack-oslo | 14:02 | |
| *** prad has joined #openstack-oslo | 14:04 | |
| dims | eezhova: tested and works? | 14:04 |
| dims | sorry a bit distracted with another thing at the moment | 14:05 |
| eezhova | dims, tested with neutron-server. will double check with nova-api | 14:05 |
| dims | thanks! | 14:05 |
| *** _amrith_ is now known as amrith | 14:06 | |
| *** stevemar has joined #openstack-oslo | 14:10 | |
| *** zz_jgrimm- is now known as jgrimm | 14:12 | |
| eezhova | dims, I ran the same test for nova-api with my patch and it works - please see logs http://paste.openstack.org/show/195378/ | 14:12 |
| dims | eezhova: awesome work! thanks | 14:13 |
| eezhova | dims, thanks for you time! do you mind if remove nova from affected projects and reset the bug's status to "In progress"? | 14:14 |
| dims | eezhova: i need it for sync-ing oslo-incubator to nova | 14:15 |
| eezhova | dims, ah, OK | 14:16 |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:16 | |
| *** ccrouch-afk is now known as ccrouch | 14:17 | |
| *** mtanino has joined #openstack-oslo | 14:21 | |
| jd__ | dhellmann: dims: knock knock | 14:22 |
| dims | jd__: pong | 14:22 |
| jd__ | I've a problem and I'd like to hear from you | 14:22 |
| sigmavirus24 | dims: re bug 1101404 we were seeing it on 2.7 but I dont' recall the steps to reproduce | 14:22 |
| openstack | bug 1101404 in oslo.log "nova syslog logging to /dev/log race condition in python 2.6 and 2.7" [Medium,Confirmed] https://launchpad.net/bugs/1101404 | 14:22 |
| dhellmann | jd__: is the parser in pip, or setuptools? | 14:22 |
| dhellmann | jd__: oh, you go first :-) | 14:22 |
| sigmavirus24 | I know that isn't particularly helpful, but we're running 2.7 and were still seeing the bug | 14:22 |
| jd__ | dhellmann: pip, setuptools does not support those tags that's why we remove them | 14:22 |
| dhellmann | jd__: I thought the whole point of this was there was a standard and some of the upstream tools now support it | 14:23 |
| jd__ | so basically I was using jsonutils happily until 5 minutes ago where I was very disapointed to discover that it relies on strtime() to encode datetime https://git.openstack.org/cgit/openstack/oslo.serialization/tree/oslo_serialization/jsonutils.py#n110 | 14:23 |
| jd__ | dhellmann: I don't know everything, but from what I got that's the state of it, don't ask me more :) | 14:24 |
| dims | sigmavirus24: ack, please add whatever you can to the bug (even just saying which version of python 2.7 you are seeing) | 14:24 |
| dhellmann | jd__: ok, I'll ask lifeless | 14:24 |
| jd__ | as you know I'm on a quest to remove strtime() from OpenStack and timeutils | 14:24 |
| jd__ | so I wrote a patch to change strtime() to isotime() in oslo.serialization and I tested on Nova | 14:24 |
| dims | jd__: k | 14:24 |
| dhellmann | jd__: on the date format thing, we need to provide a migration path that supports rolling upgrades for anything that might be sending datetime values over rpc | 14:25 |
| jd__ | and it didn't explose, it just broke 2 tests | 14:25 |
| dhellmann | it's not enough that it works, it has to work with both formats at the same time | 14:25 |
| jd__ | dhellmann: yeah I did that for context in Nova already | 14:25 |
| dhellmann | ok | 14:25 |
| jd__ | you just leave strtime() on the ouput and start reading with parse_isotime() which understands both | 14:25 |
| dhellmann | that sounds good | 14:25 |
| jd__ | and on L we can drop strtime() in favor of isotime() | 14:25 |
| jd__ | it's pretty easy actually | 14:26 |
| sigmavirus24 | dims: will do | 14:26 |
| dhellmann | or just change the return value of strtime | 14:26 |
| jd__ | dhellmann: that's an option too :) | 14:26 |
| jd__ | so back on my jsonutils thing I'm not sure what to do | 14:26 |
| dhellmann | that might make it easier, since we don't need to go update all of the projects before removing the function | 14:26 |
| jd__ | clearly it changes the API, but it also fixes it | 14:26 |
| jd__ | it's not too invasive from what I tested, but still it "breaks" things | 14:27 |
| dhellmann | yeah, we need to publicize the change | 14:27 |
| dhellmann | jd__: just a sec | 14:28 |
| dims | jd__: so accept old and new and generate new? | 14:28 |
| jd__ | dims: well in jsonutils there's no "accept" so we're safe – it's just test that check for some string value, and the new value has a Z (the timezone) at the end of it so the test fails | 14:29 |
| dhellmann | dansmith: what impact would changing the datetime format have on versionedobjects? ^^ | 14:29 |
| jd__ | it it was tested with parse_isotime() it would not fail | 14:29 |
| jd__ | so I can pre-patch Nova and other projects before we make the change to oslo.serialization if that's good | 14:29 |
| * dansmith reads | 14:29 | |
| dansmith | dhellmann: jd__: we don't rely on jsonutils to serialize our datetimes, so it doesn't affect us, yes/ | 14:30 |
| jd__ | awesome | 14:30 |
| jd__ | I see you use isotime(), good call | 14:30 |
| * jd__ high-five dansmith | 14:30 | |
| dims | yay | 14:30 |
| dansmith | woohoo! :) | 14:31 |
| dansmith | right, just for the record: https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L298-304 | 14:32 |
| dansmith | that's where we serialize and deserialize datetimes | 14:32 |
| dhellmann | dansmith: excellent, that was one of my big concerns | 14:32 |
| dansmith | er, well, I highlighted down instead of up, but you get the idea :) | 14:32 |
| dhellmann | jd__: so the other thing is the rpc calls, especially notifications, so how will this affect ceilometer? | 14:32 |
| jd__ | dhellmann: let me check that | 14:33 |
| dansmith | jd__: if it has a Z at the end, does it end up with a tz-aware datetime? | 14:33 |
| dansmith | because if so, that will cause lots of problems all over | 14:34 |
| jd__ | dhellmann: looks like we don't realy on parse_strtime() in Ceilometer (and nothing about that in oslo.messaging neither) | 14:34 |
| dhellmann | jd__: I know you hate this, but one way to publicize a format change like this will be a small spec. Pulling together the notes from this conversation and your other research into the impact would be good, and then we can also prepare an email thread | 14:34 |
| jd__ | dansmith: yes, that's the point | 14:34 |
| dhellmann | I wonder if it's worth providing a backwards compatibility flag | 14:34 |
| *** achanda has joined #openstack-oslo | 14:34 | |
| dansmith | jd__: that's going to cause issues everywhere, right? | 14:34 |
| jd__ | dansmith: that's my question – from what I tested with Nova unit tested, it only breaks 2 tests | 14:34 |
| dansmith | jd__: people that create a datetime and go to compare it with something received over the wire, and end up with a failure comparing the tz-aware and tz-unaware tests | 14:35 |
| dansmith | jd__: (a) that surprises me and (b) I'm sure there are more | 14:35 |
| dansmith | jd__: because of objects, we've converted a lot of code to tz-aware now, so that's probably why it's low, but it used to be hundreds | 14:35 |
| dansmith | jd__: I would expect other projects to choke on that pretty hard | 14:35 |
| jd__ | dansmith: that depends on how you parse the time actually, parse_isotime() always add a timezone | 14:36 |
| jd__ | so we're safe with that | 14:36 |
| dansmith | but this is changing jsonutils' output right? | 14:36 |
| jd__ | and if it relies on parse_strtime(), it will explose (and I'm trying to remove parse_strtime() usage) | 14:36 |
| jd__ | dansmith: yes it is | 14:36 |
| dansmith | right, so that's the problem I think | 14:37 |
| dansmith | can't we change jsonutils to just strip out the timezone on datetimes after parsing or something? | 14:37 |
| jd__ | yes, so we are trying to see who is using parse_strtime() to read JSON, but I don't see many places doing that – and I'm trying to fixing them – and we can actually replace parse_strtime() by parse_isotime() likely? | 14:37 |
| jd__ | dansmith: jsonutils doesn't parse | 14:37 |
| dansmith | before objects, nearly every rpc call in nova sent three datetimes that were serialized by jsonutils | 14:38 |
| dansmith | by "doesn't parse" you mean the loads() returns the stringified version of the datetime instead of the actual datetime, right | 14:40 |
| dansmith | so the problem with this plan, had you done it in nova two years ago, is that you'd have some services running new code sending things with Z on the end to code that is using the old parse_strtime(), which would choke right? | 14:41 |
| *** achanda has quit IRC | 14:41 | |
| dansmith | so, people still doing such things might not care about rolling upgrades, I dunno | 14:41 |
| jd__ | yes that's the main possible issue I see | 14:41 |
| dansmith | yep | 14:42 |
| dansmith | I expect there are actually more breaks in nova around the scheduler and resource tracker stuff that don't pop up in unit tests, but the devstack tests should hopefully shake those out | 14:43 |
| jd__ | well I'm gonna send the patch I guess and we'll discuss that there I guess | 14:43 |
| dansmith | okay | 14:43 |
| jd__ | dansmith: the thing is I don't think we can now that before releasing the lib :( which is too late obviously | 14:43 |
| jd__ | dhellmann correct me if I'm wrong | 14:43 |
| dhellmann | jd__: patching the lib should trigger a devstack-gate test job that installs the lib from source | 14:44 |
| jd__ | dhellmann: ok, cool! | 14:45 |
| *** bnemec has quit IRC | 14:46 | |
| dhellmann | jd__: if that *doesn't* happen we should make sure the test job config for the lib is right :-) | 14:46 |
| jd__ | dhellmann: dully noted, I'm waiting for the results anyway | 14:47 |
| dims | jd__: you can even run oslo libraries against juno if you want. example https://review.openstack.org/#/c/165981/ | 14:47 |
| dhellmann | dims: we should make sure we have the right jobs set up against all of the libs stable branches -- do you know if that happens automatically or if we need to do something special? | 14:48 |
| openstackgerrit | Julien Danjou proposed openstack/oslo.serialization: jsonutils: replace strtime() usage with isotime() https://review.openstack.org/166861 | 14:48 |
| dims | dhellmann: need to take an inventory. don't know for sure | 14:49 |
| dims | dhellmann: will add to my todo list | 14:49 |
| *** bnemec has joined #openstack-oslo | 14:49 | |
| dhellmann | dims: ok, thanks | 14:50 |
| openstackgerrit | Julien Danjou proposed openstack-dev/pbr: Handle PEP426 markers https://review.openstack.org/165015 | 14:51 |
| openstackgerrit | Julien Danjou proposed openstack/oslo.serialization: jsonutils: simplify simple value handling https://review.openstack.org/166866 | 14:55 |
| *** pblaho has quit IRC | 15:05 | |
| sdague | does anyone here have pull with the eventlet team? This was proposed a year ago - https://github.com/eventlet/eventlet/pull/75 - and got a "will merge soon", and never did | 15:09 |
| sdague | and would simplify the oslo logging / service story with eventlet | 15:10 |
| *** tsekiyama_ has joined #openstack-oslo | 15:13 | |
| dhellmann | sdague: I don't know anyone on that team. haypo or harlowja may, since they've been looking at alternatives | 15:17 |
| openstackgerrit | Merged openstack/oslo.db: Implement generic update-on-match feature https://review.openstack.org/146228 | 15:19 |
| haypo | sdague: good luck :-p | 15:21 |
| haypo | sdague: https://github.com/temoto looks to be actively reviewing pull requests | 15:22 |
| *** sputnik13 has quit IRC | 15:22 | |
| haypo | sdague: my latest pull request, https://github.com/eventlet/eventlet/pull/187 | 15:22 |
| sdague | ok, well he was the one that said he'd merge it last Jan | 15:23 |
| sdague | I guess I'll just be patient | 15:23 |
| dims | sdague: build failed? https://travis-ci.org/eventlet/eventlet/builds/55026102 | 15:23 |
| sdague | dims: yeh, python3 races in one of the configs | 15:23 |
| sdague | it's unrelated, and hitting all the patches last time I looked | 15:23 |
| dims | i see | 15:23 |
| dims | he's published an email address... http://temoto.ru/ | 15:24 |
| *** sdague has quit IRC | 15:26 | |
| *** bogdando has quit IRC | 15:26 | |
| haypo | dims: with his phone number, you can call him! "review my pull request, immediatly!" | 15:26 |
| *** harlowja_at_home has joined #openstack-oslo | 15:30 | |
| dims | haypo: hahaha | 15:32 |
| *** exploreshaifali has joined #openstack-oslo | 15:36 | |
| dhellmann | haypo: were you going to update https://review.openstack.org/#/c/166835/ to use the fixture? I can do it, if you have other work you need to do | 15:38 |
| *** yamahata has joined #openstack-oslo | 15:39 | |
| *** sdague has joined #openstack-oslo | 15:40 | |
| haypo | dhellmann: i never used fixtures, i don't understand them :-p | 15:41 |
| *** tsekiyama_ is now known as tsekiyama | 15:41 | |
| haypo | dhellmann: i just wanted to fix oslo.utils, feel free to rewrite my simple patch | 15:41 |
| *** jaypipes has joined #openstack-oslo | 15:42 | |
| *** tsekiyama is now known as tsekiyama_ | 15:42 | |
| haypo | (i checked quickly if other tests use something call "fixture" with mock, but i gave up) | 15:42 |
| dhellmann | haypo: ok, I'll update it | 15:44 |
| dhellmann | jd__: looking at haypo's patch, it's actually going to be pretty ugly to rewrite it to use fixtures. I'm inclined to approve the fix, and rewrite it as a separate patch. Thoughts? | 15:52 |
| dhellmann | jd__: mostly because it wants to mock six.moves.builtins.open.return_value.__enter__.return_value.read to replace a file-reading call | 15:53 |
| dhellmann | and I can refactor that into a function in netutils that we can mock instead | 15:53 |
| *** YorikSar has quit IRC | 15:56 | |
| dims | dhellmann: am cool with a separate patch | 15:56 |
| dhellmann | dims: that's on https://review.openstack.org/#/c/166835/ | 15:56 |
| *** bogdando has joined #openstack-oslo | 15:57 | |
| dims | dhellmann: ack, saw jd__'s -1 | 15:57 |
| haypo | dhellmann: hum, it sounds complex to use fixtures | 15:57 |
| openstackgerrit | Julien Danjou proposed openstack/oslo.serialization: jsonutils: encode dict keys https://review.openstack.org/166902 | 15:57 |
| e0ne | hi! i can't find agenda for today's meeting at https://wiki.openstack.org/wiki/Meetings/Oslo. can anybody help me please? | 15:57 |
| haypo | dhellmann: fixtures are not supposed to make code shorter/simpler? :-p | 15:57 |
| jd__ | dhellmann: ok, feel free to bypass my -1 :) | 15:58 |
| dhellmann | haypo: they are, but in this case the tests are doing some weird things and the fixtures aren't structured to make that easy | 15:58 |
| dhellmann | jd__: ok, I didn't want to do that without chatting about it | 15:58 |
| dims | dhellmann: Elena has the SIGHUP one ready - https://review.openstack.org/#/c/165104/ | 15:59 |
| haypo | dhellmann: i like mock API, it's simple | 15:59 |
| haypo | dhellmann: if you don't want to repeat addCleanup(patch.stop), add a call to addCleanup(patch.stopall) in the parent test class | 16:00 |
| dhellmann | haypo: the trick there is getting the ordering right, but yeah | 16:01 |
| *** tsekiyama_ has quit IRC | 16:05 | |
| *** tsekiyama has joined #openstack-oslo | 16:07 | |
| jd__ | this does not look good http://logs.openstack.org/61/166861/1/check/gate-oslo.serialization-docs/d17dd51/console.html not sure it's related to my change | 16:18 |
| *** yamahata has quit IRC | 16:19 | |
| harlowja_at_home | jd__, stop breaking all the things, haha | 16:19 |
| *** yamahata has joined #openstack-oslo | 16:19 | |
| jd__ | :( | 16:20 |
| jd__ | I'm too agile for OpenStack | 16:20 |
| harlowja_at_home | def | 16:21 |
| harlowja_at_home | lol | 16:21 |
| *** sreshetn1 has joined #openstack-oslo | 16:31 | |
| *** harlowja_at_home has quit IRC | 16:34 | |
| *** YorikSar has joined #openstack-oslo | 16:34 | |
| *** yamahata has quit IRC | 16:41 | |
| *** yamahata has joined #openstack-oslo | 16:41 | |
| *** ajo has quit IRC | 16:42 | |
| openstackgerrit | Merged openstack/oslo.messaging: Remove 'UNIQUE_ID is %s' logging https://review.openstack.org/166372 | 16:42 |
| dims | jd__: haha | 16:50 |
| *** ihrachyshka has quit IRC | 16:51 | |
| openstackgerrit | Doug Hellmann proposed openstack/oslo.utils: Clean up TestIsIPv6Enabled https://review.openstack.org/166924 | 16:52 |
| dhellmann | haypo: ^^ | 16:52 |
| dhellmann | jd__, dims : ^^ | 16:52 |
| *** yamahata has quit IRC | 16:54 | |
| *** yamahata has joined #openstack-oslo | 16:54 | |
| openstackgerrit | Doug Hellmann proposed openstack/oslo-specs: Change oslo.log to keep other libraries quiet in the logs https://review.openstack.org/164851 | 16:57 |
| *** e0ne is now known as e0ne_ | 16:57 | |
| *** e0ne_ is now known as e0ne | 16:58 | |
| haypo | jd__, dims : hey, you approved the change before i had time to send my review :-p | 17:03 |
| dims | haypo: i +2 | 17:03 |
| dims | 'ed first so it's jd__'s fault :) | 17:03 |
| haypo | dims: don't worry, it's always jd__'s fault | 17:03 |
| jd__ | I know, I know | 17:04 |
| haypo | :) | 17:04 |
| haypo | at least you know | 17:04 |
| jd__ | "but it works now!" "yeah that's jd again" | 17:04 |
| *** yamahata has quit IRC | 17:04 | |
| openstackgerrit | Merged openstack/taskflow: Add newline to avoid sphinx warning https://review.openstack.org/164006 | 17:05 |
| *** exploreshaifali_ has joined #openstack-oslo | 17:06 | |
| *** exploreshaifali has quit IRC | 17:07 | |
| *** vigneshvar has joined #openstack-oslo | 17:11 | |
| openstackgerrit | Julien Danjou proposed openstack/oslo.serialization: jsonutils: encode dict keys https://review.openstack.org/166902 | 17:15 |
| * dims bows to jd__ | 17:16 | |
| *** amotoki has quit IRC | 17:19 | |
| dhellmann | haypo, jd__ : when I use StringIO I get "StringIO instance has no attribute __exit__ | 17:19 |
| *** sputnik13 has joined #openstack-oslo | 17:21 | |
| *** harlowja has joined #openstack-oslo | 17:27 | |
| *** achanda has joined #openstack-oslo | 17:29 | |
| *** yassine has quit IRC | 17:30 | |
| krotscheck | Random historical question: Oslo libs used to be installed via copy/commit to source. How did that come about? | 17:35 |
| dhellmann | krotscheck: http://specs.openstack.org/openstack/oslo-specs/specs/policy/incubator.html | 17:36 |
| *** e0ne has quit IRC | 17:36 | |
| *** amotoki has joined #openstack-oslo | 17:36 | |
| krotscheck | dhellmann: Interesting. So it's basically an approval buffer. | 17:36 |
| dhellmann | krotscheck: right, and a way to make breaking changes to the API before declaring it stable | 17:37 |
| krotscheck | Cool. I was hoping it was a different reason, because I want to strengthen my argument to do something similar for js. | 17:37 |
| *** sreshetn1 has quit IRC | 17:38 | |
| *** khivin has quit IRC | 17:40 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add in memory filesystem clearing https://review.openstack.org/166946 | 17:46 |
| sigmavirus24 | krotscheck: use Go as your go to example for why vendoring is good | 17:46 |
| sigmavirus24 | =P | 17:46 |
| krotscheck | vendoring? | 17:46 |
| sigmavirus24 | copying a dependency into place and committing it | 17:46 |
| krotscheck | sigmavirus24: Right. That's what I was going to propose. | 17:47 |
| sigmavirus24 | krotscheck: Go is formalizing how that works | 17:47 |
| harlowja | i'm not sure everyone will agree that vendoring is good still :-P | 17:47 |
| sigmavirus24 | harlowja: sssssh | 17:47 |
| krotscheck | There are disagreements. | 17:47 |
| sigmavirus24 | breaking migration libraries in patch versions is good so why isn't vendoring? | 17:47 |
| harlowja | if google jumps off a cliff, would u | 17:47 |
| harlowja | lol | 17:47 |
| sigmavirus24 | harlowja: google wouldn't jump off a cliff though, "Do No Evil" | 17:48 |
| * sigmavirus24 is being sarcastic in case anyone missed it | 17:48 | |
| harlowja | ;) | 17:48 |
| harlowja | self-jumping off a cliff is evil? | 17:48 |
| sigmavirus24 | harlowja: according to some. there are disagreements | 17:48 |
| harlowja | :) | 17:48 |
| krotscheck | sigmavirus24: I'm confused. Do you like vendoring or not? | 17:49 |
| sigmavirus24 | krotscheck: I tolerate it | 17:49 |
| sigmavirus24 | I neither like nor dislike it | 17:49 |
| krotscheck | (in ref to 'im being sarcastic') | 17:49 |
| *** yamahata has joined #openstack-oslo | 17:49 | |
| krotscheck | Gotcha | 17:49 |
| krotscheck | I would prefer for bower to use a signed code repository. | 17:49 |
| sigmavirus24 | I recognize the fact that it is useful and that debian does it in spite of their clear guidelines not to | 17:49 |
| krotscheck | So that we can mirror it without someon doing a git history rewrite and messing everything up. | 17:50 |
| harlowja | imho if its done at the right level, then sure its ok (but i think that level is the container/vm level, not the library level) | 17:50 |
| krotscheck | But that's sadly not the case. | 17:50 |
| harlowja | vendoring stuff for your own laziness, not a good reason imho, vendoring stuff at deployment time, sure ok | 17:50 |
| sigmavirus24 | harlowja: that's a much clearer explanation. I was confused what you meant | 17:51 |
| krotscheck | I don't think it's a lazyness issue. it's a question of "Is your code trustworthy enough to package on CentOS/Debian" | 17:51 |
| krotscheck | And, well, if you do a build-time resolution against a git repository, like bower does, then no: your code isn't trustworthy. | 17:51 |
| sigmavirus24 | krotscheck: for what it's worth, I maintain requests. It has a vendoring policy set by the original author who refuses to change it, so we've made it easy for people to remove vendored dependencies | 17:52 |
| harlowja | sounds like said person needs to take some confidence drugs | 17:52 |
| sigmavirus24 | (Without munging imports that is) | 17:52 |
| harlowja | not the ones they sell u on TV, lol | 17:52 |
| krotscheck | sigmavirus24: I'm definitely a fan of a two-pronged approach :) | 17:53 |
| krotscheck | Ultimately, I'd love to cater to all relevant audiences: Packagers & the JS community. | 17:53 |
| sigmavirus24 | krotscheck: won't happen | 17:53 |
| krotscheck | I think it's possible. | 17:53 |
| openstackgerrit | Doug Hellmann proposed openstack/oslo.utils: Clean up TestIsIPv6Enabled https://review.openstack.org/166924 | 17:53 |
| sigmavirus24 | krotscheck: use JS{L,H}int and see how far you get ;P | 17:54 |
| dhellmann | jd__, haypo, dims : I found a work-around | 17:54 |
| harlowja | krotscheck https://github.com/kennethreitz/requests/pull/1812 ;) | 17:54 |
| * dhellmann didn't plan to spend his day fiddling with old unit tests | 17:54 | |
| krotscheck | sigmavirus24: You mean eslint? | 17:54 |
| krotscheck | :D | 17:54 |
| sigmavirus24 | krotscheck: the community prefers JSHint and that's the standard you should be using | 17:55 |
| sigmavirus24 | =P | 17:55 |
| sigmavirus24 | harlowja: yep | 17:55 |
| harlowja | ^ fun read ;-P | 17:55 |
| krotscheck | Your logical fallacy is straw man. We're not talking about tooling choices. | 17:55 |
| krotscheck | We're talking about making a single javascript project's codebase trustworthy enough for packagers, but still deployable to either npm or bower. | 17:56 |
| sigmavirus24 | krotscheck: there are common grounds you won't find is all I'm saying | 17:56 |
| harlowja | do packagers have trust issues? | 17:56 |
| harlowja | maybe they should see someone about that | 17:56 |
| harlowja | ^ thats my recommended fix, lol | 17:56 |
| *** achanda has quit IRC | 17:57 | |
| *** stevemar has quit IRC | 18:02 | |
| *** stevemar has joined #openstack-oslo | 18:03 | |
| *** ChuckC has quit IRC | 18:08 | |
| *** achanda has joined #openstack-oslo | 18:15 | |
| *** flashgordon is now known as jogo | 18:16 | |
| openstackgerrit | Joshua Harlow proposed openstack/oslo.messaging: Use the oslo_utils stop watch in decaying timer https://review.openstack.org/166951 | 18:17 |
| *** harlowja has quit IRC | 18:18 | |
| *** achanda has quit IRC | 18:19 | |
| *** achanda has joined #openstack-oslo | 18:20 | |
| *** harlowja has joined #openstack-oslo | 18:21 | |
| openstackgerrit | Merged openstack/taskflow: Allow using shallow copy instead of deep copy https://review.openstack.org/165693 | 18:23 |
| openstackgerrit | Merged openstack/taskflow: By default use a in memory backend (when none is provided) https://review.openstack.org/165654 | 18:27 |
| *** achanda has quit IRC | 18:27 | |
| *** achanda has joined #openstack-oslo | 18:28 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Just unify having a single requirements.txt file https://review.openstack.org/164523 | 18:34 |
| *** vigneshvar has quit IRC | 18:54 | |
| *** tsekiyama is now known as tsekiyama_ | 18:55 | |
| lifeless | dhellmann: ask me wha? | 18:56 |
| dhellmann | lifeless: jd__ has a patch up to pbr to change the way environment markers are handled, basically stripping them out before sending the values to setuptools. I thought setuptools understood them, though, so didn't know why we would do that. According to mordred most of that part of the code is really not used and could probably be deleted since pip does understand the environment markers. | 19:01 |
| *** tsekiyama_ is now known as tsekiyama | 19:02 | |
| *** mfedosin has quit IRC | 19:04 | |
| *** frickler has quit IRC | 19:04 | |
| *** SpamapS has quit IRC | 19:04 | |
| *** SpamapS has joined #openstack-oslo | 19:04 | |
| *** SpamapS has joined #openstack-oslo | 19:04 | |
| *** pasquier-s has quit IRC | 19:04 | |
| *** otherwiseguy has quit IRC | 19:04 | |
| *** frickler has joined #openstack-oslo | 19:04 | |
| *** mfedosin has joined #openstack-oslo | 19:04 | |
| *** pasquier-s has joined #openstack-oslo | 19:04 | |
| *** otherwiseguy has joined #openstack-oslo | 19:04 | |
| lifeless | dhellmann: ack. jd__ - can you consult with dstufft on the rightness of that patch? | 19:09 |
| jd__ | dhellmann: as I said on openstack-dev, as far as I can tell my patch works, if it's not clear, using pip install -e . it does what's expected | 19:11 |
| jd__ | just a reminder if it wasn't clear :) | 19:11 |
| dhellmann | jd__: yeah, I think lifeless just came online and was catching up on the discussion | 19:12 |
| *** sreshetn1 has joined #openstack-oslo | 19:18 | |
| *** rushiagr_away is now known as rushiagr | 19:23 | |
| *** prad has quit IRC | 19:32 | |
| *** rushiagr is now known as rushiagr_away | 19:34 | |
| *** alexpilotti has quit IRC | 19:39 | |
| *** ajo has joined #openstack-oslo | 19:43 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Make the storage layer more resilent to failures https://review.openstack.org/166990 | 19:57 |
| *** jgrimm is now known as zz_jgrimm | 20:05 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Make the storage layer more resilent to failures https://review.openstack.org/166990 | 20:05 |
| *** andreykurilin_ has joined #openstack-oslo | 20:15 | |
| *** kgiusti has left #openstack-oslo | 20:19 | |
| *** andreykurilin_ has quit IRC | 20:19 | |
| *** andreykurilin__ has joined #openstack-oslo | 20:19 | |
| *** e0ne has joined #openstack-oslo | 20:21 | |
| *** crc32 has joined #openstack-oslo | 20:22 | |
| *** crc32 has quit IRC | 20:23 | |
| *** amrith is now known as _amrith_ | 20:27 | |
| *** ndipanov has quit IRC | 20:29 | |
| *** crc32 has joined #openstack-oslo | 20:36 | |
| *** andreykurilin__ has quit IRC | 20:37 | |
| *** andreykurilin_ has joined #openstack-oslo | 20:37 | |
| *** exploreshaifali_ has quit IRC | 20:41 | |
| *** stpierre has quit IRC | 20:46 | |
| *** jaypipes has quit IRC | 20:47 | |
| *** stpierre has joined #openstack-oslo | 20:48 | |
| *** stpierre has quit IRC | 20:48 | |
| *** stpierre has joined #openstack-oslo | 20:48 | |
| *** vipuls is now known as vipul | 20:49 | |
| *** jaypipes has joined #openstack-oslo | 20:52 | |
| *** fesp has joined #openstack-oslo | 20:56 | |
| *** jamielennox|away is now known as jamielennox | 21:02 | |
| *** cdent has quit IRC | 21:09 | |
| *** prad has joined #openstack-oslo | 21:20 | |
| *** sreshetn1 has quit IRC | 21:23 | |
| *** yamahata has quit IRC | 21:35 | |
| *** stpierre has quit IRC | 21:35 | |
| *** yamahata has joined #openstack-oslo | 21:36 | |
| *** exploreshaifali_ has joined #openstack-oslo | 21:36 | |
| *** yamahata has quit IRC | 21:37 | |
| *** yamahata has joined #openstack-oslo | 21:38 | |
| *** exploreshaifali_ has quit IRC | 21:39 | |
| dhellmann | harlowja: I'm thinking about where to put the versionutils module when we graduate it. Can you imagine any scenario in which it would be useful to have a wrapper around debtcollector that knows about openstack versions? like, would we want to emit messages such as "this module is going away in M"? | 21:43 |
| harlowja | dhellmann either a wrapper or there could be a new decorator param 'to_log' or something that gets used also? | 21:44 |
| harlowja | or wrapper + said param | 21:45 |
| harlowja | https://docs.python.org/2/library/logging.html#logging.captureWarnings exists, but its pretty primitivie (the impl is sorta not so great imho) | 21:45 |
| dhellmann | harlowja: yeah, I wasn't thinking of adding this to debtcollector, just trying to decide if a standalone lib made sense | 21:45 |
| dhellmann | it's too small to be worth that at this point, but if we were going to add to it I could see doing that | 21:45 |
| lifeless | jd__: yeah, so I'm confused if it works if you're stripping the environment markers ;) | 21:45 |
| harlowja | could just be something small enough to include; seems well isolated | 21:45 |
| dhellmann | harlowja: it brings in a dependency on oslo.config, and ties debtcollector to openstack version names | 21:46 |
| harlowja | hmmm | 21:46 |
| dhellmann | I assumed we wouldn't want that for debtcollector as it is now | 21:46 |
| harlowja | hmmm, so whats the dependency on oslo.config for? | 21:47 |
| harlowja | guess 'fatal_deprecations' | 21:47 |
| dhellmann | right | 21:47 |
| dhellmann | harlowja: I'll think about this more tomorrow, I need to go help prep the house for guests | 21:47 |
| dhellmann | harlowja: I'm thinking at this point that we just move it to oslo.log or oslo.config | 21:48 |
| harlowja | if there is a 'logger' handle that gets passed in to debtcollector stuff, i'm wondering if that could just handle that configuration | 21:48 |
| dhellmann | one less tiny library | 21:48 |
| harlowja | and then debtcollector just doesn't know about what that logger is doing | 21:48 |
| dhellmann | harlowja: the config option has to live somewhere, though | 21:48 |
| dhellmann | and if that lives somewhere, then the associated code might as well live there, too | 21:48 |
| dhellmann | and leave debtcollector independent of openstack | 21:48 |
| * dhellmann has to run | 21:49 | |
| harlowja | kk | 21:49 |
| harlowja | until next time! | 21:49 |
| harlowja | will think about it | 21:49 |
| harlowja | why i think the logging.captureWarnings stuff not so good ( https://hg.python.org/releasing/3.4/file/8671f89107c8/Lib/logging/__init__.py#l1913 ) | 21:50 |
| harlowja | sorta meh meh, monkey patchey | 21:50 |
| harlowja | but maybe should be a oslo.version anyway, with all things releated to openstack versions/comparison functions... | 21:51 |
| *** _amrith_ is now known as amrith | 21:55 | |
| openstackgerrit | Merged openstack/oslo.utils: Fix test_netutils: stop patches https://review.openstack.org/166835 | 21:57 |
| *** sigmavirus24 is now known as sigmavirus24_awa | 22:07 | |
| *** andreykurilin_ has quit IRC | 22:15 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Make the storage layer more resilent to failures https://review.openstack.org/166990 | 22:20 |
| *** gordc has quit IRC | 22:23 | |
| openstackgerrit | Eric Brown proposed openstack/oslo.config: Allow case insensitive compare of choice values https://review.openstack.org/167025 | 22:28 |
| *** prad has quit IRC | 22:31 | |
| *** salv-orl_ has joined #openstack-oslo | 22:34 | |
| *** salv-orlando has quit IRC | 22:34 | |
| *** salv-orl_ has quit IRC | 22:38 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add ability to use tooz to find and advertise workers https://review.openstack.org/151495 | 22:40 |
| *** jecarey has quit IRC | 22:41 | |
| *** salv-orlando has joined #openstack-oslo | 22:42 | |
| *** crc32 has quit IRC | 22:53 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Allow providing a node stringify function to tree pformat https://review.openstack.org/167040 | 23:23 |
| *** tsekiyam_ has joined #openstack-oslo | 23:32 | |
| *** tsekiyama has quit IRC | 23:35 | |
| *** mtanino has quit IRC | 23:35 | |
| *** tsekiyam_ has quit IRC | 23:36 | |
| *** zzzeek has quit IRC | 23:39 | |
| *** jaosorior has quit IRC | 23:42 | |
| *** dims_ has joined #openstack-oslo | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!