Tuesday, 2014-09-16

*** bknudson has joined #openstack-oslo00:16
*** rpodolyaka1 has quit IRC00:23
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Get the basics of a process executor working  https://review.openstack.org/12128000:26
*** tsekiyama has joined #openstack-oslo00:35
*** takedakn has joined #openstack-oslo00:38
*** tsekiyama has quit IRC00:39
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Get the basics of a process executor working  https://review.openstack.org/12128000:45
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Get the basics of a process executor working  https://review.openstack.org/12128000:49
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Get the basics of a process executor working  https://review.openstack.org/12128000:50
*** zzzeek has joined #openstack-oslo01:01
*** sigmavirus24_awa is now known as sigmavirus2401:02
*** harlowja_at_home has joined #openstack-oslo01:08
*** zzzeek has quit IRC01:10
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Use the engine entrypoint function instead of direct class  https://review.openstack.org/12172501:22
*** harlowja_at_home has quit IRC01:57
*** yamahata has quit IRC02:01
*** yamahata has joined #openstack-oslo02:01
*** praneshp has quit IRC02:06
*** dims has quit IRC02:29
*** dims has joined #openstack-oslo02:29
*** dims has quit IRC02:30
*** dims has joined #openstack-oslo02:30
*** dims has quit IRC02:30
*** dims has joined #openstack-oslo02:31
*** dims has quit IRC02:35
*** praneshp has joined #openstack-oslo02:48
*** praneshp_ has joined #openstack-oslo02:49
*** praneshp has quit IRC02:52
*** praneshp_ is now known as praneshp02:52
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Ensure that failure objects a pickleable  https://review.openstack.org/12173502:53
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Ensure that failure objects are pickleable  https://review.openstack.org/12173502:54
*** amotoki has joined #openstack-oslo03:03
*** amotoki has quit IRC03:03
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Ensure that failure objects are pickleable  https://review.openstack.org/12173503:05
*** AAzza has quit IRC03:08
*** AAzza_afk has joined #openstack-oslo03:09
*** AAzza_afk is now known as AAzza03:09
*** AAzza has quit IRC03:13
*** AAzza_afk has joined #openstack-oslo03:18
*** AAzza_afk is now known as AAzza03:18
*** sigmavirus24 is now known as sigmavirus24_awa03:22
*** jeckersb is now known as jeckersb_gone03:24
*** AAzza has quit IRC03:25
*** AAzza_afk has joined #openstack-oslo03:26
*** takedakn has quit IRC03:26
*** AAzza_afk is now known as AAzza03:26
*** harlowja_at_home has joined #openstack-oslo03:26
*** noelbk has quit IRC03:39
openstackgerritYAMAMOTO Takashi proposed a change to openstack/oslo.messaging: Sync jsonutils from oslo-incubator  https://review.openstack.org/12174303:42
*** 20WABJQK7 has quit IRC03:47
*** arnaud has quit IRC03:47
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Add a futures type that can unify our future functionality  https://review.openstack.org/11648903:58
*** oomichi has joined #openstack-oslo04:16
*** arnaud has joined #openstack-oslo04:42
*** vigneshvar has joined #openstack-oslo04:49
*** harlowja_at_home has quit IRC04:50
*** arnaud has quit IRC04:52
*** harlowja_at_home has joined #openstack-oslo04:58
*** harlowja_ is now known as harlowja_away04:59
*** rpodolyaka1 has joined #openstack-oslo04:59
*** stevemar has quit IRC05:03
openstackgerritmelanie witt proposed a change to openstack-dev/pbr: add the ability to get version from METADATA file  https://review.openstack.org/12175105:05
*** takedakn has joined #openstack-oslo05:09
*** zhiyan_ is now known as zhiyan05:10
*** rpodolyaka1 has quit IRC05:22
*** rpodolyaka1 has joined #openstack-oslo05:23
*** rpodolyaka1 has quit IRC05:24
*** yamahata has quit IRC05:27
*** yamahata has joined #openstack-oslo05:28
*** cfriesen_ has quit IRC05:30
*** harlowja_at_home has quit IRC05:45
*** vigneshvar has quit IRC05:57
*** vigneshvar has joined #openstack-oslo06:00
*** gpanda has joined #openstack-oslo06:12
*** takedakn has quit IRC06:17
*** flaper87|afk is now known as flaper8706:17
*** e0ne has joined #openstack-oslo06:29
*** e0ne has quit IRC06:37
*** e0ne has joined #openstack-oslo06:41
*** e0ne has quit IRC06:43
*** i159 has joined #openstack-oslo07:09
*** r1chardj0n3s is now known as r1chardj0n3s_afk07:20
*** bogdando has quit IRC07:26
*** ihrachyshka has joined #openstack-oslo07:53
openstackgerritA change was merged to openstack-dev/pbr: Refactor oneline changelog walking.  https://review.openstack.org/10827107:58
*** e0ne has joined #openstack-oslo08:08
*** e0ne has quit IRC08:09
*** andreykurilin_ has joined #openstack-oslo08:14
*** andreykurilin_ has quit IRC08:35
*** flaper87 is now known as flaper87|afk08:37
*** flaper87|afk is now known as flaper8708:37
*** noelbk has joined #openstack-oslo08:42
*** viktors|afk is now known as viktors08:44
ttxdhellmann: oslo.rootwrap 1.3.0.0a3 (hopefully last alpha before 1.3.0) just tagged08:52
*** flaper87 is now known as flaper87|afk08:56
*** yassine has joined #openstack-oslo08:57
*** e0ne has joined #openstack-oslo09:00
*** amotoki has joined #openstack-oslo09:04
*** flaper87|afk is now known as flaper8709:05
openstackgerritJames Page proposed a change to openstack/oslo.messaging: Enable heartbeating on rabbitmq connections  https://review.openstack.org/9465609:05
*** noelbk has quit IRC09:14
*** flaper87 is now known as flaper87|afk09:57
*** e0ne has quit IRC10:01
*** pcm_ has joined #openstack-oslo10:01
openstackgerritmouad benchchaoui proposed a change to openstack/oslo-incubator: Fix using multiple instances of ProcessLauncher  https://review.openstack.org/12179810:01
*** pcm_ has quit IRC10:01
*** pcm_ has joined #openstack-oslo10:02
*** e0ne has joined #openstack-oslo10:04
*** praneshp has quit IRC10:08
*** gpanda has quit IRC10:11
*** bjornar_ has joined #openstack-oslo10:45
*** i159_ has joined #openstack-oslo10:48
*** i159 has quit IRC10:52
*** ildikov has joined #openstack-oslo10:52
*** flaper87|afk is now known as flaper8710:53
*** dims has joined #openstack-oslo10:56
*** pblaho has joined #openstack-oslo11:00
*** jamespage has joined #openstack-oslo11:13
*** tedross_ has joined #openstack-oslo11:29
*** flaper87 is now known as flaper87|afk11:38
*** bjornar_ has quit IRC11:42
*** kgiusti has joined #openstack-oslo11:43
*** flaper87 has joined #openstack-oslo11:49
*** stannie1 has joined #openstack-oslo12:01
*** stannie1 is now known as pierrepad12:02
*** pblaho has quit IRC12:03
*** flaper87 has quit IRC12:06
openstackgerritA change was merged to openstack/oslo.utils: Enable mask_password to handle byte code strings  https://review.openstack.org/12165712:07
*** pblaho has joined #openstack-oslo12:08
openstackgerritA change was merged to openstack-dev/pbr: Stop using intersphinx  https://review.openstack.org/12132912:08
viktorszhiyan: ping12:12
viktorszhiyan: sorry, wrong nickname :)12:12
*** _amrith_ is now known as amrith12:14
*** amotoki has quit IRC12:17
*** andreykurilin has quit IRC12:17
*** dims has quit IRC12:18
*** dims has joined #openstack-oslo12:19
*** tedross_ is now known as tedross12:21
*** andreykurilin has joined #openstack-oslo12:22
*** ihrachyshka_ has joined #openstack-oslo12:22
*** pblaho is now known as pblaho|afk12:22
*** pblaho|afk is now known as pblaho12:24
*** pblaho has quit IRC12:24
*** ihrachyshka has quit IRC12:25
viktorsjd__: hi! FYI - the new oslo.db was released12:28
jd__viktors: awesome, thanks12:28
*** pblaho has joined #openstack-oslo12:31
dimsviktors: w00t12:35
viktorsdims: yean, and I really hope, that it will not break anything :)12:36
dimscross our collective fingers and toes :)12:37
zhiyanviktors: ?12:38
*** zz_jgrimm is now known as jgrimm12:41
*** sigmavirus24_awa is now known as sigmavirus2412:44
*** mriedem has joined #openstack-oslo12:46
*** mriedem has quit IRC12:48
*** rpodolyaka1 has joined #openstack-oslo12:51
*** amrith is now known as _amrith_12:53
*** mriedem has joined #openstack-oslo12:56
*** jeckersb_gone is now known as jeckersb13:00
*** tedross has quit IRC13:02
*** pblaho has quit IRC13:06
*** bknudson has quit IRC13:16
*** tedross has joined #openstack-oslo13:18
*** pblaho has joined #openstack-oslo13:18
*** gordc has joined #openstack-oslo13:21
*** pblaho has quit IRC13:23
*** pblaho has joined #openstack-oslo13:24
openstackgerritTim Kelsey proposed a change to openstack/oslo.messaging: Adds message security plugins  https://review.openstack.org/10980613:25
*** jecarey has quit IRC13:34
*** e0ne has quit IRC13:36
*** miqui has joined #openstack-oslo13:42
*** e0ne has joined #openstack-oslo13:42
*** jecarey has joined #openstack-oslo13:47
*** oomichi has quit IRC13:59
*** tedross has quit IRC14:03
*** abhijeetm has joined #openstack-oslo14:04
*** pblaho_ has joined #openstack-oslo14:07
*** pblaho_ has quit IRC14:08
*** sigmavirus24 is now known as sigmavirus24_awa14:10
*** pblaho has quit IRC14:11
*** sigmavirus24_awa is now known as sigmavirus2414:12
*** abhijeetm has quit IRC14:12
*** pblaho has joined #openstack-oslo14:15
*** pblaho has quit IRC14:18
*** tedross has joined #openstack-oslo14:19
*** pblaho has joined #openstack-oslo14:20
*** pblaho has quit IRC14:20
*** pblaho has joined #openstack-oslo14:20
*** pblaho has quit IRC14:21
*** cfriesen_ has joined #openstack-oslo14:26
*** YorikSar_ is now known as YorikSar14:29
openstackgerritA change was merged to openstack/oslo.messaging: Ensure the amqp options are present in config file  https://review.openstack.org/11962714:32
*** jgrimm is now known as zz_jgrimm14:33
dhellmannkgiusti, flaper87|afk: do we need a release of oslo.messaging with ^^ ? We should cut an alpha today, unless that can wait for kilo.14:33
dhellmannsileht: ^^14:33
*** andreykurilin has quit IRC14:36
*** jeckersb is now known as jeckersb_gone14:36
silehtdhellmann, it's just doc for the AMQP1.0 experimental driver14:37
dhellmannsileht: well, it does move the options to a different group, doesn't it?14:37
dhellmannsileht: we should probably go ahead and release that to avoid having to deal with deprecation later14:39
silehtdhellmann, this one is only for AMQP1.0, the one for other drivers is not yet merged14:39
dhellmannsileht: the amqp 1.0 driver was in an earlier alpha, so it has already been 'released'14:39
dhellmannit's experimental, but we should still be careful about config changes14:40
dhellmannsileht: you handled the deprecation properly for the other drivers, so it's just the new one14:42
silehtdhellmann, the group have changed from ampq1 to oslo_messaging_amq, so we need to release it :p14:42
dhellmannsileht: yeah, I'm trying to decide if we need a patch to add the deprecated group settings. I have no idea how many people might be using that driver right now. Possibly only kgiusti ? :-)14:43
*** myatsenko has quit IRC14:43
silehtdhellmann, I guess too14:43
*** jeckersb_gone is now known as jeckersb14:44
dhellmannsileht: how much work would it be to add the deprecation settings just for the amqp1 driver?14:44
*** pblaho has joined #openstack-oslo14:45
silehtdhellmann, it just be easy14:46
dhellmannsileht: OK. Based on some of the other changes to the library, I was not planning to release a new version for juno. How about it you add the deprecation settings, and we roll that into the first kilo release?14:46
dhellmann*How about if14:47
silehtdhellmann, yes14:48
dhellmannsileht: ok, sounds good14:48
openstackgerritMehdi Abaakouk proposed a change to openstack/oslo.messaging: Add missing deprecated group amqp1  https://review.openstack.org/12188614:51
*** jeckersb is now known as jeckersb_gone14:54
*** jeckersb_gone is now known as jeckersb14:54
*** ihrachyshka_ has quit IRC14:57
*** ildikov_ has joined #openstack-oslo14:59
*** tedross has quit IRC14:59
*** ildikov has quit IRC15:01
*** viktors is now known as viktors|afk15:10
*** tsekiyama has joined #openstack-oslo15:12
openstackgerritJulien Danjou proposed a change to openstack/oslo.messaging: Switch to oslo.middleware  https://review.openstack.org/12189915:13
*** jeckersb is now known as jeckersb_gone15:13
*** jeckersb_gone is now known as jeckersb15:14
*** tedross has joined #openstack-oslo15:14
openstackgerritSergey Shnaidman proposed a change to openstack/oslo-incubator: Fix custom virtualenv directory for tempest  https://review.openstack.org/12190115:16
*** sigmavirus24 is now known as sigmavirus24_awa15:17
*** sigmavirus24_awa is now known as sigmavirus2415:19
*** zz_jgrimm is now known as jgrimm15:22
*** rpodolyaka1 has quit IRC15:25
*** rpodolyaka1 has joined #openstack-oslo15:29
openstackgerritMehdi Abaakouk proposed a change to openstack/oslo.messaging: Move each drivers options into its own group  https://review.openstack.org/12003815:30
*** flaper87|afk is now known as flaper8715:31
flaper87dhellmann: that can wait, IMHO15:32
*** noelbk has joined #openstack-oslo15:35
*** jeckersb is now known as jeckersb_gone15:35
*** jeckersb_gone is now known as jeckersb15:35
*** rpodolyaka1 has quit IRC15:37
*** viktors|afk is now known as viktors15:39
kgiustidhellmann: sorry, afk - yeah, I think I'm the only one futzing with that right now. Changing shouldn't be an issue either way IMHO.15:41
*** tedross has quit IRC15:42
dhellmannflaper87, kgiusti: ok, sounds good. we'll leave that for an early kilo alpha release15:42
*** zzzeek has joined #openstack-oslo15:43
kgiustiflaper87: btw - we've got amqp 1.0 support in devstack now :D15:43
flaper87kgiusti: yup, I noticed :D15:44
*** jeckersb is now known as jeckersb_gone15:44
flaper87kgiusti: great work there, thanks for following up15:44
*** jeckersb_gone is now known as jeckersb15:44
kgiustiAre there any existing qpidd-based gate tests for oslo.messaging?  Something I can clone + swap the backend for 1.0?15:44
dhellmannkgiusti: there *might* be some tests in oslo-incubator, but I don't think you'd be able to just swap them out15:46
*** vigneshvar has quit IRC15:47
dhellmannmigrating the tests for the drivers other than rabbit is on the todo list15:47
*** e0ne has quit IRC15:48
*** ildikov_ has quit IRC15:49
*** i159_ has quit IRC15:50
kgiustiI might be able to help with that - at least for the qpid side (both impl_qpid and amqp1.0).15:51
*** andreykurilin has joined #openstack-oslo15:56
*** tedross has joined #openstack-oslo15:57
*** viktors is now known as viktors|afk15:59
*** bknudson has joined #openstack-oslo16:00
*** kgiusti is now known as kgiusti|afk16:03
*** takedakn has joined #openstack-oslo16:10
openstackgerritJulien Danjou proposed a change to openstack/oslo-incubator: network_utils: make tests portable  https://review.openstack.org/11029316:17
*** jeckersb is now known as jeckersb_gone16:18
*** jeckersb_gone is now known as jeckersb16:22
*** palendae has quit IRC16:23
*** palendae has joined #openstack-oslo16:24
*** gordc has quit IRC16:24
*** palendae has quit IRC16:25
openstackgerritJulien Danjou proposed a change to openstack/oslo.concurrency: Remove oslo-incubator fixture  https://review.openstack.org/12192016:25
*** palendae has joined #openstack-oslo16:25
openstackgerritJulien Danjou proposed a change to openstack/oslo.concurrency: Remove oslo-incubator fixture  https://review.openstack.org/12192016:27
openstackgerritA change was merged to openstack-dev/pbr: Only consider tags that look like versions.  https://review.openstack.org/11440316:29
*** palendae has quit IRC16:30
openstackgerritJulien Danjou proposed a change to openstack/oslo.middleware: Remove oslo-incubator fixture  https://review.openstack.org/12192116:30
*** gordc has joined #openstack-oslo16:30
*** palendae has joined #openstack-oslo16:30
*** rpodolyaka1 has joined #openstack-oslo16:35
openstackgerritJulien Danjou proposed a change to openstack/oslo.messaging: Stop using importutils from oslo-incubator  https://review.openstack.org/12192416:40
openstackgerritJulien Danjou proposed a change to openstack/oslo.messaging: Switch to oslo.serialization  https://review.openstack.org/12192516:40
*** rpodolyaka1 has quit IRC16:41
*** jgrimm is now known as zz_jgrimm16:42
*** rpodolyaka1 has joined #openstack-oslo16:47
*** pierrepad has quit IRC16:48
*** ihrachyshka has joined #openstack-oslo16:53
*** sigmavirus24 is now known as sigmavirus24_awa16:59
*** kgiusti|afk is now known as kgiusti16:59
*** harlowja_away is now known as harlowja_17:04
*** vigneshvar has joined #openstack-oslo17:07
*** flaper87 is now known as flaper87|afk17:09
*** rpodolyaka1 has quit IRC17:12
*** tongli has joined #openstack-oslo17:14
*** bjornar_ has joined #openstack-oslo17:19
*** jeckersb is now known as jeckersb_gone17:19
*** jeckersb_gone is now known as jeckersb17:21
*** rpodolyaka1 has joined #openstack-oslo17:21
*** kevinbenton has quit IRC17:27
*** e0ne has joined #openstack-oslo17:32
*** kevinbenton has joined #openstack-oslo17:32
*** pblaho has quit IRC17:34
*** pblaho has joined #openstack-oslo17:35
*** vigneshvar has quit IRC17:36
*** rpodolyaka1 has quit IRC17:39
*** rpodolyaka1 has joined #openstack-oslo17:43
*** rpodolyaka1 has quit IRC17:44
*** zz_jgrimm is now known as jgrimm17:51
*** andreykurilin_ has joined #openstack-oslo17:54
*** ihrachyshka has quit IRC17:55
*** rpodolyaka1 has joined #openstack-oslo17:56
*** vigneshvar has joined #openstack-oslo17:56
*** andreykurilin has quit IRC17:56
*** yassine has quit IRC17:58
*** ihrachyshka has joined #openstack-oslo18:02
*** sigmavirus24_awa is now known as sigmavirus2418:07
*** palendae has quit IRC18:08
*** palendae has joined #openstack-oslo18:08
*** ihrachyshka has quit IRC18:12
*** yamahata has quit IRC18:20
*** ihrachyshka has joined #openstack-oslo18:20
*** flaper87|afk is now known as flaper8718:25
*** pblaho has quit IRC18:28
*** rpodolyaka1 has quit IRC18:30
*** rpodolyaka1 has joined #openstack-oslo18:31
*** rpodolyaka1 has quit IRC18:31
*** ihrachyshka has quit IRC18:32
*** e0ne has quit IRC18:38
*** ajo_ has joined #openstack-oslo18:48
*** vigneshvar has quit IRC18:53
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.vmware: Switch to use requests/urllib3  https://review.openstack.org/12195618:58
*** vigneshvar has joined #openstack-oslo19:06
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Get the basics of a process executor working  https://review.openstack.org/12128019:17
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Ensure that failure objects are pickleable  https://review.openstack.org/12173519:18
openstackgerritBrian Elliott proposed a change to openstack/oslo-incubator: Cleanup and adding timing to lockutils logging  https://review.openstack.org/12195919:21
*** harlowja_ has quit IRC19:22
*** harlowja has joined #openstack-oslo19:22
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Get the basics of a process executor working  https://review.openstack.org/12128019:29
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Ensure that failure objects are pickleable  https://review.openstack.org/12173519:29
*** ajo_ has quit IRC19:31
*** praneshp has joined #openstack-oslo19:32
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Get the basics of a process executor working  https://review.openstack.org/12128019:33
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Ensure that failure objects are pickleable  https://review.openstack.org/12173519:33
openstackgerritJoshua Harlow proposed a change to openstack/oslo.db: Start adding a environment for py34/py33  https://review.openstack.org/11996819:37
openstackgerritBrian Elliott proposed a change to openstack/oslo-incubator: Cleanup and adding timing to lockutils logging  https://review.openstack.org/12195919:39
openstackgerritJoshua Harlow proposed a change to openstack/oslo.db: Explicitly depend on six in requirements file  https://review.openstack.org/12196319:43
*** e0ne has joined #openstack-oslo19:46
openstackgerritJoshua Harlow proposed a change to openstack/oslo.db: Use the six provided iterator mix-in  https://review.openstack.org/12196619:51
*** photocyte has joined #openstack-oslo19:52
openstackgerritJoshua Harlow proposed a change to openstack/oslo.db: Use six.wraps instead of functools.wraps  https://review.openstack.org/12196819:58
*** e0ne has quit IRC20:00
openstackgerritJoshua Harlow proposed a change to openstack/oslo.db: Use __qualname__ if we can  https://review.openstack.org/12196920:03
*** e0ne has joined #openstack-oslo20:04
*** kgiusti has quit IRC20:07
*** e0ne has quit IRC20:18
*** rpodolyaka1 has joined #openstack-oslo20:34
openstackgerritBrian Elliott proposed a change to openstack/oslo-incubator: Cleanup and adding timing to lockutils logging  https://review.openstack.org/12195920:35
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.vmware: Switch to use requests/urllib3  https://review.openstack.org/12195620:36
*** vigneshvar has quit IRC20:45
harlowjadhellmann has the pylockfile repo been moved yet?20:51
dhellmannharlowja: moved? into gerrit? yes20:52
harlowjaah, got it , https://github.com/openstack/pylockfile20:52
*** photocyte has quit IRC20:53
*** tedross has quit IRC20:55
*** andreykurilin_ has quit IRC20:57
*** e0ne has joined #openstack-oslo20:58
*** r1chardj0n3s_afk is now known as r1chardj0n3s21:07
*** jeckersb is now known as jeckersb_gone21:21
*** miqui has quit IRC21:23
openstackgerritJoshua Harlow proposed a change to openstack/oslo.concurrency: Break up the logging around the lockfile release/unlock  https://review.openstack.org/12199021:24
*** flaper87 is now known as flaper87|afk21:29
*** melwitt has joined #openstack-oslo21:33
melwittdhellmann: o/21:33
dhellmannmelwitt: hi21:34
dhellmannno, I didn't mean pkg_resources, though that's a good runtime tool for doing this21:34
dhellmannpresumably setuptools has logic for reading the metadata files (or it uses distutils or something)21:34
melwittokay. I'm new to python packaging21:34
dhellmannand I wonder if we can call that directly21:34
dhellmannnot a problem, I'm just thinking out loud21:35
melwittohh, I see. yeah that would be much much better21:35
melwittbtw using 'pip list' I get setuptools (0.6c12dev-r88846)21:35
dhellmannmelwitt: otoh, I'll bet that isn't a stable API21:35
melwittdhellmann: yeah, without thinking about it I mimicked what was being done with the PKG-INFO file. I think I saw lifeless commented in that code, maybe there's a reason to do it the way it is21:37
melwitti.e. maybe I can ask him21:37
dhellmannmelwitt: good idea, though I don't know if he's online at this time of day21:37
melwittnod21:38
*** morganfainberg is now known as CaptainMorgan21:39
*** CaptainMorgan is now known as morganfainberg21:39
lifelessmelwitt: hi, whats up?21:42
lifelessdhellmann: its 0942 here, I am indeed around21:42
dhellmannlifeless: I can't to timezones in my head :-)21:43
dhellmannlifeless: looking at https://review.openstack.org/#/c/121751/1/pbr/packaging.py21:44
dhellmannI was wonder if there was any code in setuptools we could use to get the metadata, instead of looking for the file ourselves. I'm not finding anything, but I'm not that familiar with the internals21:44
melwittlifeless: hi! dhellman and I were talking about pbr reading package versions, basically right now we have an issue where there's a package metadata file called METADATA instead of the PKG-INFO file, so when there isn't PKG-INFO and there isn't git, the user gets an error when running commands bug 1369179. and we were wondering is that the right way to read the metadata21:44
melwittI thought I saw your comments in that code so I thought to ask you. I put up a patch that simply tries to look for METADATA in addition to PKG-INFO. dhellman suggested and it'd be ideal if there's a way to query it through setuptools21:47
*** zzzeek has quit IRC21:47
lifelessmelwitt: interesting. I disagree with the long term guidance btw21:49
dhellmann?21:49
lifelessmelwitt: since I've just done two pycon talks in a row where I encourage using pbr.version.VersionInfo('package').semantic_version to abstract that all away21:50
lifelessdhellmann: last para of the commit message in melwitt's proposal21:50
openstackgerritJon Grimm proposed a change to openstack/oslo.config: Warn if new and deprecated options are both present  https://review.openstack.org/12169121:50
dhellmannlifeless: I see way way too many bugs from clients trying to get pbr installed at runtime just to read a version number out of a file that pkg_resources can read for them.21:50
*** pcm_ has quit IRC21:51
lifelessdhellmann: what makes pbr fail to install at runtime?21:52
melwittlifeless: okay. I'm new to python packaging and put the few minute consensus from the infra channel in the bug report (on the pkg_resources thing)21:53
dhellmannthat's a good question, I don't have an answer to that21:53
lifelessdhellmann: so I don't want to run around lots of projects copying and pasting semanticversion code21:53
melwittlifeless: pbr installed fine, it's just when you invoke the client, it can't read the version through the pbr call because pbr couldn't find PKG-INFO *and* git wasn't installed or no git repo could be found21:53
dhellmannlifeless: if something just needs to get the version number of itself after it is installed, why does that require the semver stuff?21:54
melwittin the __init__ of the client it calls pbr for the version21:54
*** e0ne has quit IRC21:54
lifelessdhellmann: the version_info tuple translation for one21:54
melwittso pbr can't get a version and it throws the error "Exception("Versioning for this project requires either an sdist"21:54
melwittException: Versioning for this project requires either an sdist tarball, or access to an upstream git repository. Are you sure that git is installed?"21:54
dhellmannmelwitt: I'm talking about a different issue that led me to recommend using pkg_resources21:54
melwittdhellmann: oh sorry21:54
*** bjornar_ has quit IRC21:54
lifelessdhellmann: secondly to handle getting the version when there is no PKG-INFO or METADATA - when running in develop mode21:54
*** jordandh has joined #openstack-oslo21:54
dhellmannlifeless: I thought develop mode created the metadata files, doesn't it?21:55
lifelessdhellmann: characteristing 'what version am I' as 'just use pkg_resources' skips over both of those things, and neither is trivial21:55
lifelessdhellmann: it creates them once, and they are instantly wrong as soon as you do a commit or whatever21:55
dhellmannok, let's talk about that some other time, then21:56
lifelessdhellmann: so you'd immediately have a pkg_resources based approach giving different answers to what pbr sdist will do21:56
lifelessand then we'll have *those* bugs to deal with21:56
dhellmannfor now, what about melwitt's patch? is there a way to ask setuptools to read those files so we don't have to keep up with their changes, or is that not a public api?21:56
lifelessuntil they run egg_info21:56
*** bknudson has quit IRC21:56
lifelessand then the 'bug' will go away21:56
lifelessand folk will want to stab pbr for doing the wrong thing21:56
lifelessdhellmann: reviewing it atm21:57
lifelessso metadata 2.0 is in pydist.json files21:59
lifelessso this isn't metadata 2.022:00
melwittis the "METADATA" file not related to 2.0?22:00
lifelessmelwitt: no22:01
lifelessmelwitt: at least not according to http://legacy.python.org/dev/peps/pep-0426/22:01
melwittlifeless: oh. :( I tried to find what caused the changed from PKG-INFO to METADATA, it seemed related to the metadata version i.e. all the PKG-INFO I opened said Metadata 1.1 etc at the top and the METADATA I opened said Metadata 2.022:02
lifelessmelwitt: I can't find a reference for a file called METADATA22:02
openstackgerritJulien Danjou proposed a change to openstack/oslo.messaging: Switch to oslo.middleware  https://review.openstack.org/12189922:03
melwittlifeless: I couldn't either. I web searched and read docs on python packaging, metadata and such and couldn't find real docs22:03
lifelessI've pinged dstufft22:03
*** jecarey has quit IRC22:03
*** dstufft has joined #openstack-oslo22:03
* dstufft arrives22:04
lifelessPKG-INFO is defined in http://legacy.python.org/dev/peps/pep-0241/22:04
lifelessdstufft: hi22:04
lifelessdstufft: context22:04
lifelesshttps://review.openstack.org/#/c/121751/22:04
lifelessand22:04
lifelesshttps://bugs.launchpad.net/pbr/+bug/136917922:04
*** rhsu has joined #openstack-oslo22:04
*** rhsu has left #openstack-oslo22:04
dhellmannlifeless, melwitt : it's my turn to cook tonight, so I need to fire up the grill. I'll check back in later to see how you resolve this.22:04
lifelessdstufft: we have two questions I think; one is is there an API to abstract out the location (and format?) of the egg info metadata file22:05
lifelessdstufft: secondly, wtf is 'METADATA' since we can't seem to fine a PEP for it22:05
dstuffthttp://legacy.python.org/dev/peps/pep-0427/#the-dist-info-directory22:05
lifelessahha!22:06
openstackgerritJulien Danjou proposed a change to openstack/oslo.concurrency: Remove oslo-incubator fixture  https://review.openstack.org/12192022:06
lifelesswheels22:06
lifelessof course wheels22:06
dstufftthinking about the other thing22:07
dstufftthe format is just22:07
dstufftemail.parser.Parser().parsestr()22:07
dstufftmore or les22:07
dstufftpkg_resources.get_distribution("pip").has_metadata("METADATA")22:09
dstufftpkg_resources.get_distribution("pip").get_metadata("METADATA")22:10
dstufftpkg_resources.get_distribution("pip").PKG_INFO22:10
dstufftthose things should get what you want I think22:11
lifelessI think we need to do a bit of an overhaul in this area of the code22:11
lifelessin pbr22:11
dstufftin particular, dist = pkg_resources.get_distribution("pip"); dist.get_metadata(dist.PKG_INFO)22:12
lifelessmelwitt: reviewed - I'm -1 but all simple stuff to tweak22:17
lifelessdstufft: yeah so I've just observed that we use egg_info in preference to git, and we don't use pkg_resources.get_distribution ... which is where I was pinged on in the first place22:17
harlowjai've done a hybrid approach in my stuff, maybe others should also?22:18
harlowjahttps://github.com/stackforge/doc8/blob/master/doc8/version.py#L1722:18
lifelessdstufft: I think what we need to do is to do some very careful testing about behaviours like 'lookup the version for foo when sitting in the root of a project bar22:18
melwittlifeless: thanks! I'll make the changes22:18
lifelessharlowja: that shouldn't be needed, pretty sure pbr puts itself as a dep on anything it builds22:19
lifelessharlowja: IMBW of course :)22:19
harlowjaas a runtime dependency?22:19
harlowja:-/22:19
lifelessharlowja: it is implicitly as its a runtime dep of setup for the project22:19
lifelessharlowja: wheels may avoid that, but source dist installation invokes setup.py22:20
harlowjapython sux, lol22:20
lifelessharlowja: pbr has no deps itself, deliberatly, because of thise22:20
harlowjaya22:20
*** takedakn has quit IRC22:20
lifelessI am now thinking of all the ways I might be able to break pbr. sadface.22:21
*** melwitt1 has joined #openstack-oslo22:23
harlowjadon't think such things22:23
harlowjathink happy thoughts22:23
harlowjalifeless has there ever been any talk of getting pbr logic -> setuptools itself?22:24
*** melwitt has quit IRC22:24
*** melwitt1 is now known as melwitt22:25
*** takedakn has joined #openstack-oslo22:25
harlowjai wonder how many oslo libraries need to have pbr listed back in there requirements again then22:26
harlowjaafaik we were taking it out from the requirements.txt files22:26
harlowjaand assuming its a implict dependency22:26
lifelessharlowja: I don't know. Worth raising :)22:27
harlowjahmmmm, also just checked that its not inserting itself at build time22:27
harlowjahttps://pypi.python.org/packages/source/t/taskflow/taskflow-0.4.0.tar.gz doesn't seem to have it added automagically22:27
lifelessharlowja: so try installing that in a clean venv, see if pbr is sucked in22:28
harlowjawill try22:28
harlowjanegative, no work22:28
lifelessSuccessfully installed taskflow iso8601 networkx stevedore Babel six pbr futures anyjson decorator pytz22:28
lifelessah 0.3.2122:29
harlowjahttp://paste.ubuntu.com/8360937/22:29
harlowjaya, 0.4 was part of the remove pbr funtime22:29
harlowjanone of the oslo libraries afaik have pbr listed in requirements.txt anymore22:29
*** jgrimm is now known as zz_jgrimm22:29
harlowja*maybe that should be fixe22:30
harlowja*fixed22:30
lifelessharlowja: yeah, you've triggered easy_install22:30
lifelesswhich we don't want22:30
dhellmannwhy would they need it in their requirements list if they don't import it anywhere other than setup.py?22:30
lifelesseither pbr has to inject itself, or it needs to be in requirements.txt22:30
dhellmannit's in the install_requires list22:30
lifelessdhellmann: because to run setup.py it is needed22:31
lifelessdhellmann: and install_requires is triggered via easy_install which is the spawn of the devil22:31
harlowjalifeless is that only a python2.6 issue then?22:31
dhellmannthis feels like the sort of thing we need to have in documentation somewhere22:31
lifelessharlowja: that error looks like an ssl issue then ?22:31
harlowjalifeless let me try on a different box22:31
lifelessharlowja: though it might be more easy_install fail22:32
lifelessdhellmann: I'd rather put it in code22:32
lifelessdhellmann: inject it into the appropriate metadata automatically22:32
dhellmannlifeless: that'd be fine, but the docs should explain (a) that it happens and (b) why22:32
lifelessdhellmann: agreed22:32
dhellmannI just pip installed taskflow in a virtualenv without pbr installed and pbr is not installed afterwards22:33
harlowjalifeless it looks like pbr downloads itself22:33
harlowjain a different machiens venv22:33
* dhellmann goes back to tending the grill22:33
harlowja  Running setup.py egg_info for package taskflow22:33
harlowja    Installed /home/harlowja/.venv/build/taskflow/pbr-0.10.0-py2.6.egg22:33
harlowja    [pbr] Processing SOURCES.txt22:33
harlowjaah, one sec, outside machine (not in venv) had pbr installed22:34
harlowjaya, anyway, seems to suck itself down, interesting22:35
lifelessdhellmann: yes, but easy_install does this thing where it drops the file locally outside of the dist-packages tree for running shit22:35
lifelessdhellmann: which is evil22:35
lifelessdhellmann: so I am just questioning whether looking in freeze afterwards is sufficent ;)22:36
lifelessdhellmann: anyhow, I want to be simple about this - if pbr is our abstraction for versioning and packaging, it would be a lot easier to reason about if we just require it22:36
lifelessdhellmann: its 75kB, not tiny but not huge22:37
harlowjaya, explict seems better than implict (which is what seems to happen when u don't explicitly list it)22:37
harlowjaalthough i'll let u guys decide :-P22:38
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Use timeutils functions instead of misc.wallclock  https://review.openstack.org/12201222:38
openstackgerritmelanie witt proposed a change to openstack-dev/pbr: Support reading versions from wheels  https://review.openstack.org/12175122:40
lifelessmelwitt: +2 from me22:46
lifelessdhellmann: ^ melwitt's updated it if you want to +2, I can +A when it gets through CI22:47
melwittlifeless: thanks :)22:48
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Use timeutils functions instead of misc.wallclock  https://review.openstack.org/12201222:49
*** AAzza_afk has joined #openstack-oslo22:55
*** AAzzal has joined #openstack-oslo22:56
dstufftwait22:56
*** AAzza has quit IRC22:56
*** AAzzal is now known as AAzza22:56
dstufftif you just need it for setup.py why would you put it in install_requires instead of setup_requires22:57
dstufftI mean yea setup_requires is the worst thing ever, but I'm trying to fix that22:57
*** AAzza_afk has quit IRC22:59
dhellmannlifeless, melwitt: +223:01
dhellmanndstufft: sorry, I meant setup_requires23:02
*** dims has quit IRC23:06
*** dims has joined #openstack-oslo23:06
*** mriedem has quit IRC23:08
*** noelbk has quit IRC23:09
*** r1chardj0n3s is now known as r1chardj0n3s_afk23:09
*** tongli has quit IRC23:09
*** dims has quit IRC23:11
*** rpodolyaka1 has quit IRC23:21
*** sigmavirus24 is now known as sigmavirus24_awa23:22
*** rpodolyaka1 has joined #openstack-oslo23:24
*** rpodolyaka1 has quit IRC23:26
*** gordc has quit IRC23:26
*** tsekiyama has quit IRC23:31
*** r1chardj0n3s_afk is now known as r1chardj0n3s23:32
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Use timeutils functions instead of misc.wallclock  https://review.openstack.org/12201223:37
*** praneshp has quit IRC23:37
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Use timeutils functions instead of misc.wallclock  https://review.openstack.org/12201223:41
*** Guest72739 is now known as mgagne23:42
*** mgagne has joined #openstack-oslo23:42
*** ekarlso has quit IRC23:48
*** ekarlso has joined #openstack-oslo23:48
*** yamahata has joined #openstack-oslo23:56

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