| *** bknudson has joined #openstack-oslo | 00:00 | |
| *** bknudson1 has joined #openstack-oslo | 00:03 | |
| *** bknudson has quit IRC | 00:05 | |
| *** miqui_ has joined #openstack-oslo | 00:12 | |
| *** noelbk has quit IRC | 00:13 | |
| *** takedakn has joined #openstack-oslo | 00:14 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Remove 'SaveOrderTask' and test state in class variables https://review.openstack.org/147038 | 00:16 |
|---|---|---|
| *** haypo_ is now known as haypo | 00:19 | |
| *** zz_sabari is now known as sabari | 00:21 | |
| *** prad has quit IRC | 00:23 | |
| *** zzzeek has quit IRC | 00:24 | |
| *** takedakn has quit IRC | 00:25 | |
| *** david-lyle has joined #openstack-oslo | 00:25 | |
| *** takedakn has joined #openstack-oslo | 00:33 | |
| *** rodrigod` is now known as rodrigods | 00:41 | |
| *** takedakn has quit IRC | 00:45 | |
| *** dims has quit IRC | 00:46 | |
| *** sabari is now known as zz_sabari | 00:50 | |
| *** noelbk has joined #openstack-oslo | 00:51 | |
| *** zz_sabari is now known as sabari | 00:51 | |
| *** salv-orlando has quit IRC | 00:53 | |
| *** zzzeek has joined #openstack-oslo | 00:59 | |
| *** achanda has joined #openstack-oslo | 01:01 | |
| *** achanda has quit IRC | 01:01 | |
| *** achanda_ has quit IRC | 01:04 | |
| *** noelbk has quit IRC | 01:04 | |
| *** david-lyle has quit IRC | 01:08 | |
| openstackgerrit | Michael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module https://review.openstack.org/138215 | 01:08 |
| *** zzzeek has quit IRC | 01:13 | |
| *** tsekiyama has joined #openstack-oslo | 01:16 | |
| *** mtanino has quit IRC | 01:16 | |
| *** tsekiyam_ has quit IRC | 01:19 | |
| *** tsekiyama has quit IRC | 01:20 | |
| *** david-lyle has joined #openstack-oslo | 01:26 | |
| *** stevemar has joined #openstack-oslo | 01:27 | |
| *** dims has joined #openstack-oslo | 01:29 | |
| *** boris-42 has quit IRC | 01:33 | |
| *** openstack has joined #openstack-oslo | 01:35 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Tidy up the WBE cache (now WBE types) module https://review.openstack.org/103245 | 01:39 |
| *** dims has quit IRC | 01:42 | |
| *** dims has joined #openstack-oslo | 01:42 | |
| *** dims has quit IRC | 01:43 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Tidy up the WBE cache (now WBE types) module https://review.openstack.org/103245 | 01:44 |
| *** kgiusti has quit IRC | 01:46 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Tidy up the WBE cache (now WBE types) module https://review.openstack.org/103245 | 01:47 |
| *** dims has joined #openstack-oslo | 01:50 | |
| *** david-lyle has quit IRC | 02:01 | |
| *** takedakn has joined #openstack-oslo | 02:25 | |
| *** dims has quit IRC | 02:29 | |
| *** takedakn has quit IRC | 02:29 | |
| *** takedakn has joined #openstack-oslo | 02:39 | |
| *** stevemar has quit IRC | 02:41 | |
| *** david-lyle has joined #openstack-oslo | 02:42 | |
| *** david-lyle has quit IRC | 02:42 | |
| *** stevemar has joined #openstack-oslo | 02:42 | |
| *** takedakn has quit IRC | 02:44 | |
| *** yamahata has quit IRC | 02:45 | |
| *** harlowja is now known as harlowja_away | 02:57 | |
| openstackgerrit | Paul Belanger proposed openstack/oslo.utils: Add microsecond support to timeutils.utcnow_ts() https://review.openstack.org/147736 | 03:24 |
| *** salv-orlando has joined #openstack-oslo | 03:27 | |
| *** salv-orlando has quit IRC | 03:30 | |
| *** salv-orlando has joined #openstack-oslo | 03:35 | |
| *** salv-orlando has quit IRC | 03:39 | |
| *** salv-orlando has joined #openstack-oslo | 03:43 | |
| *** salv-orlando has quit IRC | 03:48 | |
| *** burgobianco has joined #openstack-oslo | 03:56 | |
| *** rushiagr_away is now known as rushiagr | 04:11 | |
| *** amotoki has quit IRC | 04:23 | |
| *** amotoki has joined #openstack-oslo | 04:42 | |
| *** harlowja_at_home has joined #openstack-oslo | 04:42 | |
| *** boris-42 has joined #openstack-oslo | 04:43 | |
| *** achanda has joined #openstack-oslo | 04:44 | |
| *** yamahata has joined #openstack-oslo | 04:44 | |
| *** achanda has quit IRC | 04:46 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Use constants for runner state machine event names https://review.openstack.org/147698 | 04:56 |
| *** achanda has joined #openstack-oslo | 04:57 | |
| *** achanda has quit IRC | 04:57 | |
| *** harlowja_at_home has quit IRC | 04:59 | |
| *** achanda has joined #openstack-oslo | 05:19 | |
| *** sabari is now known as zz_sabari | 05:24 | |
| *** takedakn has joined #openstack-oslo | 05:53 | |
| *** vigneshvar_ has joined #openstack-oslo | 06:02 | |
| *** achanda has quit IRC | 06:03 | |
| *** achanda has joined #openstack-oslo | 06:05 | |
| *** rushiagr is now known as rushiagr_away | 06:05 | |
| *** rpodolyaka1 has joined #openstack-oslo | 06:05 | |
| *** takedakn has quit IRC | 06:06 | |
| *** rushiagr_away is now known as rushiagr | 06:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.vmware: Imported Translations from Transifex https://review.openstack.org/147755 | 06:09 |
| *** rpodolyaka1 has quit IRC | 06:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.messaging: Imported Translations from Transifex https://review.openstack.org/143256 | 06:14 |
| *** rpodolyaka1 has joined #openstack-oslo | 06:15 | |
| *** achanda has quit IRC | 06:16 | |
| *** achanda has joined #openstack-oslo | 06:23 | |
| *** zz_sabari is now known as sabari | 06:25 | |
| *** inc0 has joined #openstack-oslo | 06:41 | |
| inc0 | good morning | 06:42 |
| openstackgerrit | Merged openstack/taskflow: Use constants for runner state machine event names https://review.openstack.org/147698 | 06:43 |
| *** takedakn has joined #openstack-oslo | 06:44 | |
| *** takedakn has quit IRC | 06:45 | |
| *** takedakn has joined #openstack-oslo | 06:46 | |
| *** achanda has quit IRC | 06:46 | |
| *** takedakn has quit IRC | 06:50 | |
| *** salv-orlando has joined #openstack-oslo | 06:56 | |
| *** rpodolyaka1 has quit IRC | 06:57 | |
| *** rpodolyaka1 has joined #openstack-oslo | 07:03 | |
| *** salv-orlando has quit IRC | 07:17 | |
| *** flaper87 has quit IRC | 07:19 | |
| *** flaper87 has joined #openstack-oslo | 07:19 | |
| *** burgobianco has quit IRC | 07:44 | |
| *** burgobianco has joined #openstack-oslo | 07:45 | |
| *** noelbk has joined #openstack-oslo | 08:04 | |
| *** rpodolyaka1 has quit IRC | 08:05 | |
| *** stevemar has quit IRC | 08:17 | |
| *** oomichi has quit IRC | 08:18 | |
| *** yamahata has quit IRC | 08:19 | |
| *** burgobianco has quit IRC | 08:21 | |
| *** cburgess_ has quit IRC | 08:21 | |
| *** morganfainberg has quit IRC | 08:21 | |
| *** morganfainberg has joined #openstack-oslo | 08:21 | |
| *** dtantsur|afk is now known as dtantsur | 08:22 | |
| *** sabari is now known as zz_sabari | 08:22 | |
| *** cburgess has joined #openstack-oslo | 08:23 | |
| *** burgobianco has joined #openstack-oslo | 08:26 | |
| haypo | sileht: hi. i have questions for your listener test patch, https://review.openstack.org/#/c/145798/5/oslo_messaging/tests/notify/test_listener.py | 08:36 |
| sileht | haypo, yep | 08:37 |
| haypo | sileht: i don't know if there are bug or not, but my suggestions may avoid tricky race conditions? | 08:38 |
| sileht | haypo, oh yes, I'm fixed them | 08:41 |
| openstackgerrit | Vipin Balachandran proposed openstack/oslo.vmware: Fix race during session creation https://review.openstack.org/146090 | 08:42 |
| *** ajo has joined #openstack-oslo | 08:44 | |
| *** dtantsur is now known as dtantsur|brb | 08:46 | |
| openstackgerrit | Victor Stinner proposed openstack/oslo.messaging: Adjust tests for the new namespace https://review.openstack.org/147549 | 08:47 |
| openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.messaging: Fixes test_two_pools_three_listener https://review.openstack.org/145798 | 08:48 |
| haypo | sileht: you don't want to protect the read access with the lock (wait_for_messages) | 08:49 |
| haypo | ? | 08:49 |
| sileht | haypo, not really important, it retries :) | 08:49 |
| haypo | sileht: ok | 08:49 |
| haypo | sileht: did you run the tests? (i hesite to vote before jenkins ran the tests) | 08:50 |
| inc0 | hi, quick question about graduating. So I'm cloning oslo-incubator and another copy of it with name of library. When do I copy actual library content? How do I point out files in other project I want to extract to oslo? | 08:50 |
| sileht | haypo, yes | 08:50 |
| *** jaosorior has joined #openstack-oslo | 09:07 | |
| *** takedakn has joined #openstack-oslo | 09:20 | |
| *** dulek has joined #openstack-oslo | 09:20 | |
| *** pblaho has joined #openstack-oslo | 09:25 | |
| openstackgerrit | Julien Danjou proposed openstack/tooz: Add a file based driver https://review.openstack.org/146507 | 09:31 |
| *** Guest49876 is now known as d0ugal | 09:36 | |
| *** d0ugal has quit IRC | 09:36 | |
| *** d0ugal has joined #openstack-oslo | 09:36 | |
| openstackgerrit | Victor Stinner proposed openstack/oslo.concurrency: Port processutils to Python 3 (new try) https://review.openstack.org/146984 | 09:37 |
| openstackgerrit | Victor Stinner proposed openstack/oslo.concurrency: Port processutils to Python 3 (new try) https://review.openstack.org/146984 | 09:40 |
| haypo | dhellmann: hi. ^^ here is my updated patch for oslo.concurrency if you want to take a look. it's rebased on master | 09:40 |
| haypo | i tried to sum up the python 2/3 and mask_password() issues in the changelog | 09:41 |
| *** inc0 has quit IRC | 09:44 | |
| *** david-lyle has joined #openstack-oslo | 09:46 | |
| haypo | jd__: hi. can you please review https://review.openstack.org/#/c/136653/ ? "Add a new aioeventlet executor" | 09:49 |
| haypo | jd__: sileht already voted +2 | 09:49 |
| haypo | dhellmann was supposed to take a look yesterday, but i guess that he had more urgent things to do ;) | 09:50 |
| jd__ | haypo: sure | 09:51 |
| jd__ | haypo: though tests are failing? | 09:51 |
| haypo | jd__: the fail on python3 is unrelated to my change, | 09:52 |
| haypo | jd__: it's the random test_two_pools_three_listener failure. sileht already proposed a fix, https://review.openstack.org/145798 | 09:52 |
| jd__ | ok | 09:52 |
| haypo | jd__: you may also review this one ;) | 09:52 |
| jd__ | haypo: btw I wanted to tell I used trollius in Gnocchi too ;) | 09:52 |
| haypo | jd__: oh? really? great! | 09:52 |
| jd__ | haypo: https://review.openstack.org/#/c/146117/4/gnocchi/statsd.py FWIW | 09:53 |
| haypo | i expected asyncio/trollius to be stable, but last days i got a lot of bug reports, especially resource leaks | 09:53 |
| haypo | and race conditions when a task is cancelled because of a timeout | 09:54 |
| haypo | i just finished to fix most of these (or all) issues, in asyncio and trollius | 09:54 |
| jd__ | cool | 09:54 |
| jd__ | if you fixed things in asyncio that's good sign too :) | 09:54 |
| haypo | jd__: a few months ago, there were minor additions, or even subtle changes, in the API | 09:55 |
| haypo | jd__: now we are now focused on fixing issues | 09:55 |
| haypo | i guess that we start to get real bug reports because more and more people are using asyncio | 09:55 |
| haypo | including people deploying asyncio in production | 09:56 |
| jd__ | very likely indeed | 09:56 |
| haypo | (i know that asyncio is used on some large projects deployed on production) | 09:56 |
| haypo | jd__: so, do you like asyncio API? is it better than eventlet or just completly different? | 09:56 |
| haypo | i didn't design the API, but i really like it. it's well designed and smarter than all callback-based API like Twisted, Javascript things, etc. | 09:57 |
| haypo | sileht: another test_two_pools_three_listener failure on the py33 gate for https://review.openstack.org/#/c/147512/ :-/ | 09:58 |
| sileht | ahahharg | 09:58 |
| haypo | sileht: if your fix for test_two_pools_three_listener is accepted, will it be added to the end of the queue of patches to be merged? | 09:58 |
| haypo | sileht: is it possible to ask to add it at the beginning of the queue? | 09:59 |
| *** andreykurilin_ has joined #openstack-oslo | 09:59 | |
| haypo | (i just added "reverify no bug") | 10:00 |
| sileht | haypo, it's not really a problem, when the fix have been merged, we just have to do recheck | 10:00 |
| sileht | the patch will be rebased automatically | 10:00 |
| haypo | sileht: does jenkins try to merge approved changes the exact order of the acceptance? | 10:01 |
| sileht | haypo, nop | 10:01 |
| haypo | sileht: oh cool | 10:01 |
| sileht | haypo, you can check the queue http://status.openstack.org/zuul/ | 10:02 |
| haypo | sileht: yeah, i saw the eventlet 0.16 peak on this page, on the graph at the bottom | 10:02 |
| haypo | (well, i think that the peak was related to the eventlet 0.16 issue on nova) | 10:03 |
| *** ihrachyshka has joined #openstack-oslo | 10:04 | |
| haypo | jd__: there are too many issues with python 3, so my new executor is only usable with python 2 right now | 10:05 |
| jd__ | haypo: ok | 10:05 |
| haypo | jd__: i'm trying to fix remaining eventlet issues, i have a pending pull request on eventlet | 10:05 |
| jd__ | haypo: but that means trollius will be installed and used with Python 3 anyway, is this a good idea? | 10:06 |
| jd__ | because oslo.messaging is Python 3 enabled | 10:06 |
| haypo | jd__: trollius works on python 3, even on python 3.4+ | 10:06 |
| jd__ | haypo: will it work better than asyncio here? | 10:06 |
| *** e0ne has joined #openstack-oslo | 10:07 | |
| haypo | jd__: if you want to use asyncio or trollius depending on the python version, you cannot write coroutines | 10:07 |
| haypo | http://trollius.readthedocs.org/asyncio.html#differences-between-trollius-and-tulip | 10:07 |
| haypo | jd__: whereas it really like coroutines | 10:07 |
| *** david-lyle has quit IRC | 10:08 | |
| haypo | i* really like | 10:08 |
| *** takedakn has quit IRC | 10:08 | |
| jd__ | haypo: ok thanks, that makes sense | 10:09 |
| haypo | jd__: aioeventlet uses asyncio if available, or fallback to trollius | 10:11 |
| openstackgerrit | Vipin Balachandran proposed openstack/oslo.vmware: Fix race during session creation https://review.openstack.org/146090 | 10:21 |
| haypo | jd__: oh. you approved my change, but it depends on aioeventlet which is not in global requirements yet | 10:21 |
| haypo | jd__: please see https://review.openstack.org/#/c/138750/ for global requirements | 10:22 |
| jd__ | I can do that too | 10:25 |
| haypo | jd__: i'm not sure that it's possible to use aioeventlet with trollius on python 3.4+ | 10:26 |
| haypo | jd__: it looks like aioeventlet prefers asyncio if available, over trollius | 10:26 |
| haypo | i will investigate this issue to check if it's really an issue or not :) | 10:26 |
| jd__ | yeah you still have plenty of time to test and fix that | 10:26 |
| jd__ | you're far in advance :p | 10:26 |
| haypo | jd__: what? python 3 is the present! even python 3.4 | 10:27 |
| *** e0ne is now known as e0ne_ | 10:28 | |
| openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.middleware: Add healthcheck middleware https://review.openstack.org/147805 | 10:28 |
| openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.messaging: Deprecates the localcontext API https://review.openstack.org/140616 | 10:37 |
| *** e0ne_ has quit IRC | 10:38 | |
| *** noelbk has quit IRC | 10:52 | |
| *** salv-orlando has joined #openstack-oslo | 11:06 | |
| openstackgerrit | Merged openstack/oslo.messaging: fix qpid test issue with eventlet monkey patching https://review.openstack.org/147512 | 11:07 |
| *** dulek has quit IRC | 11:13 | |
| *** e0ne has joined #openstack-oslo | 11:16 | |
| *** david-lyle has joined #openstack-oslo | 11:17 | |
| *** andreykurilin_ has quit IRC | 11:18 | |
| *** alexpilotti has joined #openstack-oslo | 11:23 | |
| *** salv-orlando has quit IRC | 11:35 | |
| *** salv-orlando has joined #openstack-oslo | 11:35 | |
| *** dtantsur|brb is now known as dtantsur | 11:48 | |
| *** dims__ has joined #openstack-oslo | 11:53 | |
| *** salv-orlando has quit IRC | 11:55 | |
| *** dulek has joined #openstack-oslo | 12:01 | |
| *** kbyrne has quit IRC | 12:06 | |
| *** kbyrne has joined #openstack-oslo | 12:08 | |
| openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.messaging: Declare DirectPublisher exchanges with passive=True https://review.openstack.org/109373 | 12:26 |
| *** inc0 has joined #openstack-oslo | 12:34 | |
| *** rushiagr is now known as rushiagr_away | 12:48 | |
| *** salv-orlando has joined #openstack-oslo | 13:05 | |
| *** david-lyle has quit IRC | 13:11 | |
| *** ihrachyshka has quit IRC | 13:13 | |
| *** ihrachyshka has joined #openstack-oslo | 13:13 | |
| *** ihrachyshka has quit IRC | 13:22 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/141509 | 13:28 |
| openstackgerrit | Merged openstack/oslo.messaging: Fixes test_two_pools_three_listener https://review.openstack.org/145798 | 13:34 |
| *** jaypipes has joined #openstack-oslo | 13:35 | |
| *** salv-orlando has quit IRC | 13:36 | |
| *** gordc has joined #openstack-oslo | 13:40 | |
| *** ihrachyshka has joined #openstack-oslo | 13:41 | |
| haypo | sileht: "Merged openstack/oslo.messaging: Fixes test_two_pools_three_listener" cool :) | 13:43 |
| *** rpodolyaka1 has joined #openstack-oslo | 13:44 | |
| *** rpodolyaka1 has quit IRC | 13:44 | |
| *** rpodolyaka1 has joined #openstack-oslo | 13:45 | |
| *** rushiagr_away is now known as rushiagr | 13:46 | |
| haypo | sdague: hi. can you please review/accept https://review.openstack.org/#/c/138750/ ? "Add aioeventlet dependency" | 13:47 |
| haypo | sdague: tell me if you need more context | 13:47 |
| *** jaypipes is now known as leakypipes | 13:49 | |
| *** mriedem has joined #openstack-oslo | 13:52 | |
| openstackgerrit | Dan Prince proposed openstack/oslo.vmware: Make setup.cfg packages include oslo.vmware https://review.openstack.org/147854 | 13:54 |
| sdague | haypo: yeh... so I put a big comment block in there. Have we had the conversation on the global level that all of openstack should move to this programming model? | 13:54 |
| openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.messaging: Deprecates the localcontext API https://review.openstack.org/140616 | 13:57 |
| haypo | sdague: the rationale is in the blueprint, which was approved for juno | 13:57 |
| *** kgiusti has joined #openstack-oslo | 13:57 | |
| haypo | sdague: https://blueprints.launchpad.net/oslo.messaging/+spec/greenio-executor | 13:58 |
| haypo | sdague: oh, there is also https://blueprints.launchpad.net/oslo.messaging/+spec/asyncio-executor | 13:58 |
| sdague | unrelated, apparently oslo.messaging isn't in the openstack group on lp, so the bp links don't work | 13:58 |
| openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.messaging: NotifyPublisher need handle amqp_auto_delete https://review.openstack.org/138723 | 13:59 |
| haypo | sdague: "oslo.messaging isn't in the openstack group on lp" i don't understand, are you talking about a field of the blueprints? | 14:00 |
| sdague | so here's the issue, while there is clearly a lot of complaining about eventlet, something like "get rid of eventlet in openstack" feels like a project wide policy direction that needs to be made. And feels like we should probably agree as a project that that's a global statement. | 14:01 |
| haypo | sdague: i discussed asyncio with mark mcloughlin who designed oslo messaging, he's the author of the greenio executor blueprint | 14:01 |
| sdague | haypo: I get that. Do you understand my concern though? | 14:01 |
| haypo | sdague: with dhellmann & jd__, we decided to "experiment" asyncio in ceilometer, to see if it is possible to switch to asyncio and then discuss if other projects should switch to asyncio or not | 14:02 |
| haypo | sdague: i don't think that we need to decide at this point | 14:03 |
| *** vigneshvar_ has quit IRC | 14:03 | |
| haypo | sdague: i don't think that it's possible to make such decision, there is too many technical questions to say it would be possible to replace eventlet with asyncio in nova for example | 14:03 |
| sdague | and what happens if ceilometer converts across, then we bring it to the TC, and it's rejected as future direction? | 14:04 |
| haypo | (well, the short answer is no, it's not possible) | 14:04 |
| sdague | will ceilometer revert? | 14:04 |
| *** salv-orlando has joined #openstack-oslo | 14:05 | |
| openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.middleware: Add healthcheck middleware https://review.openstack.org/147805 | 14:05 |
| haypo | sdague: sorry, i don't know the technical committee | 14:05 |
| haypo | oh, i see that dhellmann & you are part of it http://www.openstack.org/foundation/tech-committee/ | 14:06 |
| sdague | haypo: yeh, but this isn't actually about what the TC decision will be. Let's leave that in the realm of all possibilities. | 14:06 |
| haypo | sdague: i already discussed such question (programming model) on the mailing list, but no real decision was taken | 14:07 |
| sdague | right, which is what concerns me, because all that means is that we'll end up with openstack projects that all use different programming models | 14:07 |
| haypo | sdague: IMO there is only one good model, asyncio :) it's just take it takes time to replace eventlet with asyncio in *all* projects | 14:08 |
| sdague | haypo: right, I understand you feel that way | 14:08 |
| sdague | but lots of other people and projects don't | 14:08 |
| haypo | sdague: do you mean that you may prefer to don't touch this part and continue to handle eventlet issues and work around them? | 14:09 |
| haypo | sdague: i wrote blueprints and discuss it on the mailing list, what should i except of that? | 14:09 |
| sdague | well, that's what I recommended in the review, this probably really does need a TC statement of direction here, and actually think through the ramifications | 14:10 |
| haypo | sdague: does it sound wrong to you to use eventlet in some projects and trollius in other projects? | 14:10 |
| sdague | haypo: yes, it massively increases the debugability problem | 14:10 |
| sdague | because as you move between projects the way the code is written is much different | 14:11 |
| haypo | sdague: in this case, it's just impossible to replace with anything else, and you must continue to use eventlet | 14:11 |
| haypo | sdague: it's impossible to replace eventlet with something else in "one shot" | 14:11 |
| haypo | i will take a lot of time, probably more than 1 year | 14:11 |
| sdague | well, there can be transition periods. If this was defined direction then you knew everyone was going to do it, that's different | 14:11 |
| sdague | but it seems like this is starting down a journey without knowing if there is an end | 14:12 |
| sdague | anyway, my suggested next step is in the review comment | 14:13 |
| *** dims__ is now known as dimsum__ | 14:13 | |
| haypo | sdague: i wrote my whole roadmap (in multiple steps) in http://aioeventlet.readthedocs.org/openstack.html | 14:14 |
| ttx | dhellmann: I triaged all the two rootwrap bugs, nothing is in the pipe, so we can cut the new release whenever convenient. | 14:24 |
| haypo | sdague: should i add the point to the agenda of the next meeting? | 14:25 |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:30 | |
| *** jecarey has joined #openstack-oslo | 14:30 | |
| sdague | haypo: yeh, that's my suggestion | 14:32 |
| haypo | sdague: ah ok, i was not sure that i'm allowed to modify this agenda | 14:32 |
| sdague | yeh, anyone is allowed to add stuff | 14:33 |
| sdague | and I think this is pretty appropriate for that | 14:33 |
| *** stevemar has joined #openstack-oslo | 14:37 | |
| *** inc0 has quit IRC | 14:42 | |
| *** rpodolyaka1 has quit IRC | 14:42 | |
| haypo | sdague: in july you approved my patch greenio dependency, https://review.openstack.org/#/c/108637/ | 14:43 |
| haypo | sdague: "Personally I'd like to have seen the unit tests in oslo.messaging work before this lands, however if this is defined oslo direction, so be it." | 14:43 |
| haypo | sdague: did you change your mind? or greenio is differen than aioeventlet? | 14:43 |
| sdague | haypo: I feel like we're at a point where we need to checkpoint on this process and get the TC to weigh in one way or another about where it's all going | 14:44 |
| haypo | sdague: ok. anyway, we will discuss that during the meeting ;) | 14:53 |
| *** amotoki has quit IRC | 14:57 | |
| *** mtanino has joined #openstack-oslo | 15:03 | |
| *** russellb is now known as rustlebee | 15:04 | |
| *** bnemec has quit IRC | 15:06 | |
| *** bnemec has joined #openstack-oslo | 15:09 | |
| openstackgerrit | Merged openstack/oslo.middleware: Updated from global requirements https://review.openstack.org/145895 | 15:11 |
| *** bnemec is now known as beekneemech | 15:26 | |
| *** dulek has quit IRC | 15:29 | |
| *** e0ne has quit IRC | 15:38 | |
| dhellmann | ttx: how about monday morning? I'd rather not release anything on a Friday | 15:43 |
| dhellmann | haypo, sdague : good discussion -- we should probably put together a cross-project spec based on the one for oslo.messaging with the broader scope | 15:43 |
| ttx | dhellmann: I'll be on a plane at that point. But you don't need me to release | 15:44 |
| dhellmann | ttx: sure, I can take care of it | 15:44 |
| ttx | I should land late afternoon, in case something goes wrong | 15:44 |
| dhellmann | ttx: ok, cool | 15:44 |
| *** e0ne has joined #openstack-oslo | 15:45 | |
| *** tsekiyama has joined #openstack-oslo | 15:45 | |
| *** rushiagr is now known as rushiagr_away | 15:47 | |
| openstackgerrit | Julien Danjou proposed openstack/oslo.serialization: jsonutils: add set() tests and simplify recursive code https://review.openstack.org/147195 | 15:53 |
| openstackgerrit | Julien Danjou proposed openstack/oslo.serialization: jsonutils: support UUID encoding https://review.openstack.org/147198 | 15:53 |
| *** akamyshnikova has joined #openstack-oslo | 15:55 | |
| *** salv-orlando has quit IRC | 15:56 | |
| *** zz_sabari is now known as sabari | 16:03 | |
| openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.messaging: PoC: Remove usage of contentmanager for executors https://review.openstack.org/140985 | 16:09 |
| openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.messaging: PoC: Remove usage of contentmanager for executors https://review.openstack.org/140985 | 16:10 |
| *** salv-orlando has joined #openstack-oslo | 16:15 | |
| *** stevemar has quit IRC | 16:15 | |
| *** salv-orlando has joined #openstack-oslo | 16:15 | |
| *** amrith is now known as _amrith_ | 16:21 | |
| *** dimsum__ has quit IRC | 16:25 | |
| *** dimsum__ has joined #openstack-oslo | 16:28 | |
| *** achanda has joined #openstack-oslo | 16:30 | |
| *** dims_ has joined #openstack-oslo | 16:32 | |
| *** dimsum__ has quit IRC | 16:37 | |
| openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.messaging: PoC: Remove usage of contentmanager for executors https://review.openstack.org/140985 | 16:37 |
| *** e0ne is now known as e0ne_ | 16:39 | |
| *** Guest29310 is now known as redrobot | 16:41 | |
| *** achanda has quit IRC | 16:50 | |
| openstackgerrit | Merged openstack/oslo.messaging: Add oslo.messaging._drivers.common for heat tests https://review.openstack.org/146872 | 16:50 |
| *** salv-orlando has quit IRC | 16:53 | |
| *** pblaho has quit IRC | 16:53 | |
| *** amotoki has joined #openstack-oslo | 16:58 | |
| *** vigneshvar_ has joined #openstack-oslo | 17:01 | |
| *** raildo has joined #openstack-oslo | 17:01 | |
| dhellmann | sileht: now that ^^ merged we're ready for an oslo.messaging release, but I think I'll wait for Monday for that one, too | 17:04 |
| *** e0ne_ is now known as e0ne | 17:07 | |
| *** amotoki has quit IRC | 17:11 | |
| *** amotoki has joined #openstack-oslo | 17:15 | |
| *** vigneshvar_ has quit IRC | 17:16 | |
| *** e0ne is now known as e0ne_ | 17:18 | |
| *** e0ne_ is now known as e0ne | 17:21 | |
| *** dims_ has quit IRC | 17:26 | |
| *** amotoki has quit IRC | 17:27 | |
| *** dimsum__ has joined #openstack-oslo | 17:30 | |
| *** noelbk has joined #openstack-oslo | 17:32 | |
| dimsum__ | dhellmann: want me to take over that oslo.log/nova patch? let me know | 17:34 |
| dhellmann | dimsum__: yeah, that would be great if you have time | 17:34 |
| dhellmann | dimsum__: I'll see if I can figure out why the concurrency changes we made still aren't working for neutron | 17:35 |
| *** amotoki has joined #openstack-oslo | 17:36 | |
| dimsum__ | dhellmann: sounds good. one question i had was, some of the imports had "from oslo_log import log as logging", folks usually expect logging to have DEBUG. no? | 17:37 |
| dimsum__ | so do we change the way we do the imports (avoid "logging")? | 17:38 |
| beekneemech | dhellmann: Did neutron update to reference the group location of the concurrency opts? | 17:38 |
| beekneemech | That's the other thing we changed in the graduation. | 17:38 |
| dhellmann | beekneemech: hmm, likely not | 17:38 |
| dhellmann | that may be a problem, since this is the interpolation reference | 17:39 |
| *** vigneshvar has joined #openstack-oslo | 17:39 | |
| *** e0ne has quit IRC | 17:39 | |
| dhellmann | the deprecation logic should deal with that, though | 17:39 |
| dimsum__ | dhellmann: you may have missed my query ^^^ :) | 17:40 |
| dhellmann | dimsum__: sorry, I was trying to minimize the changes. We don't re-expose the python logging levels, but I guess we could. | 17:42 |
| dhellmann | dimsum__: in a couple of places I imported both modules, since that was also being done already | 17:42 |
| dimsum__ | then we go back to fix oslo_log? | 17:43 |
| dhellmann | I'm not sure we want to pretend that oslo_log is python's logging module. | 17:43 |
| dhellmann | maybe the more correct fix is to change the name that's imported under in nova? | 17:44 |
| dhellmann | it makes it a bigger change, but it's also more explicit | 17:44 |
| dimsum__ | right, i'll go with that | 17:44 |
| dhellmann | dimsum__: let's see how bad that makes the migration, if it starts to turn really messy we can talk more | 17:45 |
| dimsum__ | dhellmann: ack will work it up and let you know | 17:45 |
| openstackgerrit | Paul Belanger proposed openstack/oslo.utils: Add microsecond support to iso8601_from_timestamp https://review.openstack.org/147929 | 17:45 |
| dhellmann | dimsum__: good, thanks | 17:50 |
| *** achanda has joined #openstack-oslo | 17:50 | |
| *** salv-orlando has joined #openstack-oslo | 17:53 | |
| *** achanda has quit IRC | 17:59 | |
| *** salv-orl_ has joined #openstack-oslo | 17:59 | |
| sileht | dhellmann, ok for Monday | 18:00 |
| *** salv-orlando has quit IRC | 18:01 | |
| *** dtantsur is now known as dtantsur|afk | 18:01 | |
| *** achanda has joined #openstack-oslo | 18:02 | |
| openstackgerrit | Merged openstack/taskflow: Remove 'SaveOrderTask' and test state in class variables https://review.openstack.org/147038 | 18:03 |
| *** achanda has quit IRC | 18:03 | |
| *** zzzeek has joined #openstack-oslo | 18:07 | |
| *** amotoki has quit IRC | 18:15 | |
| *** harlowja_away is now known as harlowja | 18:18 | |
| *** mriedem has quit IRC | 18:20 | |
| harlowja | sileht https://review.openstack.org/#/c/140985/ is pretty cool | 18:21 |
| harlowja | even passed the gates :-P | 18:21 |
| harlowja | i'm not sure what to do about the futures module though; its not really openstack specific, maybe its yet another mini-library | 18:22 |
| *** amotoki has joined #openstack-oslo | 18:22 | |
| harlowja | something to ponder about i guess :-P | 18:22 |
| *** prad has joined #openstack-oslo | 18:28 | |
| *** e0ne has joined #openstack-oslo | 18:28 | |
| sileht | harlowja, thanks to your executor ! | 18:30 |
| *** rpodolyaka1 has joined #openstack-oslo | 18:33 | |
| harlowja | sileht woot, ha | 18:35 |
| *** rpodolyaka1 has quit IRC | 18:35 | |
| openstackgerrit | Merged openstack/oslosphinx: Add run_cross_tests.sh from incubator https://review.openstack.org/140363 | 18:36 |
| harlowja | sileht then a next question becomes if u can just use a asyncio executor | 18:38 |
| *** stevemar has joined #openstack-oslo | 18:38 | |
| harlowja | i believe it has a nearly api compat impl | 18:38 |
| harlowja | or if it doesn't one could be made? | 18:38 |
| harlowja | since https://docs.python.org/3/library/asyncio.html at least has a nearly-same future's object | 18:39 |
| harlowja | i guess haypo would be the expert there | 18:39 |
| sileht | yep | 18:40 |
| sileht | bbl | 18:40 |
| *** sabari is now known as zz_sabari | 18:40 | |
| dhellmann | sileht, flaper87 : I'm sorry I missed your bootstrapping session today. I hope it went well. I'll queue up the video to watch. | 18:45 |
| *** alexpilotti has quit IRC | 18:52 | |
| *** amotoki has quit IRC | 18:58 | |
| *** amotoki has joined #openstack-oslo | 19:01 | |
| openstackgerrit | Merged openstack/tooz: Add a file based driver https://review.openstack.org/146507 | 19:05 |
| openstackgerrit | Doug Hellmann proposed openstack/oslo-incubator: Add pre_release_tests.py https://review.openstack.org/147681 | 19:07 |
| openstackgerrit | Doug Hellmann proposed openstack/oslo-incubator: Refactor oslo.conf handling for tool scripts https://review.openstack.org/147680 | 19:07 |
| dimsum__ | dhellmann: pattern i followed was | 19:14 |
| dimsum__ | "import logging as std_logging" if we needed standard logging | 19:14 |
| *** yamahata has joined #openstack-oslo | 19:14 | |
| dimsum__ | 3 spots in 2 files, was not too bad | 19:15 |
| openstackgerrit | Merged openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/145891 | 19:15 |
| *** ajo has quit IRC | 19:15 | |
| *** dims_ has joined #openstack-oslo | 19:16 | |
| *** dimsum__ has quit IRC | 19:19 | |
| *** ajo has joined #openstack-oslo | 19:28 | |
| openstackgerrit | Doug Hellmann proposed openstack/oslo-incubator: Add tox env for running release notes script https://review.openstack.org/147963 | 19:30 |
| openstackgerrit | Doug Hellmann proposed openstack/oslo-incubator: Adjust release note template whitespace https://review.openstack.org/147964 | 19:30 |
| dhellmann | harlowja: ^^ | 19:32 |
| harlowja | cool | 19:32 |
| harlowja | ya, damn jinja2 special whitespace stuff | 19:32 |
| harlowja | lol | 19:32 |
| harlowja | special "-" that u can place to try to tweak stuff | 19:32 |
| dhellmann | yeah, it can be tricky to get that right (note I also rstrip() the contents of the noteable file) | 19:35 |
| dhellmann | harlowja: by the way, I think that should be "notable" (no e) | 19:35 |
| harlowja | ah | 19:35 |
| harlowja | ya, seems like u are right, english ftw | 19:36 |
| *** zz_sabari is now known as sabari | 19:37 | |
| *** rpodolyaka1 has joined #openstack-oslo | 19:37 | |
| *** yamahata has quit IRC | 19:38 | |
| *** yamahata has joined #openstack-oslo | 19:38 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Tidy up the WBE cache (now WBE types) module https://review.openstack.org/103245 | 19:39 |
| *** amotoki has quit IRC | 19:41 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add back a 'eventlet_utils' helper utility module https://review.openstack.org/146304 | 19:54 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Allow loading conductors via entrypoints https://review.openstack.org/124617 | 19:59 |
| *** achanda has joined #openstack-oslo | 20:00 | |
| openstackgerrit | Joshua Harlow proposed openstack/oslo-incubator: Fix spelling of noteables -> notables https://review.openstack.org/147974 | 20:04 |
| harlowja | dhellmann is there a meeting on monday? | 20:08 |
| harlowja | *being its MLK (in us) | 20:08 |
| *** rpodolyaka1 has quit IRC | 20:09 | |
| * harlowja maybe i missed an email/note, ha | 20:09 | |
| *** salv-orl_ has quit IRC | 20:10 | |
| *** achanda has quit IRC | 20:10 | |
| *** andreykurilin_ has joined #openstack-oslo | 20:12 | |
| *** achanda has joined #openstack-oslo | 20:12 | |
| openstackgerrit | Doug Hellmann proposed openstack/oslo-incubator: Add pre_release_tests.py https://review.openstack.org/147681 | 20:13 |
| dhellmann | harlowja: oh, yeah, that's a holiday isn't it? | 20:13 |
| harlowja | for some folks i think | 20:13 |
| dhellmann | we should probably skip | 20:13 |
| dhellmann | I'll send email | 20:13 |
| harlowja | k | 20:14 |
| dhellmann | harlowja: done, and thanks for bringing that up, it completely slipped my mind | 20:15 |
| harlowja | np | 20:15 |
| *** achanda has quit IRC | 20:17 | |
| *** e0ne is now known as e0ne_ | 20:18 | |
| dhellmann | ihrachyshka: it looks like you got neutron and oslo.concurrency working in https://review.openstack.org/#/c/139717/12 and the changes look pretty straightforward -- did you just need to remove that set_override() call? | 20:21 |
| *** raildo has quit IRC | 20:21 | |
| *** andreykurilin_ has quit IRC | 20:22 | |
| *** rushiagr_away is now known as rushiagr | 20:22 | |
| openstackgerrit | Doug Hellmann proposed openstack/oslo.rootwrap: Add cross-testing script https://review.openstack.org/145582 | 20:25 |
| *** salv-orlando has joined #openstack-oslo | 20:26 | |
| *** prad has quit IRC | 20:28 | |
| *** sabari has left #openstack-oslo | 20:33 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Mirror the task executor methods in the retry action https://review.openstack.org/147982 | 20:34 |
| *** sabari has joined #openstack-oslo | 20:35 | |
| *** rpodolyaka1 has joined #openstack-oslo | 20:36 | |
| *** achanda has joined #openstack-oslo | 20:37 | |
| *** wendar_ is now known as wendar | 20:38 | |
| *** achanda has quit IRC | 20:42 | |
| *** achanda has joined #openstack-oslo | 20:43 | |
| *** noelbk has quit IRC | 20:43 | |
| *** ajo has quit IRC | 20:43 | |
| ihrachyshka | dhellmann, yes | 20:45 |
| ihrachyshka | dhellmann, that was enough. I guess it's just not correct way to set it (instead, we would use set_defaults(), right? though we don't really need it, so removed) | 20:45 |
| *** andreykurilin_ has joined #openstack-oslo | 20:50 | |
| ihrachyshka | dhellmann, hm, I planned to have a chat on Mon with you on overall oslo/neutron direction in terms for Kilo, but since it's holiday... will you have ~10-15 mins for me today, or no luck? | 20:50 |
| dhellmann | ihrachyshka: I can chat now if you like | 20:51 |
| dhellmann | either set_defaults() or set_override() should work, but then for the expansion to work you have to have registered the option it is referencing | 20:52 |
| ihrachyshka | dhellmann, cool, so I've talked to Kyle yesterday, and we decided that we need to come up with some oslo/neutron priority list to align with other plans for kilo (neutron is going thru huge refactoring, and adding oslo patches to change lots of imports etc. should be considered in terms of additional pain it creates) | 20:53 |
| *** e0ne_ has quit IRC | 20:53 | |
| ihrachyshka | dhellmann, so I want to talk to you to see what neutron may expect from oslo till the end of the cycle, and whether we can align new libraries/changes in imports/adoption of fixtures/whatever else with other stuff | 20:54 |
| ihrachyshka | dhellmann, atm in my todo list I only have namespace changes (waiting for oslo.messaging before going thru it) and completing oslo.concurrency migration. I wonder whether we have some milestones for other new graduated libs, and other adoptions (maybe completely new libraries you would like to see consumed/tested) | 20:55 |
| dhellmann | ihrachyshka: my goal is to have versions of all of the libraries with the namespace package changes released by k-2 so you have the rest of the cycle to migrate | 20:56 |
| dhellmann | ihrachyshka: I would like to have that done by the end of the cycle, but I recognize that might not be possible, so a more realistic goal might be L-1 | 20:56 |
| dhellmann | that would give us time to release versions of the libraries without the old namespace packages by L-2 | 20:56 |
| ihrachyshka | dhellmann, if oslo.messaging is released till kilo-2, we'll be able to consume most of them, if not all | 20:57 |
| harlowja | who can kick https://review.openstack.org/#/c/146225/ ?? :-P | 20:57 |
| ihrachyshka | I just wait for oslo.messaging to avoid import reshuffling for pep8 | 20:57 |
| dhellmann | ihrachyshka: oslo.messaging should be released monday, unless I discover that it's going to break something | 20:57 |
| dhellmann | harlowja: fungi or clarkb in -infra should be able to | 20:58 |
| ihrachyshka | dhellmann, ok, cool. any more graduations on horizon? | 20:58 |
| harlowja | k, let me poke them | 20:58 |
| dhellmann | harlowja: some of them are traveling this week, so I don't know who's "on call" | 20:58 |
| harlowja | kk | 20:58 |
| dhellmann | ihrachyshka: ayoung is working on policy, and https://launchpad.net/oslo/+milestone/kilo-3 lists a few as well | 20:59 |
| dhellmann | ihrachyshka: the plan is to do policy for K-2 as well, if we can | 20:59 |
| ihrachyshka | dhellmann, oslo.cache - is it a graduation or merely adding dogpile backend? | 21:00 |
| *** vigneshvar has quit IRC | 21:00 | |
| dhellmann | ihrachyshka: a little of both; there is going to be a new API wrapping dogpile | 21:00 |
| *** stevemar has quit IRC | 21:00 | |
| ihrachyshka | dhellmann, ack. we use it in one place only, I hope it won't be hard to adopt new api | 21:03 |
| ihrachyshka | versionedobjects will definitely wait till L at least neutron-wise | 21:04 |
| ihrachyshka | any other requests from neutron? maybe some adoptions on testing side? | 21:04 |
| *** gordc has quit IRC | 21:06 | |
| ihrachyshka | dhellmann, ok, I already used my 15 mins, thanks for the chat :) | 21:07 |
| *** achanda has quit IRC | 21:09 | |
| openstackgerrit | Merged openstack/oslo-incubator: Replace the 'release_notes' shell script with a python variant https://review.openstack.org/141939 | 21:10 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Have the serial task executor shutdown/restart its executor https://review.openstack.org/147994 | 21:14 |
| openstackgerrit | Merged openstack/oslo-incubator: Add the ability to skip requirement commits https://review.openstack.org/142916 | 21:15 |
| *** vigneshvar has joined #openstack-oslo | 21:15 | |
| dhellmann | ihrachyshka: sorry, local distraction -- I think you're doing a good job with adoptions, so don't stress about it. We expect a cycle delay between release and adoption in a lot of cases. | 21:17 |
| openstackgerrit | Merged openstack/taskflow: Fix unused and conflicting variables https://review.openstack.org/147590 | 21:19 |
| *** achanda has joined #openstack-oslo | 21:19 | |
| *** achanda has quit IRC | 21:20 | |
| openstackgerrit | Sean Dague proposed openstack/oslo.log: wip: let LOG.debug take a stack= parameter https://review.openstack.org/147998 | 21:23 |
| sdague | dhellmann: so that's my random friday "what if" patch ^^^ | 21:23 |
| ihrachyshka | dhellmann, well, I'm not stressing about adoptions. though I'd like to move them further to avoid potential backports-thru-oslo-incubator as much as possible ;) | 21:24 |
| dhellmann | sdague: cool, I'll take a look | 21:26 |
| dhellmann | ihrachyshka: sure, that's the goal, I just meant I think you're on track for what we've done so far | 21:26 |
| sdague | dhellmann: it's definitely not quite right yet, and doesn't pass all the tests, but it's a thought experiment | 21:26 |
| dhellmann | ihrachyshka: we'll take a more careful look at adoption across all projects after k-2 | 21:27 |
| dhellmann | sdague: ok, sure | 21:27 |
| *** e0ne has joined #openstack-oslo | 21:28 | |
| *** e0ne has quit IRC | 21:29 | |
| dhellmann | sdague: while we're doing experiments: https://review.openstack.org/147681 | 21:30 |
| sdague | dhellmann: so I think the difference I'd make with that is to do the git clone as well | 21:32 |
| dhellmann | sdague: we already have a separate script for that | 21:32 |
| sdague | oh, ok | 21:32 |
| dhellmann | we have this chicken and egg problem of figuring out which repos need their tests run and figuring out which repos to check out | 21:32 |
| dhellmann | so we have a script that just checks out everything, and then the other tools assume that's what you've done | 21:33 |
| sdague | ah, gotcha | 21:33 |
| dhellmann | that way the other tools also work on a CI node, theoretically | 21:33 |
| dhellmann | tools/clone_openstack.sh checks out everything openstack*/* from gerrit | 21:33 |
| openstackgerrit | Merged openstack/oslo.rootwrap: Add cross-testing script https://review.openstack.org/145582 | 21:34 |
| dims_ | dhellmann: another api problem possibly | 21:37 |
| dims_ | dhellmann: AttributeError: 'KeywordArgumentAdapter' object has no attribute 'level' | 21:37 |
| dhellmann | dims_: what's looking at that? | 21:38 |
| dims_ | dhellmann: http://paste.openstack.org/show/158316/ | 21:38 |
| *** leakypipes has quit IRC | 21:38 | |
| dhellmann | dims_: ok, that looks like a case where we want a fixture to set log levels at the start of a test and reset them when the test is odne | 21:38 |
| dhellmann | done | 21:38 |
| sdague | yeh, it's just that one migrations test, it's just crufty | 21:39 |
| dhellmann | let me look at the docs and see if level is part of the public api of the logger | 21:39 |
| sdague | dims_: also, I am not actually convinced we need that piece of that test any more given the logging fixture that we have | 21:40 |
| dims_ | sdague: agree, was staring at it a bit. by i expect others to do the same hence the question | 21:41 |
| dims_ | s/others/others may/ | 21:41 |
| dhellmann | dims_, sdague : yeah, I don't see that attribute described in the public docs for loggers or adapters | 21:41 |
| openstackgerrit | Merged openstack/oslo.utils: Add microsecond support to timeutils.utcnow_ts() https://review.openstack.org/147736 | 21:41 |
| dims_ | dhellmann: cool, i'll nuke it then | 21:42 |
| dhellmann | a new fixture to manage the level for a given set of loggers by name, and that uses the stdlib getlogger instead of the wrapper, would be pretty easy if we need to do this | 21:42 |
| sdague | dims_: I believe this - https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L141-L143 covers the case that was supposed to | 21:42 |
| openstackgerrit | Merged openstack/oslo.vmware: Make setup.cfg packages include oslo.vmware https://review.openstack.org/147854 | 21:42 |
| openstackgerrit | Merged openstack/taskflow: Switch to using 'oslo_utils' vs 'oslo.utils' https://review.openstack.org/146297 | 21:42 |
| dhellmann | dims_: you could also try changing the tests to use getEffectiveLevel() and setLevel() instead of manipulating the attribute directly | 21:42 |
| dims_ | sdague: ah, nice | 21:42 |
| dims_ | dhellmann: see above link from sdague there's another way this was already handled | 21:43 |
| dhellmann | dims_: even better | 21:43 |
| dims_ | so the code here is not needed | 21:43 |
| dims_ | thanks sdague | 21:43 |
| sdague | dims_: yeh, I think so. Don't 100% trust me on that | 21:43 |
| dims_ | sdague: i'll watch for it :) | 21:43 |
| openstackgerrit | Merged openstack/oslo.serialization: Add a messagepack utils helper module https://review.openstack.org/147353 | 21:46 |
| *** gordc has joined #openstack-oslo | 21:54 | |
| *** gordc has quit IRC | 22:00 | |
| *** kgiusti has quit IRC | 22:08 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Tidy up the WBE cache (now WBE types) module https://review.openstack.org/103245 | 22:10 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Use monotonic time when/if available https://review.openstack.org/144212 | 22:12 |
| *** rpodolyaka1 has quit IRC | 22:12 | |
| *** achanda has joined #openstack-oslo | 22:14 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Rework the sqlalchemy backend https://review.openstack.org/126110 | 22:14 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Use the new table length constants https://review.openstack.org/126166 | 22:15 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Move the jobboard/job bases to a jobboard/base module https://review.openstack.org/143246 | 22:17 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Move implementation(s) to there own sections https://review.openstack.org/143249 | 22:18 |
| dims_ | harlowja: fyi - setuptools has declared war - All your base are belong to us - action on infra | 22:21 |
| harlowja | lol | 22:21 |
| harlowja | yet another fun release | 22:22 |
| harlowja | lol | 22:22 |
| openstackgerrit | Brant Knudson proposed openstack/oslo.utils: Add TimeFixture https://review.openstack.org/146719 | 22:22 |
| harlowja | guess all my rebases won't work, lol | 22:22 |
| dims_ | yep! | 22:23 |
| harlowja | whoever does this setuptools stuff makes me shudder more and more, lol | 22:24 |
| harlowja | something doesn't seem right there, lol | 22:24 |
| * harlowja and i'm just an observer ... | 22:25 | |
| harlowja | *especially the releases on fridays... | 22:25 |
| *** andreykurilin_ has quit IRC | 22:30 | |
| *** achanda has quit IRC | 22:33 | |
| *** achanda has joined #openstack-oslo | 22:40 | |
| *** dims_ has quit IRC | 22:40 | |
| *** salv-orlando has quit IRC | 22:43 | |
| *** achanda has quit IRC | 22:43 | |
| *** achanda has joined #openstack-oslo | 22:49 | |
| *** noelbk has joined #openstack-oslo | 22:52 | |
| *** andreykurilin_ has joined #openstack-oslo | 22:53 | |
| *** ihrachyshka has quit IRC | 22:55 | |
| *** salv-orlando has joined #openstack-oslo | 23:01 | |
| *** jaosorior has quit IRC | 23:03 | |
| *** sabari is now known as zz_sabari | 23:08 | |
| *** achanda has quit IRC | 23:09 | |
| dhellmann | the pre-release test for oslo.i18n will run the unit tests of 37 projects. I think I'm going to let that run for a while. :-) | 23:13 |
| openstackgerrit | Ben Nemec proposed openstack/oslo.config: Log a warning when deprecated opts are used https://review.openstack.org/148020 | 23:17 |
| *** alexpilotti has joined #openstack-oslo | 23:17 | |
| *** openstackgerrit has quit IRC | 23:19 | |
| *** openstackgerrit has joined #openstack-oslo | 23:19 | |
| *** dimsum__ has joined #openstack-oslo | 23:41 | |
| *** dimsum__ has quit IRC | 23:47 | |
| *** zzzeek has quit IRC | 23:47 | |
| *** zzzeek has joined #openstack-oslo | 23:49 | |
| *** zz_sabari is now known as sabari | 23:52 | |
| *** sabari is now known as zz_sabari | 23:54 | |
| *** zz_sabari is now known as sabari | 23:55 | |
| *** andreykurilin_ has quit IRC | 23:55 | |
| *** vigneshvar has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!