Wednesday, 2015-05-06

*** tsekiyam_ has joined #openstack-oslo00:11
*** sputnik13 has quit IRC00:14
*** tsekiyama has quit IRC00:15
*** mtanino has quit IRC00:15
*** tsekiyam_ has quit IRC00:16
*** achanda has quit IRC00:18
*** achanda has joined #openstack-oslo00:19
*** zzzeek has quit IRC00:21
*** ozialien has joined #openstack-oslo00:21
*** achanda has quit IRC00:23
*** dims_ has joined #openstack-oslo00:32
*** dims has quit IRC00:33
*** mriedem has quit IRC00:34
-openstackstatus- NOTICE: Restarted gerrit due to stuck stream-events connections. Events since 23:49 were missed and changes uploaded since then will need to be rechecked.00:43
*** vilobhmm1 has quit IRC00:46
*** vilobhmm1 has joined #openstack-oslo00:47
*** arnaud____ has joined #openstack-oslo00:53
*** arnaud____ has quit IRC00:58
*** alexpilotti has quit IRC01:08
openstackgerritJoshua Harlow proposed openstack/taskflow: Speed up memory backend via a path -> node reverse mapping  https://review.openstack.org/18037301:23
*** alexpilotti has joined #openstack-oslo01:30
*** vilobhmm1 has quit IRC01:34
*** alexpilotti has quit IRC01:48
*** browne has quit IRC01:51
*** haigang has joined #openstack-oslo01:53
*** salv-orlando has joined #openstack-oslo01:58
*** salv-orlando has quit IRC02:04
*** haigang has quit IRC02:05
*** arnaud____ has joined #openstack-oslo02:09
*** arnaud____ has quit IRC02:14
*** vilobhmm1 has joined #openstack-oslo02:15
*** sigmavirus24 is now known as sigmavirus24_awa02:19
*** yamahata has quit IRC02:22
*** dims_ has quit IRC02:24
*** vilobhmm1 has quit IRC02:26
*** browne has joined #openstack-oslo02:28
*** achanda has joined #openstack-oslo02:40
*** sputnik13 has joined #openstack-oslo03:04
*** yamahata has joined #openstack-oslo03:11
*** dims has joined #openstack-oslo03:24
*** dims has quit IRC03:30
*** links has joined #openstack-oslo03:41
*** frickler has quit IRC03:42
*** rushiagr_away is now known as rushiagr03:44
*** Kennan2 has joined #openstack-oslo03:45
*** Kennan has quit IRC03:46
*** sdake has joined #openstack-oslo03:51
*** arnaud____ has joined #openstack-oslo03:55
*** sdake_ has joined #openstack-oslo03:57
*** arnaud____ has quit IRC03:59
*** sdake has quit IRC04:00
*** sdake has joined #openstack-oslo04:01
*** sdake__ has joined #openstack-oslo04:04
*** sdake_ has quit IRC04:04
*** sdake has quit IRC04:07
*** rushiagr is now known as rushiagr_away04:12
*** sdake__ is now known as sdake04:13
openstackgerritMerged openstack-dev/pbr: Issue #1451976: handle commits with non-utf8 text  https://review.openstack.org/18026504:20
*** salv-orlando has joined #openstack-oslo04:35
*** salv-orlando has quit IRC04:38
*** vilobhmm1 has joined #openstack-oslo04:45
*** nkrinner has joined #openstack-oslo04:51
*** vilobhmm1 has quit IRC04:52
*** arnaud____ has joined #openstack-oslo04:55
*** vilobhmm1 has joined #openstack-oslo04:57
*** vilobhmm1 has quit IRC05:00
*** arnaud____ has quit IRC05:00
*** vilobhmm1 has joined #openstack-oslo05:01
*** harlowja_at_home has joined #openstack-oslo05:03
*** rushiagr_away is now known as rushiagr05:03
*** crc32 has quit IRC05:17
*** sdake_ has joined #openstack-oslo05:23
*** sdake has quit IRC05:27
*** subscope_ has joined #openstack-oslo05:35
openstackgerritJoshua Harlow proposed openstack/taskflow: Deprecate upgrade (that now is moved to initialize)  https://review.openstack.org/18035105:39
*** rushiagr is now known as rushiagr_away05:40
openstackgerritAngus Lees proposed openstack/oslo.db: Test that concurrent sqlalchemy transactions don't block  https://review.openstack.org/10443605:47
*** dulek has joined #openstack-oslo05:48
*** frickler has joined #openstack-oslo05:48
*** frickler has quit IRC05:49
*** frickler has joined #openstack-oslo05:52
*** subscope_ has quit IRC06:02
*** arnaud____ has joined #openstack-oslo06:06
*** sheeprine has quit IRC06:07
*** salv-orlando has joined #openstack-oslo06:10
openstackgerritJoshua Harlow proposed openstack/taskflow: Speed up memory backend via a path -> node reverse mapping  https://review.openstack.org/18037306:11
*** sheeprine has joined #openstack-oslo06:13
openstackgerritJoshua Harlow proposed openstack/taskflow: Speed up memory backend via a path -> node reverse mapping  https://review.openstack.org/18037306:18
*** rushiagr_away is now known as rushiagr06:22
*** vilobhmm1 has quit IRC06:30
*** harlowja_at_home has quit IRC06:33
*** vilobhmm1 has joined #openstack-oslo06:35
openstackgerritMoshe Levi proposed openstack/oslo.concurrency: Add timeout parameter to execute  https://review.openstack.org/17703006:38
*** tteggel has quit IRC06:41
openstackgerritJoshua Harlow proposed openstack/tooz: Expose redis client class constants  https://review.openstack.org/17955106:41
*** yamahata has quit IRC06:41
*** tteggel has joined #openstack-oslo06:41
*** yamahata has joined #openstack-oslo06:48
*** viktors|afk is now known as viktors06:48
openstackgerritJoshua Harlow proposed openstack/oslo.utils: Add exception_to_unicode() function  https://review.openstack.org/16350706:50
*** shardy_z is now known as shardy06:54
*** achanda has quit IRC07:07
*** achanda has joined #openstack-oslo07:11
*** achanda has quit IRC07:11
*** frickler_ has joined #openstack-oslo07:14
*** frickler_ has quit IRC07:14
*** jaosorior has joined #openstack-oslo07:23
*** arnaud____ has quit IRC07:39
*** rushiagr has quit IRC07:56
openstackgerritRick van de Loo proposed openstack/taskflow: Allow same deps for requires and provides in task  https://review.openstack.org/18043108:00
*** browne has quit IRC08:05
*** ndipanov has quit IRC08:06
*** stpierre has quit IRC08:06
*** yassine has joined #openstack-oslo08:07
*** ndipanov has joined #openstack-oslo08:08
openstackgerritMehdi Abaakouk (sileht) proposed openstack/oslo.messaging: deprecate usage of transport aliases  https://review.openstack.org/18043408:10
*** rushiagr has joined #openstack-oslo08:12
*** salv-orlando has quit IRC08:18
*** salv-orlando has joined #openstack-oslo08:18
*** e0ne has joined #openstack-oslo08:22
*** sputnik13 has quit IRC08:26
*** sputnik13 has joined #openstack-oslo08:29
*** dguitarbite has joined #openstack-oslo08:32
*** stpierre has joined #openstack-oslo08:35
*** stpierre_ has joined #openstack-oslo08:37
*** ozamiatin has joined #openstack-oslo08:39
*** stpierre has quit IRC08:40
*** e0ne has quit IRC08:43
*** dbite has joined #openstack-oslo08:44
*** sputnik13 has quit IRC08:46
*** sputnik13 has joined #openstack-oslo08:48
*** sputnik13 has quit IRC08:53
*** tsufiev has quit IRC08:58
*** _amrith_ has quit IRC08:58
*** shardy_ has joined #openstack-oslo08:59
*** shardy has quit IRC09:00
*** tsufiev has joined #openstack-oslo09:00
*** _amrith_ has joined #openstack-oslo09:00
*** zqfan has joined #openstack-oslo09:01
zqfanhi, I come across an issue about MultiStrOpt, this type of config option seems cannot handled by crudini. Is there any way migrate this type to ListOpt and keep backward compatibility?09:03
*** shardy_ has quit IRC09:04
openstackgerritMerged openstack/tooz: Expose memcache coord. class constants  https://review.openstack.org/17996309:05
*** shardy has joined #openstack-oslo09:05
*** rushiagr is now known as rushiagr_away09:11
*** cdent has joined #openstack-oslo09:16
openstackgerritMehdi Abaakouk (sileht) proposed openstack/oslo.messaging: rabbit: fixes a logging issue  https://review.openstack.org/18045309:17
*** _amrith_ is now known as amrith09:18
*** e0ne has joined #openstack-oslo09:25
*** pblaho has joined #openstack-oslo09:25
*** ozamiatin has quit IRC09:29
*** ozamiatin has joined #openstack-oslo09:29
*** stpierre__ has joined #openstack-oslo09:31
*** stpierre_ has quit IRC09:34
*** inc0 has joined #openstack-oslo09:35
*** mfedosin has joined #openstack-oslo09:41
*** ozamiatin has quit IRC09:49
*** yassine has quit IRC09:55
*** dulek has quit IRC09:59
*** e0ne is now known as e0ne_09:59
openstackgerritMerged openstack/tooz: Expose redis client class constants  https://review.openstack.org/17955109:59
*** dulek has joined #openstack-oslo10:03
*** e0ne_ is now known as e0ne10:05
*** dims has joined #openstack-oslo10:07
*** rushiagr_away is now known as rushiagr10:15
*** dulek has quit IRC10:18
*** dulek has joined #openstack-oslo10:20
*** arnaud____ has joined #openstack-oslo10:39
*** dulek has quit IRC10:40
*** arnaud____ has quit IRC10:44
*** dulek has joined #openstack-oslo10:44
*** yassine has joined #openstack-oslo11:02
*** dulek has quit IRC11:03
*** dulek has joined #openstack-oslo11:03
*** dulek has quit IRC11:11
*** dulek has joined #openstack-oslo11:12
*** boris-42 has joined #openstack-oslo11:27
*** yamahata has quit IRC11:30
*** ozamiatin has joined #openstack-oslo11:30
*** stpierre__ is now known as stpierre11:31
*** stpierre has joined #openstack-oslo11:31
*** yassine has quit IRC11:40
*** yassine has joined #openstack-oslo11:41
*** yassine has joined #openstack-oslo11:41
*** rakhmerov has joined #openstack-oslo11:43
rakhmerovhi, can anybody consult me on pbr?11:43
*** shardy_ has joined #openstack-oslo11:45
*** shardy has quit IRC11:46
*** shardy_ has quit IRC11:50
*** shardy has joined #openstack-oslo11:51
*** alexpilotti has joined #openstack-oslo11:57
*** kgiusti has joined #openstack-oslo11:59
*** bknudson has joined #openstack-oslo12:04
*** sdake has joined #openstack-oslo12:07
*** sdake_ has quit IRC12:12
*** e0ne is now known as e0ne_12:22
*** e0ne_ is now known as e0ne12:29
viktorssileht: hi!12:30
silehtviktors, hi12:31
silehtlets me bet, oslo.msg ?12:31
viktorssileht: wow! :)12:31
*** zqfan is now known as zqfan_afk12:31
sileht:)12:31
viktorsyes, I wanted to ask you about amqpdriver -  why do we need to call self._send_reply() twice at https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/_drivers/amqpdriver.py#L74-L75 ?12:33
silehtviktors, good questions I have no clue, it looks like we don't trust the underlying driver and send a message to be sure or something like that12:34
viktorshm, ok12:34
viktorssileht: also I see an old abandoned patch, which removed second _send_reply() call - https://review.openstack.org/#/c/112669/12:34
viktorsis it a sense to continue this work?12:35
silehtviktors, no12:35
viktorssileht, why?12:36
silehtviktors, the ReplyWaiters have been updated to use a thread instead of wierd code we have before12:36
silehtviktors, but we can remove the second call of _send_reply I thinks12:36
viktorssileht: I'm just trying to get the reason of this call at first12:37
silehtviktors, i means using replacing the thread that handle replies by futures is a waste of time, remove the second _send_reply is not, if we could figure why that exists12:37
viktorssileht: ok, I'll try to instigate this12:38
viktorscan I ask someone else about this?12:38
silehtviktors, the only guys that have worked on that is markmc, but it have not work on openstack code since a while now, so  no really :(12:39
silehtviktors, only 'git log' and me are your friends12:40
viktorssileht: got it, thanks!12:41
*** ozamiatin has quit IRC12:41
*** ozamiatin has joined #openstack-oslo12:42
rakhmerovguys, do you know who is the best person to talk to on pbr?12:44
*** ozamiatin has quit IRC12:45
*** ozamiatin has joined #openstack-oslo12:45
silehtviktors, I got it I think, we can remove it !12:52
viktorssileht: that's cool!12:52
viktorssileht: I'll land a patch, if you don't mind12:52
*** dims has quit IRC12:52
*** openstackgerrit has quit IRC12:53
*** openstackgerrit has joined #openstack-oslo12:53
silehtviktors, https://github.com/openstack/oslo-incubator/blob/stable/icehouse/openstack/common/rpc/amqp.py#L46412:53
*** dims has joined #openstack-oslo12:53
silehtviktors, yeah that old :p12:53
viktorsbrrr...12:54
silehtviktors, before 'reply' allows generator as payload to send multiple messages on one 'rpc.call'12:54
viktorsaha12:54
*** links has quit IRC12:54
silehtviktors, but this is no more possible12:54
silehtviktors, I think we need that for backward compat12:55
silehtviktors, so if you want to remove it, that will be long tasks, with deprecation stuff and oc :)12:55
viktorssileht: so another projects expect for a such behavior? I supposed, that this is an internal stuff...12:57
*** gordc has joined #openstack-oslo12:58
silehtviktors, but if we remove it we must handle the case of the previous oslo.msg version talk with the new one12:58
* viktors thinking...12:59
silehtviktors, that looks more tricky that I have though at the first glance :(12:59
viktorssileht: anyway, IMO it may be a good idea - to reduce amount of RPC calls13:01
silehtviktors, sure13:01
*** rushiagr is now known as rushiagr_away13:02
viktorssileht: so I'll try and ping you, when I'll have some results13:02
*** rushiagr_away is now known as rushiagr13:02
silehtviktors, cool thanks a lot !, it's not everyday that people try to dig in oslo.msg :p13:03
viktorssileht: :)13:03
*** rushiagr is now known as rushiagr_away13:06
*** e0ne is now known as e0ne_13:19
*** e0ne_ is now known as e0ne13:21
*** mriedem has joined #openstack-oslo13:34
openstackgerritVictor Sergeyev proposed openstack/oslo.messaging: WIP: Remove second _send_reply() call  https://review.openstack.org/18054213:34
*** sigmavirus24_awa is now known as sigmavirus2413:35
*** openstackgerrit has quit IRC13:38
*** openstackgerrit has joined #openstack-oslo13:38
*** lifeless has quit IRC13:42
*** sdake_ has joined #openstack-oslo13:42
*** sdake__ has joined #openstack-oslo13:43
openstackgerritMerged openstack/tooz: Use a serialization/deserialization specific exception  https://review.openstack.org/18027513:44
*** sdake has quit IRC13:44
*** mriedem has left #openstack-oslo13:46
*** mriedem has joined #openstack-oslo13:47
*** sdake_ has quit IRC13:48
openstackgerritSean McGinnis proposed openstack/oslo.i18n: Misplaced parenthesis causing confusion  https://review.openstack.org/18055013:51
*** inc0 has quit IRC13:52
*** dulek has quit IRC14:04
*** shakamunyi has joined #openstack-oslo14:09
*** lifeless has joined #openstack-oslo14:10
*** shakamunyi has quit IRC14:13
openstackgerritGrzegorz Grasza (xek) proposed openstack/oslo.versionedobjects: Get the indirection_api from the current VersionedObject class  https://review.openstack.org/18056014:13
openstackgerritVictor Sergeyev proposed openstack/oslo.messaging: Add RequestContextSerializer  https://review.openstack.org/17977214:14
openstackgerritVictor Sergeyev proposed openstack/oslo.messaging: Add RequestContextSerializer  https://review.openstack.org/17977214:16
*** inc0 has joined #openstack-oslo14:18
*** rushiagr_away is now known as rushiagr14:21
*** nkrinner has quit IRC14:26
*** e0ne is now known as e0ne_14:27
openstackgerritMerged openstack/oslo.messaging: rabbit/qpid: simplify the consumer loop  https://review.openstack.org/17896114:34
*** shakamunyi has joined #openstack-oslo14:35
*** shakamunyi has quit IRC14:36
*** shakamunyi has joined #openstack-oslo14:36
*** stevemar has joined #openstack-oslo14:37
*** e0ne_ is now known as e0ne14:37
*** dulek has joined #openstack-oslo14:47
*** browne has joined #openstack-oslo14:48
*** bknudson has quit IRC14:55
*** zzzeek has joined #openstack-oslo15:06
*** sputnik13 has joined #openstack-oslo15:12
*** sdake has joined #openstack-oslo15:14
*** sdake__ has quit IRC15:17
*** tsekiyama has joined #openstack-oslo15:18
*** ozamiatin has quit IRC15:21
*** exploreshaifali has joined #openstack-oslo15:22
*** sputnik13 has quit IRC15:24
viktorssileht: still around?15:24
silehtviktors, yes15:25
viktorssileht: is there any example with deprecation stuff in messaging?15:25
silehtviktors, we have some stuffs in localcontext, but I don't think the is a good example15:26
viktorsit doesn't seems to be, that we can easy deprecate something :(15:27
silehtviktors, we don't have any example of deprecation of this kind of stuffs15:27
openstackgerritVictor Sergeyev proposed openstack/oslo.messaging: Send a single reply to call message  https://review.openstack.org/18058315:36
viktorssileht: at the moment, there is only one-way compatibility there ^15:36
*** openstackgerrit has quit IRC15:37
*** openstackgerrit has joined #openstack-oslo15:37
silehtviktors, I will test that tomorrow15:38
silehtviktors, I don't think we should support the old generator stuffs, but that must be discuted in meeting to be sure15:38
silehtviktors, I will just test compat pre <-> post this patch15:39
silehtviktors, always sending ending looks a good idea15:39
viktorssileht: but the old _process_reply() will parse reply with `ending=True` incorrectly15:40
silehtviktors, just see that15:41
*** inc0 has quit IRC15:42
silehtviktors, unfortunatly I don't think we can remove the double send now15:42
* viktors snips15:42
*** browne has quit IRC15:43
silehtviktors, fixing the receving part is a good start15:43
viktorssileht: and remove the double send in the next release?15:45
silehtviktors, I have some other changes that very to get working with backward compat15:45
*** yamahata has joined #openstack-oslo15:45
silehtvery/very hard15:46
*** bknudson has joined #openstack-oslo15:46
silehtbecause I want to change some attributes on rabbit queue15:47
silehtwe need to find something for this kind of low level amqp design change15:47
viktorssileht: maybe we can add a `version` to rpc-message to know, how can we proceed it?15:49
silehtviktors, I think this is a good idea, we need to check with dims , if we can do that in L+1 or if we must wait L+215:49
silehtviktors, we already have a version15:49
silehtviktors, used for the payload format15:49
viktorssileht: oh, I'm still not familiar with all msg stuff15:49
silehtviktors, perhaps we can extend it for that kind of stuffs15:49
silehtfor your case that can works15:50
silehtfor mine (queue attribute change), infortunatly no15:50
*** ashleighfarnham has joined #openstack-oslo15:51
viktorssileht, is there any best practice for a such cases in openstack?15:51
viktorsversionutils should handle this, afaik15:51
viktorssileht: anyway, thanks a lot for your help, I'll continue this work tomorrow with a fresh head :)15:54
*** viktors is now known as viktors|afk15:54
silehtviktors, cool thanks for helping on that15:54
dimssileht: write it up first? then we can decide the timeline15:57
silehtdims, the fix ?15:58
silehtdims, I means the deprecation patch  ?15:58
*** shakamunyi has quit IRC15:58
dimssileht: slowly reading the chatter and saw "design change"15:59
dimssileht: what exactly do we need to change?16:00
silehtdims, the idea is to remove useless amqp messages, but because of a oddity in the current code, this backward incompatible16:00
silehtdims, so we need to fix the oddity, wait some version and then remove the useless message16:00
openstackgerritAndrey Kurilin proposed openstack/oslo.db: WIP: add requirements checker  https://review.openstack.org/18059216:01
silehtdims, I just don't known how many time we need to wait16:01
silehtI think two releases is safer, but this so long ........16:01
*** vilobhmm1 has joined #openstack-oslo16:04
*** yassine has quit IRC16:07
openstackgerritOpenStack Proposal Bot proposed openstack/oslo-incubator: Updated from global requirements  https://review.openstack.org/18060216:08
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.log: Updated from global requirements  https://review.openstack.org/17842216:08
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements  https://review.openstack.org/18060316:08
*** bknudson has quit IRC16:09
dimssileht: can you and viktors|afk add a blueprint at least so we can point it to people and debate?16:10
silehtdims, ok16:11
dimssileht: thanks16:11
*** bknudson has joined #openstack-oslo16:14
*** arnaud____ has joined #openstack-oslo16:15
*** vilobhmm1 has quit IRC16:23
*** dulek has quit IRC16:29
*** sputnik13 has joined #openstack-oslo16:37
*** browne has joined #openstack-oslo16:41
*** e0ne has quit IRC16:43
*** shakamunyi has joined #openstack-oslo16:49
*** arnaud____ has quit IRC16:52
*** vilobhmm1 has joined #openstack-oslo16:53
*** crc32 has joined #openstack-oslo16:55
*** gordc has quit IRC16:57
*** sdake_ has joined #openstack-oslo16:57
openstackgerritDoug Hellmann proposed openstack/oslo-incubator: Add svg files for team mascot  https://review.openstack.org/18062717:00
*** sdake has quit IRC17:00
dhellmanndims, bnemec, jd__, sileht, harlowja: ^^17:01
dhellmannviktors|afk, rpodolyaka, haypo: ^^17:01
* bnemec doesn't speak svg well :-)17:02
dimsbnemec: open in a browser :)17:03
dhellmannbnemec: click the links in the commit message17:03
bnemecYeah, actually my browser is doing the right thing if I just click the download link on the diff page.17:04
dhellmannah, cool17:04
dimsanyone remember vrml? cosmoplayer?17:05
*** shakamunyi has quit IRC17:05
bnemecVaguely17:06
dhellmannoh, vrml! Man, I haven't heard that one in ages.17:07
dimsdhellmann: bnemec: the only copy of the source code for cosmoplayer ended up on an unplugged irix box under my teammate's desk back in Computer Associates :)17:08
dimsa long time ago17:08
dhellmanndims: that's epic17:08
bnemecNice :-)17:08
*** sdake has joined #openstack-oslo17:10
*** yamahata has quit IRC17:12
*** sdake_ has quit IRC17:14
*** sdake__ has joined #openstack-oslo17:14
*** achanda has joined #openstack-oslo17:17
dhellmanndims: we need a name for the mascot ;-)17:17
*** sdake has quit IRC17:18
*** shakamunyi has joined #openstack-oslo17:19
dimsha!17:22
dimsanyone have ideas?17:22
*** shardy has quit IRC17:22
openstackgerritVilobh Meshram proposed openstack/tooz: Expose Zookeeper client class constants  https://review.openstack.org/18063917:23
dimsdhellmann: "Bucky" :)17:24
*** exploreshaifali has quit IRC17:27
*** boris-42 has quit IRC17:28
dimsBullwinkle?17:29
* dims is learning a lot about moose today!17:35
*** alexpilotti has quit IRC17:41
openstackgerritJoshua Harlow proposed openstack/taskflow: Ensure we allow unseen UNCLAIMED -> COMPLETE job transition  https://review.openstack.org/18065117:44
openstackgerritJoshua Harlow proposed openstack/taskflow: Ensure we allow unseen UNCLAIMED -> COMPLETE job transition  https://review.openstack.org/18065117:47
*** e0ne has joined #openstack-oslo17:50
*** alexpilotti has joined #openstack-oslo17:53
*** kgiusti has quit IRC17:53
openstackgerritMerged openstack/oslo-incubator: Updated from global requirements  https://review.openstack.org/18060217:57
openstackgerritJoshua Harlow proposed openstack/taskflow: Ensure we allow unseen UNCLAIMED -> COMPLETE job transition  https://review.openstack.org/18065117:57
*** browne has quit IRC17:58
*** browne has joined #openstack-oslo18:00
openstackgerritJoshua Harlow proposed openstack/taskflow: Ensure we allow unseen UNCLAIMED -> COMPLETE job transition  https://review.openstack.org/18065118:01
openstackgerritDoug Hellmann proposed openstack/oslo-incubator: Add svg files for team mascot  https://review.openstack.org/18062718:07
dimsharlowja: need a name for the mascot ^^18:09
dhellmanndims: markus moose? http://www.nordicnames.de/wiki/Norwegian_Statistics_of_201418:11
dhellmannmagnus is on that list, too18:11
harlowjahttps://review.openstack.org/cat/180627%2C1%2Cdoc/source/_images/oslo-moose-color.svg%5E018:14
harlowjaniceeee18:14
harlowjahaha18:14
* harlowja needs to learn how to draw18:14
dimshow about "Odin"?18:14
harlowjalooks good to me +20018:14
harlowjavrml, a little, not so much18:15
*** e0ne_ has joined #openstack-oslo18:15
*** e0ne has quit IRC18:15
harlowjahaha18:15
dimsmarkus, magnus, odin, bullwinkle, bucky...so far18:16
dhellmannI like the alliteration of the m names, but odin is a good suggestion, too18:17
*** e0ne has joined #openstack-oslo18:17
dimssome more m* names http://www.behindthename.com/names/usage/norwegian/218:18
dimsi like markus18:18
harlowja"Doug"18:19
harlowjalol18:19
harlowja^ might be a little awkward, lol18:19
* harlowja must not cough up coffee due to laughing, lol18:19
dimslol18:19
*** e0ne_ has quit IRC18:20
*** sdake has joined #openstack-oslo18:20
*** sdake_ has joined #openstack-oslo18:23
*** rushiagr is now known as rushiagr_away18:23
*** sdake__ has quit IRC18:24
*** sdake has quit IRC18:26
elmikohey all, i'm curious about best practices with regards to oslo.config usage in a library type package18:27
harlowjahmmm18:27
harlowjalet me consult the person that makes good names18:27
* harlowja let me see if he's online in internal y! irc18:27
harlowjawasn't that from gladiator, lol18:28
elmikobasically, i'm using a package that is adding a config option to the global CONF object and i want to change the default in our usage. is there any advice about this?18:28
harlowja^ nothing wrong with that btw18:28
harlowjadims i'll get back to u with some names i can gather from y! folks (that don't seem to go on public irc that much)18:28
harlowjapeople that make better names than me :-P18:28
harlowjacan we than make a movie out of him?18:28
harlowjai'll be odin?18:28
harlowjadims u can be thor18:28
harlowjaor maybe u should be odin18:28
harlowjaidk18:28
harlowjahaha18:28
dimslove it!18:28
harlowjaseems fine with me18:29
harlowjaalthough bucky is cool to18:29
harlowjau could really confuse people and call it rudolf (the moose)18:29
harlowjainstead of rudolf the reindeer18:29
harlowja*rudolph (guess its spelled that way offically)18:29
harlowjaput a red nose on the moose and profit18:30
elmikoside question, is it appropriate for libs to register_opts themselves or is it preferrable for them to create list_opts style functions that can be added by consumers through an entry_point?18:30
harlowjahmmm18:31
harlowjaelmiko thats an interesting question, can u describe the library, and how u are thinking of using it18:31
harlowjaand how widely it will be used (a guess is ok)18:31
elmikoharlowja: this is the castellan library i'm talking about18:31
harlowjaelmiko ideally the library/package u are using wouldn't be using the global CONF object (but would at least take a conf object as a __init__ param)18:32
elmikoit basically has a package that registers a ConfigOpt into the global CONF on import18:32
*** mtanino has joined #openstack-oslo18:32
harlowjathat way u can on usage of that library/package provide a conf object that u have created/configured/tweaked as u want18:32
harlowja*imho*18:32
harlowjaelmiko i think the preferrable way is entrypoint18:32
harlowjakk, looking at18:32
elmikoyea, that's kinda what i thought too18:32
* harlowja didn't know about that one, ha18:32
elmikokinda new18:32
harlowjafor https://github.com/openstack/castellan/blob/master/castellan/context.py#L45 ?18:32
harlowjais that the only real need for that?18:33
elmikono, for the catellan.key_manager.__init__18:33
*** yamahata has joined #openstack-oslo18:33
harlowjaah i see https://github.com/openstack/castellan/blob/master/castellan/key_manager/__init__.py#L1818:33
elmikohttps://review.openstack.org/#/c/171918/ look at that too18:33
harlowjahmmm18:33
harlowjawhy not just use full on entrypoints instead18:33
elmikoespecially the changes to __init__18:33
harlowjaaka, is oslo.config really needed at all?18:34
elmikothat's what i'm thinking, but i wanted to do all the due dilligence before i start making proposals18:34
harlowjadepends on really how widely distributed u want this library to be imho, and how tight of integration u want with oslo config settings18:34
harlowjathis one doesn't really seem to need that18:34
elmikowhat i'm most concerned about is the library setting options that clients might want to override, for things like running oslo-config-generator18:35
harlowjafor example u could have https://github.com/openstack/taskflow/blob/master/taskflow/persistence/backends/__init__.py#L26 (which uses stevedore + namespaces)18:35
elmikowell, it will be a library to help with integrating an openstack key manager service18:35
elmikoharlowja: thanks, i think you've confirmed some of what i was wondering about18:36
elmikoand good luck with the moose naming ;)18:36
elmiko(it looks awesome btw)18:37
harlowjaelmiko so my thought (not everyone agress with me here) is that you make an API (base class) and have your key manager code/thing fetch a key manager object using regular entrypoints, if those things that your entrypoint fetches use oslo.config thats there purgative (not yours) right?18:37
harlowjasame could be done with the other usage of oslo.config @ https://github.com/openstack/castellan/blob/master/castellan/context.py#L45 (fetch it from an entrypoint, or require it to be provided, if whoever constructs that object wants to use oslo.config, have at it)18:38
elmikoyea, that makes sense. i think they want to be able to have the lib configured via the normal conf file that a project might use. akin to how keystonemiddleware has options in a projcet.18:38
elmiko(for example)18:39
harlowjaright, i think u can make both work, if u want, since castellan is more tightly coupled with openstack either path imho seems ok18:40
elmikook, thanks!18:41
elmikoi'm pretty sure we need to keep the config in the conf files, so that might make the entry_point changes a little less desirable18:41
harlowjaelmiko i start to feel that stuff like barbican_opts in that review should just be __init__ arguments (then u can have docstrings that describe there types, defaults...)18:41
harlowjabut maybe thats to radical, idk18:42
elmikoyea i dunno, these options for the api_class and related would only be set once on the conf file for each deployment and i'm not immediately sure on the best approach for providing all these config options but also giving end-users the ability to define in their conf files.18:43
elmikoi mean, the lib has to provide the ConfigOpts, but the client would need to send them to the __init__18:44
harlowjaya, that's to me where it becomes not really a generic library, but an openstack semi-library18:44
harlowja^ which might be fine with castellan18:44
elmikoit's definitely openstack specific18:44
harlowjak, then its prob fine18:44
elmikoso, would it be acceptable in that guise to create a list_opts style function that consumers could incorporate in their entry_points to help inform oslo.config?18:45
harlowjathinks so18:45
elmikocool, thanks again for the advice =)18:46
harlowjanp18:46
*** openstackgerrit has quit IRC18:47
*** tsufiev has quit IRC18:48
*** amrith has quit IRC18:48
*** yamahata has quit IRC18:48
*** e0ne has quit IRC18:48
*** browne has quit IRC18:48
*** rpodolyaka1 has quit IRC18:48
*** pasquier-s has quit IRC18:48
*** morganfainberg has quit IRC18:48
*** vipul has quit IRC18:48
*** mriedem has quit IRC18:48
*** hogepodge has quit IRC18:48
*** yamahata has joined #openstack-oslo18:49
*** e0ne has joined #openstack-oslo18:49
*** browne has joined #openstack-oslo18:49
*** mriedem has joined #openstack-oslo18:49
*** rpodolyaka1 has joined #openstack-oslo18:49
*** hogepodge has joined #openstack-oslo18:49
*** vipul has joined #openstack-oslo18:49
*** morganfainberg has joined #openstack-oslo18:49
*** pasquier-s has joined #openstack-oslo18:49
*** openstackgerrit has joined #openstack-oslo18:49
*** tsufiev has joined #openstack-oslo18:49
*** amrith has joined #openstack-oslo18:49
dhellmannelmiko, harlowja : I think you're mixing up the use case for the list_opts entry point and registering options18:50
harlowjapossibly :-P18:50
dhellmannlist_opts is really just meant for the config generator, since we don't want to scan source files any more18:50
dhellmannelmiko, harlowja : code that needs options should register those options itself, at runtime, using a config object passed to it18:51
dhellmannalthough if you have a way to define the library without needing config options, that's fine, too18:51
elmikodhellmann: ok, so better if we pass a config to the library and let it register?18:51
dhellmannelmiko: yes, that's what we did in oslo.db and oslo.messaging, for example18:51
elmikook, great. i'll study those too18:51
dhellmannas harlowja said, for a lib meant to be used outside of openstack you want to avoid that pattern, but that's what we do inside the project where we know oslo.config is being used18:52
elmikodhellmann: what about if we want to change a default that the library is defining in a ConfigOpt, is there a good pattern for this?18:52
dhellmannelmiko: the library defines a default and the app wants to override it?18:52
elmikowell, that's what is being proposed, but i think the lib should not define a default18:52
dhellmannelmiko: we have a few set_defaults() functions in some of the oslo libraries that deal with that case. I think there's one in oslo.log, for example18:52
elmikooh, cool!18:53
elmikothe tough part is that the lib needs to define some config options, but a client might want to override a default. for example, when creating a sample config with oslo-config-generator18:53
elmikoi'm kinda curious if we have any patterns for this type of behavior18:54
dhellmannelmiko: if you have different defaults in all of the users of the library, it sounds like the library doesn't own that configuration setting18:55
dhellmannmaybe the value should be passed to the library, instead?18:55
elmikogood point18:55
openstackgerritMerged openstack/oslo.log: Updated from global requirements  https://review.openstack.org/17842218:55
elmikodhellmann, harlowja, thanks again. i've got some thinking to do lol18:56
harlowjanp, hopefully didn't confuse u to much :-P18:56
dhellmannelmiko: sure thing, and drop by again if you want to talk more18:56
elmikocool, i will. and it's not confusing, i think it's just an odd problem for the case i want.18:56
harlowjak18:57
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.cache: [WIP] just a test review - please ignore  https://review.openstack.org/18067518:58
openstackgerritJoshua Harlow proposed openstack/taskflow: Add speed-test tools script  https://review.openstack.org/18067818:59
harlowjadims ok got a good one from internal, another confusing/tricky one, lol18:59
harlowja'name it "teddy" (after Teddy Roosevelt, founder of the Bull Moose Party)'18:59
harlowjahaha18:59
harlowjawe can than say we didn't know that teddy was usually connected with bears, lol19:00
dimshehe19:00
* harlowja might have to soon change all those badge links (showing pypi downloads) to something else19:02
harlowjahttps://github.com/badges/pypipins/issues/37 doesn't seem to be getting fixed...19:02
harlowjashield.io seems to do the same thing (and isn't dead/down)19:03
harlowja* http://shields.io/19:03
* harlowja will wait a little longer on that one19:04
*** gordc has joined #openstack-oslo19:06
*** achanda has quit IRC19:07
*** ndipanov has quit IRC19:11
*** kgiusti has joined #openstack-oslo19:14
*** boris-42 has joined #openstack-oslo19:14
*** zul has quit IRC19:14
*** zul has joined #openstack-oslo19:16
*** salv-orl_ has joined #openstack-oslo19:16
*** yamahata has quit IRC19:18
*** e0ne has quit IRC19:18
*** browne has quit IRC19:18
*** rpodolyaka1 has quit IRC19:18
*** pasquier-s has quit IRC19:18
*** morganfainberg has quit IRC19:18
*** vipul has quit IRC19:18
*** yamahata has joined #openstack-oslo19:19
*** e0ne has joined #openstack-oslo19:19
*** browne has joined #openstack-oslo19:19
*** rpodolyaka1 has joined #openstack-oslo19:19
*** vipul has joined #openstack-oslo19:19
*** morganfainberg has joined #openstack-oslo19:19
*** pasquier-s has joined #openstack-oslo19:19
*** salv-orlando has quit IRC19:19
*** morganfainberg has quit IRC19:20
*** bogdando has quit IRC19:21
*** morganfainberg has joined #openstack-oslo19:21
openstackgerritMerged openstack/oslo.policy: Updated from global requirements  https://review.openstack.org/17842419:29
harlowjabtw dims salv-orl_ dhellmann bnemec i put a big braindump on https://review.openstack.org/#/c/132127/ (might make sense to go that route, idk)19:32
* harlowja doesn't want the quota stuff to stall out again :(19:33
harlowja^if u get some time to think about it19:33
dimsharlowja: ack, will do. reminds me to start etherpads too19:33
dhellmannharlowja: the issue with having a lib own part of a database schema is getting the migration scripts to run exactly once19:34
harlowjai got one about it to19:34
harlowjaha19:34
harlowjawell seperate the database schema part to a 'pluggable backend'19:35
harlowjahave the in-tree-library one just be a 'in-memory' version19:35
harlowjahave nova provide its own 'backend' (and later consulate them all)19:36
harlowja^ if that can be done19:36
*** sdake has joined #openstack-oslo19:36
dhellmannwe still need to solve the problem of actually running the migrations, though, and that's the thing that started turning into an ops issue that led us to just follow the pattern of having an application own the things19:36
dhellmannthat's not to say there aren't other ways, but it's more consistent with the way other databases are managed, so...19:36
harlowjaya, but i'm wondering if we can simulatenously do both, work on the library (assuming/using in-memory backend) and let another group of people work on 'database backend' + solve/figureout/document migrations19:37
dhellmannmaybe? I'm not sure the library would be seen as useful until the second part is solved, so we might as well figure that out first19:38
dhellmannof course my solution to that is to not do it, so I may not be the right person to drive the work :-)19:39
*** sdake_ has quit IRC19:39
harlowjaother thing that i wonder is why does this backend have to be tied to a DB with schema, doesn't seem like it does (a no-sql consistent backend, without schemas, as long as its transcational seems like it would work just fine)19:41
harlowjaand zookeeper afaik is one of those :-P19:42
dhellmannharlowja: selling operators on yet another service is difficult19:42
dhellmannis zookeeper persistent?19:42
harlowjayup19:42
dhellmannk19:42
harlowjaand transcational19:42
dimsharlowja: dhellmann: "replicated synchronization service with eventual consistency"19:43
harlowjahmmmm, zookeeper shouldn't be eventual19:43
dhellmannthat doesn't sound inspiring for quota management19:43
harlowja+119:43
* harlowja doesn't like eventual consistent things that cost $$19:44
harlowja*and/or impact $$19:44
dimshttp://zookeeper.apache.org/doc/trunk/zookeeperProgrammers.html#ch_zkGuarantees19:44
*** sdake_ has joined #openstack-oslo19:44
dimsguessing we are talking about "Simultaneously Consistent Cross-Client Views"19:44
*** ajo has quit IRC19:44
harlowja'gets notified of the change'19:45
harlowjathats watches, slightly different19:45
harlowjaoh, nm, reading19:45
harlowjaguess just have to call sync() then19:46
*** sdake has quit IRC19:47
harlowjai guess thats because when a leader commits a value, it ensures that a quoroum of other servers also have commited it, and the server u may be connected to (the B in that case) may be connected to one that isn't in that quorum (and that server Z will get updated to that value soon when it learns it needs to update); http://zookeeper-user.578899.n2.nabble.com/Consistency-in-zookeeper-td7578531.html (has info about this topic)19:52
*** arnaud____ has joined #openstack-oslo19:53
harlowjabetter than mongodb i guess for quotas :-P19:54
harlowjahttps://aphyr.com/posts/322-call-me-maybe-mongodb-stale-reads (recent)19:55
harlowja^ scares me, lol19:55
dimsharlowja: we should all use DB2 :)19:55
harlowja:)19:55
harlowjaor pay oracle or something19:56
harlowjai'm also a core on https://github.com/python-zk/kazoo so we should use it for quotas, haha19:57
*** arnaud____ has quit IRC19:57
* harlowja isn't biased or anything, hahaha19:59
*** browne has quit IRC20:03
harlowjaanyways, something to think about20:06
*** pabelanger has joined #openstack-oslo20:06
pabelangergreetings20:06
pabelangerrunning into an issue with oslo-config-generator: see http://pastebin.com/U0msTnGi20:07
pabelangernote the section naming20:07
pabelangerhere is the my code: http://pastebin.com/KXQwCiQe20:07
*** browne has joined #openstack-oslo20:10
*** yamahata has quit IRC20:14
harlowjadims http://paste.openstack.org/show/215514/ also happens (u can ensure that u have read the latest i think via this mechanism to; since writes go to the leader, who will have a different version... and blowup at the client)20:15
harlowjaretries and stuff can then be done with that bad version exception (if so desired)20:16
harlowjathen u don't need to sync all the time20:17
*** e0ne has quit IRC20:26
*** browne has quit IRC20:27
*** browne has joined #openstack-oslo20:28
*** e0ne has joined #openstack-oslo20:34
*** browne has quit IRC20:35
*** bogdando has joined #openstack-oslo20:36
pabelangerYa, looks like a bug20:37
*** achanda has joined #openstack-oslo20:38
*** browne has joined #openstack-oslo20:41
*** browne has quit IRC20:42
*** browne has joined #openstack-oslo20:42
openstackgerritPaul Belanger proposed openstack/oslo.config: Fix issue when generating code with cfg.OptGroup  https://review.openstack.org/18073520:42
pabelangerI _think_ that will fix the problem20:43
pabelangertest coverage none the less20:43
*** browne1 has joined #openstack-oslo20:45
*** achanda has quit IRC20:45
*** browne has quit IRC20:45
*** e0ne has quit IRC20:45
*** pabelanger has quit IRC20:48
*** browne has joined #openstack-oslo20:54
*** browne1 has quit IRC20:57
*** Rockyg has joined #openstack-oslo21:13
*** sdake has joined #openstack-oslo21:14
*** sdake_ has quit IRC21:18
*** jgrimm_ has quit IRC21:22
*** jgrimm has joined #openstack-oslo21:23
*** david-lyle has quit IRC21:29
*** kgiusti has left #openstack-oslo21:31
*** cdent has quit IRC21:33
openstackgerritJoshua Harlow proposed openstack/tooz: Move more string constants to class constants with docstrings  https://review.openstack.org/18076421:45
*** achanda has joined #openstack-oslo21:46
*** alexpilotti has quit IRC21:46
*** achanda has quit IRC21:51
harlowjasileht https://review.openstack.org/#/q/status:open+project:openstack/oslo.messaging+branch:master+topic:sileht/refactor,n,z woah, ha21:52
*** sdake has quit IRC21:53
*** sdake has joined #openstack-oslo21:53
*** stevemar has quit IRC21:57
*** stpierre has quit IRC22:00
*** vilobhmm1 has quit IRC22:01
*** achanda has joined #openstack-oslo22:01
*** achanda has quit IRC22:02
*** vilobhmm1 has joined #openstack-oslo22:02
*** Kennan2 has quit IRC22:02
*** achanda has joined #openstack-oslo22:03
*** Kennan has joined #openstack-oslo22:03
*** jgrimm has quit IRC22:07
*** gordc has quit IRC22:07
*** jgrimm has joined #openstack-oslo22:09
*** sdake has quit IRC22:17
*** sdake has joined #openstack-oslo22:17
*** sdake has quit IRC22:17
*** sdake has joined #openstack-oslo22:17
*** jaosorior has quit IRC22:22
openstackgerritMerged openstack/oslo.log: Change misleading TRACE to ERROR  https://review.openstack.org/16653722:36
*** salv-orl_ has quit IRC22:38
*** salv-orlando has joined #openstack-oslo22:40
*** bknudson has quit IRC22:58
*** browne1 has joined #openstack-oslo23:02
*** browne has quit IRC23:05
*** Rockyg has quit IRC23:32
*** dims_ has joined #openstack-oslo23:36
*** dims has quit IRC23:38
*** david-lyle has joined #openstack-oslo23:47

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