Monday, 2015-01-12

*** sabari is now known as zz_sabari00:05
*** jaypipes has joined #openstack-oslo00:30
*** yamahata has quit IRC00:31
*** david-lyle has joined #openstack-oslo00:35
*** zz_sabari is now known as sabari01:16
*** david-lyle has quit IRC01:18
*** david-lyle has joined #openstack-oslo01:25
*** salv-orlando has quit IRC01:26
*** david-lyle has quit IRC01:27
*** david-lyle has joined #openstack-oslo01:28
*** sabari is now known as zz_sabari01:40
*** zz_sabari is now known as sabari01:41
*** sabari is now known as zz_sabari01:46
*** david-lyle has quit IRC01:59
*** salv-orlando has joined #openstack-oslo02:00
openstackgerritJoshua Harlow proposed openstack/taskflow: Use platform neutral line separator(s)  https://review.openstack.org/14634702:07
*** salv-orlando has quit IRC02:12
*** salv-orlando has joined #openstack-oslo02:18
*** salv-orlando has quit IRC02:28
*** salv-orlando has joined #openstack-oslo02:37
*** salv-orlando has quit IRC02:41
*** salv-orlando has joined #openstack-oslo03:10
*** sigmavirus24_awa is now known as sigmavirus2403:12
*** salv-orlando has quit IRC03:13
*** arnaud___ has joined #openstack-oslo03:35
*** david-lyle has joined #openstack-oslo03:44
openstackgerritMerged openstack/oslo.utils: fix link to bug tracker in README  https://review.openstack.org/14578903:49
*** david-lyle has quit IRC04:50
*** david-lyle has joined #openstack-oslo04:50
*** salv-orlando has joined #openstack-oslo05:00
*** david-lyle has quit IRC05:06
*** salv-orlando has quit IRC05:11
*** zz_sabari is now known as sabari05:18
*** sigmavirus24 is now known as sigmavirus24_awa05:21
*** harlowja_at_home has joined #openstack-oslo05:27
*** sabari is now known as zz_sabari05:28
*** dulek has joined #openstack-oslo05:33
*** dulek_ has joined #openstack-oslo05:37
*** dulek has quit IRC05:37
*** harlowja_at_home has quit IRC05:41
*** harlowja_at_home has joined #openstack-oslo05:41
*** dulek_ has quit IRC05:51
*** salv-orlando has joined #openstack-oslo05:53
*** salv-orlando has quit IRC05:57
*** salv-orlando has joined #openstack-oslo06:06
*** zz_sabari is now known as sabari06:08
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Imported Translations from Transifex  https://review.openstack.org/14325606:10
*** salv-orlando has quit IRC06:11
openstackgerritJoshua Harlow proposed openstack/taskflow: Add executor statistics (WIP)  https://review.openstack.org/14638206:21
*** vigneshvar has joined #openstack-oslo06:22
*** harlowja_at_home has quit IRC06:26
openstackgerritJoshua Harlow proposed openstack/taskflow: Add executor statistics (WIP)  https://review.openstack.org/14638206:35
openstackgerritJoshua Harlow proposed openstack/taskflow: Add executor statistics (WIP)  https://review.openstack.org/14638206:46
openstackgerritJoshua Harlow proposed openstack/taskflow: Add executor statistics (WIP)  https://review.openstack.org/14638206:55
*** rushiagr_away is now known as rushiagr06:58
openstackgerritMerged openstack/taskflow: Update the README.rst with accurate requirements  https://review.openstack.org/14628807:14
*** salv-orlando has joined #openstack-oslo07:37
*** subscope has joined #openstack-oslo07:37
*** arnaud___ has quit IRC07:41
*** arnaud___ has joined #openstack-oslo07:42
*** arnaud___ has quit IRC07:47
*** salv-orlando has quit IRC07:49
*** salv-orlando has joined #openstack-oslo07:50
openstackgerritMerged openstack/oslo.serialization: Updated from global requirements  https://review.openstack.org/14616107:53
*** salv-orlando has quit IRC07:54
*** ihrachyshka has joined #openstack-oslo08:06
*** e0ne has joined #openstack-oslo08:07
*** e0ne has quit IRC08:10
*** pblaho has joined #openstack-oslo08:31
*** eezhova has joined #openstack-oslo08:38
*** ihrachyshka has quit IRC08:49
openstackgerritLouis Taylor proposed openstack/oslo-incubator: Change apiclient to use oslo_utils.uuidutils  https://review.openstack.org/14571708:49
*** jaosorior has joined #openstack-oslo08:58
*** andreykurilin has joined #openstack-oslo08:59
*** sabari is now known as zz_sabari09:00
*** rpodolyaka1 has quit IRC09:02
*** rpodolyaka1 has joined #openstack-oslo09:02
*** yamahata has joined #openstack-oslo09:04
*** achanda has joined #openstack-oslo09:06
*** andreykurilin has quit IRC09:28
*** jprovazn has joined #openstack-oslo09:28
*** andreykurilin has joined #openstack-oslo09:29
*** achanda has quit IRC09:30
*** yamahata has quit IRC09:34
*** ihrachyshka has joined #openstack-oslo09:38
*** xek has joined #openstack-oslo09:41
*** salv-orlando has joined #openstack-oslo09:42
*** andreykurilin has quit IRC09:50
*** achanda has joined #openstack-oslo10:02
*** achanda has quit IRC10:15
openstackgerritOleksii Zamiatin proposed openstack/oslo-specs: oslo.messaging examples proposal  https://review.openstack.org/14031810:25
openstackgerritLi Ma proposed openstack/oslo.messaging: Add matchmaker rings for OpenStack projects  https://review.openstack.org/14572210:26
*** e0ne has joined #openstack-oslo10:27
openstackgerritOleksii Zamiatin proposed openstack/oslo-specs: oslo.messaging examples proposal  https://review.openstack.org/14031810:30
*** ajo has quit IRC10:32
*** ajo has joined #openstack-oslo10:34
*** sreshetnyak has joined #openstack-oslo10:38
*** andreykurilin has joined #openstack-oslo10:56
*** pcaruana has joined #openstack-oslo10:59
*** andreykurilin has quit IRC11:10
*** viktors|afk is now known as viktors11:12
viktorsdhellmann: +1 from me :)11:12
*** david-lyle has joined #openstack-oslo11:32
openstackgerritJulien Danjou proposed openstack/tooz: IPC simplification  https://review.openstack.org/14643111:55
*** e0ne is now known as e0ne_11:56
*** e0ne_ is now known as e0ne12:04
*** mfedosin has joined #openstack-oslo12:16
*** dims has joined #openstack-oslo12:35
*** alexpilotti has joined #openstack-oslo12:39
*** alexpilotti has quit IRC12:44
*** miqui has quit IRC12:49
*** rushiagr is now known as rushiagr_away12:56
*** alexpilotti has joined #openstack-oslo12:58
*** noelbk has quit IRC13:04
*** inc0 has joined #openstack-oslo13:06
inc0dhellmann, ping13:06
*** e0ne is now known as e0ne_13:11
*** dims has quit IRC13:14
*** dims__ has joined #openstack-oslo13:18
*** kgiusti has joined #openstack-oslo13:25
*** rushiagr_away is now known as rushiagr13:33
openstackgerritChris Dent proposed openstack/tooz: Update sentinel support to allow multiple sentinel hosts  https://review.openstack.org/14646313:35
openstackgerritChris Dent proposed openstack/tooz: Update sentinel support to allow multiple sentinel hosts  https://review.openstack.org/14646313:38
*** e0ne_ is now known as e0ne13:45
*** gordc has joined #openstack-oslo13:45
*** jprovazn has quit IRC13:48
*** bknudson has quit IRC13:51
*** vigneshvar has quit IRC13:55
*** jeckersb_gone is now known as jeckersb13:58
*** tedross has joined #openstack-oslo13:58
*** e0ne is now known as e0ne_14:03
*** e0ne_ is now known as e0ne14:11
openstackgerritChris Dent proposed openstack/tooz: Update sentinel support to allow multiple sentinel hosts  https://review.openstack.org/14646314:19
*** mriedem has joined #openstack-oslo14:36
*** noelbk has joined #openstack-oslo14:36
*** e0ne is now known as e0ne_14:39
*** e0ne_ is now known as e0ne14:42
*** jeckersb is now known as jeckersb_gone14:42
*** salv-orl_ has joined #openstack-oslo14:42
*** noelbk has quit IRC14:42
*** salv-orlando has quit IRC14:43
*** bknudson has joined #openstack-oslo14:45
*** tedross has quit IRC14:46
*** jeckersb_gone is now known as jeckersb14:48
*** jecarey has quit IRC14:49
*** haypo has joined #openstack-oslo15:00
*** sigmavirus24_awa is now known as sigmavirus2415:02
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for ampq/controller module  https://review.openstack.org/14648415:04
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for ampq/driver module  https://review.openstack.org/14648515:04
*** mtanino has joined #openstack-oslo15:07
*** beekneemech is now known as bnemec15:16
*** jecarey has joined #openstack-oslo15:17
*** palendae has quit IRC15:18
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for amqp/eventloop module  https://review.openstack.org/14649215:20
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/amqpdriver  module  https://review.openstack.org/14649315:20
*** palendae has joined #openstack-oslo15:23
*** palendae has quit IRC15:29
*** e0ne is now known as e0ne_15:30
*** palendae has joined #openstack-oslo15:31
*** palendae has quit IRC15:31
dhellmanninc0: pong15:34
dhellmannviktors: ok, good. Let's talk about when to do it during the meeting.15:34
inc0hi dhellmann, I'm digging into oslo.versionedobjects15:35
inc0can I assign whole thing to myself?15:35
inc0is there any concurrent work being done in that matter?15:36
dhellmanninc0: I'd be happy to have help. We talked about this last week, right?15:37
inc0yup15:37
dhellmannI was going to try to get it going this week, so maybe we can work on it together.15:37
dhellmannThe first thing we need is a script to export the nova objects code, based on oslo-incubator/tools/graduate.sh15:38
inc0well, right now I'm trying to get this code working in heat, so we'll have scope of changes which we need to do15:38
inc0and prototype15:38
silehtdhellmann, I have seen that the namespace patch for oslo.messaging doesn't work with a strange import issue:15:39
dhellmanndo 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' works15:40
inc0dhellmann, for heat I had to add new field type for example15:40
inc0thats a minor change, but I'm still deep into the woods15:40
*** e0ne_ has quit IRC15:41
inc0good thing that besides naming there aren't many nova-specific references15:41
dhellmannsileht: yeah, I started looking into that friday but didn't find the answer15:41
dhellmanninc0: ok, we can add new field types to the library after we have it in its own repository15:41
*** yamahata has joined #openstack-oslo15:42
bnemecPython namespaces: the gift you keep trying to give but can never get rid of.15:42
dhellmannbnemec: yeah15:42
*** palendae has joined #openstack-oslo15:43
*** palendae has quit IRC15:43
*** zz_jgrimm is now known as jgrimm15:44
inc0dhellmann, 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 project15:44
inc0because this often won't be straightforward15:44
*** palendae has joined #openstack-oslo15:45
dhellmanninc0: agreed15:45
inc0well, 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 that15:46
inc0I'll try to get graduate.sh asap and continue to push heat implementation to get full scope of changes needed15:47
inc0sounds good to you?15:48
*** msdubov_ has joined #openstack-oslo15:49
dhellmanninc0: 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 up15:49
inc0ok, then I'll keep doing that15:50
openstackgerritJulien Danjou proposed openstack/tooz: Allow to pass arguments to retry()  https://review.openstack.org/14650615:51
openstackgerritJulien Danjou proposed openstack/tooz: Add a file based driver  https://review.openstack.org/14650715:51
viktorsdhellmann: I'll miss meeting today, you can ask db-related questions to rpodolyaka115:51
*** e0ne has joined #openstack-oslo15:51
*** viktors is now known as viktors|afk15: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
dhellmannviktors|afk: ok15:53
dhellmannmsdubov_: 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 IRC15:55
msdubov_dhellmann: Thanks! I'll be grateful if you write a reply15:58
dhellmannmsdubov_: which project is that for?15:58
*** zzzeek has joined #openstack-oslo15:58
msdubov_dhellmann: Rally15:58
*** e0ne_ has joined #openstack-oslo15:59
msdubov_dhellmann: We're refactoring our docs and have this problem here https://review.openstack.org/12001815:59
*** e0ne has quit IRC16:02
*** zz_sabari is now known as sabari16:03
denis_makogonyes16:05
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/common module  https://review.openstack.org/14653616:06
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/impl_qpid.py module  https://review.openstack.org/14653716:06
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/impl_rabbit.py module  https://review.openstack.org/14653816:06
*** e0ne_ is now known as e0ne16:12
*** stevemar has joined #openstack-oslo16:15
*** tsekiyama has joined #openstack-oslo16:21
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Import workaround  https://review.openstack.org/14654716:23
*** yassine has joined #openstack-oslo16:27
*** rushiagr is now known as rushiagr_away16:29
*** e0ne_ has joined #openstack-oslo16:37
*** salv-orl_ has quit IRC16:37
*** salv-orlando has joined #openstack-oslo16:38
*** e0ne has quit IRC16:40
*** dhellmann has quit IRC16:42
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/impl_zmq.py module  https://review.openstack.org/14656016:42
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/matchmaker.py module  https://review.openstack.org/14656116:42
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/matchmaker_redis.py module  https://review.openstack.org/14656216:42
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for _drivers/matchmaker_ring.py module  https://review.openstack.org/14656316:42
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for _executors, notify, rpc  packages  https://review.openstack.org/14656416:42
*** dhellmann has joined #openstack-oslo16:42
*** sabari has quit IRC16:43
*** sabari has joined #openstack-oslo16:43
*** msdubov_ has quit IRC16:47
*** msdubov_ has joined #openstack-oslo16:48
openstackgerritVictor Stinner proposed openstack/oslo.messaging: fix qpid test issue with eventlet monkey patching  https://review.openstack.org/14657417:00
hayposileht: ^^ this change should fix the current random qpid hang in oslo messaging tests17:01
ihrachyshkadhellmann, was there any email notice on 'release oslo.concurrency DONE'?17:03
openstackgerritVictor Stinner proposed openstack/oslo.messaging: Add a new aioeventlet executor  https://review.openstack.org/13665317:07
morganfainbergdhellmann, ping. is there an LP project created for each olso lib?17:16
openstackgerritMerged openstack/oslo.db: Make sure sort_key_attr is QueryableAttribute when query  https://review.openstack.org/14363217:17
*** yassine has quit IRC17:18
openstackgerritMerged openstack/oslo.db: Fix slowest test output after test run  https://review.openstack.org/14634917:21
openstackgerritMerged openstack/oslo.vmware: Use ToggleLazy fixture from oslo.i18n  https://review.openstack.org/14592317:21
dhellmannzzzeek, rpodolyaka1 : I'm going to grab lunch, and then tag the release in a bit.17:22
dhellmannmorganfainberg: yes, we need separate bug trackers for each project17:23
morganfainbergdhellmann, i was looking at the wrong place, it's already there17:23
morganfainberg:)17:23
dhellmannmorganfainberg: k17:23
*** vigneshvar has joined #openstack-oslo17:24
*** ihrachyshka has quit IRC17:25
rpodolyaka1dhellmann: ack, thanks!17:26
*** pcaruana is now known as pcaruana|off|17:27
*** msdubov_ has quit IRC17:28
*** andreykurilin has joined #openstack-oslo17:34
silehtdhellmann, https://review.openstack.org/#/c/146547/ works, you can squash it, if the solution is acceptable for you17:35
*** e0ne_ has quit IRC17:36
*** jecarey has quit IRC17:38
*** pblaho has quit IRC17:39
*** yamahata has quit IRC17:43
*** sabari is now known as zz_sabari17:58
dhellmannsileht: excellent, thanks for working that out18:00
*** dtantsur is now known as dtantsur|afk18:01
openstackgerritDoug Hellmann proposed openstack/oslo.messaging: Move files out of the namespace package  https://review.openstack.org/14479418:02
dhellmannsileht: ^^18:04
silehtdhellmann, thanks18:04
* sileht have to plan a rebase day now :p18:05
*** jeckersb is now known as jeckersb_gone18:11
*** yamahata has joined #openstack-oslo18:14
dhellmannsileht: yeah, sorry :-/18:19
*** isviridov is now known as isviridov_away18:22
dhellmannzzzeek, rpodolyaka1 : oslo.db 1.4.0 released18:27
*** ajo has quit IRC18:31
openstackgerritDoug Hellmann proposed openstack/oslo.vmware: Move files out of the namespace package  https://review.openstack.org/14592418:34
dims__dhellmann: '_' is getting quite a workout :)18:37
dhellmanndims__: 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 harlowja18:39
*** jraim_ is now known as jraim18:50
*** jecarey has joined #openstack-oslo18:51
*** noelbk has joined #openstack-oslo18:52
*** e0ne has joined #openstack-oslo18:54
*** harlowja has quit IRC19:00
*** harlowja has joined #openstack-oslo19:00
*** harlowja has quit IRC19:00
openstackgerritDenis M. proposed openstack/oslo.messaging: Applying i18n for _executors, notify, rpc  packages  https://review.openstack.org/14656419:06
*** raildo has joined #openstack-oslo19:06
*** e0ne is now known as e0ne_19:09
*** zz_sabari is now known as sabari19:10
*** raildo has quit IRC19:11
openstackgerritJames Carey  proposed openstack/oslo.vmware: Update oslo.i18n version to one that includes ToggleLazy fixture  https://review.openstack.org/14661519:12
*** e0ne_ has quit IRC19:15
amrithI'm assuming that __dims__ is some special thing in the global namespace?19:18
*** e0ne has joined #openstack-oslo19:20
dims__amrith: :)19:24
amriththat's ::amrith:: to you ;)19:24
amrithdims__, are you planning to attend magnum mid-cycle?19:24
amrithI just sent in my rsvp19:24
dims__amrith: not sure yet (which means probably not)19:26
*** jeckersb_gone is now known as jeckersb19:32
*** pabelanger has left #openstack-oslo19:49
*** harlowja has joined #openstack-oslo19:52
*** e0ne has quit IRC20:00
openstackgerritBen Nemec proposed openstack/oslo.concurrency: Bump to hacking 0.10  https://review.openstack.org/14662620:00
*** david-lyle has quit IRC20:02
openstackgerritBen Nemec proposed openstack/oslo.serialization: Bump to hacking 0.10  https://review.openstack.org/14662720:02
*** e0ne has joined #openstack-oslo20: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
openstackgerritBen Nemec proposed openstack/oslo.config: Bump to hacking 0.10  https://review.openstack.org/14662820:08
dhellmanndims__: looking20:09
dhellmanndims__: shouldn't that go in oslo.utils somewhere?20:09
dhellmanndims__: and is there any particular reason for re-implementing it instead of copying nested() from the python 2.7 stdlib?20:12
dhellmanndims__: left a comment on the review20:15
*** yamahata has quit IRC20:17
*** yamahata has joined #openstack-oslo20:18
*** lbragstad has quit IRC20:23
*** wendar has quit IRC20:24
*** wendar has joined #openstack-oslo20:25
*** dims__ has quit IRC20:25
*** jaosorior has quit IRC20:25
*** boris-42 has quit IRC20:25
*** freyes has quit IRC20:25
*** russellb has quit IRC20:25
*** dims__ has joined #openstack-oslo20:26
*** vishy has quit IRC20:26
*** jroll has quit IRC20:26
*** yamahata has quit IRC20:26
*** yamahata has joined #openstack-oslo20:26
* dhellmann notices that about 2 hrs have passed since the oslo.db release without any alarms sounding20:26
*** jraim has quit IRC20:28
*** dims__ has quit IRC20:28
*** dims__ has joined #openstack-oslo20:29
*** lbragstad has joined #openstack-oslo20:29
*** russellb has joined #openstack-oslo20:29
*** yamahata has quit IRC20:29
*** jraim has joined #openstack-oslo20:29
*** rm_work has joined #openstack-oslo20:30
*** jroll has joined #openstack-oslo20:30
*** yamahata has joined #openstack-oslo20:30
*** jaosorior has joined #openstack-oslo20:30
*** boris-42 has joined #openstack-oslo20:30
*** freyes has joined #openstack-oslo20:30
*** andreykurilin has quit IRC20:31
*** rm_work has left #openstack-oslo20:32
*** vishy has joined #openstack-oslo20: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
dhellmanndims__: we should check; it might be safer to keep the reimplementation20:36
dhellmanndims__: I definitely think oslo.utils is better though20:36
dims__right. k will re-file20:36
dhellmanndims__: cool, and nice addition20:40
*** alexpilotti has quit IRC20:41
openstackgerritVictor Stinner proposed openstack/oslo.messaging: fix qpid test issue with eventlet monkey patching  https://review.openstack.org/14657420:49
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.utils: custom context manger for py27/py3x compatability  https://review.openstack.org/14663920:54
*** david-lyle has joined #openstack-oslo20:57
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module  https://review.openstack.org/13821521:01
openstackgerritMichael Bayer proposed openstack/oslo.db: Ensure DBConnectionError is raised on failed revalidate  https://review.openstack.org/13973821:02
*** andreykurilin has joined #openstack-oslo21:02
*** sabari has quit IRC21:03
openstackgerritBen Nemec proposed openstack/oslo.serialization: Bump to hacking 0.10  https://review.openstack.org/14662721:08
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow failure objects exceptions to be replaced  https://review.openstack.org/14664221:09
openstackgerritBen Nemec proposed openstack/oslo.config: Bump to hacking 0.10  https://review.openstack.org/14662821:10
openstackgerritBen Nemec proposed openstack/oslo.concurrency: Bump to hacking 0.10  https://review.openstack.org/14662621:10
*** git-harry has quit IRC21:14
*** git-harry has joined #openstack-oslo21:14
*** e0ne has quit IRC21:15
*** git-harry has left #openstack-oslo21:15
*** david-lyle has quit IRC21:16
*** david-ly_ has joined #openstack-oslo21:16
*** marun has joined #openstack-oslo21:16
marundhellmann (or anyone else): so, an oslo.db change has broken the neutron functional job: https://bugs.launchpad.net/neutron/+bug/140990921:17
marunhere is the offending commit: https://github.com/openstack/oslo.db/commit/75b402be3b8497d12bd21f8c371b52427931952d21:17
marunMy question - is there anything in place that should have caught this?21:17
marunAnd if not, do we need to introduce one?21:17
*** sabari has joined #openstack-oslo21:18
*** armax has joined #openstack-oslo21:18
bnemecHmm, we shouldn't have removed that without a deprecation period.21:19
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow failure objects exceptions to be replaced  https://review.openstack.org/14664221:20
marunbnemec: Does oslo.db gate on dependent projects?21:20
bnemecmarun: Not unit tests.21:20
marunbnemec: The tests in question are part of our functional (system-interacting) job.21:20
bnemecI believe there are devstack tempest jobs, but cross-gating everything in oslo turned out to be impractical.21:21
marunbnemec: 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
marunit's -> its21:22
marunbnemec: Would it be possible to have oslo.db gate on it?21:22
bnemecmarun: 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
marunbnemec: Or is there another coordination strategy you'd prefer?21:23
marunbnemec: If break/fix is the way to do it for the short-term, I'm fine with that.21:23
bnemecIt's possible we could set up some specific cross-testing to help with this.21:23
bnemecmarun: Yeah, break/fix is kind of where we are right now.21:23
marunbnemec: ok.21:23
*** jeckersb is now known as jeckersb_gone21:23
marunbnemec: 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
dhellmannmarun: thanks for the bug report. I ran the unit tests, and tempest, but didn't realize the functional tests were already split out21:24
bnemecdhellmann: 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
marunbnemec: 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 change21:25
zzzeekah, hm21:25
zzzeekwow21:25
bnemecWell, it was still a removal of something in the public api that wasn't deprecated.21:25
bnemecBut if neutron wants to work around it to unblock themselves that is fine too obviously. :-)21:26
marundhellmann: ah, makes sense.  Yeah, we finally moved the db tests to the functional job recently.21:26
marundhellmann: where they belong, imho :)21:26
dhellmannyeah, my mistake for not checking on that21:26
zzzeekbnemec: it would be nice if neutron could work around this21:26
dhellmannalthough this looks like a method with a public name that should have been deprecated instead of removed immediately21:26
zzzeekbnemec: by “work around” i mean “modernize” on their end21:26
marunzzzeek: armax is working on a fix, so I think we're fine.21:27
armaxmarun, zzzeek: posting now21:27
zzzeekdhellmann: 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
dhellmannzzzeek: for the namespace changes, yes, I have been manually running unit tests against a bunch of projects before release21:28
zzzeekdhellmann: so its manual, OK21:28
dhellmannthe 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 tests21:28
bnemecMaybe we could create a few cross-test jobs for projects that we know are consuming libs.21:29
dhellmannnormally it would have been caught by someone noticing that it was a public api in review, though21:29
dhellmannyeah, we may need to add some21:29
zzzeekdhellmann: so with this, i wouldnt want to revert, id restore the check_foreign_keys() method alone21:29
zzzeekdhellmann: im assuming they’re calling that directly, if i recall correctly21:29
dhellmannzzzeek: ok, why not just revert it?21:30
zzzeektest_models_sync should continue to use alembic21:30
dhellmannah, ok21:30
zzzeekdhellmann: because the newer version is using an alembic feature that works better than the method that is here21:30
marunthis calls for some intelligence in how we run tests...21:30
dhellmannzzzeek: can you work with rpodolyaka1 and viktors|afk to put something like that together, and include a deprecation warning21:30
zzzeekdhellmann: OK, so we want to fix olso.db here and not just wait for neutron, right?21:31
dhellmannI 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 issue21:31
dhellmannzzzeek: 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 this21:31
dhellmannI don't know this contributor's irc nick to include them in the conversation, do you?21:32
zzzeekdhellmann: i dont21:32
dhellmannok21:32
zzzeekbut we +2’ed it so its us anyway21:32
dhellmanntrue21:32
zzzeekzap!  I did not +2 this! :)21:33
zzzeekbut ill push something up :)21:33
dhellmannmarun: 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 jobs21:33
dhellmannzzzeek: thanks!21:33
*** mriedem has left #openstack-oslo21:34
marundhellmann: 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
marundhellmann: 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
marundhellmann: This capability would also allow us to skip running tempest jobs for doc-only change.s21:35
* marun wants a pony, too.21:36
sdaguemarun: 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 checking21:37
*** jroll has quit IRC21:38
*** jroll has joined #openstack-oslo21:38
marunsdague: jaypipes started working on this pre-summit, so maybe I'll see where he ended up21:38
sdagueok, that's the first I heard of jay working on code there.21:39
marunsdague: Would you happen to have a pointer to where the non-inverse capability is defined so I can investigate?21:39
dhellmannmarun: 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
jaypipesmarun: I haven't gotten anywhere with that :(21:39
marunjaypipes: oh, nuts21:39
jaypipessorry :(21:39
marunjaypipes: and here I was hoping to avoid work :)21:40
jaypipes:)21:40
marundhellmann: I'm hoping we could do something smarter...21:40
dhellmannyeah21:40
*** david-ly_ has quit IRC21:40
marundhellmann: collecting the dependency graph either on-demand or periodically is probably the tough part of making it smarter, though.21:41
marundhellmann: 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-oslo21:41
dhellmannright21:42
sdaguemarun: 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 match21:44
sdaguebut 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 additive21:44
marunsdague: wouldn't it be possible to not add though?21:45
*** rpodolyaka2 has joined #openstack-oslo21:45
marunsdague: so, where are the objects used in this method defined?21:46
marunsdague: ah, I should clone and introspect21:46
sdagueso, realistically, follow on questions are best for jeblair and infra folks21:46
sdagueI only roughly understand this part21:46
*** david-lyle has quit IRC21:49
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement generic update-on-match feature  https://review.openstack.org/14622821:49
marunsdague: ok, thanks for pointer!21:49
*** david-lyle has joined #openstack-oslo21:50
openstackgerritMerged openstack/oslo.concurrency: Updated from global requirements  https://review.openstack.org/14616021:51
*** jgrimm is now known as zz_jgrimm21:52
marunsdague: out of curiosity, why do you understand this part?21:55
marunsdague: Were you working on something around this?21:55
openstackgerritDoug Hellmann proposed openstack-dev/pbr: Be more aggressive about building wheels  https://review.openstack.org/14665021:56
*** jeckersb_gone is now known as jeckersb21:57
sdaguemarun: 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
sdaguethat would save a lot of thundering herds21:58
marunsdague: ah, makes sense.  and did you end up implementing that capability?21:59
sdagueno, I got as far as vaguely understanding where changes might be needed21:59
marunsdague: ah, ok.22:01
*** ajo has joined #openstack-oslo22:02
*** andreykurilin has quit IRC22:06
*** andreykurilin has joined #openstack-oslo22:07
openstackgerritRoman Podoliaka proposed openstack/oslo.db: Revert "Remove check_foreign_keys from ModelsMigrationsSync"  https://review.openstack.org/14665822:10
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.utils: custom context manger for py27/py3x compatability  https://review.openstack.org/14663922:10
*** mriedem has joined #openstack-oslo22:10
*** e0ne has joined #openstack-oslo22:17
dims__">=0.10" vs ">=0.10.0" bnemec? weird22:18
dims__jogo: ^^^22:18
dims__jogo: see bnemec's email to the mailing list "Hacking 0.10 released"22:18
bnemecdims__: Yeah, see the requirements job results for https://review.openstack.org/#/c/146627/22:18
*** stevemar has quit IRC22:18
dims__bnemec: either we fix /usr/local/jenkins/slave_scripts/project-requirements-change.py or we should fix global-requirements txt22:20
dims__right?22:20
*** ajo has quit IRC22:23
*** ajo has joined #openstack-oslo22:24
bnemecdims__: 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 IRC22:27
openstackgerritMichael Bayer proposed openstack/oslo.db: Restore the check_foreign_keys() method.  https://review.openstack.org/14666322:27
*** rpodolyaka2 has quit IRC22:33
*** rpodolyaka2 has joined #openstack-oslo22:35
*** rpodolyaka2 has quit IRC22:39
*** ajo has quit IRC22:44
*** mriedem has quit IRC22:45
*** rossella_s has quit IRC22:46
openstackgerritMerged openstack/tooz: IPC simplification  https://review.openstack.org/14643122:47
*** rossella_s has joined #openstack-oslo22:47
openstackgerritMerged openstack/tooz: Update sentinel support to allow multiple sentinel hosts  https://review.openstack.org/14646322:48
*** raildo has joined #openstack-oslo22:48
*** armax has quit IRC22:50
*** gordc has quit IRC22:52
jogodims__ bnemec: odd, thanks for pointing that out22:52
*** e0ne has quit IRC22:54
*** vigneshvar has quit IRC22:56
*** raildo has quit IRC23:01
*** ihrachyshka has joined #openstack-oslo23:03
ihrachyshkadims__, hey23:07
ihrachyshkadims__, first, thanks for oslo namespace hacking check.23:07
ihrachyshkadims__, though it seems incomplete (it does not catch e.g. 'from oslo import messaging')23:07
ihrachyshkaI 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
dhellmannzzzeek, 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-oslo23:12
zzzeekdhellmann: are we collecting pymysql eval somewhere?  wiki page ?23:12
dhellmannzzzeek: I don't have a page going, I thought you all were going to start something23:12
dhellmannbased on our discussion from the meeting last week, right?23:13
*** jaosorior has quit IRC23:13
zzzeekrrrrright :)23:13
* zzzeek cues dog ate his homework23:13
dhellmann:-)23:14
zzzeekdhellmann: 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 predictable23:14
*** dims__ has quit IRC23:14
zzzeekdhellmann: once we swallow the eventlet pill here, we are opened up to a whole host of weird new issues that can arise23:14
dhellmannzzzeek: I'll add that to my review list for tomorrow23:14
*** raildo has joined #openstack-oslo23:14
dhellmannright23:14
*** dims__ has joined #openstack-oslo23:15
*** vigneshvar has quit IRC23:15
zzzeekb.c. right now we’re just blockoing on DB access.   though if we keep the connecvtion pool sizes low, it wont be as bad23:15
dhellmannzzzeek: 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 changes23:15
zzzeekok23:15
dhellmannzzzeek: how about if we add this to the meeting agenda for next week?23:16
zzzeekdhellmann: oh but anyway, oracle is a no-reply, so mysql-connector is out23:16
zzzeekdhellmann: sure23:16
dhellmannzzzeek: yeah, the lack of response from oracle is an important data point23:16
zzzeekdhellmann: yup23:16
zzzeekdhellmann: id really not mind having a mysql driver on stackforge, like pymysql itself or a fork, whats the position on that23:17
zzzeekdhellmann: just beacuse we really need somethign that can respond to issues quickly23:17
dhellmannthat'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 us23:18
dhellmannwe should also talk to them about moving, rather than forking, like we did with wsme and pecan23:18
*** dims__ has quit IRC23:19
zzzeekdhellmann: well anyway id need to look and see how responsive they are to start with, i havent logged who maintains it yet23:20
*** david-lyle has quit IRC23:20
dhellmannzzzeek: right, that's all part of the info we need to collect on this one23:20
ihrachyshkadhellmann, 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-oslo23:25
dhellmannihrachyshka: do you mean the thing that removed the filter?23:26
ihrachyshkadhellmann, 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 misunderstanding23:26
dhellmannihrachyshka: 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
dhellmannihrachyshka: 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 it23:27
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.utils: custom context manger for py27/py3x compatability  https://review.openstack.org/14663923:27
openstackgerritJoshua Harlow proposed openstack/taskflow: Add executor statistics  https://review.openstack.org/14638223:27
ihrachyshkasure thing, thanks :)23:27
dhellmannihrachyshka: sorry about the delay on this one, I really thought that change was in 0.3.023:28
ihrachyshkathat'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-oslo23:32
ihrachyshkazzzeek, "AttributeError: 'TestModelsMigrationsPsql' object has no attribute 'check_foreign_keys'" in neutron. is it due to oslo.db release?23:33
dhellmannihrachyshka: yes, marun and others are working on a fix23:33
zzzeekihrachyshka: yes see https://review.openstack.org/#/c/146663/, though neutron is uspposed to be fixing independently23:33
ihrachyshkaah, ok, thanks23:33
zzzeekihrachyshka: also, I’m writing up on PyMySQL, Mysql-connector is out23:33
dhellmannzzzeek: I missed that. +2a.23:34
*** bknudson has joined #openstack-oslo23:34
ihrachyshkazzzeek, ack, tnx23:34
openstackgerritJoshua Harlow proposed openstack/taskflow: Add executor statistics  https://review.openstack.org/14638223:34
*** dims__ has quit IRC23:35
*** andreykurilin has quit IRC23:35
*** dims_ has quit IRC23:36
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow failure objects exceptions to be replaced  https://review.openstack.org/14664223:45
*** mriedem has joined #openstack-oslo23:52
zzzeekihrachyshka: here’s my start: https://wiki.openstack.org/wiki/PyMySQL_evaluation23:54

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!