Wednesday, 2016-05-04

openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Redesign router proxy  https://review.openstack.org/30865300:19
*** ozamiatin has quit IRC00:22
*** mtanino has quit IRC00:28
openstackgerritDmitriy Ukhlov proposed openstack/oslo.messaging: Implements configurable connection factory  https://review.openstack.org/31066400:30
*** yamahata has joined #openstack-oslo00:39
*** zz_dimtruck is now known as dimtruck01:10
*** dukhlov has quit IRC01:13
*** dukhlov has joined #openstack-oslo01:15
*** yamamoto has quit IRC01:21
*** EinstCrazy has joined #openstack-oslo01:24
*** EinstCrazy has quit IRC01:28
*** EinstCrazy has joined #openstack-oslo01:29
*** EinstCrazy has quit IRC01:29
*** EinstCrazy has joined #openstack-oslo01:30
*** EinstCra_ has joined #openstack-oslo01:32
*** EinstCrazy has quit IRC01:33
*** EinstCrazy has joined #openstack-oslo01:33
*** yamamoto has joined #openstack-oslo01:36
*** EinstCra_ has quit IRC01:37
*** yamahata has quit IRC01:38
*** EinstCrazy has quit IRC01:39
*** EinstCrazy has joined #openstack-oslo01:41
*** yamamoto has quit IRC01:41
*** yamamoto has joined #openstack-oslo01:41
*** yamamoto has quit IRC01:42
*** ozialien10 has joined #openstack-oslo01:46
*** ozialien10 has quit IRC01:46
*** ozialien10 has joined #openstack-oslo01:47
openstackgerritDmitriy Ukhlov proposed openstack/oslo.messaging: Implements configurable connection factory  https://review.openstack.org/31066401:54
*** EinstCrazy has quit IRC01:59
*** EinstCrazy has joined #openstack-oslo02:00
*** dukhlov has quit IRC02:00
*** EinstCra_ has joined #openstack-oslo02:04
*** EinstCrazy has quit IRC02:08
*** yamahata has joined #openstack-oslo02:12
*** EinstCra_ has quit IRC02:22
*** jaypipes has quit IRC02:40
*** yamamoto has joined #openstack-oslo02:43
*** EinstCrazy has joined #openstack-oslo02:44
*** EinstCrazy has quit IRC02:45
*** ozialien10 has quit IRC02:52
*** ozialien10 has joined #openstack-oslo02:52
*** yamamoto has quit IRC02:54
*** EinstCrazy has joined #openstack-oslo02:57
*** EinstCrazy has quit IRC02:58
*** EinstCrazy has joined #openstack-oslo03:00
*** EinstCrazy has quit IRC03:01
*** EinstCrazy has joined #openstack-oslo03:03
*** EinstCrazy has quit IRC03:07
*** EinstCrazy has joined #openstack-oslo03:09
*** EinstCra_ has joined #openstack-oslo03:12
*** EinstCra_ has quit IRC03:15
*** EinstCrazy has quit IRC03:15
*** EinstCrazy has joined #openstack-oslo03:18
*** EinstCra_ has joined #openstack-oslo03:23
*** EinstCrazy has quit IRC03:23
*** jamielennox is now known as jamielennox|away03:28
*** dukhlov has joined #openstack-oslo03:31
*** dukhlov has quit IRC03:36
*** jamielennox|away is now known as jamielennox03:37
*** dukhlov has joined #openstack-oslo03:44
*** yamamoto has joined #openstack-oslo03:57
*** e0ne has joined #openstack-oslo03:59
*** e0ne has quit IRC04:01
*** e0ne has joined #openstack-oslo04:02
*** EinstCra_ has quit IRC04:03
*** EinstCrazy has joined #openstack-oslo04:04
*** e0ne has quit IRC04:05
*** EinstCrazy has quit IRC04:05
*** dukhlov has quit IRC04:05
*** dukhlov has joined #openstack-oslo04:06
*** EinstCrazy has joined #openstack-oslo04:08
*** yamamoto has quit IRC04:09
*** EinstCrazy has quit IRC04:09
*** itisha has quit IRC04:09
*** EinstCrazy has joined #openstack-oslo04:10
*** EinstCrazy has quit IRC04:10
*** EinstCrazy has joined #openstack-oslo04:12
*** EinstCrazy has quit IRC04:13
*** EinstCrazy has joined #openstack-oslo04:14
*** EinstCrazy has quit IRC04:14
*** EinstCrazy has joined #openstack-oslo04:15
*** EinstCrazy has quit IRC04:15
*** EinstCrazy has joined #openstack-oslo04:16
*** EinstCrazy has quit IRC04:16
*** EinstCrazy has joined #openstack-oslo04:17
*** EinstCrazy has quit IRC04:19
*** EinstCrazy has joined #openstack-oslo04:21
*** EinstCra_ has joined #openstack-oslo04:22
*** EinstCrazy has quit IRC04:22
*** EinstCra_ has quit IRC04:22
*** sputnik13 has quit IRC04:23
*** yamamoto has joined #openstack-oslo04:24
*** EinstCrazy has joined #openstack-oslo04:24
*** EinstCrazy has quit IRC04:24
*** links has joined #openstack-oslo04:25
*** yamamoto has quit IRC04:26
*** EinstCra_ has joined #openstack-oslo04:27
*** EinstCra_ has quit IRC04:28
*** dukhlov has quit IRC04:29
*** yamamoto has joined #openstack-oslo04:30
*** hoonetorg has quit IRC04:32
*** EinstCrazy has joined #openstack-oslo04:32
*** EinstCrazy has quit IRC04:33
*** hoonetorg has joined #openstack-oslo04:33
*** yamamoto has quit IRC04:36
*** EinstCra_ has joined #openstack-oslo04:37
*** EinstCra_ has quit IRC04:38
*** EinstCra_ has joined #openstack-oslo04:38
*** EinstCra_ has quit IRC04:38
*** dimtruck is now known as zz_dimtruck04:43
*** EinstCrazy has joined #openstack-oslo04:43
*** EinstCrazy has quit IRC04:44
*** EinstCrazy has joined #openstack-oslo04:45
*** EinstCrazy has quit IRC04:45
*** EinstCrazy has joined #openstack-oslo04:46
*** EinstCrazy has quit IRC04:46
*** EinstCrazy has joined #openstack-oslo04:46
*** harlowja_at_home has joined #openstack-oslo04:46
*** EinstCra_ has joined #openstack-oslo04:49
*** EinstCrazy has quit IRC04:50
*** EinstCra_ has quit IRC04:53
*** EinstCrazy has joined #openstack-oslo04:58
*** EinstCrazy has quit IRC04:58
*** EinstCrazy has joined #openstack-oslo04:58
*** EinstCrazy has quit IRC04:59
*** harlowja_at_home has quit IRC05:02
*** EinstCrazy has joined #openstack-oslo05:05
*** EinstCrazy has quit IRC05:06
*** EinstCrazy has joined #openstack-oslo05:07
*** EinstCrazy has quit IRC05:07
*** hoonetorg has quit IRC05:07
*** EinstCrazy has joined #openstack-oslo05:07
*** bnemec has quit IRC05:17
*** hoonetorg has joined #openstack-oslo05:23
*** yamamoto has joined #openstack-oslo05:37
*** jamielennox is now known as jamielennox|away05:40
*** yamamoto has quit IRC05:45
*** david-lyle has quit IRC06:05
*** salv-orlando has joined #openstack-oslo06:07
*** openstackgerrit has quit IRC06:17
*** openstackgerrit has joined #openstack-oslo06:18
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.utils: Imported Translations from Zanata  https://review.openstack.org/31193606:33
*** salv-orlando has quit IRC06:36
*** eliqiao has quit IRC06:38
*** eliqiao has joined #openstack-oslo06:38
*** lpetrut has joined #openstack-oslo06:40
*** e0ne has joined #openstack-oslo06:41
*** sputnik13 has joined #openstack-oslo06:49
*** sputnik13 has quit IRC06:55
*** sputnik13 has joined #openstack-oslo06:56
*** tesseract has joined #openstack-oslo07:01
*** tesseract is now known as Guest2331907:01
*** sputnik13 has quit IRC07:09
*** jamielennox|away is now known as jamielennox07:22
*** jpena|off is now known as jpena07:23
*** deray has joined #openstack-oslo07:25
*** salv-orlando has joined #openstack-oslo07:36
*** lpetrut has quit IRC07:40
*** salv-orlando has quit IRC07:41
openstackgerritJiaJunsu proposed openstack/oslo.messaging: reset head to master  https://review.openstack.org/30122307:45
openstackgerritHaifeng Yan proposed openstack/oslo.log: Fix example issue  https://review.openstack.org/31235107:46
*** toabctl has quit IRC07:48
openstackgerritJiaJunsu proposed openstack/oslo.messaging: Add parse.unquote to transport_url  https://review.openstack.org/30122307:48
openstackgerritHaifeng Yan proposed openstack/oslo.log: Fix example issue  https://review.openstack.org/31235107:49
*** toabctl has joined #openstack-oslo07:52
*** lpetrut has joined #openstack-oslo07:54
*** shardy has joined #openstack-oslo07:56
*** EinstCra_ has joined #openstack-oslo07:57
*** lpetrut has quit IRC07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-oslo08:00
*** EinstCrazy has quit IRC08:00
*** yamahata has quit IRC08:11
*** ozamiatin has joined #openstack-oslo08:13
*** ozialien10 has quit IRC08:28
*** ozialien10 has joined #openstack-oslo08:29
*** markus_z has joined #openstack-oslo08:33
*** yassine has joined #openstack-oslo08:37
*** salv-orlando has joined #openstack-oslo08:38
*** yassine is now known as Guest1452908:38
*** salv-orlando has quit IRC08:42
*** yamahata has joined #openstack-oslo08:46
*** eezhova has quit IRC08:50
*** cdent has joined #openstack-oslo08:54
openstackgerritJamie Lennox proposed openstack/oslo.context: Allow deprecated headers in from_environ  https://review.openstack.org/31238808:57
openstackgerritJamie Lennox proposed openstack/oslo.context: Strip roles in from_environ  https://review.openstack.org/31238908:57
openstackgerritYulia Portnova proposed openstack/oslo.messaging: Fix simulator stat printing  https://review.openstack.org/31239609:17
*** yamamoto has joined #openstack-oslo09:28
*** sdague has joined #openstack-oslo09:33
*** yamamoto has quit IRC09:34
*** cdent has quit IRC09:36
*** lpetrut has joined #openstack-oslo09:38
*** salv-orlando has joined #openstack-oslo09:38
*** salv-orlando has quit IRC09:43
*** yamamoto has joined #openstack-oslo09:44
*** yamamoto has quit IRC09:50
*** Guest14529 has quit IRC09:51
*** jpena is now known as jpena|brb10:01
*** jecarey has joined #openstack-oslo10:05
*** e0ne has quit IRC10:06
*** e0ne has joined #openstack-oslo10:06
*** jecarey has quit IRC10:09
*** ozialien10 has quit IRC10:18
*** ozialien10 has joined #openstack-oslo10:18
*** yamamoto has joined #openstack-oslo10:21
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Redesign router proxy  https://review.openstack.org/30865310:30
*** EinstCra_ has quit IRC10:30
*** ri0 has joined #openstack-oslo10:38
*** salv-orlando has joined #openstack-oslo10:40
*** links has quit IRC10:45
*** salv-orlando has quit IRC10:45
*** lucasagomes is now known as lucas-hungry10:56
*** david-lyle has joined #openstack-oslo10:58
*** deray has quit IRC11:07
*** ri0 is now known as radio11:20
*** radio is now known as ri011:21
*** yamamoto has quit IRC11:22
*** yamamoto has joined #openstack-oslo11:25
*** jpena|brb is now known as jpena11:29
*** salv-orlando has joined #openstack-oslo11:40
*** cdent has joined #openstack-oslo11:43
*** salv-orlando has quit IRC11:46
*** yamahata has quit IRC11:54
*** lucas-hungry is now known as lucasagomes11:55
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Redesign router proxy  https://review.openstack.org/30865311:56
*** links has joined #openstack-oslo11:59
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Redesign router proxy  https://review.openstack.org/30865312:04
*** e0ne has quit IRC12:05
*** jaypipes has joined #openstack-oslo12:11
openstackgerritKirill Bespalov proposed openstack/oslo.messaging: Use single producer and avoid exchange redeclaration  https://review.openstack.org/31098412:11
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Redesign router proxy  https://review.openstack.org/30865312:12
*** yassine has joined #openstack-oslo12:12
*** yassine is now known as Guest4137012:12
*** davidlenwell has quit IRC12:13
*** haypo has joined #openstack-oslo12:14
*** davidlenwell has joined #openstack-oslo12:15
*** Guest41370 is now known as yassou12:15
*** e0ne has joined #openstack-oslo12:22
*** kgiusti has joined #openstack-oslo12:22
*** binarin_ has quit IRC12:24
*** ozialien10 has quit IRC12:24
*** ozialien10 has joined #openstack-oslo12:25
*** yamamoto has quit IRC12:30
*** yamamoto has joined #openstack-oslo12:31
*** pradk has quit IRC12:34
*** jpena is now known as jpena|lunch12:34
*** salv-orlando has joined #openstack-oslo12:41
*** salv-orlando has quit IRC12:46
*** links has quit IRC12:46
*** rlrossit has joined #openstack-oslo12:55
*** andymaier has joined #openstack-oslo12:56
*** jecarey has joined #openstack-oslo13:03
*** jecarey has quit IRC13:03
*** jecarey has joined #openstack-oslo13:03
*** jecarey has quit IRC13:03
*** jecarey has joined #openstack-oslo13:04
*** salv-orlando has joined #openstack-oslo13:09
*** krotscheck_dcm is now known as krotscheck13:10
*** salv-orlando has quit IRC13:13
*** ri0 has quit IRC13:15
openstackgerritMerged openstack/oslo.messaging: Remove Beta development status from classifiers  https://review.openstack.org/31179913:15
ozamiatindims, dukhlov: Hi, could you please take a look at https://review.openstack.org/#/c/308653 and https://review.openstack.org/#/c/311065/ when you get chance?13:15
patchbotozamiatin: patch 311065 - oslo.messaging - [zmq] Add Sentinel instructions to deployment guide13:15
*** ri0 has joined #openstack-oslo13:15
dimsozamiatin : spelling mistake in last line of https://review.openstack.org/#/c/311065/1/doc/source/zmq_driver.rst13:16
patchbotdims: patch 311065 - oslo.messaging - [zmq] Add Sentinel instructions to deployment guide13:16
ozamiatindims: got it, thanks13:17
dimsozamiatin : the router redesign is scary big :)13:18
ozamiatindims: yeah I know it, I believe it is the last huge redesign :)13:18
dimsozamiatin : will work through it today. thanks13:18
*** ri0 has quit IRC13:19
*** mixos has joined #openstack-oslo13:20
openstackgerritMerged openstack/oslo.concurrency: processutils: add support for missing process limits  https://review.openstack.org/30781313:24
*** mixos has quit IRC13:24
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Add Sentinel instructions to deployment guide  https://review.openstack.org/31106513:24
openstackgerritMerged openstack/oslo.config: Add enfore_type parameter for set_default,  https://review.openstack.org/28700913:25
*** tonytan4ever has joined #openstack-oslo13:26
openstackgerritMerged openstack/oslo.config: Avoid duplicate sections in generator output  https://review.openstack.org/30480213:33
*** dkehn__ has joined #openstack-oslo13:37
openstackgerritMerged openstack/oslo.log: Allow reload of 'debug' option  https://review.openstack.org/25482113:38
*** dkehn_ has quit IRC13:39
*** yamamoto has quit IRC13:45
*** d0ugal has quit IRC13:46
*** d0ugal has joined #openstack-oslo13:47
openstackgerritMerged openstack/oslo.service: Make _spawn_service more flexible  https://review.openstack.org/28987613:50
jd__harlowja: can I get a oslo.middleware release boss?13:52
openstackgerritMichael Krotscheck proposed openstack/oslo.middleware: Set default allow methods to those defined in RFC 2616  https://review.openstack.org/29356713:53
openstackgerritMichael Krotscheck proposed openstack/oslo.middleware: Added PATCH method to default config  https://review.openstack.org/29393013:53
openstackgerritMichael Krotscheck proposed openstack/oslo.middleware: Removed default headers from CORS_OPTS  https://review.openstack.org/29356613:53
openstackgerritMichael Krotscheck proposed openstack/oslo.middleware: Added set_defaults method to oslo_middleware.cors  https://review.openstack.org/28536813:53
*** jpena|lunch is now known as jpena13:55
krotscheckdims: Thanks for doing those rechecks, but the main issue was that all those patches were waiting on one with a -1 that I haven't addressed yet.13:56
krotscheckI've gone ahead and rebased them all so that one's not blocking naymore.13:56
*** sigmavirus24_awa is now known as sigmavirus2413:56
dimskrotscheck : ah i was in the auto-recheck mode :)13:56
krotscheckdims: Been there :D13:57
krotscheckOn a related note, what's the process for deprecating an API? The Latent Property feature in the cors middleware is better supported in oslo-config's new genconfig default hooks.14:00
krotscheck(I'm asking form a code perspective)14:00
krotscheckLike, what should I do - add a warning logger letting people know that it's deprecated? Is there an annotation or something that I can use (I know Java has @deprecated)14:00
dimskrotscheck : will let harlowja handle that :)14:01
*** itisha has joined #openstack-oslo14:02
*** gordc has joined #openstack-oslo14:09
*** salv-orlando has joined #openstack-oslo14:10
*** salv-orlando has quit IRC14:15
*** mixos has joined #openstack-oslo14:16
*** yamahata has joined #openstack-oslo14:20
*** bnemec has joined #openstack-oslo14:21
*** zhenq has joined #openstack-oslo14:24
*** salv-orlando has joined #openstack-oslo14:24
*** zz_dimtruck is now known as dimtruck14:27
*** mtanino has joined #openstack-oslo14:28
*** salv-orlando has quit IRC14:29
*** ozialien10 has quit IRC14:31
*** pradk has joined #openstack-oslo14:31
*** yamamoto has joined #openstack-oslo14:32
*** yamamoto has quit IRC14:32
*** yamamoto has joined #openstack-oslo14:33
haypo:vs14:42
*** m1r4nt15_b0y is now known as d34dh0r5314:44
*** e0ne has quit IRC14:50
*** yamahata has quit IRC14:52
*** yamahata has joined #openstack-oslo14:53
*** salv-orlando has joined #openstack-oslo14:54
*** salv-orlando has quit IRC14:59
gibihi! I have a spec up for ovo https://review.openstack.org/#/c/311194/ versionedobjects: add json schema generation that needs some core attention. Could you check it please?15:01
patchbotgibi: patch 311194 - oslo-specs - versionedobjects: add json schema generation15:01
*** yamamoto has quit IRC15:04
*** sputnik13 has joined #openstack-oslo15:06
*** sputnik13 has quit IRC15:12
*** lpetrut has quit IRC15:18
*** lpetrut has joined #openstack-oslo15:23
rbradforkrotscheck, depending on what exactly you are deprecating, debtcollector is a means to annotate.15:28
openstackgerritStephen Finucane proposed openstack/oslo.config: Add 'DEPRECATED: ' prefix for deprecated opts  https://review.openstack.org/30768515:37
*** deray has joined #openstack-oslo15:41
openstackgerritAndrew Laski proposed openstack/oslo-specs: Allow policy registration from code  https://review.openstack.org/30915215:43
openstackgerritAndrew Laski proposed openstack/oslo-specs: Add capability to generate a sample policy.json  https://review.openstack.org/30915315:43
*** salv-orlando has joined #openstack-oslo15:44
*** Guest23319 has quit IRC15:46
*** e0ne has joined #openstack-oslo15:49
*** andymaier has quit IRC15:52
*** yassou has quit IRC15:54
krotscheckrbradfor Cool, will look into that15:54
*** salv-orlando has quit IRC15:59
openstackgerritMerged openstack/oslo.middleware: Removed default headers from CORS_OPTS  https://review.openstack.org/29356615:59
openstackgerritMerged openstack/oslo.service: Offer mutate_config_files  https://review.openstack.org/26331215:59
*** salv-orlando has joined #openstack-oslo16:00
*** lucasagomes is now known as lucas-afk16:01
*** amotoki has joined #openstack-oslo16:02
*** lpetrut has quit IRC16:02
*** salv-orlando has quit IRC16:03
*** yamamoto has joined #openstack-oslo16:05
openstackgerritRonald Bradford proposed openstack/oslo.config: Update Beta development status from classifiers  https://review.openstack.org/31180016:06
*** yamamoto has quit IRC16:14
*** deray has quit IRC16:20
openstackgerritMerged openstack/oslo.middleware: Set default allow methods to those defined in RFC 2616  https://review.openstack.org/29356716:20
*** sputnik13 has joined #openstack-oslo16:24
*** ozialien10 has joined #openstack-oslo16:29
harlowjakrotscheck ya try debtcollector16:31
harlowjajd__  is https://review.openstack.org/#/c/311812/ fine with u ;)16:33
patchbotharlowja: patch 311812 - releases - Oslo releases for week of may 2nd16:33
harlowjaoslo middleware in that16:33
harlowjai can split it out if u really want16:34
*** vilobhmm11 has joined #openstack-oslo16:35
krotscheckharlowja: Roger wilco16:36
harlowjabig trucker over16:36
harlowjalol16:36
krotscheckIncidentally, I've got 2x+2 on a couple of reviews that lost their +A because rebase. Can I impose on someone to go check those out? https://review.openstack.org/#/c/285368/ https://review.openstack.org/#/c/293930/316:43
patchbotkrotscheck: patch 285368 - oslo.middleware - Added set_defaults method to oslo_middleware.cors16:43
patchbotkrotscheck: patch 293930 - oslo.middleware - Added PATCH method to default config16:43
*** e0ne has quit IRC16:46
*** e0ne has joined #openstack-oslo16:47
*** markus_z has quit IRC16:51
*** e0ne has left #openstack-oslo16:53
*** lpetrut has joined #openstack-oslo17:07
*** bigdogstl has joined #openstack-oslo17:09
*** salv-orlando has joined #openstack-oslo17:12
*** bigdogstl has quit IRC17:15
openstackgerritMichael Krotscheck proposed openstack/oslo.middleware: Deprecate multiple config block parsing.  https://review.openstack.org/31264517:16
openstackgerritMichael Krotscheck proposed openstack/oslo.middleware: Deprecate using String as valid value for allowed_origin.  https://review.openstack.org/31264617:16
openstackgerritMichael Krotscheck proposed openstack/oslo.middleware: Deprecated set_latent  https://review.openstack.org/31264917:24
openstackgerritMichael Krotscheck proposed openstack/oslo.middleware: Removed simple headers from documentation examples  https://review.openstack.org/31265017:26
*** jpena is now known as jpena|off17:28
krotscheckSo, a question about deprecating things. I have time to write the code that will _eventually_ land as soon as the deprecation period expires.17:30
krotscheckI can't guarantee that I'll be around two versions down the road though.17:31
harlowjawhereu going17:33
harlowja:-P17:33
*** tonytan4ever has quit IRC17:34
*** bigdogstl has joined #openstack-oslo17:35
*** vilobhmm111 has joined #openstack-oslo17:36
harlowjakrotscheck i think the best that can be done is add TODO and do the depreciation, i don't think we have self-removing code yet17:36
harlowjabut maybe someday17:36
*** vilobhmm11 has quit IRC17:37
krotscheckharlowja: Well, I can create the patch, but it'll likely bitrot17:38
*** rlrossit has quit IRC17:38
harlowjaright, if u want, feel free, if not i think that's ok17:38
krotscheckharlowja: What's oslos deprecation cycle, 2 cycles?17:38
harlowjahttp://lists.openstack.org/pipermail/openstack-dev/2016-April/093527.html i think is the new agreement :)17:39
harlowja(which may change due to some upcoming TC discussion)17:40
krotscheckharlowja: Cool, I'll keep my eyes out.17:43
harlowja:)17:43
krotscheckharlowja: Though, knowing that I only have to wait until Ocata starts makes me happy.17:43
harlowjadepends when those depreciation things get merged :)17:44
harlowja(and released)17:44
harlowjasince i think that is time zero17:44
harlowjakrotscheck  are those depreciation messages mostly for operators consumption ?17:44
*** bigdogstl has quit IRC17:45
*** tonytan4ever has joined #openstack-oslo17:45
krotscheckharlowja: ....mmmmostly. The multiple-config-block one definitely is. The set_latent impacts devs as well as operators. The allow_origin one is for devs only.17:46
krotscheckharlowja: Ultimately I just want to collapse all the multitude of configuration choices into set_defaults and some API method on top of CORS_OPTS17:47
harlowjakk, i think what we've been aiming for is that for operator facing deprecations it goes in https://github.com/openstack/oslo.log/blob/master/oslo_log/versionutils.py and dev facing ones use debtcollector17:47
harlowjai gotta write up that policy somewhere, since it seems sorta not well known17:48
krotscheckMostly because I tried to write tests for https://review.openstack.org/#/c/291381/ and found out that none of the config paths are even remotely consistent.17:48
patchbotkrotscheck: patch 291381 - oslo.middleware - Retain reference to GroupAttrs in CORS middleware17:48
harlowja:-/17:49
* krotscheck is annoyed at his past self because of this.17:49
krotscheckharlowja: It seems like versionutils only work for callables. I'm guessing I should extract the impacted code into a private method and annotate that?17:54
rbradforharlowja, krotscheck I have started that policy at https://review.openstack.org/#/c/288720/. However as pointed out versionutils has a small subset of capabilities17:55
patchbotrbradfor: patch 288720 - oslo-specs - [WIP] Add policy for managing deprecation17:55
harlowjarbradfor thx, i thought i remembered there was something17:55
rbradforharlowja, WIP, but no feedback to date, and it was something I'd proposed for discussion at summit.17:56
harlowjarbradfor  i'll get that moving :-P17:57
krotscheckHrm. Oslo middleware doesn't depend on oslo log.17:57
rbradforkrotscheck, well that will be fun17:58
*** salv-orlando has quit IRC18:01
krotscheckARGH too many patch chains in oslo_middleware, can't keep 'em apart.18:05
*** hoonetorg has quit IRC18:07
harlowjaya, hmmm, i need to figure out how to drop the version_utils stuff and hook something into debtcollector so that one thing can do both here (vs 2 things)18:13
krotscheckharlowja: Sooo... I should not send up my versionutils thing for now?18:14
harlowjanah i think u can18:15
harlowjafor operator focused messages i think using oslo.log will be the way to go18:15
harlowjauntil i can figure out a good plan to collapse these 2 things18:15
krotscheckYou got it.18:17
*** Kennan has quit IRC18:19
*** Kennan has joined #openstack-oslo18:20
*** hoonetorg has joined #openstack-oslo18:23
*** bigdogstl has joined #openstack-oslo18:32
*** shardy has quit IRC18:34
*** bigdogstl has quit IRC18:39
*** bigdogstl has joined #openstack-oslo18:43
*** sputnik13 has quit IRC18:44
openstackgerritDavanum Srinivas (dims) proposed openstack-dev/pbr: Remember the insertion order for pbr.json  https://review.openstack.org/30740518:47
openstackgerritJoshua Harlow proposed openstack/debtcollector: Allow providing a redirect function for generated messages  https://review.openstack.org/31268718:48
harlowjakrotscheck rbradfor ^ that should start to make it possible to say provide a logging redirection function18:48
*** bigdogstl has quit IRC18:48
harlowjato say target operators (vs the default stderr)18:48
harlowjabut ya, we need to flush out that policy18:49
krotscheckharlowja: Hrm, that doesn't seem too crazy.18:49
harlowjaya, its a start18:50
krotscheckIs there a meanigful difference between dev deprecation and op deprecation?18:51
krotscheckWhat I mean is: If they all go to log...18:51
*** salv-orlando has joined #openstack-oslo18:51
krotscheck...is the world going to explode?18:51
harlowjaimho, no :-P18:51
harlowjathe nice thing is that https://docs.python.org/2/library/logging.html#logging.captureWarnings exists for this18:52
harlowjamaybe that'd be a good thing, so that people actually pay attention to these warnings18:52
harlowjacause they by default got to stder, which probably goes to dev/null18:52
harlowjaso thats the other option, use debtcollector for this kind of stuff (non-config deprecations) and just say, captureWarnings should be used18:53
krotscheckThe other thign we could do is create a deprecation log level.18:53
harlowja:-/18:53
krotscheckBut that kinda breaks the hierarchy of severity18:53
harlowjaright18:53
harlowjacustom log levels scare me18:53
harlowjawell not really scare me, u get the point18:54
harlowjaha18:54
krotscheckI get it18:54
krotscheckIt's kinda meh18:54
*** rlrossit has joined #openstack-oslo18:54
harlowjaya18:54
*** vilobhmm111 has quit IRC18:54
harlowjaseems like django just turns that logging.captureWarnings on by default18:55
harlowjahttps://docs.djangoproject.com/en/1.9/_modules/django/utils/log/ (configure_logging func there)18:56
*** salv-orlando has quit IRC18:56
harlowjamaybe we should just see if that is possible in all the openstack projects ;)18:56
krotscheckWhat could possibly go wrong?18:56
*** salv-orlando has joined #openstack-oslo18:56
rbradforharlowja, added my 2 cents, but the package is a bit above my understanding.18:56
rbradforkrotscheck, FWIW, i just used  debtcollector.deprecate() to emit a message for a usage I wanted to deprecate, but I suspect your case is more detailed. https://review.openstack.org/#/c/274188/6/oslo_log/formatters.py18:58
patchbotrbradfor: patch 274188 - oslo.log - Use new logging specific method for context info18:58
krotscheckrbradfor: That removal version says 4... I thought we did a full version release with each release, so it'd be 5?18:59
krotscheckOr are we doing 3.7+ until newton?19:00
krotscheck(end of newton)19:00
rbradforkrotscheck, good eye, this was proposed before freeze19:00
rbradforand hence is out of date19:00
krotscheckrbradfor; Yeah, your use case is about equivalent to mine.19:00
krotscheckJust a code path I need to check on.19:00
*** salv-orlando has quit IRC19:07
*** salv-orlando has joined #openstack-oslo19:08
*** itisha has quit IRC19:09
harlowjakrotscheck i'll ponder on that question of just enable the warnings -> log thingy, it'd be nice, and i wonder if people would then pay more attention to warnings :-P19:10
krotscheckOr less.19:11
harlowjalol19:11
krotscheckMore noise usually == less attention, that's why splunk can charge such ridiculous prices.19:11
harlowjaright but if people are typically redirecting stderr > /dev/null  that make zero attention, ha19:12
*** salv-orlando has quit IRC19:13
harlowjaand less > zero :-P19:14
krotscheckPoint19:16
*** tonytan4ever has quit IRC19:17
*** bigdogstl has joined #openstack-oslo19:28
*** salv-orlando has joined #openstack-oslo19:30
*** sputnik13 has joined #openstack-oslo19:32
*** bigdogstl has quit IRC19:33
*** njohnston has joined #openstack-oslo19:34
*** tonytan4ever has joined #openstack-oslo19:38
*** openstackgerrit has quit IRC19:48
*** openstackgerrit has joined #openstack-oslo19:48
*** yamahata has quit IRC19:58
*** salv-orlando has quit IRC20:00
njohnstonharlowja: Hi, I am having an issue with oslo.service and fwaas, and xgerman suggested that I ask you for help. :-)20:00
harlowjahmmmm, njohnston whats the issue, maybe myself (or others) here can help20:01
njohnstonpy27 and py34 tests are failing on the master for fwaas, and it looks like it's an odd issue that is either inside oslo.service or how we're using it.  Here are the details: http://paste.openstack.org/show/496146/20:01
njohnstonActually that paste is wrong; I hand-jammed something in .tox/py27/lib/python2.7/site-packages/oslo_service/periodic_task.py line 177, I added the 'conf' as argument.20:06
*** sdague has quit IRC20:07
njohnstonIf I do `rm -rf .tox .testrepository ; tox -e py27` then I get:20:07
njohnstonhttp://paste.openstack.org/show/496148/20:08
harlowjabb20:09
njohnston"TypeError: __init__() takes exactly 2 arguments (1 given)" from oslo_service/periodic_task.py line 17720:09
*** harlowja has quit IRC20:09
njohnstonI am unsure how to proceed; googling found http://freshfoo.com/blog/object__init__takes_no_parameters but little else to guide me.20:10
*** salv-orlando has joined #openstack-oslo20:11
*** sdague has joined #openstack-oslo20:14
*** bigdogstl has joined #openstack-oslo20:22
*** bigdogstl has quit IRC20:27
*** tonytan4ever has quit IRC20:33
*** salv-orlando has quit IRC20:35
*** bigdogstl has joined #openstack-oslo20:38
*** yamahata has joined #openstack-oslo20:38
*** bigdogstl has quit IRC20:43
*** salv-orlando has joined #openstack-oslo20:44
*** sputnik13 has quit IRC20:47
njohnstonharlowja: any thoughts?20:50
rbradfornjohnston, just so I can try to reproduce, are you referring to the openstack/neutron-fwaas project?20:52
njohnstonyes20:53
njohnstonrbradfor: Thanks for taking a look at this.  Do `git clone https://github.com/openstack/neutron-fwaas.git && cd neutron-fwaas && tox -e py27` and you will see the issue.20:54
njohnstonwhich is 6 failures of type "TypeError: __init__() takes exactly 2 arguments (1 given)" occurring at oslo_service/periodic_task.py line 17720:55
rbradfornjohnston, running now20:55
njohnstonrbradfor: Thanks much!20:55
*** jaypipes has quit IRC20:56
rbradforlots of deprecationwarnings? some easy rework oneday.20:58
njohnstonI hope to get to those once I get the tests passing at least20:58
rbradfornjohnston,  ok, so I can at least reproduce the error to verify the problem20:59
njohnstonrbradfor: Thanks, that helps me feel less insane.20:59
rbradfornjohnston, np, no promises I can fix it, but I'll poke around some more.21:00
njohnstonrbradfor: Thanks very much, I'll be lurking.  :-)21:00
*** sputnik13 has joined #openstack-oslo21:02
*** njohnsto_ has joined #openstack-oslo21:03
*** harlowja has joined #openstack-oslo21:10
harlowjanjohnsto_ did u figure that one out, sorry had a sprint planning stuffs21:11
harlowjai don't recall the periodic task stuff having changed arguments21:11
*** ozamiatin has quit IRC21:12
njohnsto_harlowja: No luck yet, but rbradfor confirmed he could replicate.21:12
njohnsto_harlowja: Do `git clone https://github.com/openstack/neutron-fwaas.git && cd neutron-fwaas && tox -e py27` and you will see the issue.21:13
harlowjakk21:13
njohnsto_(ignore the deprecation warnings, I will fix those once the tests are working.)21:14
rbradforharlowja, attempting my basic debugging skills on the problem now, but happy for advanced help!21:16
harlowjaya, let me just see what changed in oslo.service that might have triggered that21:17
harlowjahttps://github.com/openstack/oslo.service/blob/master/oslo_service/periodic_task.py hasn't been touched in a while so it makes me wonder21:17
njohnsto_Do very many components make use of oslo.service?21:19
*** rlrossit has quit IRC21:19
harlowjaya21:19
harlowjamost do :-P21:19
*** sputnik13 has quit IRC21:20
njohnsto_OK, I can never tell :-)21:20
harlowja:)21:20
harlowjaya, usage metrics are an ongoing task21:20
*** jecarey has quit IRC21:21
njohnsto_Is there a change that most projects could have made that neutron-FWaaS is just behind the times on?  The FWaaS code base has not been tended as actively as it ought of late.21:22
harlowjaya, i'm looking into the fwaas stuff to see21:22
njohnsto_I can't imagine what it would be, but I am trying to think outside the box.21:22
rbradforharlowja, njohnsto_ so I can step through the debugger and get the same problem, but not yet any wiser of problem21:24
harlowjaya, i'm just looking through the code to see if i notice anything off21:24
*** vilobhmm11 has joined #openstack-oslo21:26
*** vilobhmm111 has joined #openstack-oslo21:28
harlowjalet me trying running tox and see here21:29
harlowjanothing code wise looks really off :-/21:29
njohnsto_Ok, thanks21:30
*** vilobhmm11 has quit IRC21:31
*** bigdogstl has joined #openstack-oslo21:34
*** sputnik13 has joined #openstack-oslo21:36
rbradforharlowja, perhaps a good advance debugging lesson step. I'd be up for learning more21:36
harlowjaya, if u want rbradfor  :)21:36
harlowjaif we can just get one test to fail, without having to run the full tox -epy27 that will likely help21:36
rbradforharlowja, I can do that easily.21:37
rbradforsource .tox/py27/bin/activate21:37
harlowjaand then another question is what oslo.service version is installed21:37
harlowjayup21:37
rbradforpython -m testtools.run neutron_fwaas.tests.unit.services.firewall.agents.varmour.test_varmour_router.TestVarmourRouter.test_agent_snat_enable21:37
rbradforset applicable break points21:37
harlowjayup yup21:37
harlowjasometimes i switch into nosetests instead of testtools for single test runs, but either or21:37
rbradforhmmm, my oslo.service is 1.8.0, I don'21:38
njohnsto_I believe requirements.text specifies ">1.0.0" for this21:38
rbradfort see that at http://git.openstack.org/cgit/openstack/oslo.service/log/21:38
harlowjahttps://github.com/openstack/oslo.service/tree/1.8.0 ?21:38
harlowjagithub has nicer tag listing i think :-P21:39
rbradforbut tag exists http://git.openstack.org/cgit/openstack/oslo.service/tag/?h=1.8.021:39
harlowjaor http://git.openstack.org/cgit/openstack/oslo.service/refs/ i guess21:39
harlowjamight be interesting to uninstall 1.821:39
harlowjainstall 1.721:39
harlowjasee if it worked there, that will isolate which changes could have causd issues21:39
*** bigdogstl has quit IRC21:39
harlowjamy slow vm is still cloning that repo, lol21:40
rbradforharlowja, doing it now21:41
harlowjaso i just tried the following and it seemed to work21:50
harlowjahttp://paste.openstack.org/show/496162/21:50
harlowjalet me make sure my neutron repo is up to date21:50
njohnsto_I am interested to see if using 1.7 fixes the issue; I was trying downgrades but it never fixed the situation, but then21:50
njohnsto_but then I may have been doing it wrong.21:51
*** sdague has quit IRC21:51
rbradforharlowja, downgrading to 1.7 didn't work for me either.21:51
harlowjaya, ok, so i upgraded my neutron and it failed21:52
njohnsto_harlowja: does the fact that running nose tests works mean the venv is suspect?21:52
njohnsto_Ok21:52
harlowjadamn, let me see if i can get back to the neutron version that failed, ha21:52
harlowjashould of got the sha, lol21:52
harlowjaok at least neutron @ git checkout 0fae3beb does work21:53
rbradforharlowja, ok, well I'm about to21:53
harlowjaand that was about a week ago21:53
harlowjasooo something in neutron changed since then to cause this21:53
rbradforsign off, prepare dinner, will like to look into seeing the real problem tomorrow21:54
harlowjak, njohnsto_ so that's something u can investigate i guess, or i can, bisecting what in neutron caused this by trying the commits after 0fae3beb21:54
harlowjavia `git log 0fae3beb..HEAD `21:55
harlowjaits like 10 commits in that log21:55
harlowjaso not to many21:56
njohnsto_Great, thanks!21:56
harlowjanp21:56
*** jecarey has joined #openstack-oslo21:57
*** njohnsto_ has quit IRC21:58
rbradforharlowja, why can I not find that hash in my git log?22:00
harlowjain neutron :)22:00
rbradforok, not  -fwaas22:01
*** jecarey has quit IRC22:01
harlowjayuppers22:03
harlowjaseems neutron is required for -fwwas22:03
harlowjaand altering the neutron version sha seemed to fix it22:03
harlowjato a apr 24th sha22:04
harlowjaso something apr 24 --> now changed this22:04
*** jecarey has joined #openstack-oslo22:09
*** kgiusti has left #openstack-oslo22:10
*** jecarey has quit IRC22:14
*** dimtruck is now known as zz_dimtruck22:19
*** gordc has quit IRC22:20
*** sigmavirus24 is now known as sigmavirus24_awa22:21
*** mixos has quit IRC22:30
*** lpetrut has quit IRC22:36
*** cdent has quit IRC22:38
*** gordc has joined #openstack-oslo22:50
*** harlowja has quit IRC22:52
*** harlowja has joined #openstack-oslo22:53
*** pradk has quit IRC23:01
*** sputnik13 has quit IRC23:07
*** sputnik13 has joined #openstack-oslo23:07
*** sputnik13 has quit IRC23:11
*** sputnik13 has joined #openstack-oslo23:13
*** sputnik13 has quit IRC23:20
*** sputnik13 has joined #openstack-oslo23:20
*** gordc has quit IRC23:20
*** bapalm_ has quit IRC23:30
*** sputnik13 has quit IRC23:34
*** salv-orlando has quit IRC23:39
*** dkehn__ is now known as dkehn_23:49
*** mixos has joined #openstack-oslo23:57

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