*** sabari is now known as zz_sabari | 00:05 | |
*** jaypipes has joined #openstack-oslo | 00:30 | |
*** yamahata has quit IRC | 00:31 | |
*** david-lyle has joined #openstack-oslo | 00:35 | |
*** zz_sabari is now known as sabari | 01:16 | |
*** david-lyle has quit IRC | 01:18 | |
*** david-lyle has joined #openstack-oslo | 01:25 | |
*** salv-orlando has quit IRC | 01:26 | |
*** david-lyle has quit IRC | 01:27 | |
*** david-lyle has joined #openstack-oslo | 01:28 | |
*** sabari is now known as zz_sabari | 01:40 | |
*** zz_sabari is now known as sabari | 01:41 | |
*** sabari is now known as zz_sabari | 01:46 | |
*** david-lyle has quit IRC | 01:59 | |
*** salv-orlando has joined #openstack-oslo | 02:00 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Use platform neutral line separator(s) https://review.openstack.org/146347 | 02:07 |
---|---|---|
*** salv-orlando has quit IRC | 02:12 | |
*** salv-orlando has joined #openstack-oslo | 02:18 | |
*** salv-orlando has quit IRC | 02:28 | |
*** salv-orlando has joined #openstack-oslo | 02:37 | |
*** salv-orlando has quit IRC | 02:41 | |
*** salv-orlando has joined #openstack-oslo | 03:10 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 03:12 | |
*** salv-orlando has quit IRC | 03:13 | |
*** arnaud___ has joined #openstack-oslo | 03:35 | |
*** david-lyle has joined #openstack-oslo | 03:44 | |
openstackgerrit | Merged openstack/oslo.utils: fix link to bug tracker in README https://review.openstack.org/145789 | 03:49 |
*** david-lyle has quit IRC | 04:50 | |
*** david-lyle has joined #openstack-oslo | 04:50 | |
*** salv-orlando has joined #openstack-oslo | 05:00 | |
*** david-lyle has quit IRC | 05:06 | |
*** salv-orlando has quit IRC | 05:11 | |
*** zz_sabari is now known as sabari | 05:18 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 05:21 | |
*** harlowja_at_home has joined #openstack-oslo | 05:27 | |
*** sabari is now known as zz_sabari | 05:28 | |
*** dulek has joined #openstack-oslo | 05:33 | |
*** dulek_ has joined #openstack-oslo | 05:37 | |
*** dulek has quit IRC | 05:37 | |
*** harlowja_at_home has quit IRC | 05:41 | |
*** harlowja_at_home has joined #openstack-oslo | 05:41 | |
*** dulek_ has quit IRC | 05:51 | |
*** salv-orlando has joined #openstack-oslo | 05:53 | |
*** salv-orlando has quit IRC | 05:57 | |
*** salv-orlando has joined #openstack-oslo | 06:06 | |
*** zz_sabari is now known as sabari | 06:08 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.messaging: Imported Translations from Transifex https://review.openstack.org/143256 | 06:10 |
*** salv-orlando has quit IRC | 06:11 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add executor statistics (WIP) https://review.openstack.org/146382 | 06:21 |
*** vigneshvar has joined #openstack-oslo | 06:22 | |
*** harlowja_at_home has quit IRC | 06:26 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add executor statistics (WIP) https://review.openstack.org/146382 | 06:35 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add executor statistics (WIP) https://review.openstack.org/146382 | 06:46 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add executor statistics (WIP) https://review.openstack.org/146382 | 06:55 |
*** rushiagr_away is now known as rushiagr | 06:58 | |
openstackgerrit | Merged openstack/taskflow: Update the README.rst with accurate requirements https://review.openstack.org/146288 | 07:14 |
*** salv-orlando has joined #openstack-oslo | 07:37 | |
*** subscope has joined #openstack-oslo | 07:37 | |
*** arnaud___ has quit IRC | 07:41 | |
*** arnaud___ has joined #openstack-oslo | 07:42 | |
*** arnaud___ has quit IRC | 07:47 | |
*** salv-orlando has quit IRC | 07:49 | |
*** salv-orlando has joined #openstack-oslo | 07:50 | |
openstackgerrit | Merged openstack/oslo.serialization: Updated from global requirements https://review.openstack.org/146161 | 07:53 |
*** salv-orlando has quit IRC | 07:54 | |
*** ihrachyshka has joined #openstack-oslo | 08:06 | |
*** e0ne has joined #openstack-oslo | 08:07 | |
*** e0ne has quit IRC | 08:10 | |
*** pblaho has joined #openstack-oslo | 08:31 | |
*** eezhova has joined #openstack-oslo | 08:38 | |
*** ihrachyshka has quit IRC | 08:49 | |
openstackgerrit | Louis Taylor proposed openstack/oslo-incubator: Change apiclient to use oslo_utils.uuidutils https://review.openstack.org/145717 | 08:49 |
*** jaosorior has joined #openstack-oslo | 08:58 | |
*** andreykurilin has joined #openstack-oslo | 08:59 | |
*** sabari is now known as zz_sabari | 09:00 | |
*** rpodolyaka1 has quit IRC | 09:02 | |
*** rpodolyaka1 has joined #openstack-oslo | 09:02 | |
*** yamahata has joined #openstack-oslo | 09:04 | |
*** achanda has joined #openstack-oslo | 09:06 | |
*** andreykurilin has quit IRC | 09:28 | |
*** jprovazn has joined #openstack-oslo | 09:28 | |
*** andreykurilin has joined #openstack-oslo | 09:29 | |
*** achanda has quit IRC | 09:30 | |
*** yamahata has quit IRC | 09:34 | |
*** ihrachyshka has joined #openstack-oslo | 09:38 | |
*** xek has joined #openstack-oslo | 09:41 | |
*** salv-orlando has joined #openstack-oslo | 09:42 | |
*** andreykurilin has quit IRC | 09:50 | |
*** achanda has joined #openstack-oslo | 10:02 | |
*** achanda has quit IRC | 10:15 | |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo-specs: oslo.messaging examples proposal https://review.openstack.org/140318 | 10:25 |
openstackgerrit | Li Ma proposed openstack/oslo.messaging: Add matchmaker rings for OpenStack projects https://review.openstack.org/145722 | 10:26 |
*** e0ne has joined #openstack-oslo | 10:27 | |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo-specs: oslo.messaging examples proposal https://review.openstack.org/140318 | 10:30 |
*** ajo has quit IRC | 10:32 | |
*** ajo has joined #openstack-oslo | 10:34 | |
*** sreshetnyak has joined #openstack-oslo | 10:38 | |
*** andreykurilin has joined #openstack-oslo | 10:56 | |
*** pcaruana has joined #openstack-oslo | 10:59 | |
*** andreykurilin has quit IRC | 11:10 | |
*** viktors|afk is now known as viktors | 11:12 | |
viktors | dhellmann: +1 from me :) | 11:12 |
*** david-lyle has joined #openstack-oslo | 11:32 | |
openstackgerrit | Julien Danjou proposed openstack/tooz: IPC simplification https://review.openstack.org/146431 | 11:55 |
*** e0ne is now known as e0ne_ | 11:56 | |
*** e0ne_ is now known as e0ne | 12:04 | |
*** mfedosin has joined #openstack-oslo | 12:16 | |
*** dims has joined #openstack-oslo | 12:35 | |
*** alexpilotti has joined #openstack-oslo | 12:39 | |
*** alexpilotti has quit IRC | 12:44 | |
*** miqui has quit IRC | 12:49 | |
*** rushiagr is now known as rushiagr_away | 12:56 | |
*** alexpilotti has joined #openstack-oslo | 12:58 | |
*** noelbk has quit IRC | 13:04 | |
*** inc0 has joined #openstack-oslo | 13:06 | |
inc0 | dhellmann, ping | 13:06 |
*** e0ne is now known as e0ne_ | 13:11 | |
*** dims has quit IRC | 13:14 | |
*** dims__ has joined #openstack-oslo | 13:18 | |
*** kgiusti has joined #openstack-oslo | 13:25 | |
*** rushiagr_away is now known as rushiagr | 13:33 | |
openstackgerrit | Chris Dent proposed openstack/tooz: Update sentinel support to allow multiple sentinel hosts https://review.openstack.org/146463 | 13:35 |
openstackgerrit | Chris Dent proposed openstack/tooz: Update sentinel support to allow multiple sentinel hosts https://review.openstack.org/146463 | 13:38 |
*** e0ne_ is now known as e0ne | 13:45 | |
*** gordc has joined #openstack-oslo | 13:45 | |
*** jprovazn has quit IRC | 13:48 | |
*** bknudson has quit IRC | 13:51 | |
*** vigneshvar has quit IRC | 13:55 | |
*** jeckersb_gone is now known as jeckersb | 13:58 | |
*** tedross has joined #openstack-oslo | 13:58 | |
*** e0ne is now known as e0ne_ | 14:03 | |
*** e0ne_ is now known as e0ne | 14:11 | |
openstackgerrit | Chris Dent proposed openstack/tooz: Update sentinel support to allow multiple sentinel hosts https://review.openstack.org/146463 | 14:19 |
*** mriedem has joined #openstack-oslo | 14:36 | |
*** noelbk has joined #openstack-oslo | 14:36 | |
*** e0ne is now known as e0ne_ | 14:39 | |
*** e0ne_ is now known as e0ne | 14:42 | |
*** jeckersb is now known as jeckersb_gone | 14:42 | |
*** salv-orl_ has joined #openstack-oslo | 14:42 | |
*** noelbk has quit IRC | 14:42 | |
*** salv-orlando has quit IRC | 14:43 | |
*** bknudson has joined #openstack-oslo | 14:45 | |
*** tedross has quit IRC | 14:46 | |
*** jeckersb_gone is now known as jeckersb | 14:48 | |
*** jecarey has quit IRC | 14:49 | |
*** haypo has joined #openstack-oslo | 15:00 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 15:02 | |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for ampq/controller module https://review.openstack.org/146484 | 15:04 |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for ampq/driver module https://review.openstack.org/146485 | 15:04 |
*** mtanino has joined #openstack-oslo | 15:07 | |
*** beekneemech is now known as bnemec | 15:16 | |
*** jecarey has joined #openstack-oslo | 15:17 | |
*** palendae has quit IRC | 15:18 | |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for amqp/eventloop module https://review.openstack.org/146492 | 15:20 |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/amqpdriver module https://review.openstack.org/146493 | 15:20 |
*** palendae has joined #openstack-oslo | 15:23 | |
*** palendae has quit IRC | 15:29 | |
*** e0ne is now known as e0ne_ | 15:30 | |
*** palendae has joined #openstack-oslo | 15:31 | |
*** palendae has quit IRC | 15:31 | |
dhellmann | inc0: pong | 15:34 |
dhellmann | viktors: ok, good. Let's talk about when to do it during the meeting. | 15:34 |
inc0 | hi dhellmann, I'm digging into oslo.versionedobjects | 15:35 |
inc0 | can I assign whole thing to myself? | 15:35 |
inc0 | is there any concurrent work being done in that matter? | 15:36 |
dhellmann | inc0: I'd be happy to have help. We talked about this last week, right? | 15:37 |
inc0 | yup | 15:37 |
dhellmann | I was going to try to get it going this week, so maybe we can work on it together. | 15:37 |
dhellmann | The first thing we need is a script to export the nova objects code, based on oslo-incubator/tools/graduate.sh | 15:38 |
inc0 | well, right now I'm trying to get this code working in heat, so we'll have scope of changes which we need to do | 15:38 |
inc0 | and prototype | 15:38 |
sileht | dhellmann, I have seen that the namespace patch for oslo.messaging doesn't work with a strange import issue: | 15:39 |
dhellmann | do you anticipate needing API changes to the objects code? | 15:40 |
sileht | 'from oslo import messaging ; messaging.rpc;' doesn't works, but 'from oslo.messaging import rpc; rpc' works | 15:40 |
inc0 | dhellmann, for heat I had to add new field type for example | 15:40 |
inc0 | thats a minor change, but I'm still deep into the woods | 15:40 |
*** e0ne_ has quit IRC | 15:41 | |
inc0 | good thing that besides naming there aren't many nova-specific references | 15:41 |
dhellmann | sileht: yeah, I started looking into that friday but didn't find the answer | 15:41 |
dhellmann | inc0: ok, we can add new field types to the library after we have it in its own repository | 15:41 |
*** yamahata has joined #openstack-oslo | 15:42 | |
bnemec | Python namespaces: the gift you keep trying to give but can never get rid of. | 15:42 |
dhellmann | bnemec: yeah | 15:42 |
*** palendae has joined #openstack-oslo | 15:43 | |
*** palendae has quit IRC | 15:43 | |
*** zz_jgrimm is now known as jgrimm | 15:44 | |
inc0 | dhellmann, my thoughts exactly. There is a lot of cosmetics needed and we'd need to write good docs of how to implement that into your project | 15:44 |
inc0 | because this often won't be straightforward | 15:44 |
*** palendae has joined #openstack-oslo | 15:45 | |
dhellmann | inc0: agreed | 15:45 |
inc0 | well, to summarize (I won't be able to attend meeting fully), I'll assign whole thing to myself and push the topic, ok? I'd love tho if you could create repo for that | 15:46 |
inc0 | I'll try to get graduate.sh asap and continue to push heat implementation to get full scope of changes needed | 15:47 |
inc0 | sounds good to you? | 15:48 |
*** msdubov_ has joined #openstack-oslo | 15:49 | |
dhellmann | inc0: the script is needed to create the repository for import (https://wiki.openstack.org/wiki/Oslo/CreatingANewLibrary) so that really is our first step. I'll try to work on it this week. If you keep identifying API changes, then you'll be able to submit those patches when we get the repo set up | 15:49 |
inc0 | ok, then I'll keep doing that | 15:50 |
openstackgerrit | Julien Danjou proposed openstack/tooz: Allow to pass arguments to retry() https://review.openstack.org/146506 | 15:51 |
openstackgerrit | Julien Danjou proposed openstack/tooz: Add a file based driver https://review.openstack.org/146507 | 15:51 |
viktors | dhellmann: I'll miss meeting today, you can ask db-related questions to rpodolyaka1 | 15:51 |
*** e0ne has joined #openstack-oslo | 15:51 | |
*** viktors is now known as viktors|afk | 15:52 | |
msdubov_ | dhellmann: hi! Have you seen the "How to change the default Sphinx theme in a docs gate job?" topic in openstack-dev? | 15:52 |
msdubov_ | dhellmann: Could you please advise us something to resolve this issue with global requirements? | 15:52 |
dhellmann | viktors|afk: ok | 15:53 |
dhellmann | msdubov_: I haven't, yet. I'm about to go into the oslo meeting, and I'll catch up on email after. | 15:54 |
*** inc0 has quit IRC | 15:55 | |
msdubov_ | dhellmann: Thanks! I'll be grateful if you write a reply | 15:58 |
dhellmann | msdubov_: which project is that for? | 15:58 |
*** zzzeek has joined #openstack-oslo | 15:58 | |
msdubov_ | dhellmann: Rally | 15:58 |
*** e0ne_ has joined #openstack-oslo | 15:59 | |
msdubov_ | dhellmann: We're refactoring our docs and have this problem here https://review.openstack.org/120018 | 15:59 |
*** e0ne has quit IRC | 16:02 | |
*** zz_sabari is now known as sabari | 16:03 | |
denis_makogon | yes | 16:05 |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/common module https://review.openstack.org/146536 | 16:06 |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/impl_qpid.py module https://review.openstack.org/146537 | 16:06 |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/impl_rabbit.py module https://review.openstack.org/146538 | 16:06 |
*** e0ne_ is now known as e0ne | 16:12 | |
*** stevemar has joined #openstack-oslo | 16:15 | |
*** tsekiyama has joined #openstack-oslo | 16:21 | |
openstackgerrit | Mehdi Abaakouk proposed openstack/oslo.messaging: Import workaround https://review.openstack.org/146547 | 16:23 |
*** yassine has joined #openstack-oslo | 16:27 | |
*** rushiagr is now known as rushiagr_away | 16:29 | |
*** e0ne_ has joined #openstack-oslo | 16:37 | |
*** salv-orl_ has quit IRC | 16:37 | |
*** salv-orlando has joined #openstack-oslo | 16:38 | |
*** e0ne has quit IRC | 16:40 | |
*** dhellmann has quit IRC | 16:42 | |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/impl_zmq.py module https://review.openstack.org/146560 | 16:42 |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/matchmaker.py module https://review.openstack.org/146561 | 16:42 |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/matchmaker_redis.py module https://review.openstack.org/146562 | 16:42 |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/matchmaker_ring.py module https://review.openstack.org/146563 | 16:42 |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for _executors, notify, rpc packages https://review.openstack.org/146564 | 16:42 |
*** dhellmann has joined #openstack-oslo | 16:42 | |
*** sabari has quit IRC | 16:43 | |
*** sabari has joined #openstack-oslo | 16:43 | |
*** msdubov_ has quit IRC | 16:47 | |
*** msdubov_ has joined #openstack-oslo | 16:48 | |
openstackgerrit | Victor Stinner proposed openstack/oslo.messaging: fix qpid test issue with eventlet monkey patching https://review.openstack.org/146574 | 17:00 |
haypo | sileht: ^^ this change should fix the current random qpid hang in oslo messaging tests | 17:01 |
ihrachyshka | dhellmann, was there any email notice on 'release oslo.concurrency DONE'? | 17:03 |
openstackgerrit | Victor Stinner proposed openstack/oslo.messaging: Add a new aioeventlet executor https://review.openstack.org/136653 | 17:07 |
morganfainberg | dhellmann, ping. is there an LP project created for each olso lib? | 17:16 |
openstackgerrit | Merged openstack/oslo.db: Make sure sort_key_attr is QueryableAttribute when query https://review.openstack.org/143632 | 17:17 |
*** yassine has quit IRC | 17:18 | |
openstackgerrit | Merged openstack/oslo.db: Fix slowest test output after test run https://review.openstack.org/146349 | 17:21 |
openstackgerrit | Merged openstack/oslo.vmware: Use ToggleLazy fixture from oslo.i18n https://review.openstack.org/145923 | 17:21 |
dhellmann | zzzeek, rpodolyaka1 : I'm going to grab lunch, and then tag the release in a bit. | 17:22 |
dhellmann | morganfainberg: yes, we need separate bug trackers for each project | 17:23 |
morganfainberg | dhellmann, i was looking at the wrong place, it's already there | 17:23 |
morganfainberg | :) | 17:23 |
dhellmann | morganfainberg: k | 17:23 |
*** vigneshvar has joined #openstack-oslo | 17:24 | |
*** ihrachyshka has quit IRC | 17:25 | |
rpodolyaka1 | dhellmann: ack, thanks! | 17:26 |
*** pcaruana is now known as pcaruana|off| | 17:27 | |
*** msdubov_ has quit IRC | 17:28 | |
*** andreykurilin has joined #openstack-oslo | 17:34 | |
sileht | dhellmann, https://review.openstack.org/#/c/146547/ works, you can squash it, if the solution is acceptable for you | 17:35 |
*** e0ne_ has quit IRC | 17:36 | |
*** jecarey has quit IRC | 17:38 | |
*** pblaho has quit IRC | 17:39 | |
*** yamahata has quit IRC | 17:43 | |
*** sabari is now known as zz_sabari | 17:58 | |
dhellmann | sileht: excellent, thanks for working that out | 18:00 |
*** dtantsur is now known as dtantsur|afk | 18:01 | |
openstackgerrit | Doug Hellmann proposed openstack/oslo.messaging: Move files out of the namespace package https://review.openstack.org/144794 | 18:02 |
dhellmann | sileht: ^^ | 18:04 |
sileht | dhellmann, thanks | 18:04 |
* sileht have to plan a rebase day now :p | 18:05 | |
*** jeckersb is now known as jeckersb_gone | 18:11 | |
*** yamahata has joined #openstack-oslo | 18:14 | |
dhellmann | sileht: yeah, sorry :-/ | 18:19 |
*** isviridov is now known as isviridov_away | 18:22 | |
dhellmann | zzzeek, rpodolyaka1 : oslo.db 1.4.0 released | 18:27 |
*** ajo has quit IRC | 18:31 | |
openstackgerrit | Doug Hellmann proposed openstack/oslo.vmware: Move files out of the namespace package https://review.openstack.org/145924 | 18:34 |
dims__ | dhellmann: '_' is getting quite a workout :) | 18:37 |
dhellmann | dims__: Maybe I'll bring a bunch of extra _ keys to the summit to pass out to folks as swag. | 18:37 |
dims__ | +1 :) | 18:37 |
*** harlowja_away is now known as harlowja | 18:39 | |
*** jraim_ is now known as jraim | 18:50 | |
*** jecarey has joined #openstack-oslo | 18:51 | |
*** noelbk has joined #openstack-oslo | 18:52 | |
*** e0ne has joined #openstack-oslo | 18:54 | |
*** harlowja has quit IRC | 19:00 | |
*** harlowja has joined #openstack-oslo | 19:00 | |
*** harlowja has quit IRC | 19:00 | |
openstackgerrit | Denis M. proposed openstack/oslo.messaging: Applying i18n for _executors, notify, rpc packages https://review.openstack.org/146564 | 19:06 |
*** raildo has joined #openstack-oslo | 19:06 | |
*** e0ne is now known as e0ne_ | 19:09 | |
*** zz_sabari is now known as sabari | 19:10 | |
*** raildo has quit IRC | 19:11 | |
openstackgerrit | James Carey proposed openstack/oslo.vmware: Update oslo.i18n version to one that includes ToggleLazy fixture https://review.openstack.org/146615 | 19:12 |
*** e0ne_ has quit IRC | 19:15 | |
amrith | I'm assuming that __dims__ is some special thing in the global namespace? | 19:18 |
*** e0ne has joined #openstack-oslo | 19:20 | |
dims__ | amrith: :) | 19:24 |
amrith | that's ::amrith:: to you ;) | 19:24 |
amrith | dims__, are you planning to attend magnum mid-cycle? | 19:24 |
amrith | I just sent in my rsvp | 19:24 |
dims__ | amrith: not sure yet (which means probably not) | 19:26 |
*** jeckersb_gone is now known as jeckersb | 19:32 | |
*** pabelanger has left #openstack-oslo | 19:49 | |
*** harlowja has joined #openstack-oslo | 19:52 | |
*** e0ne has quit IRC | 20:00 | |
openstackgerrit | Ben Nemec proposed openstack/oslo.concurrency: Bump to hacking 0.10 https://review.openstack.org/146626 | 20:00 |
*** david-lyle has quit IRC | 20:02 | |
openstackgerrit | Ben Nemec proposed openstack/oslo.serialization: Bump to hacking 0.10 https://review.openstack.org/146627 | 20:02 |
*** e0ne has joined #openstack-oslo | 20:03 | |
dims__ | dhellmann: not sure if you saw this, wanted your opinion when you have a chance - https://review.openstack.org/#/c/146205/ | 20:08 |
openstackgerrit | Ben Nemec proposed openstack/oslo.config: Bump to hacking 0.10 https://review.openstack.org/146628 | 20:08 |
dhellmann | dims__: looking | 20:09 |
dhellmann | dims__: shouldn't that go in oslo.utils somewhere? | 20:09 |
dhellmann | dims__: and is there any particular reason for re-implementing it instead of copying nested() from the python 2.7 stdlib? | 20:12 |
dhellmann | dims__: left a comment on the review | 20:15 |
*** yamahata has quit IRC | 20:17 | |
*** yamahata has joined #openstack-oslo | 20:18 | |
*** lbragstad has quit IRC | 20:23 | |
*** wendar has quit IRC | 20:24 | |
*** wendar has joined #openstack-oslo | 20:25 | |
*** dims__ has quit IRC | 20:25 | |
*** jaosorior has quit IRC | 20:25 | |
*** boris-42 has quit IRC | 20:25 | |
*** freyes has quit IRC | 20:25 | |
*** russellb has quit IRC | 20:25 | |
*** dims__ has joined #openstack-oslo | 20:26 | |
*** vishy has quit IRC | 20:26 | |
*** jroll has quit IRC | 20:26 | |
*** yamahata has quit IRC | 20:26 | |
*** yamahata has joined #openstack-oslo | 20:26 | |
* dhellmann notices that about 2 hrs have passed since the oslo.db release without any alarms sounding | 20:26 | |
*** jraim has quit IRC | 20:28 | |
*** dims__ has quit IRC | 20:28 | |
*** dims__ has joined #openstack-oslo | 20:29 | |
*** lbragstad has joined #openstack-oslo | 20:29 | |
*** russellb has joined #openstack-oslo | 20:29 | |
*** yamahata has quit IRC | 20:29 | |
*** jraim has joined #openstack-oslo | 20:29 | |
*** rm_work has joined #openstack-oslo | 20:30 | |
*** jroll has joined #openstack-oslo | 20:30 | |
*** yamahata has joined #openstack-oslo | 20:30 | |
*** jaosorior has joined #openstack-oslo | 20:30 | |
*** boris-42 has joined #openstack-oslo | 20:30 | |
*** freyes has joined #openstack-oslo | 20:30 | |
*** andreykurilin has quit IRC | 20:31 | |
*** rm_work has left #openstack-oslo | 20:32 | |
*** vishy has joined #openstack-oslo | 20:35 | |
dims__ | dhellmann: just needs a home, anywhere is ok w/ me. i was not sure about licenses etc when copying from stdlib, is it kosher to do that? | 20:35 |
dhellmann | dims__: we should check; it might be safer to keep the reimplementation | 20:36 |
dhellmann | dims__: I definitely think oslo.utils is better though | 20:36 |
dims__ | right. k will re-file | 20:36 |
dhellmann | dims__: cool, and nice addition | 20:40 |
*** alexpilotti has quit IRC | 20:41 | |
openstackgerrit | Victor Stinner proposed openstack/oslo.messaging: fix qpid test issue with eventlet monkey patching https://review.openstack.org/146574 | 20:49 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.utils: custom context manger for py27/py3x compatability https://review.openstack.org/146639 | 20:54 |
*** david-lyle has joined #openstack-oslo | 20:57 | |
openstackgerrit | Michael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module https://review.openstack.org/138215 | 21:01 |
openstackgerrit | Michael Bayer proposed openstack/oslo.db: Ensure DBConnectionError is raised on failed revalidate https://review.openstack.org/139738 | 21:02 |
*** andreykurilin has joined #openstack-oslo | 21:02 | |
*** sabari has quit IRC | 21:03 | |
openstackgerrit | Ben Nemec proposed openstack/oslo.serialization: Bump to hacking 0.10 https://review.openstack.org/146627 | 21:08 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Allow failure objects exceptions to be replaced https://review.openstack.org/146642 | 21:09 |
openstackgerrit | Ben Nemec proposed openstack/oslo.config: Bump to hacking 0.10 https://review.openstack.org/146628 | 21:10 |
openstackgerrit | Ben Nemec proposed openstack/oslo.concurrency: Bump to hacking 0.10 https://review.openstack.org/146626 | 21:10 |
*** git-harry has quit IRC | 21:14 | |
*** git-harry has joined #openstack-oslo | 21:14 | |
*** e0ne has quit IRC | 21:15 | |
*** git-harry has left #openstack-oslo | 21:15 | |
*** david-lyle has quit IRC | 21:16 | |
*** david-ly_ has joined #openstack-oslo | 21:16 | |
*** marun has joined #openstack-oslo | 21:16 | |
marun | dhellmann (or anyone else): so, an oslo.db change has broken the neutron functional job: https://bugs.launchpad.net/neutron/+bug/1409909 | 21:17 |
marun | here is the offending commit: https://github.com/openstack/oslo.db/commit/75b402be3b8497d12bd21f8c371b52427931952d | 21:17 |
marun | My question - is there anything in place that should have caught this? | 21:17 |
marun | And if not, do we need to introduce one? | 21:17 |
*** sabari has joined #openstack-oslo | 21:18 | |
*** armax has joined #openstack-oslo | 21:18 | |
bnemec | Hmm, we shouldn't have removed that without a deprecation period. | 21:19 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Allow failure objects exceptions to be replaced https://review.openstack.org/146642 | 21:20 |
marun | bnemec: Does oslo.db gate on dependent projects? | 21:20 |
bnemec | marun: Not unit tests. | 21:20 |
marun | bnemec: The tests in question are part of our functional (system-interacting) job. | 21:20 |
bnemec | I believe there are devstack tempest jobs, but cross-gating everything in oslo turned out to be impractical. | 21:21 |
marun | bnemec: The tempest jobs, I get. We'll be using oslo.db testing infrastructure in our functional job, though, and it's execution footprint is far lighter and less succeptible to false positives than any tempest-based job. | 21:22 |
marun | it's -> its | 21:22 |
marun | bnemec: Would it be possible to have oslo.db gate on it? | 21:22 |
bnemec | marun: Yeah, we know it's a gap in our CI testing right now. We try to do some manual cross-tests before releasing, but evidently that didn't happen in this case. | 21:23 |
marun | bnemec: Or is there another coordination strategy you'd prefer? | 21:23 |
marun | bnemec: If break/fix is the way to do it for the short-term, I'm fine with that. | 21:23 |
bnemec | It's possible we could set up some specific cross-testing to help with this. | 21:23 |
bnemec | marun: Yeah, break/fix is kind of where we are right now. | 21:23 |
marun | bnemec: ok. | 21:23 |
*** jeckersb is now known as jeckersb_gone | 21:23 | |
marun | bnemec: I'll leave it with you then. If this becomes a problem we can investigate whether gating on neutron-dsvm-functional would be a reasonable imposition. | 21:24 |
dhellmann | marun: thanks for the bug report. I ran the unit tests, and tempest, but didn't realize the functional tests were already split out | 21:24 |
bnemec | dhellmann: viktors|afk rpodolyaka1 zzzeek: We should probably revert that change and re-release the new oslo.db. Maybe add a deprecation warning first so we can remove it next cycle. | 21:24 |
marun | bnemec: We were looking at posting a fix to Neutron, not sure if reverting is the fastest path. | 21:25 |
* zzzeek looks, assuming it’s his change | 21:25 | |
zzzeek | ah, hm | 21:25 |
zzzeek | wow | 21:25 |
bnemec | Well, it was still a removal of something in the public api that wasn't deprecated. | 21:25 |
bnemec | But if neutron wants to work around it to unblock themselves that is fine too obviously. :-) | 21:26 |
marun | dhellmann: ah, makes sense. Yeah, we finally moved the db tests to the functional job recently. | 21:26 |
marun | dhellmann: where they belong, imho :) | 21:26 |
dhellmann | yeah, my mistake for not checking on that | 21:26 |
zzzeek | bnemec: it would be nice if neutron could work around this | 21:26 |
dhellmann | although this looks like a method with a public name that should have been deprecated instead of removed immediately | 21:26 |
zzzeek | bnemec: by “work around” i mean “modernize” on their end | 21:26 |
marun | zzzeek: armax is working on a fix, so I think we're fine. | 21:27 |
armax | marun, zzzeek: posting now | 21:27 |
zzzeek | dhellmann: so..how is this usually caught? someone manually tests the oslo.db release against all consuming projects? there’s nothing automated for this ? | 21:28 |
dhellmann | zzzeek: for the namespace changes, yes, I have been manually running unit tests against a bunch of projects before release | 21:28 |
zzzeek | dhellmann: so its manual, OK | 21:28 |
dhellmann | the cross-product of all unit test suites and all oslo libs would consume all of the available nodes in the test infrastructure, so we don't automate the tests | 21:28 |
bnemec | Maybe we could create a few cross-test jobs for projects that we know are consuming libs. | 21:29 |
dhellmann | normally it would have been caught by someone noticing that it was a public api in review, though | 21:29 |
dhellmann | yeah, we may need to add some | 21:29 |
zzzeek | dhellmann: so with this, i wouldnt want to revert, id restore the check_foreign_keys() method alone | 21:29 |
zzzeek | dhellmann: im assuming they’re calling that directly, if i recall correctly | 21:29 |
dhellmann | zzzeek: ok, why not just revert it? | 21:30 |
zzzeek | test_models_sync should continue to use alembic | 21:30 |
dhellmann | ah, ok | 21:30 |
zzzeek | dhellmann: because the newer version is using an alembic feature that works better than the method that is here | 21:30 |
marun | this calls for some intelligence in how we run tests... | 21:30 |
dhellmann | zzzeek: can you work with rpodolyaka1 and viktors|afk to put something like that together, and include a deprecation warning | 21:30 |
zzzeek | dhellmann: OK, so we want to fix olso.db here and not just wait for neutron, right? | 21:31 |
dhellmann | I want to go through the right process, even if neutron is going to work around the problem, because I want to make sure everyone involved learns from the issue | 21:31 |
dhellmann | zzzeek: right, if neutron works around it too that's great, but i want the db team to understand that they can't make changes to the public APIs of their classes like this | 21:31 |
dhellmann | I don't know this contributor's irc nick to include them in the conversation, do you? | 21:32 |
zzzeek | dhellmann: i dont | 21:32 |
dhellmann | ok | 21:32 |
zzzeek | but we +2’ed it so its us anyway | 21:32 |
dhellmann | true | 21:32 |
zzzeek | zap! I did not +2 this! :) | 21:33 |
zzzeek | but ill push something up :) | 21:33 |
dhellmann | marun: yeah, we may need to pick a few projects and test suites to gate against like this. We built a bunch of tools, but never implemented the actual gate jobs | 21:33 |
dhellmann | zzzeek: thanks! | 21:33 |
*** mriedem has left #openstack-oslo | 21:34 | |
marun | dhellmann: I've been thinking of something a bit more advanced, such that you can run a subset of tests based on various criteria. | 21:34 |
marun | dhellmann: In this case, it should be possible to determine a dependency graph such that only tests that interact with a given oslo library could be targeted. | 21:35 |
marun | dhellmann: This capability would also allow us to skip running tempest jobs for doc-only change.s | 21:35 |
* marun wants a pony, too. | 21:36 | |
sdague | marun: if you have hacking time on that, it would need to be in zuul. But it would basically be the inverse of the "add a job if this part of the tree changes" which is used for requirements consistency checking | 21:37 |
*** jroll has quit IRC | 21:38 | |
*** jroll has joined #openstack-oslo | 21:38 | |
marun | sdague: jaypipes started working on this pre-summit, so maybe I'll see where he ended up | 21:38 |
sdague | ok, that's the first I heard of jay working on code there. | 21:39 |
marun | sdague: Would you happen to have a pointer to where the non-inverse capability is defined so I can investigate? | 21:39 |
dhellmann | marun: I do have a script that can be used to run tests through tox using a lib installed from source, so maybe that could be used along with some sort of tagging of tests. Of course, then the problem is on the app developers to correctly tag their test suite, which shifts the burden without really making it simpler. | 21:39 |
jaypipes | marun: I haven't gotten anywhere with that :( | 21:39 |
marun | jaypipes: oh, nuts | 21:39 |
jaypipes | sorry :( | 21:39 |
marun | jaypipes: and here I was hoping to avoid work :) | 21:40 |
jaypipes | :) | 21:40 |
marun | dhellmann: I'm hoping we could do something smarter... | 21:40 |
dhellmann | yeah | 21:40 |
*** david-ly_ has quit IRC | 21:40 | |
marun | dhellmann: collecting the dependency graph either on-demand or periodically is probably the tough part of making it smarter, though. | 21:41 |
marun | dhellmann: it reminds me of coverage, though, and we don't seem to generate that because of the considerable increase in execution time. | 21:41 |
*** david-lyle has joined #openstack-oslo | 21:41 | |
dhellmann | right | 21:42 |
sdague | marun: https://github.com/openstack-infra/zuul/blob/master/zuul/model.py#L494-L501 this is the piece that adds a job to the execution if it had a file match | 21:44 |
sdague | but you'd need to chat with jeblair to get a better idea on how to invert the model, because today I think zuul is basically only additive | 21:44 |
marun | sdague: wouldn't it be possible to not add though? | 21:45 |
*** rpodolyaka2 has joined #openstack-oslo | 21:45 | |
marun | sdague: so, where are the objects used in this method defined? | 21:46 |
marun | sdague: ah, I should clone and introspect | 21:46 |
sdague | so, realistically, follow on questions are best for jeblair and infra folks | 21:46 |
sdague | I only roughly understand this part | 21:46 |
*** david-lyle has quit IRC | 21:49 | |
openstackgerrit | Michael Bayer proposed openstack/oslo.db: Implement generic update-on-match feature https://review.openstack.org/146228 | 21:49 |
marun | sdague: ok, thanks for pointer! | 21:49 |
*** david-lyle has joined #openstack-oslo | 21:50 | |
openstackgerrit | Merged openstack/oslo.concurrency: Updated from global requirements https://review.openstack.org/146160 | 21:51 |
*** jgrimm is now known as zz_jgrimm | 21:52 | |
marun | sdague: out of curiosity, why do you understand this part? | 21:55 |
marun | sdague: Were you working on something around this? | 21:55 |
openstackgerrit | Doug Hellmann proposed openstack-dev/pbr: Be more aggressive about building wheels https://review.openstack.org/146650 | 21:56 |
*** jeckersb_gone is now known as jeckersb | 21:57 | |
sdague | marun: because I looked at it briefly especially to skip the tempest jobs on requirements updates (which are by definition pointless, because we just ran them in g-r) | 21:58 |
sdague | that would save a lot of thundering herds | 21:58 |
marun | sdague: ah, makes sense. and did you end up implementing that capability? | 21:59 |
sdague | no, I got as far as vaguely understanding where changes might be needed | 21:59 |
marun | sdague: ah, ok. | 22:01 |
*** ajo has joined #openstack-oslo | 22:02 | |
*** andreykurilin has quit IRC | 22:06 | |
*** andreykurilin has joined #openstack-oslo | 22:07 | |
openstackgerrit | Roman Podoliaka proposed openstack/oslo.db: Revert "Remove check_foreign_keys from ModelsMigrationsSync" https://review.openstack.org/146658 | 22:10 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.utils: custom context manger for py27/py3x compatability https://review.openstack.org/146639 | 22:10 |
*** mriedem has joined #openstack-oslo | 22:10 | |
*** e0ne has joined #openstack-oslo | 22:17 | |
dims__ | ">=0.10" vs ">=0.10.0" bnemec? weird | 22:18 |
dims__ | jogo: ^^^ | 22:18 |
dims__ | jogo: see bnemec's email to the mailing list "Hacking 0.10 released" | 22:18 |
bnemec | dims__: Yeah, see the requirements job results for https://review.openstack.org/#/c/146627/ | 22:18 |
*** stevemar has quit IRC | 22:18 | |
dims__ | bnemec: either we fix /usr/local/jenkins/slave_scripts/project-requirements-change.py or we should fix global-requirements txt | 22:20 |
dims__ | right? | 22:20 |
*** ajo has quit IRC | 22:23 | |
*** ajo has joined #openstack-oslo | 22:24 | |
bnemec | dims__: Probably should. I'm not sure whether it was done intentionally because of the setuptools version sorting weirdness or if it's just a bug. | 22:26 |
*** bknudson has quit IRC | 22:27 | |
openstackgerrit | Michael Bayer proposed openstack/oslo.db: Restore the check_foreign_keys() method. https://review.openstack.org/146663 | 22:27 |
*** rpodolyaka2 has quit IRC | 22:33 | |
*** rpodolyaka2 has joined #openstack-oslo | 22:35 | |
*** rpodolyaka2 has quit IRC | 22:39 | |
*** ajo has quit IRC | 22:44 | |
*** mriedem has quit IRC | 22:45 | |
*** rossella_s has quit IRC | 22:46 | |
openstackgerrit | Merged openstack/tooz: IPC simplification https://review.openstack.org/146431 | 22:47 |
*** rossella_s has joined #openstack-oslo | 22:47 | |
openstackgerrit | Merged openstack/tooz: Update sentinel support to allow multiple sentinel hosts https://review.openstack.org/146463 | 22:48 |
*** raildo has joined #openstack-oslo | 22:48 | |
*** armax has quit IRC | 22:50 | |
*** gordc has quit IRC | 22:52 | |
jogo | dims__ bnemec: odd, thanks for pointing that out | 22:52 |
*** e0ne has quit IRC | 22:54 | |
*** vigneshvar has quit IRC | 22:56 | |
*** raildo has quit IRC | 23:01 | |
*** ihrachyshka has joined #openstack-oslo | 23:03 | |
ihrachyshka | dims__, hey | 23:07 |
ihrachyshka | dims__, first, thanks for oslo namespace hacking check. | 23:07 |
ihrachyshka | dims__, though it seems incomplete (it does not catch e.g. 'from oslo import messaging') | 23:07 |
ihrachyshka | I have a local fix to catch that, once we get oslo_messaging released, I'll post it for review (in case you're interested) | 23:08 |
dhellmann | zzzeek, rpodolyaka1, viktors|afk: please include clarkb in your pymysql eval notes; he has some input and I want to make sure all voices with useful comments are heard. | 23:10 |
*** vigneshvar has joined #openstack-oslo | 23:12 | |
zzzeek | dhellmann: are we collecting pymysql eval somewhere? wiki page ? | 23:12 |
dhellmann | zzzeek: I don't have a page going, I thought you all were going to start something | 23:12 |
dhellmann | based on our discussion from the meeting last week, right? | 23:13 |
*** jaosorior has quit IRC | 23:13 | |
zzzeek | rrrrright :) | 23:13 |
* zzzeek cues dog ate his homework | 23:13 | |
dhellmann | :-) | 23:14 |
zzzeek | dhellmann: we need to test it but also i really would likje https://review.openstack.org/#/c/138215/ merged which will help with making it more predictable | 23:14 |
*** dims__ has quit IRC | 23:14 | |
zzzeek | dhellmann: once we swallow the eventlet pill here, we are opened up to a whole host of weird new issues that can arise | 23:14 |
dhellmann | zzzeek: I'll add that to my review list for tomorrow | 23:14 |
*** raildo has joined #openstack-oslo | 23:14 | |
dhellmann | right | 23:14 |
*** dims__ has joined #openstack-oslo | 23:15 | |
*** vigneshvar has quit IRC | 23:15 | |
zzzeek | b.c. right now we’re just blockoing on DB access. though if we keep the connecvtion pool sizes low, it wont be as bad | 23:15 |
dhellmann | zzzeek: that sort of info should go into the write-up -- this might end up turning into a cross-project spec at least as a way to publicize the changes | 23:15 |
zzzeek | ok | 23:15 |
dhellmann | zzzeek: how about if we add this to the meeting agenda for next week? | 23:16 |
zzzeek | dhellmann: oh but anyway, oracle is a no-reply, so mysql-connector is out | 23:16 |
zzzeek | dhellmann: sure | 23:16 |
dhellmann | zzzeek: yeah, the lack of response from oracle is an important data point | 23:16 |
zzzeek | dhellmann: yup | 23:16 |
zzzeek | dhellmann: id really not mind having a mysql driver on stackforge, like pymysql itself or a fork, whats the position on that | 23:17 |
zzzeek | dhellmann: just beacuse we really need somethign that can respond to issues quickly | 23:17 |
dhellmann | that's an approach we should explore -- I don't have a lot of enthusiasm for forking if we can find an upstream project that will work with us | 23:18 |
dhellmann | we should also talk to them about moving, rather than forking, like we did with wsme and pecan | 23:18 |
*** dims__ has quit IRC | 23:19 | |
zzzeek | dhellmann: well anyway id need to look and see how responsive they are to start with, i havent logged who maintains it yet | 23:20 |
*** david-lyle has quit IRC | 23:20 | |
dhellmann | zzzeek: right, that's all part of the info we need to collect on this one | 23:20 |
ihrachyshka | dhellmann, I'm sorry if I missed update on oslo.concurrency. So was a new release that would include 'ConfigParser -> ConfigOpts' change? | 23:24 |
*** dims__ has joined #openstack-oslo | 23:25 | |
dhellmann | ihrachyshka: do you mean the thing that removed the filter? | 23:26 |
ihrachyshka | dhellmann, yes. I thought that was the goal of the action item from prev meeting, but you've said it's DONE today, though I haven't seen any release since then. so probably there is some misunderstanding | 23:26 |
dhellmann | ihrachyshka: it looks like that hasn't been released -- the version with the namespace package change was released, but the change you wanted came later (I thought it was included, sorry) | 23:27 |
dhellmann | ihrachyshka: I'll do a release in the morning -- I'm going offline shortly and don't want to release it and have things break when I'm not around to deal with it | 23:27 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.utils: custom context manger for py27/py3x compatability https://review.openstack.org/146639 | 23:27 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add executor statistics https://review.openstack.org/146382 | 23:27 |
ihrachyshka | sure thing, thanks :) | 23:27 |
dhellmann | ihrachyshka: sorry about the delay on this one, I really thought that change was in 0.3.0 | 23:28 |
ihrachyshka | that's fine, I have stuff to do in the meantime. Like moving out of namespaces. (and that one depends on oslo.messaging release with namespace change since I don't want to shuffle imports due to pep8 ordering change) | 23:29 |
*** dims_ has joined #openstack-oslo | 23:32 | |
ihrachyshka | zzzeek, "AttributeError: 'TestModelsMigrationsPsql' object has no attribute 'check_foreign_keys'" in neutron. is it due to oslo.db release? | 23:33 |
dhellmann | ihrachyshka: yes, marun and others are working on a fix | 23:33 |
zzzeek | ihrachyshka: yes see https://review.openstack.org/#/c/146663/, though neutron is uspposed to be fixing independently | 23:33 |
ihrachyshka | ah, ok, thanks | 23:33 |
zzzeek | ihrachyshka: also, I’m writing up on PyMySQL, Mysql-connector is out | 23:33 |
dhellmann | zzzeek: I missed that. +2a. | 23:34 |
*** bknudson has joined #openstack-oslo | 23:34 | |
ihrachyshka | zzzeek, ack, tnx | 23:34 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add executor statistics https://review.openstack.org/146382 | 23:34 |
*** dims__ has quit IRC | 23:35 | |
*** andreykurilin has quit IRC | 23:35 | |
*** dims_ has quit IRC | 23:36 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Allow failure objects exceptions to be replaced https://review.openstack.org/146642 | 23:45 |
*** mriedem has joined #openstack-oslo | 23:52 | |
zzzeek | ihrachyshka: here’s my start: https://wiki.openstack.org/wiki/PyMySQL_evaluation | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!