Friday, 2015-02-13

*** salv-orlando has joined #openstack-oslo00:01
openstackgerritJoe Gordon proposed openstack/debtcollector: Move to hacking 0.10  https://review.openstack.org/15551700:02
openstackgerritJoe Gordon proposed openstack/taskflow: Move to hacking 0.10  https://review.openstack.org/15552100:03
*** zzzeek has quit IRC00:05
*** zzzeek has joined #openstack-oslo00:12
*** mtanino has quit IRC00:26
*** david-lyle is now known as david-lyle_afk00:37
*** zzzeek has quit IRC00:41
*** ViswaV has quit IRC00:51
*** daniel3_ has quit IRC01:04
openstackgerritIan Wienand proposed openstack/oslo.policy: Deprecate default value for "policy_dirs"  https://review.openstack.org/15474201:05
openstackgerritJoshua Harlow proposed openstack/taskflow: Remove delayed decorator and replace with nicer method  https://review.openstack.org/15555301:06
*** bknudson has joined #openstack-oslo01:13
greghaynesHello! I am trying to use oslo.db with pymysql and it seems that the integritey error matching is not working as it should, which results in a DBError being raised rather than DBReferenceError01:14
greghayneshttp://logs.openstack.org/39/152339/3/check/gate-storyboard-python27/1e768db/console.html#_2015-02-12_02_13_07_620 is a link to where this happens in a test01:15
greghaynescurious if 1) it seems that is in fact what is going on 2) this is a known issue?01:15
openstackgerritJoshua Harlow proposed openstack/taskflow: Remove delayed decorator and replace with nicer method  https://review.openstack.org/15555301:16
greghaynesdhellmann: git blame implicates you ;)01:18
*** achanda has quit IRC01:19
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow loading conductors via entrypoints  https://review.openstack.org/12461701:22
*** noelbk has quit IRC01:24
openstackgerritJoshua Harlow proposed openstack/taskflow: Mark conductor 'stop' method deprecation kwarg with versions  https://review.openstack.org/15556201:28
*** dims_ has joined #openstack-oslo01:32
*** dims_ has quit IRC01:32
*** dims_ has joined #openstack-oslo01:32
*** dims__ has quit IRC01:33
*** tsekiyam_ has joined #openstack-oslo01:34
*** dims__ has joined #openstack-oslo01:36
*** tsekiyama has quit IRC01:37
openstackgerritJoshua Harlow proposed openstack/taskflow: Mark conductor 'stop' method deprecation kwarg with versions  https://review.openstack.org/15556201:38
*** tsekiyam_ has quit IRC01:38
*** dims_ has quit IRC01:39
*** ChuckC has quit IRC01:45
*** salv-orl_ has joined #openstack-oslo01:46
*** salv-orlando has quit IRC01:48
*** ChuckC has joined #openstack-oslo01:52
openstackgerritgreghaynes proposed openstack/oslo.db: Fix PyMySQL reference error detection  https://review.openstack.org/15557401:53
greghaynes^ wow that was fun to debug01:53
greghaynesnot sure where that error actually lies in terms of upstreams, I think PyMySQL?01:53
harlowja_looks like a unicode thing01:54
harlowja_fun fun01:54
openstackgerritJoshua Harlow proposed openstack/tooz: Allow coordinator non-string options and use them  https://review.openstack.org/15560202:11
*** mriedem is now known as mriedem_zz02:16
*** salv-orl_ has quit IRC02:17
openstackgerritIan Wienand proposed openstack/oslo.policy: Deprecate default value for "policy_dirs"  https://review.openstack.org/15474202:27
openstackgerritJoshua Harlow proposed openstack/taskflow: Remove delayed decorator and replace with nicer method  https://review.openstack.org/15555302:27
*** yamahata has quit IRC03:04
*** amotoki has joined #openstack-oslo03:04
*** dims__ has quit IRC03:07
*** salv-orlando has joined #openstack-oslo03:18
*** harlowja_ is now known as harlowja_away03:27
*** sdague has quit IRC03:31
*** sdague has joined #openstack-oslo03:31
*** ChuckC_ has joined #openstack-oslo03:40
*** ChuckC has quit IRC03:44
*** liusheng has joined #openstack-oslo04:05
*** dims__ has joined #openstack-oslo04:07
*** dims__ has quit IRC04:12
*** amotoki has quit IRC04:20
*** liusheng has quit IRC04:26
*** liusheng has joined #openstack-oslo04:26
*** yamahata has joined #openstack-oslo04:35
*** harlowja_at_home has joined #openstack-oslo04:48
openstackgerritJoshua Harlow proposed openstack/taskflow: Mark conductor 'stop' method deprecation kwarg with versions  https://review.openstack.org/15556204:50
*** achanda has joined #openstack-oslo05:02
*** harlowja_at_home has quit IRC05:05
*** ViswaV has joined #openstack-oslo05:06
*** amotoki has joined #openstack-oslo05:09
*** salv-orlando has quit IRC05:10
*** ViswaV has quit IRC05:21
*** ankit_ag has joined #openstack-oslo05:22
*** ViswaV has joined #openstack-oslo05:23
openstackgerritSyed Ismail Faizan Barmawer proposed openstack/oslo-incubator: Add common Ironic modules as libironic  https://review.openstack.org/15519005:27
*** sheeprine has quit IRC05:35
*** sheeprine has joined #openstack-oslo05:36
openstackgerritJoshua Harlow proposed openstack/taskflow: Chain exceptions correctly on py3.x  https://review.openstack.org/15516805:39
openstackgerritJoshua Harlow proposed openstack/taskflow: Use the ability to chain exceptions correctly  https://review.openstack.org/15518705:39
*** achanda has quit IRC06:03
*** achanda has joined #openstack-oslo06:11
*** yamahata has quit IRC06:13
*** yamahata has joined #openstack-oslo06:14
openstackgerritMerged openstack/taskflow: Clarify documentation related to inputs  https://review.openstack.org/15509306:18
*** salv-orlando has joined #openstack-oslo06:20
*** ViswaV has quit IRC06:29
openstackgerritSyed Ismail Faizan Barmawer proposed openstack/oslo-incubator: Add common Ironic modules as libironic  https://review.openstack.org/15519006:48
*** noelbk has joined #openstack-oslo06:50
*** ViswaV has joined #openstack-oslo07:05
*** dims__ has joined #openstack-oslo07:10
*** ankit_ag has quit IRC07:13
*** ankit_ag has joined #openstack-oslo07:13
openstackgerritMehdi Abaakouk proposed openstack/tooz: Two locks acquired from one coord must works  https://review.openstack.org/15530307:14
*** dims__ has quit IRC07:15
openstackgerritJoshua Harlow proposed openstack/taskflow: Improve multilock class and its associated unit test  https://review.openstack.org/15566307:21
*** ViswaV has quit IRC07:21
openstackgerritMehdi Abaakouk proposed openstack/tooz: Two locks acquired from one coord must works  https://review.openstack.org/15530307:26
openstackgerritMichal Jastrzebski (inc0) proposed openstack/oslo.versionedobjects: New field type - Json  https://review.openstack.org/15482007:30
openstackgerritMehdi Abaakouk proposed openstack/tooz: Two locks acquired from one coord must works  https://review.openstack.org/15530307:37
*** andreykurilin_ has joined #openstack-oslo07:42
*** salv-orlando has quit IRC07:43
*** andreykurilin_ has quit IRC07:50
openstackgerritMehdi Abaakouk proposed openstack/tooz: Two locks acquired from one coord must works  https://review.openstack.org/15530307:50
*** andreykurilin_ has joined #openstack-oslo07:50
silehtjd__, ^^ is ready07:51
jd__ok07:53
*** stevemar has quit IRC07:56
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: NotifyPublisher need handle amqp_auto_delete  https://review.openstack.org/13872307:56
openstackgerritMerged openstack/taskflow: Move to hacking 0.10  https://review.openstack.org/15552108:03
*** andreykurilin_ has quit IRC08:07
*** andreykurilin_ has joined #openstack-oslo08:11
openstackgerritMichal Jastrzebski (inc0) proposed openstack/oslo.versionedobjects: Fixes for heat implementation  https://review.openstack.org/15483508:14
*** inc0 has joined #openstack-oslo08:15
inc0good morning08:16
openstackgerritMichal Jastrzebski (inc0) proposed openstack/oslo.versionedobjects: Fixes for heat implementation  https://review.openstack.org/15483508:19
openstackgerritMichal Jastrzebski (inc0) proposed openstack/oslo.versionedobjects: New field type - Json  https://review.openstack.org/15482008:19
*** achanda has quit IRC08:21
*** achanda has joined #openstack-oslo08:23
*** dulek has joined #openstack-oslo08:24
*** ihrachyshka has joined #openstack-oslo08:24
openstackgerritqiaojian proposed openstack/oslo.messaging: Fix matchmaker_redis ack_alive fails with KeyError  https://review.openstack.org/15567308:30
openstackgerritqiaojian proposed openstack/oslo.messaging: Fix matchmaker_redis ack_alive fails with KeyError  https://review.openstack.org/15567308:34
*** achanda has quit IRC08:38
*** openstackgerrit has quit IRC08:42
*** openstackgerrit has joined #openstack-oslo08:42
openstackgerritMichal Jastrzebski (inc0) proposed openstack/oslo.versionedobjects: Fixes for heat implementation  https://review.openstack.org/15483508:43
*** achanda has joined #openstack-oslo08:44
*** vigneshvar has joined #openstack-oslo08:50
*** MasterPiece has joined #openstack-oslo09:05
*** dtantsur|afk is now known as dtantsur09:06
*** MasterPiece has quit IRC09:11
*** achanda has quit IRC09:14
*** e0ne has joined #openstack-oslo09:26
openstackgerritMichal Jastrzebski (inc0) proposed openstack/oslo.versionedobjects: New field type - Json  https://review.openstack.org/15482009:32
*** amotoki has quit IRC09:34
*** salv-orlando has joined #openstack-oslo09:40
openstackgerritMerged openstack/oslo.versionedobjects: Compatibility with Python 3  https://review.openstack.org/15440709:56
*** MasterPiece has joined #openstack-oslo10:09
*** dims__ has joined #openstack-oslo10:12
*** alexpilotti has joined #openstack-oslo10:14
*** dims__ has quit IRC10:17
*** e0ne is now known as e0ne_10:23
openstackgerritMichal Jastrzebski (inc0) proposed openstack/oslo.versionedobjects: Remove more Nova-isms  https://review.openstack.org/15539310:30
*** e0ne_ is now known as e0ne10:32
*** inc0 has quit IRC11:01
*** dulek_ has joined #openstack-oslo11:03
*** salv-orlando has quit IRC11:05
*** dulek has quit IRC11:06
*** dulek_ is now known as dulek11:07
*** dims__ has joined #openstack-oslo11:12
openstackgerritMehdi Abaakouk proposed openstack/tooz: Releases locks in tests  https://review.openstack.org/15571511:35
openstackgerritMehdi Abaakouk proposed openstack/tooz: Two locks acquired from one coord must works  https://review.openstack.org/15530311:35
*** e0ne is now known as e0ne_11:38
*** ankit_ag has quit IRC11:41
*** ankit_ag has joined #openstack-oslo11:41
*** yamahata has quit IRC11:43
*** e0ne_ has quit IRC11:43
openstackgerritMehdi Abaakouk proposed openstack/tooz: Two locks acquired from one coord must works  https://review.openstack.org/15530311:47
openstackgerritMehdi Abaakouk proposed openstack/tooz: Two locks acquired from one coord must works  https://review.openstack.org/15530311:50
*** e0ne has joined #openstack-oslo11:52
*** dims__ has quit IRC12:01
*** dulek_ has joined #openstack-oslo12:02
*** dims__ has joined #openstack-oslo12:02
*** salv-orlando has joined #openstack-oslo12:04
*** e0ne is now known as e0ne_12:04
*** dulek has quit IRC12:04
*** dims__ has quit IRC12:06
*** dims__ has joined #openstack-oslo12:07
*** jaypipes has quit IRC12:13
*** e0ne_ has quit IRC12:14
*** jaypipes has joined #openstack-oslo12:16
*** ajo_ has joined #openstack-oslo12:23
*** ajo has quit IRC12:26
*** dims__ has quit IRC12:32
*** pblaho__ has quit IRC12:37
*** dulek_ has quit IRC12:40
*** ankit_ag has quit IRC12:46
*** jaosorior has joined #openstack-oslo12:57
*** dims__ has joined #openstack-oslo13:13
*** dims___ has joined #openstack-oslo13:16
*** dims__ has quit IRC13:17
*** inc0 has joined #openstack-oslo13:24
openstackgerritVipin Balachandran proposed openstack/oslo.vmware: PBM utility method to retrieve profiles of a VM  https://review.openstack.org/15574513:31
*** vigneshvar has quit IRC13:39
jd__sileht: the IPC random failure at http://logs.openstack.org/03/155303/13/check/gate-tooz-tox-py27-postgresql/52ba2d4/ is awesome13:42
jd__errno 43 is EIDRM it's not really documented as a possible errno AFAICS13:42
jd__ah it is in shmctl() so maybe the binding is using that13:43
jd__bug in sysv_ipc?13:43
* jd__ don't want to look13:43
openstackgerritMerged openstack/tooz: Releases locks in tests  https://review.openstack.org/15571514:03
*** miqui_away is now known as miqui14:21
*** david-lyle_afk is now known as david-lyle14:22
*** e0ne has joined #openstack-oslo14:26
*** sigmavirus24_awa is now known as sigmavirus2414:40
*** mriedem_zz has quit IRC14:44
*** mriedem has joined #openstack-oslo14:44
*** e0ne is now known as e0ne_14:45
haypoi read that subprocess is not used with monkey-patching in openstack14:46
haypobut is there a way to control asynchronously a subprocess?14:46
dansmithinc0: you think we should keep the persistent object?14:49
inc0dansmith, I don't know really, I haven't been using it14:50
dansmithinc0: I guess nova-spawned projects may all still have those fields, but I didn't think it was universal enough to really include in the base layer so I was going to leave it out14:51
dansmithI'd propose we leave it out and if a bunch of projects define it locally and identically, then we can add it back in14:52
inc0well, if its in nova, its always possible to make a patch with it14:52
*** pblaho__ has joined #openstack-oslo14:53
*** pblaho__ is now known as pblaho14:53
inc0my thoughts exactly...let's not add to general confusion about this project:)14:53
*** pblaho has quit IRC14:53
*** pblaho has joined #openstack-oslo14:53
inc0dansmith, also, I've made patchset with jsonschema14:54
inc0in test you have sample of usage, what do you think?14:54
*** stevemar has joined #openstack-oslo14:54
dansmithinc0: I think it doesn't change my arguments at all, but I'll have a look in a bit :)14:55
inc0it will kill 2 birds with one stone (oh how I hate this morbid metaphore). Will allow to have in-place strict schema if needed and allow just enough flexibility to suit all needs14:55
*** e0ne_ is now known as e0ne14:55
*** jaypipes is now known as leakypipes14:56
*** zz_jgrimm is now known as jgrimm14:56
*** jecarey has joined #openstack-oslo14:56
inc0if we're going to break strict schemas anyway (in heat we have to), we might as well do it in a way we can control14:56
inc0if we close this doors permanently, people will find the way around, and that's scary14:57
*** dansmith is now known as superdan14:58
*** liusheng has quit IRC15:01
*** liusheng has joined #openstack-oslo15:01
*** MasterPiece has quit IRC15:14
*** MasterPiece has joined #openstack-oslo15:15
*** dims___ is now known as dimsum__15:15
openstackgerritTerry Howe proposed openstack/cliff: Fix pep8 tests for lambda  https://review.openstack.org/15577815:18
*** prad has joined #openstack-oslo15:18
*** pblaho has quit IRC15:21
*** dulek has joined #openstack-oslo15:28
openstackgerritTerry Howe proposed openstack/cliff: Change the argument passed to __init__ for help  https://review.openstack.org/15578415:30
*** vigneshvar has joined #openstack-oslo15:37
*** e0ne is now known as e0ne_15:38
*** e0ne_ is now known as e0ne15:43
*** vigneshvar has quit IRC15:47
*** MasterPiece has quit IRC15:48
bnemechaypo: I have no idea, but for some reason eventlet has a green subprocess module that we were using in processutils: https://github.com/openstack/oslo.concurrency/commit/394fbf4f046bcaa4e0591238b9fc06f1769e6ac5#diff-3faee77cf16073f0a68123aaa0bc3d64L2815:51
bnemecIt may have to do with how subprocess does io or waits for processes /pure speculation15:52
*** MasterPiece has joined #openstack-oslo15:57
*** inc0 has quit IRC16:03
*** ajo_ has quit IRC16:05
*** ajo has joined #openstack-oslo16:05
*** mriedem1 has joined #openstack-oslo16:06
*** mriedem has quit IRC16:08
*** jgrimm_ has joined #openstack-oslo16:10
*** dulek has quit IRC16:10
*** dulek has joined #openstack-oslo16:11
*** bknudson has quit IRC16:12
*** jgrimm- has joined #openstack-oslo16:13
*** jgrimm has quit IRC16:13
*** dulek has quit IRC16:16
*** jgrimm_ has quit IRC16:17
*** MasterPiece has quit IRC16:20
*** MasterPiece has joined #openstack-oslo16:24
*** bknudson has joined #openstack-oslo16:27
*** noelbk has quit IRC16:30
*** zzzeek has joined #openstack-oslo16:35
*** prad has quit IRC16:38
bnemecdhellmann: So you're familiar with python-openstackclient, right?16:44
* bnemec realizes how much he's about to type in IRC and wonders if an e-mail would have been better16:45
bnemecThe reason I ask is that we're starting to have serious discussions about our user interface for deployment, and there's been a number of suggestions that we create 'one CLI to rule them all' and wrap all the openstack services.16:46
bnemecWhich is basically python-openstackclient as I understand it.16:46
bnemecThe only catch is we want to add some deployment-specific stuff, but it looks like openstackclient would allow us to plug in another client library.16:47
*** ihrachyshka has quit IRC16:47
bknudsonbnemec: openstackclient folks hang out in -sdks.16:48
stevemarbknudson, bnemec we have out here too16:48
bnemecIf that's the case, then I think it should work for us and I'm going to push hard for us to use it.16:48
bknudsoneveryone should use it.16:48
bknudsonall the other clis should be deprecated.16:49
stevemarbknudson, thats the tyrannical spirit we need16:50
bnemecbknudson: Yeah, that's basically what I'm thinking.16:52
bnemecJust trying to make sure it works the way I think it does before I start insisting. :-)16:52
bknudsonI actually don't know about the plugins.16:52
bnemecI'll go ask in -sdks16:53
*** ihrachyshka has joined #openstack-oslo16:54
*** yamahata has joined #openstack-oslo16:54
*** MasterPiece has quit IRC16:55
*** e0ne is now known as e0ne_16:58
*** MasterPiece has joined #openstack-oslo17:00
*** e0ne_ is now known as e0ne17:03
*** MasterPiece has quit IRC17:10
*** e0ne is now known as e0ne_17:11
*** stevemar has quit IRC17:11
*** noelbk has joined #openstack-oslo17:12
*** alexpilotti has quit IRC17:13
openstackgerritMerged openstack/debtcollector: Move to hacking 0.10  https://review.openstack.org/15551717:14
*** alexpilotti has joined #openstack-oslo17:15
*** alexpilotti has quit IRC17:16
*** e0ne_ is now known as e0ne17:16
*** daniel3_ has joined #openstack-oslo17:21
*** dtantsur is now known as dtantsur|afk17:23
*** ekarlso has quit IRC17:25
*** ekarlso has joined #openstack-oslo17:25
*** ViswaV has joined #openstack-oslo17:28
openstackgerritValeriy Ponomaryov proposed openstack/oslo.config: Add exception handling for entry points  https://review.openstack.org/15386317:32
openstackgerritMerged openstack/oslo.db: Fix PyMySQL reference error detection  https://review.openstack.org/15557417:32
*** ViswaV has quit IRC17:44
*** harlowja_away is now known as harlowja_17:48
*** jogo is now known as flashgordon17:54
*** stevemar has joined #openstack-oslo17:54
*** MasterPiece has joined #openstack-oslo17:55
*** e0ne has quit IRC17:56
*** krotscheck has quit IRC18:00
*** krotscheck has joined #openstack-oslo18:00
*** mtanino has joined #openstack-oslo18:01
greghaynesbnemec: have you looked at shade?18:03
bnemecgreghaynes: Nope, what is it?18:04
greghaynesIts and infra repo for doing what you describe18:05
* bnemec looks18:05
openstackgerritJoe Gordon proposed openstack-dev/cookiecutter: Bump cookiecutter's hacking to 0.10  https://review.openstack.org/15584318:06
*** tsekiyama has joined #openstack-oslo18:07
greghaynesbnemec: not sure how far along it is but I think they just want to make a single OS client for their needs which seems like a similar issue18:08
openstackgerritJoe Gordon proposed openstack-dev/cookiecutter: Bump cookiecutter's hacking to 0.10  https://review.openstack.org/15584318:09
bnemecgreghaynes: Yeah, interesting.  I wonder if that should just be part of openstackclient as an "easy mode" or something.18:09
greghaynesMight be worth asking mordred why they didnt do that18:09
openstackgerritJoe Gordon proposed openstack-dev/oslo-cookiecutter: Bump cookiecutter's hacking to 0.10  https://review.openstack.org/15584618:10
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.concurrency: Ability to set working directory  https://review.openstack.org/15584718:12
*** pblaho has joined #openstack-oslo18:16
*** vigneshvar has joined #openstack-oslo18:17
*** jaosorior has quit IRC18:21
*** dtantsur|afk has quit IRC18:24
*** achanda has joined #openstack-oslo18:25
*** liusheng has quit IRC18:25
*** liusheng has joined #openstack-oslo18:25
*** dtantsur has joined #openstack-oslo18:26
*** ViswaV has joined #openstack-oslo18:28
*** pblaho has quit IRC18:28
*** jaosorior has joined #openstack-oslo18:29
*** andreykurilin_ has quit IRC18:32
greghaynesSo, who do I need to beg for a release to be cut for oslo.db :)18:35
greghaynesbeg and or buy a tasty beverage18:35
*** vigneshvar has quit IRC18:38
*** vigneshvar has joined #openstack-oslo18:39
bnemecgreghaynes: I would start with rpodolyaka zzzeek and dhellmann18:41
zzzeekbnemec: thought we were doing one monday ?18:41
bnemeczzzeek: Could be. :-)18:42
bnemecgreghaynes: ^18:42
zzzeekdhellmann: oslo.db release monday?18:42
greghaynesThatd make me happy :)18:42
bnemecBut yeah, we definitely won't release on a Friday.18:42
bnemecUnless it's critical, of course.18:42
greghaynesTheres a bugfix id really like out but waiting until monday is totally fine with me18:43
morganfainbergdhellmann, how awful would it be to break the default behavior in oslo.policy right now [quick bug-fix release] re: https://review.openstack.org/#/c/154742/ i think we're headed down a "doesn18:52
morganfainberg't fix anything" path18:52
morganfainbergwith those warning changes.18:52
morganfainbergdhellmann, i only ask because oslo.policy is so newly released18:52
*** MasterPiece has quit IRC18:54
*** liusheng has quit IRC18:57
*** liusheng has joined #openstack-oslo18:58
*** ViswaV has quit IRC19:00
*** MasterPiece has joined #openstack-oslo19:01
*** yamahata has quit IRC19:12
*** stevemar has quit IRC19:13
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Remove more Nova-isms  https://review.openstack.org/15539319:19
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo-incubator: Switch from oslo.config to oslo_config  https://review.openstack.org/15586919:19
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo-incubator: Switch from oslo.config to oslo_config  https://review.openstack.org/15586919:28
*** noelbk has quit IRC19:28
*** stevemar has joined #openstack-oslo19:33
bnemecmorganfainberg: See my comments on the review.  oslo.policy isn't released yet. :-)19:38
morganfainbergbnemec, oh it isn't19:38
dimsum__superdan: do we have to keep both object_data? http://git.openstack.org/cgit/openstack/oslo.versionedobjects/tree/oslo_versionedobjects/tests/test_objects.py#n114319:38
bnemecAlso the deprecation warning is temporary, and will only be logged if people are relying on that default behavior already.19:38
morganfainbergi saw on pypi a 1.0.0 thing19:38
dimsum__superdan: 1143 and 114819:38
morganfainbergbnemec, i would rather break the compatibility before we release and make part of moving to the lib setting a sane default here?19:39
bnemec97 downloads in the last month <- Huh?19:39
bnemecWhat are you playing at, pypi?19:39
morganfainbergbnemec, i mean, the deprecation sounds kindof insane if it's not been released here19:39
superdandimsum__: technically not, but lets leave it until we figure out how to make this test a template for other projects19:39
superdandimsum__: if that's okay19:39
morganfainbergbnemec, especially since it could result in a) noise, and b) if we end up with people relying on that behavior in projects that weren't relying on it to begin with19:40
bnemecmorganfainberg: Right, but services with the policy code have been released.19:40
dimsum__superdan: yep, works for me, just checking19:40
bnemecAnd this is user-facing behavior so we can't just say "fix it when you adopt oslo.policy"19:40
morganfainbergbnemec, so they set a default as part of moving from oslo-incubator -> policy lib?19:40
morganfainbergbnemec, the services could override this in a set_defaults manner that could be sourced in a compatible way19:40
bnemecmorganfainberg: No, it's in the original incubator code: https://github.com/openstack/oslo-incubator/blob/stable/juno/openstack/common/policy.py#L10319:41
bnemecSo users may be relying on that behavior already.19:41
morganfainbergbnemec, then we don't deprecate it19:41
*** noelbk has joined #openstack-oslo19:41
morganfainbergbnemec, my point is if people may be relying on this behavior, don't deprecate it. document we need that directory - live with the warning19:41
morganfainbergchanging this behavior is either: a) services need to handle the change in defaults cleanly, or b) we don't change the behavior19:42
morganfainbergi think my comment wasn't clear that was what i was advocating19:42
bnemecmorganfainberg: Actually, I wonder if we could just create that directory if it doesn't exist.  At least for the default case.19:42
morganfainbergbnemec, unlikely, i wouldn't assume you have write access19:43
bnemecmorganfainberg: Yeah19:43
morganfainbergperhaps changing the warning to "you should either override this or create the directory" would be better19:43
morganfainbergand we make the services [e.g. what devstack does] ensure the directory is there19:43
morganfainbergi don't like changing the behavior to be the same but instead of a useful warning a scary deprecation warning19:44
bnemecmorganfainberg: I guess the thing is that I was fine with it being an info level message.19:44
morganfainbergfor arguably a perfectly reasonable expectation19:44
bnemecIt was other people who had a problem with that.19:44
morganfainbergbnemec, well i'm leaning towards we don't break this or we *really* break it. but not do something wierd with it.19:45
morganfainbergand i'd rather not break it19:45
openstackgerritJoshua Harlow proposed openstack/taskflow: Improve multilock class and its associated unit test  https://review.openstack.org/15566319:45
morganfainbergif we really break it, services moving to oslo.policy would need to handle the "expected default behavior" which tells me that changing the default is not resulting in a sane default19:46
morganfainbergbnemec, so if that makes sense.19:47
*** inc0 has joined #openstack-oslo19:49
bnemecWait, this was being logged 37000 times in a devstack run?19:49
bnemecNo wonder they were bothered by it, even at info level.19:49
bnemecmorganfainberg: I think we should talk to Ian and Jordan from https://review.openstack.org/#/c/152082/ and see if just fixing that repeated logging would be a sufficient fix.19:50
bnemecI've only ever seen it logged once so I didn't realize it was this big a problem.19:51
* bnemec brb19:52
harlowja_dhellmann dimsum__ https://review.openstack.org/#/c/155163/ if u guys get a chance (this should also fix the six.moves problem found a while ago by me [noted in the commit message], that was fixed in six)19:56
harlowja_the six.raise_from helper will help do exceptions better in py2/py3 compat projects19:57
harlowja_my guess is other projects should use it to (tooz...)19:57
*** noelbk has quit IRC19:59
*** inc0 has quit IRC19:59
harlowja_chained exceptions can be quite nice...20:02
sigmavirus24== harlowja_20:03
sigmavirus24they're super confusing for people who're used to Python 2 and start seeing them in Python 3 though20:03
harlowja_hopefully said people can get used to it; thats the least python3.x changes :-P20:03
sigmavirus24I can find an issue in requests where people still comment because the first part of the traceback is similar even though all of their problems are radically different20:04
*** MasterPiece has quit IRC20:04
harlowja_sounds like a traceback formatting issue no?20:04
sigmavirus24No. It's just that people read the first traceback which is the one that was being handled when the other(s) was(were) raised20:05
harlowja_sounds like a UI issue though and people need to adapt :-P20:05
* harlowja_ wonders if http://bugs.python.org/issue17911 will help here (maybe)20:05
harlowja_it seems to be adjusting some of that stuff20:05
sigmavirus24Yeah. It explicitly says "An exception was raised while this exception was being handled"20:05
sigmavirus24In other words: "The treasure is on a different island"20:06
harlowja_ya, so maybe better wording, formatting....20:06
sigmavirus24I just chalk it up to people being lazy because that issue has been receiving comments for >1.5 years now20:06
harlowja_lifeless u working on http://bugs.python.org/issue17911 i think right?20:06
harlowja_ya, me too, its the same kind of crap when facebook (or yahoo, or others) changes there layout20:06
harlowja_people always complain20:07
harlowja_haters gonna hate, lol20:07
sigmavirus24Something like that =)20:08
*** MasterPiece has joined #openstack-oslo20:18
*** ihrachyshka has quit IRC20:21
*** yamahata has joined #openstack-oslo20:25
*** achanda has quit IRC20:31
*** bnemec is now known as beekneemech20:37
*** achanda has joined #openstack-oslo20:43
openstackgerritJoshua Harlow proposed openstack/oslo.versionedobjects: Use six.wraps for py2/py3 compat  https://review.openstack.org/15588220:45
dimsum__harlowja_: +121:02
harlowja_thx21:02
openstackgerritJoshua Harlow proposed openstack/taskflow: Make the atom class an abstract class  https://review.openstack.org/15588721:02
openstackgerritJoshua Harlow proposed openstack/taskflow: Make the atom class an abstract class  https://review.openstack.org/15588721:03
*** e0ne has joined #openstack-oslo21:17
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.concurrency: Ability to set working directory  https://review.openstack.org/15584721:18
openstackgerritJoshua Harlow proposed openstack/taskflow: Automatically mutate the docstring for deprecated moves  https://review.openstack.org/15589621:24
openstackgerritJoshua Harlow proposed openstack/taskflow: Automatically mutate the docstring for deprecated moves  https://review.openstack.org/15589621:24
*** e0ne has quit IRC21:26
*** jecarey has quit IRC21:35
*** e0ne has joined #openstack-oslo21:37
*** david-lyle has quit IRC21:42
*** david-lyle has joined #openstack-oslo21:42
*** ajo_ has joined #openstack-oslo21:43
*** leakypipes has quit IRC21:45
*** stevemar has quit IRC21:45
*** ajo has quit IRC21:46
*** david-lyle has quit IRC21:47
*** e0ne has quit IRC21:54
*** mriedem1 has quit IRC22:02
*** mriedem has joined #openstack-oslo22:02
* dhellmann catches up on all the scrollback after taking an unexpected personal day22:17
dhellmannzzzeek, greghaynes: yes, we'll do releases of a bunch of libs, including oslo.db, monday and tuesday22:19
dhellmannmorganfainberg: unless someone did it when I wasn't looking, oslo.policy wasn't released at all yet so we should consider changing behavior if that's the right thing -- we need some artifact to document the reason for the change (bug, spec, etc.)22:20
dhellmannmorganfainberg: although I see you had a longer convo with bnemec and his points seem valid, so maybe not22:21
*** achanda has quit IRC22:21
beekneemechdhellmann: Yeah, I think the real problem there is that we were logging 37k times in a single devstack run.22:22
beekneemechI'm hoping if we just fix that it will be good enough.22:22
*** achanda has joined #openstack-oslo22:23
*** harlowja_ is now known as harlowja_away22:24
dhellmannbeekneemech: yeah, so what's the fix? not to log? keep track of whether we've logged?22:25
dhellmannand if the latter, can we do that using the warnings module instead of reinventing that tracking ourselves22:25
beekneemechdhellmann: Yeah, I think that.  The only problem I've seen with warnings is that they don't go to the logs by default.22:26
dhellmannwe can fix that in oslo.log22:26
beekneemechYeah, works for me then.22:28
*** harlowja_away is now known as harlowja_22:28
dhellmannbeekneemech: did you find what you needed about openstackclient?22:33
beekneemechdhellmann: Yeah, I think so.  At least enough to convince me I was right to insist that we use it. :-)22:34
dhellmannbeekneemech: cool. dtroyer is the main person driving that, although it's why we have cliff, too22:34
beekneemechYep, he was the one I ended up talking to.22:35
greghaynesdhellmann: awesome, ty22:35
*** gordc has quit IRC22:35
dhellmanngreghaynes: we were blocked on the capping stuff for stable/juno, but now that it's done we're just waiting for it to be the beginning of a week22:35
greghaynesah, yep, that was fun22:36
*** jaosorior has quit IRC22:41
lifelessharlowja_: I am :)22:49
harlowja_lifeless cools22:50
harlowja_lifeless do u know if http://bugs.python.org/issue17911 will end up with an object that is pickleable and retains the traceback to? (thats something that u can't do afaik)22:53
harlowja_and then can be unpickled and super stuff like that22:53
lifelessyes22:56
harlowja_woot22:56
lifelessthey are POPOs22:56
harlowja_thx u!22:56
lifelessstringified local variables22:57
lifelessstringified everything22:57
harlowja_cools22:57
harlowja_then i can get rid of http://docs.openstack.org/developer/taskflow/types.html#module-taskflow.types.failure i think22:57
lifelesswe're just debating the constructor API now AFAICT22:57
harlowja_which is sorta similar afaik22:57
*** jecarey has joined #openstack-oslo22:58
harlowja_which is guess is slightly similar to twisted version (although afaik twisted did way more, haha)23:03
harlowja_aka, even making fake frame objects...23:03
harlowja_crazy, mad things, ha23:03
*** miqui is now known as miqui_away23:03
harlowja_http://twistedmatrix.com/trac/browser/trunk/twisted/python/failure.py#L115 momma, lol23:04
harlowja_* woah momma23:04
*** mriedem1 has joined #openstack-oslo23:07
*** daniel3_ has quit IRC23:07
*** mriedem has quit IRC23:09
*** shardy has quit IRC23:26
*** mriedem1 has quit IRC23:27
*** zigo has quit IRC23:27
*** zigo has joined #openstack-oslo23:28
*** mriedem has joined #openstack-oslo23:29
*** sigmavirus24 is now known as sigmavirus24_awa23:30
openstackgerritJoshua Harlow proposed openstack/taskflow: Automatically mutate the docstring for deprecations  https://review.openstack.org/15589623:32
*** mriedem has quit IRC23:34
*** mriedem has joined #openstack-oslo23:35
*** mriedem has quit IRC23:39
*** mriedem has joined #openstack-oslo23:41
openstackgerritJoshua Harlow proposed openstack/taskflow: Automatically mutate the docstring for deprecations  https://review.openstack.org/15589623:42
*** MasterPiece has quit IRC23:42
*** superdan has quit IRC23:44
*** dansmith has joined #openstack-oslo23:45
openstackgerritManish Godara proposed openstack/taskflow: Fix log statement  https://review.openstack.org/15592423:49
openstackgerritManish Godara proposed openstack/taskflow: Update Flow::__str__  https://review.openstack.org/15592623:54
*** dimsum__ has quit IRC23:57

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