Friday, 2014-09-26

*** bknudson has quit IRC00:01
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Document more function/class/method params  https://review.openstack.org/12422800:02
*** oomichi has joined #openstack-oslo00:03
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Document more function/class/method params  https://review.openstack.org/12422800:05
*** boden has joined #openstack-oslo00:10
*** boden has quit IRC00:15
*** tsekiyama has quit IRC00:15
*** mtanino has quit IRC00:17
*** dims has quit IRC00:20
*** dims has joined #openstack-oslo00:21
*** dims has quit IRC00:25
*** rpodolyaka1 has quit IRC00:27
*** rpodolyaka1 has joined #openstack-oslo00:28
*** alexpilotti has quit IRC00:30
*** yamahata_ has joined #openstack-oslo00:41
*** alexpilotti has joined #openstack-oslo00:44
*** alexpilotti has quit IRC00:49
*** yamahata_ has quit IRC00:59
*** dims has joined #openstack-oslo01:07
*** dims_ has joined #openstack-oslo01:09
*** boden has joined #openstack-oslo01:11
*** dims has quit IRC01:13
*** boden has quit IRC01:15
*** praneshp has quit IRC01:15
*** rpodolyaka1 has quit IRC01:21
*** arnaud__ has quit IRC01:23
*** arnaud has quit IRC01:23
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Always return copies from memory backend  https://review.openstack.org/12424201:32
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Always return copies from memory backend  https://review.openstack.org/12424201:35
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.vmware: Switch to use requests/urllib3 and enable cacert validation  https://review.openstack.org/12195601:46
*** HenryG has joined #openstack-oslo01:49
*** praneshp has joined #openstack-oslo02:10
*** zzzeek has quit IRC02:10
*** praneshp_ has joined #openstack-oslo02:11
*** praneshp has quit IRC02:14
*** praneshp_ is now known as praneshp02:14
*** harlowja is now known as harlowja_away02:38
*** yamahata_ has joined #openstack-oslo02:40
*** dims_ has quit IRC03:11
*** arnaud has joined #openstack-oslo03:15
*** alexpilotti has joined #openstack-oslo03:46
*** zhiyan has quit IRC03:54
*** zhiyan has joined #openstack-oslo03:57
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo.messaging: Updated from global requirements  https://review.openstack.org/12426103:59
*** alexpilotti has quit IRC04:00
*** arnaud has quit IRC04:01
*** yamahata_ has quit IRC04:08
*** yamahata_ has joined #openstack-oslo04:08
*** rpodolyaka1 has joined #openstack-oslo04:09
*** dims has joined #openstack-oslo04:12
*** boden has joined #openstack-oslo04:13
*** dims has quit IRC04:17
*** boden has quit IRC04:17
*** exploreshaifali has joined #openstack-oslo04:25
*** takedakn has joined #openstack-oslo04:34
*** harlowja_at_home has joined #openstack-oslo04:36
*** takedakn has quit IRC04:40
*** exploreshaifali has quit IRC04:48
*** rpodolyaka1 has quit IRC05:08
*** boden has joined #openstack-oslo05:14
*** tsekiyama has joined #openstack-oslo05:18
*** boden has quit IRC05:18
*** tsekiyama has quit IRC05:19
*** takedakn has joined #openstack-oslo05:28
*** rpodolyaka1 has joined #openstack-oslo05:33
*** yamahata_ has quit IRC05:43
*** yamahata_ has joined #openstack-oslo05:43
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Always return copies from memory backend  https://review.openstack.org/12424205:45
*** harlowja_at_home has quit IRC05:48
*** cfriesen_ has quit IRC05:59
*** AAzza_afk is now known as AAzza06:00
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo.middleware: Imported Translations from Transifex  https://review.openstack.org/12428006:00
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo.i18n: Imported Translations from Transifex  https://review.openstack.org/12428206:01
*** AAzza is now known as AAzza_afk06:03
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo.db: Imported Translations from Transifex  https://review.openstack.org/12428306:04
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo.log: Imported Translations from Transifex  https://review.openstack.org/12428406:06
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo.vmware: Imported Translations from Transifex  https://review.openstack.org/12428706:08
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo.concurrency: Imported Translations from Transifex  https://review.openstack.org/12428806:08
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo.messaging: Imported Translations from Transifex  https://review.openstack.org/12429306:11
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo.utils: Imported Translations from Transifex  https://review.openstack.org/12429406:11
*** boden has joined #openstack-oslo06:14
*** AAzza_afk is now known as AAzza06:18
*** boden has quit IRC06:19
*** stevemar has quit IRC06:26
*** burgobianco has quit IRC06:29
*** SergeyLukjanov has quit IRC06:29
*** burgobianco has joined #openstack-oslo06:29
*** SergeyLukjanov has joined #openstack-oslo06:35
*** e0ne has joined #openstack-oslo06:36
*** k4n0 has joined #openstack-oslo06:37
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Always return copies from memory backend  https://review.openstack.org/12424206:40
*** e0ne has quit IRC06:41
*** pblaho has joined #openstack-oslo06:43
*** ihrachyshka has joined #openstack-oslo06:59
*** dims has joined #openstack-oslo07:02
*** amotoki has joined #openstack-oslo07:03
*** takedakn has quit IRC07:03
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Always return copies from memory backend  https://review.openstack.org/12424207:05
*** flaper87|afk is now known as flaper8707:06
*** dims has quit IRC07:07
*** jaosorior has joined #openstack-oslo07:08
*** takedakn has joined #openstack-oslo07:11
*** ihrachyshka has quit IRC07:14
*** boris-42 has quit IRC07:15
*** burgobianco has quit IRC07:15
*** boden has joined #openstack-oslo07:15
*** takedakn has quit IRC07:16
*** burgobianco has joined #openstack-oslo07:16
*** rpodolyaka1 has quit IRC07:17
*** boden has quit IRC07:19
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Always return copies from memory backend  https://review.openstack.org/12424207:26
*** ihrachyshka has joined #openstack-oslo07:35
*** boden has joined #openstack-oslo07:56
*** boden has quit IRC08:00
*** yassine has joined #openstack-oslo08:08
*** takedakn has joined #openstack-oslo08:09
*** oomichi has quit IRC08:11
*** takedakn has quit IRC08:16
*** andreykurilin_ has joined #openstack-oslo08:23
openstackgerritIlya Pekelny proposed a change to openstack/oslo.messaging: Required pyzmq for 0mq driver  https://review.openstack.org/12369108:44
openstackgerritIlya Pekelny proposed a change to openstack/oslo.messaging: Reduce global CONF usage in zmq driver  https://review.openstack.org/12371108:44
openstackgerritIlya Pekelny proposed a change to openstack/oslo.messaging: `publisher`function separated into ZmqProxy method  https://review.openstack.org/12409808:44
openstackgerritIlya Pekelny proposed a change to openstack/oslo.messaging: Helper method to check and restrict socket name  https://review.openstack.org/12409908:44
openstackgerritIlya Pekelny proposed a change to openstack/oslo.messaging: Helper method to check and restrict socket name  https://review.openstack.org/12409908:47
*** boden has joined #openstack-oslo08:51
*** e0ne has joined #openstack-oslo09:02
*** pblaho has quit IRC09:04
*** praneshp has quit IRC09:08
*** yamahata_ has quit IRC09:23
openstackgerritA change was merged to openstack/taskflow: Switch to using oslo.utils and oslo.serialization  https://review.openstack.org/12063009:41
*** vigneshvar has joined #openstack-oslo09:54
*** viktors has joined #openstack-oslo09:58
*** exploreshaifali has joined #openstack-oslo10:00
exploreshaifaliQuestion: trying to make glanceclient exception messages into i18n, but getting "Import error for i18n"10:02
exploreshaifaliwhile running ./run_test.sh10:02
exploreshaifalieven though I have installeld oslo.i18n on my laptop10:03
*** pcm_ has joined #openstack-oslo10:09
*** pcm__ has joined #openstack-oslo10:10
*** pcm_ has quit IRC10:13
*** amotoki has quit IRC10:20
*** vigneshvar has quit IRC10:26
*** boden has quit IRC10:32
*** vigneshvar has joined #openstack-oslo10:32
*** AAzza is now known as AAzza_afk10:36
*** dims has joined #openstack-oslo10:37
openstackgerritA change was merged to openstack/taskflow: Expand toctree to three levels  https://review.openstack.org/12295710:40
*** AAzza_afk is now known as AAzza10:47
*** AAzza is now known as AAzza_afk10:53
*** andreykurilin_ has quit IRC10:55
*** e0ne has quit IRC10:57
*** e0ne has joined #openstack-oslo10:58
*** AAzza_afk is now known as AAzza11:05
*** pblaho has joined #openstack-oslo11:06
*** pblaho has quit IRC11:07
*** pblaho has joined #openstack-oslo11:07
*** mkoderer_cloud has joined #openstack-oslo11:26
*** AAzza is now known as AAzza_afk11:26
openstackgerritA change was merged to openstack/oslotest: Handle tempfile content encoding  https://review.openstack.org/12391911:34
*** k4n0 has quit IRC11:46
*** viktors is now known as viktors|afk11:58
*** andreykurilin_ has joined #openstack-oslo11:59
*** boden has joined #openstack-oslo12:06
*** AAzza_afk is now known as AAzza12:11
*** AAzza is now known as AAzza_afk12:18
*** AAzza_afk is now known as AAzza12:30
*** kgiusti has joined #openstack-oslo12:54
*** vigneshvar has quit IRC12:56
*** dulek has joined #openstack-oslo12:57
*** ochuprykov_ has joined #openstack-oslo12:57
dulekGuys, is there someone experienced with oslo notifications?12:57
dulekI'm trying to get simple publisher-subscriber example done.12:57
*** gordc has joined #openstack-oslo12:57
dulekThe message is getting into the Rabbit queue, but notification listener isn't consuming it.12:58
*** dims has quit IRC12:58
dulekI've already checked and it listens on right exchange and queue.12:58
dulekAny ideas?12:58
*** dims has joined #openstack-oslo12:59
dhellmann_dulek: I'm behind on my mailing list reading, have you posted your question to openstack-dev? Most of the messaging team is in europe, so depending on where you are overlap on IRC might be minimal.13:13
dhellmann_exploreshaifali: did you install oslo.i18n into the same virtualenv where the tests are running?13:13
*** dhellmann_ is now known as dhellmann13:13
dulekdhellmann_: Okay, thanks, I'll try a few things more and if I won't succeed, then I'll post question to openstack-dev.13:14
*** tedross has joined #openstack-oslo13:15
dhellmanndulek: you might also want to look at the ceilometer code, I think they're using the oslo.messaging notification subscriber13:15
*** kgiusti has quit IRC13:15
exploreshaifalidhellmann_, yes I am using devstack in VMware, there I installed oslo.i18n and in same env I am running test cases13:16
dhellmannexploreshaifali: ok, the latest devstack should have installed oslo.i18n for you. I'll bet you have the namespace package problem from installing some oslo packages "editable" (the way devstack does) and some from regular pip or system packages13:16
dhellmannexploreshaifali: are you using the latest head from the master branch of devstack?13:18
exploreshaifalidhellmann: yes even though devstasck has already installed it, I tried to reinstall it using "pip install oslo.i18n"13:18
*** kgiusti has joined #openstack-oslo13:19
*** yamahata_ has joined #openstack-oslo13:19
exploreshaifalidhellmann, "git clone https://github.com/openstack-dev/devstack.git" gives latest head13:20
*** mriedem has joined #openstack-oslo13:21
exploreshaifalidhellmann, see https://ask.openstack.org/en/question/48420/import-error-no-module-named-i18n/13:21
*** Kiall has quit IRC13:24
*** kgiusti has quit IRC13:25
openstackgerritMatthew Gilliard proposed a change to openstack/oslo.config: Check config default value is correct type  https://review.openstack.org/10545013:26
*** kgiusti has joined #openstack-oslo13:29
*** kgiusti has joined #openstack-oslo13:29
*** kgiusti has quit IRC13:29
*** kgiusti has joined #openstack-oslo13:30
dhellmannexploreshaifali: reinstalling it with pip is what broke things. Remove the installed version by running "pip uninstall oslo.i18n" until it tells you the package isn't installed (probably at least 2 runs) then install from the checked out source tree with "pip install -e /opt/stack/oslo.i18n"13:34
exploreshaifalidhellmann: ok, Thanks!!13:35
dhellmannexploreshaifali: let me know if that doesn't fix it, and I'll help you debug further, but that should do it13:35
exploreshaifalidhellmann, okay13:36
exploreshaifali:)13:36
*** sigmavirus24_awa is now known as sigmavirus2413:39
openstackgerritA change was merged to openstack/oslo-specs: Remove specs not completed in juno  https://review.openstack.org/12009513:39
*** dulek has quit IRC13:40
dimsdhellmann: oslo.log today? i seem to have 2 things there still - https://review.openstack.org/#/q/status:open+project:openstack/oslo.log,n,z13:41
openstackgerritIlya Pekelny proposed a change to openstack/oslo.messaging: Reduce global CONF usage in zmq driver  https://review.openstack.org/12371113:42
openstackgerritIlya Pekelny proposed a change to openstack/oslo.messaging: `publisher`function separated into ZmqProxy method  https://review.openstack.org/12409813:42
openstackgerritIlya Pekelny proposed a change to openstack/oslo.messaging: Helper method to check and restrict socket name  https://review.openstack.org/12409913:42
dhellmanndims: I'm working on a spec about cleaning up the oslo.log API before we release it13:42
*** mkoderer_cloud has quit IRC13:43
dhellmanndims: you and I are reviewing each other's work -- we need a third person :-)13:43
ihrachyshkadhellmann: hey. do you have 5 mins on oslo.msg question?13:43
dimsdhellmann: yeah!... bnemec, jd__ around?13:43
dhellmannihrachyshka: I can try13:43
*** cfriesen_ has joined #openstack-oslo13:44
ihrachyshkadhellmann: ok, so there is conf.rpc_response_timeout in oslo.msg, and it's used to limit waiting for reply on waiter side, but it's not used when pushing messages in the queue. meaning, they are not purged from queues at all. shouldn't we also set timeout when calling send()?13:44
dhellmannihrachyshka: do you mean set a TTL for the message itself?13:48
ihrachyshkayeah13:49
ihrachyshkawaiters won't fetch them after that timeout, so I think we should recycle those unfetched messages in reasonable time13:49
dhellmannthat's a good question. possibly? we would have to be careful about avoiding races between the TTL of the message and the timeout of the call being made13:49
bnemecdims: dhellmann: Approved the rest of the pending oslo.log changes.13:51
openstackgerritDoug Hellmann proposed a change to openstack/oslo-specs: Replace default bp URLs with portable examples  https://review.openstack.org/12020513:51
openstackgerritDoug Hellmann proposed a change to openstack/oslo-specs: Fix the blueprint URLs in existing specs  https://review.openstack.org/12020313:51
dimsbnemec: thanks!13:52
ihrachyshkadhellmann: do you see place for races here?.. waiter call()ed to us before we send() reply, so waiter will stop waiting before our reply expires in any case13:52
*** zzzeek has joined #openstack-oslo13:54
dhellmannihrachyshka: if the values are the same, and the 2 servers count time slightly differently, the sender could decide a message has timed out when a client picks it up and starts working on it13:54
dhellmannihrachyshka: which I guess is the case we have now already13:54
ihrachyshkadhellmann: ok, how about * 2 for sure? ;)13:55
*** rpodolyaka1 has joined #openstack-oslo13:55
dhellmannihrachyshka: which would you double? :-)13:56
ihrachyshkathe replier side13:56
ihrachyshkaafaik atm we don't set any ttls, so I don't get where we may get the case now13:56
openstackgerritA change was merged to openstack/oslo-specs: remove templates from toctree  https://review.openstack.org/12019213:57
openstackgerritA change was merged to openstack/oslo-specs: use the current date to get the copyright  https://review.openstack.org/12019313:57
jd__o/13:59
openstackgerritA change was merged to openstack/oslo-specs: use yasfb for rss feeds  https://review.openstack.org/12019413:59
dhellmannihrachyshka: without a TTL, if the message does go out a receive will process it even if the sender has given up on them14:01
*** tedross has quit IRC14:01
dhellmannjd__: hi!14:01
*** andreykurilin_ has quit IRC14:01
*** kgiusti has quit IRC14:01
jd__hey dhellmann!14:02
jd__I don't think I'm qualified to answer oslo.messaging questions but sileht might be14:02
dhellmannjd__: oh, I thought you were in the club for that one14:03
*** e0ne has quit IRC14:03
jd__yeah but my skills are pretty low compared to our guru sileht14:03
jd__I'm just confident with the notification part, for some reasons ;)14:03
jd__did someone noticed the bug on _i18n already?14:05
*** e0ne has joined #openstack-oslo14:05
*** kgiusti has joined #openstack-oslo14:05
openstackgerritJulien Danjou proposed a change to openstack/oslo-incubator: Fix i18n import  https://review.openstack.org/12441114:09
jd__I guess not, here's a patch ^^14:09
*** e0ne has quit IRC14:09
*** e0ne has joined #openstack-oslo14:10
*** yamahata_ has quit IRC14:10
*** exploreshaifali has quit IRC14:15
*** tedross has joined #openstack-oslo14:17
*** yamahata_ has joined #openstack-oslo14:18
*** dims has quit IRC14:19
*** AAzza has quit IRC14:20
*** dims has joined #openstack-oslo14:21
*** AAzza_afk has joined #openstack-oslo14:21
*** AAzza_afk is now known as AAzza14:21
*** stevemar has joined #openstack-oslo14:22
*** AAzza_afk has joined #openstack-oslo14:22
silehtihrachyshka, when you says 'they are not purged from queues at all.' do you talk about the in-memmory queue on the ReplyWaiters or the real queue on rabbit ?14:22
*** zz_jgrimm is now known as jgrimm14:23
ihrachyshkasileht: should be real queue14:24
ihrachyshka==rabbit14:24
*** sigmavirus24 is now known as sigmavirus24_awa14:24
*** dims has quit IRC14:25
*** AAzza has quit IRC14:25
*** AAzza_afk is now known as AAzza14:25
*** jecarey has joined #openstack-oslo14:26
dhellmannjd__: oh, does it break during a sync?14:26
jd__dhellmann: yay :(14:26
dhellmannjd__: good catch14:26
jd__I tried to upgrade Gnocchi and I hit that14:26
dhellmannjd__: maybe we need to build a test for syncing modules. I'm not sure how we'd test the results of the sync, though14:26
jd__dhellmann: sounds like a lot of effort for something that's going to disapear pretty soon anyway14:27
dhellmannjd__: I don't expect it to go away, just be used less14:27
jd__that's why I didn't even try to fix/enhance update.py, I picked the easily solution :)14:27
*** sigmavirus24_awa is now known as sigmavirus2414:27
jd__s/ly//14:27
jd__dhellmann: let me be optimistic14:27
dhellmannjd__: I read a story that BBC listeners mistook "there's a van fire on the M25" as "there's a VAMPIRE on the M25" so my optimism is sort of low today14:28
jd__dhellmann: mouhahaha14:31
*** jaypipes has joined #openstack-oslo14:31
*** dims has joined #openstack-oslo14:32
*** dims has quit IRC14:32
*** dims has joined #openstack-oslo14:33
*** dims has quit IRC14:34
*** dims has joined #openstack-oslo14:35
*** dims has quit IRC14:35
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.middleware: Avoid depending on oslo-incubator copy of context.py  https://review.openstack.org/12441714:36
*** sreshetnyak has joined #openstack-oslo14:39
*** rpodolyaka1 has quit IRC14:40
*** rpodolyaka1 has joined #openstack-oslo14:40
*** zoresvit has joined #openstack-oslo14:41
*** zoresvit has quit IRC14:41
*** dims has joined #openstack-oslo14:42
*** dims has quit IRC14:42
silehtihrachyshka, I don't catch the case of this occurs14:42
silehtihrachyshka, for the case of the caller send a message and died before the reply and never comeback ?14:43
*** dimsum_ has joined #openstack-oslo14:43
*** dimsum_ has quit IRC14:43
openstackgerritA change was merged to openstack/oslo-specs: Fix the blueprint URLs in existing specs  https://review.openstack.org/12020314:44
*** dimsum_ has joined #openstack-oslo14:45
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.concurrency: Avoid depending on oslo-incubator copy of fileutils  https://review.openstack.org/12442414:45
openstackgerritA change was merged to openstack/oslo-specs: Replace default bp URLs with portable examples  https://review.openstack.org/12020514:52
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.db: Use fixture from oslo.config instead of oslo-incubator  https://review.openstack.org/12442714:52
ihrachyshkasileht: it can die, or its own waiting timeout can expire, and it will probably won't check for the reply anymore, so reply will stay there in the queue waiting for nothing14:55
silehtihrachyshka, the reply is done with the DirectPublisher (that set auto-delete=True) so if it dies the queue is deleted14:57
*** andreykurilin_ has joined #openstack-oslo14:58
silehtihrachyshka, if this is a timeout, the next call/cast/whatever should cleanup the queue14:59
silehthum only call in fact15:01
*** marun has joined #openstack-oslo15:01
*** rpodolyaka1 has quit IRC15:03
*** mtanino has joined #openstack-oslo15:06
*** haypo has joined #openstack-oslo15:06
*** rpodolyaka1 has joined #openstack-oslo15:07
*** jaypipes is now known as leakypipes15:08
ihrachyshkasileht: I need to admit that I'm not a messaging guy. so I just try to communicate things that are experienced by our QA. here is what one of them had to say about neutron/havana (=oslo-incubator rpc): "The core neutron routines never set a timeout and thus a TTL (Time To Live) on any of the qpid messages. Meanwhile the waiters use the rpc_response_timeout to determine when to give up on a response.As a result, message15:15
ihrachyshkas that are no longer recognized can build up in the queues. This causes addition, wasted processing overhead and contributes to a longer backlog." Does it make sense?15:15
*** jogo is now known as flashgordon15:17
silehtihrachyshka, ok I known some issue around that have been fixed in qpid driver in oslo.msg, I will take a look if the 'oslo-incubator rpc' have this issue15:17
zzzeekhey dhellmann15:18
zzzeeklooks like i set the house on fire again, sorry   https://bugs.launchpad.net/oslo.db/+bug/137449715:18
*** rpodolyaka1 has quit IRC15:19
ihrachyshkasileht: I'm looking fwd to times when there won't be 'oslo-incubator rpc' in any supported projects...15:20
*** anteaya has quit IRC15:23
*** kgiusti has quit IRC15:25
silehtihrachyshka, Perhaps this issue: https://bugs.launchpad.net/oslo.messaging/+bug/131874215:26
silehtihrachyshka, neutron icehouse is not fixed: https://github.com/openstack/neutron/blob/stable/icehouse/neutron/openstack/common/rpc/impl_qpid.py#L22715:26
*** takedakn has joined #openstack-oslo15:26
*** kgiusti has joined #openstack-oslo15:28
ihrachyshkasileht: great. we also tracked that fix and was going to backport it to neutron icehouse/havana, but here is what Mark (the guy from QE) has to say on that: "[16:46]  <mwagner_wfh> [10:33] <kgiusti> mwagner_wfh: I'm wary about removing the 'autodelete' - if the clients never come back, those queues will stick around, possibly filling up, no?"15:29
*** takedakn has quit IRC15:29
ihrachyshkasileht: is that 'died-and-never-returned' case serious enough to have special handling?15:30
silehtihrachyshka, oh I got it, on rabbit autodelete is True, on qpid it's false ...15:31
dhellmannzzzeek: is there a commit we can revert to fix that for now?15:31
zzzeekdhellmann: no15:31
dhellmannzzzeek: why not?15:31
zzzeekdhellmann: however, it is easy to restore that function to its former functionality15:32
dhellmannzzzeek: ok, that's what I really mean I guess15:32
zzzeekdhellmann: yes :)15:32
dhellmannzzzeek: let's set that up, and merged to master. then we can go back and branch oslo.db to release a patch update with the fix as well.15:32
ihrachyshkasileht: oslo-incubator only?15:32
zzzeekdhellmann: question is why werent we aware that projects are not using transactions in all cases.15:32
silehtihrachyshka, both15:33
dhellmannzzzeek: there are too many projects for us to keep up with details like that :-/15:33
zzzeekdhellmann: ok15:33
zzzeekdhellmann: this is why we need more stringent patterns with less variability15:33
dhellmannzzzeek: there isn't a single "way" anything is done in openstack15:33
zzzeekdhellmann: that has to change if they are using oslo.db for DB connectivity15:33
zzzeekdhellmann: not that there wont be variability but everyone should ultimately use some pattern we are aware of and is officially supported15:34
sileht:bn15:34
ihrachyshkasileht: isn't it set in ConsumerBase.__init__15:35
ihrachyshka?15:35
silehtihrachyshka, the DirectConsumer force to conf.amqp_auto_delete that is False by default15:36
*** viktors has joined #openstack-oslo15:37
ihrachyshkaindeed15:37
silehtI wonder why rabbit and qpid have not the same behavior for DirectConsumer15:38
ihrachyshkasileht: because no one cares about keeping them in sync? ;)15:39
sileht:)15:40
silehtunfortunatly15:40
ihrachyshkasileht: so how will we handle that? should I create a bug or fix it myself (somehow)?15:45
openstackgerritOleksii Chuprykov proposed a change to openstack/oslo.db: Implement db.ModelBase.iteritems() as a generator  https://review.openstack.org/12444615:47
silehtihrachyshka, I fix we should create a bug and fix it15:48
ihrachyshkasileht: cool. let me create a bug for you15:48
silehtihrachyshka, I have checked in the past, the bug have been introduced when we have added oslo.cfg.amqp_auto_delete15:48
silehtihrachyshka, all the consumer have been updated except the DirectConsumer of rabbitmq15:49
silehtihrachyshka, but we have changed the DirectConsumer of qpid, and introduce the issue15:49
ihrachyshkawhat's that amqp_auto_delete btw? can we just set it in our distribution to True to fix the issue in the meantime?15:50
*** tsekiyama has joined #openstack-oslo15:51
*** takedakn has joined #openstack-oslo15:51
silehtihrachyshka, we it will clean all the queues each times sometimes go wrong15:51
*** noelbk has joined #openstack-oslo15:51
silehtihrachyshka, forced it to True for the DirectConsumer is easy and preferable15:51
silehtthe change that introduce the issue: https://github.com/openstack/oslo-incubator/commit/5ff534d1#diff-372094c4bfc6319d22875a970aa6b730R19015:51
*** harlowja_at_home has joined #openstack-oslo15:51
silehtihrachyshka, we clearly see the unit test change for qpid and DirectConsumer but shouldn't15:52
silehtihrachyshka, I need to go, bbl15:52
ihrachyshkayeah, probably changed unit test because of failures instead of fixing failures :)15:52
sileht:)15:52
*** ochuprykov_ has quit IRC15:53
ihrachyshkasileht: I don't know how to assign it to you, but here it is: https://bugs.launchpad.net/neutron/+bug/137451915:59
*** bnemec is now known as beekneemech16:00
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Add hacking import exception for i18n  https://review.openstack.org/12445016:04
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Expose lockutils opts to config generator  https://review.openstack.org/12445116:04
*** arnaud has joined #openstack-oslo16:09
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Add docs for oslo.config opts  https://review.openstack.org/12445416:11
*** jaosorior has quit IRC16:12
openstackgerritZhiQiang Fan proposed a change to openstack/oslo.i18n: Use same indentation in doc/source/usage  https://review.openstack.org/12445516:14
*** andreykurilin_ has quit IRC16:21
*** mriedem is now known as mriedem_burgers16:27
*** takedakn has quit IRC16:30
*** anteaya has joined #openstack-oslo16:34
*** zzzeek has quit IRC16:39
*** zzzeek has joined #openstack-oslo16:40
*** takedakn has joined #openstack-oslo16:48
*** ihrachyshka has quit IRC16:49
*** takedakn has quit IRC16:54
*** rpodolyaka1 has joined #openstack-oslo16:55
*** yamahata_ has quit IRC16:57
dhellmannbeekneemech: there are lots of calls to set_defaults()?17:01
beekneemechharlowja_away: Yeah, I definitely want to be involved in the pylockfile/lockutils refactoring.17:01
harlowja_at_homecool, check it out, bb17:01
*** harlowja_at_home has quit IRC17:01
beekneemechdhellmann: No, but we use the conf object in a bunch of the different decorators/functions, so it would be awkward to have to pass it all the time.17:01
beekneemechUnless we save a reference in set_defaults and require that to be called before using locks.17:02
*** e0ne has quit IRC17:02
beekneemechWhich is probably a reasonable option.17:02
dhellmannbeekneemech: yeah, I was just thinking something similar, though not that function17:02
beekneemech*set_defaults or something :-)17:03
beekneemechdhellmann: For both lockutils and log we would need to make sure that can happen early enough so that no one tries to use those things before the conf is set.17:04
dhellmannwith oslo.log, we have setup() that they have to call to make logging work, so that's a natural place to ask for a config object, but then we don't need it any more17:04
dhellmanngood point17:04
beekneemechI'm thinking maybe it needs to be done ASAP after eventlet monkey patching.17:05
beekneemechCan't be done before or we might end up mixing locks.17:05
dhellmannbeekneemech: another approach would be what we did with i18n, and have a factory to create the lock decorator function17:05
dhellmannthat feels like overkill, though17:06
dhellmannmaybe this is just a place where we have to assume the global config17:06
dhellmannbeekneemech: do you plan to provide a console script to call lockutils.main()?17:07
dhellmann"oslo-concurrency-lock foo" instead of "python -m oslo.concurrency.lockutils ..."17:07
*** sigmavirus24 is now known as sigmavirus24_awa17:08
beekneemechdhellmann: Oh, yeah.  That was on the graduation list wasn't it.17:08
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Move begin ping listener to a connect listener  https://review.openstack.org/12446617:11
beekneemechHmm, the spreadsheet claims oslo.concurrency has an api.rst, but I'm not seeing it in the generated docs...17:15
dhellmannbeekneemech: there's no api.rst, but dimsum_ added the auto-generation stuff from pbr so there are some modules listed on http://docs.openstack.org/developer/oslo.concurrency/17:18
dimsum_beekneemech: if you run docs you will see api/autoindex.rst17:18
dhellmanntime for lunch, bbiab17:18
*** dhellmann is now known as dhellmann_17:18
beekneemechdhellmann_: dimsum_: Huh, apparently that doesn't get included when you generate docs locally.  I guess since it's pbr it only happens for builds.17:20
dimsum_beekneemech: "tox -e docs"?17:20
beekneemechdimsum_: Yeah17:20
dimsum_weird, that's how i tested it17:20
beekneemechI get no Code Documentation section.17:20
beekneemechMaybe I have an old pbr or something?17:21
beekneemechHmm, no, says I'm on the latest already.17:22
dimsum_dims@dims-mac:~/openstack/oslo.concurrency$ find . -name au*.rst17:22
dimsum_./doc/source/api/autoindex.rst17:22
dimsum_pip freeze gives me pbr==0.10.017:22
*** arnaud has quit IRC17:25
beekneemechdimsum_: Oh, PEBKAC.  My master branch was out of date so I didn't have that change yet locally.17:25
*** andreykurilin_ has joined #openstack-oslo17:26
dimsum_ah cool17:26
*** praneshp has joined #openstack-oslo17:26
dimsum_:)17:26
beekneemechHmm, then I don't need to explicitly add opts to the index either.17:26
beekneemechdhellmann_: When you get back, we want to use the config filter for lib options too, don't we?17:29
*** tflower has joined #openstack-oslo17:31
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Expose lockutils opts to config generator  https://review.openstack.org/12445117:33
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Add hacking import exception for i18n  https://review.openstack.org/12445017:33
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Make lockutils main() a console entry point  https://review.openstack.org/12447517:33
*** dhellmann_ is now known as dhellmann17:36
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Use ConfigFilter for opts  https://review.openstack.org/12447617:38
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Use ConfigFilter for opts  https://review.openstack.org/12447617:42
*** jecarey has quit IRC17:45
rpodolyaka1zzzeek: engine_connect() will effectively be called on every engine.connect(), right?17:46
rpodolyaka1zzzeek: ( the listener set by engine_connect() )17:46
*** exploreshaifali has joined #openstack-oslo17:49
*** arnaud has joined #openstack-oslo17:53
*** andreykurilin_ has quit IRC17:53
exploreshaifalidhellmann, still getting "importerror no module name i18n"17:54
exploreshaifalidhellmann,I did all what you mentioned, first "pip uninstall oslo.i18n " then "pip install -e /opt/stack/oslo.i18n"17:55
*** arnaud__ has joined #openstack-oslo17:55
*** rpodolyaka1 has quit IRC17:55
*** yassine has quit IRC17:58
*** tedross has quit IRC17:58
*** rpodolyaka1 has joined #openstack-oslo17:59
*** harlowja_away is now known as harlowja17:59
*** viktors has quit IRC17:59
*** sigmavirus24_awa is now known as sigmavirus2418:02
*** e0ne has joined #openstack-oslo18:15
zzzeekrpodolyaka: yes18:16
*** e0ne has quit IRC18:16
*** kgiusti has quit IRC18:19
*** kgiusti has joined #openstack-oslo18:19
*** e0ne has joined #openstack-oslo18:20
*** e0ne has quit IRC18:24
*** rpodolyaka1 has quit IRC18:24
*** e0ne has joined #openstack-oslo18:28
*** rpodolyaka1 has joined #openstack-oslo18:30
*** e0ne has quit IRC18:30
openstackgerritMichael Bayer proposed a change to openstack/oslo.db: Move begin ping listener to a connect listener  https://review.openstack.org/12446618:31
*** mriedem_burgers is now known as mriedem18:32
dhellmannexploreshaifali: sorry, was at lunch18:33
exploreshaifalidhellmann, that's fine18:34
dhellmannexploreshaifali: you're using run_tests.sh, without a venv, right?18:34
exploreshaifalidhellmann, but your soln didn't work18:34
exploreshaifaliyes18:34
dhellmannexploreshaifali: ok. Which OS?18:34
exploreshaifaliusing devstack18:34
exploreshaifaliUbuntu 12.0418:35
dhellmannok, what do you have in /usr/lib/python2.7/dist-packages?18:35
exploreshaifalijust a min, I have to open vm18:36
exploreshaifalito check it18:36
dhellmannexploreshaifali: is this a vm you could easily recreate?18:36
*** stannie has joined #openstack-oslo18:39
*** pcm__ has quit IRC18:40
*** e0ne has joined #openstack-oslo18:41
dhellmanndimsum_, beekneemech, jd__ : before we make any more libraries, we have a few changes to go into the cookiecutter repo: https://review.openstack.org/#/q/status:open+project:openstack-dev/oslo-cookiecutter,n,z18:42
harlowjabunch of rebases incoming, watch out, ha18:43
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Document more function/class/method params  https://review.openstack.org/12422818:43
harlowjahmmmm, http://lists.openstack.org/pipermail/openstack-dev/2014-September/047280.html hmmmmm18:45
harlowja'No more alpha libraries' hmmm18:45
*** e0ne has quit IRC18:49
dhellmannexploreshaifali: I have to run an errand. You can try removing all oslo libraries and reinstalling the source, unless you want to wait until I'm back to spend the time to find which library is installed in the wrong mode and causing your problem.18:49
beekneemechMy kingdom for sane Python namespaces. ;-)18:50
*** e0ne has joined #openstack-oslo18:50
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Move failure to its own type specific module  https://review.openstack.org/10209118:51
exploreshaifalidhellmann, just one thing to confirm18:51
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Hoist the notifier to its own module  https://review.openstack.org/10237218:51
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Mark 'task_notifier' as renamed to 'atom_notifier'  https://review.openstack.org/10328618:51
exploreshaifalidhellnamm, to reinstall libs again ""pip install -e /opt/stack/oslo.i18n"18:51
dhellmannexploreshaifali: yes, but you need to do that for every library separately18:52
exploreshaifalibut it is not working18:52
dhellmannexploreshaifali: "it"?18:52
exploreshaifaliI mean by doing tht, I am getting oslo.i18n is working correctly18:52
exploreshaifaliit is installed correctly18:52
exploreshaifalibut now it is not working iPython cmmand line mode18:53
dhellmannexploreshaifali: You have at least one library installed in "editable" mode and at least one in non-editable mode. That configuration does not work. It may not be oslo.i18n. We can either go through the libraries one by one and figure out which is wrong, or you can install them all again in the same mode.18:53
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Expose only `ensure_atom` from storage  https://review.openstack.org/10337818:53
exploreshaifaliso you mean I should uninstall all oslo.x libs18:54
dhellmannexploreshaifali: Unfortunately, I have to step away for a bit, so I can't help with the more intensive debugging, so if this is a development VM it might be simplest to just rebuild it or remove all oslo libraries and reinstall them.18:54
exploreshaifalione by one and then reinstall it18:54
dhellmannexploreshaifali: yes18:54
*** marun has quit IRC18:55
dhellmannyou should remove them all and then let devstack install them in the right order (or look at the order it installs them and do it by hand if you don't want to unstack)18:55
exploreshaifaliokay18:55
dhellmannbecause if you install them from source in the wrong order, their requirements files will bring in package dependencies and that will just break you system again :-/18:55
exploreshaifalihmm... fine :)18:56
exploreshaifaliso I should do ./stack again18:56
exploreshaifaliafter ./unstack18:56
dhellmannunstack won't remove the libs, though, so you have to do that by hand18:57
dhellmannI have to run. bbiab.18:57
*** dhellmann is now known as dhellmann_18:57
exploreshaifalik18:57
exploreshaifalitata18:57
exploreshaifali:)18:57
*** e0ne has quit IRC19:01
*** e0ne has joined #openstack-oslo19:05
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Remove attrdict and just use existing types  https://review.openstack.org/11619119:05
*** e0ne has quit IRC19:11
*** rpodolyaka1 has quit IRC19:14
*** e0ne has joined #openstack-oslo19:15
*** e0ne has quit IRC19:18
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Clean up doc header  https://review.openstack.org/12449519:18
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Remove hard dep on eventlet  https://review.openstack.org/12449619:18
*** gordc has quit IRC19:19
*** zzzeek has quit IRC19:21
*** rpodolyaka1 has joined #openstack-oslo19:21
*** e0ne has joined #openstack-oslo19:22
*** andreykurilin_ has joined #openstack-oslo19:24
*** e0ne has quit IRC19:26
*** boris-42 has joined #openstack-oslo19:27
*** e0ne has joined #openstack-oslo19:31
*** e0ne has quit IRC19:35
*** rpodolyaka1 has quit IRC19:41
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Just use the natural module name instead of aliasing to tt  https://review.openstack.org/11918619:41
*** rpodolyaka1 has joined #openstack-oslo19:45
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Move some of the custom requirements out of tox.ini  https://review.openstack.org/12016819:49
*** rpodolyaka1 has quit IRC19:49
*** rpodolyaka1 has joined #openstack-oslo19:49
*** rpodolyaka1 has quit IRC19:52
*** andreykurilin_ has quit IRC19:53
*** andreykurilin_ has joined #openstack-oslo19:53
*** dhellmann_ is now known as dhellmann19:55
*** gordc has joined #openstack-oslo19:55
*** exploreshaifali has quit IRC20:01
openstackgerritgordon chung proposed a change to openstack/oslo-incubator: remove caching param from prettytable call  https://review.openstack.org/12450520:02
*** rpodolyaka1 has joined #openstack-oslo20:05
*** kgiusti has left #openstack-oslo20:07
*** boden has quit IRC20:10
*** boden has joined #openstack-oslo20:10
*** zzzeek has joined #openstack-oslo20:13
*** boden has quit IRC20:15
*** pcm_ has joined #openstack-oslo20:16
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Use oslotest to provide our base test case class  https://review.openstack.org/12029420:17
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Use the features that the oslotest mock base class provides  https://review.openstack.org/12062020:18
openstackgerritArnaud Legendre proposed a change to openstack/oslo.vmware: Add API to get the entity inventory path  https://review.openstack.org/12252220:30
*** gordc has quit IRC20:36
morganfainbergzzzeek, dhellmann, so i take it that https://bugs.launchpad.net/keystone/+bug/1374497 is something we're going to need to fix in keystone as another release of oslo.db is not in the books for this cycle?20:36
*** morganfainberg is now known as morgan20:36
zzzeeknot sure20:37
*** morgan is now known as morganfainberg20:37
dhellmannmorganfainberg: when the fix lands in oslo.db master we can make a branch, back port it, and release a patch release20:37
morganfainbergdhellmann, ok, i am totally on board with fixing keystone, but it's not likely to be a small fix20:37
dhellmannyeah, that sounds like a kilo change20:38
morganfainbergi'd much rather get oslo.db fixed if we can swing that and work on making keystone better in kilo20:38
* dhellmann wonders how to add multiple versions of oslo.db to the bug20:38
morganfainbergok, so i'll mark this as a Kilo target and reference the discussion here (for keystone).20:39
morganfainbergand mark the other bug as a dupe of this one.20:39
morganfainbergsince we have 2 bugs referencing this issue20:39
*** jgrimm is now known as zz_jgrimm20:52
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Get the basics of a process executor working  https://review.openstack.org/12128020:55
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Ensure that failure objects are pickleable  https://review.openstack.org/12173520:55
*** dimsum_ has quit IRC20:56
*** dimsum_ has joined #openstack-oslo20:56
*** dimsum_ has quit IRC21:00
openstackgerritJoshua Harlow proposed a change to openstack/oslo.db: Create a nested helper function that will work on py3.x  https://review.openstack.org/12019521:04
openstackgerritJoshua Harlow proposed a change to openstack/oslo.db: Fix more invalid encoding and scoping issues  https://review.openstack.org/12019821:04
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Add and use a new simple helper logging module  https://review.openstack.org/12300721:05
*** haypo has left #openstack-oslo21:07
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Expose lockutils opts to config generator  https://review.openstack.org/12445121:12
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Remove hard dep on eventlet  https://review.openstack.org/12449621:12
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Clean up doc header  https://review.openstack.org/12449521:12
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Make lockutils main() a console entry point  https://review.openstack.org/12447521:12
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Use ConfigFilter for opts  https://review.openstack.org/12447621:12
beekneemechWell now, that turned out to be a whole can of worms.21:12
beekneemechOne of the changes broke the unit tests in a way that masked some other issues with other changes.21:12
*** stevemar has quit IRC21:20
*** dimsum_ has joined #openstack-oslo21:21
*** mtanino has quit IRC21:24
*** rpodolyaka1 has quit IRC21:24
*** tflower has quit IRC21:27
*** mtanino has joined #openstack-oslo21:27
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Always return copies from memory backend  https://review.openstack.org/12424221:28
*** mriedem has quit IRC21:31
*** pcm_ has quit IRC21:37
*** boden has joined #openstack-oslo21:41
*** marun has joined #openstack-oslo21:45
*** boden has quit IRC21:45
*** leakypipes has quit IRC21:49
openstackgerritJoshua Harlow proposed a change to openstack/oslo.messaging: Add a thread + futures executor based executor  https://review.openstack.org/7091421:50
*** boden has joined #openstack-oslo21:57
*** AAzza is now known as AAzza_afk21:59
*** boden has quit IRC22:01
*** sigmavirus24 is now known as sigmavirus24_awa22:13
*** arnaud__ has quit IRC22:14
*** arnaud has quit IRC22:14
*** arnaud has joined #openstack-oslo22:14
*** arnaud__ has joined #openstack-oslo22:14
*** hartsocks has joined #openstack-oslo22:19
*** hartsocks has left #openstack-oslo22:20
*** rpodolyaka1 has joined #openstack-oslo22:31
openstackgerritA change was merged to openstack/oslo.log: Test formatting errors with log level being emitted  https://review.openstack.org/11961822:41
*** dhellmann is now known as dhellmann_22:42
*** dimsum_ has quit IRC22:43
*** dimsum_ has joined #openstack-oslo22:44
*** rpodolyaka1 has quit IRC22:47
*** rpodolyaka1 has joined #openstack-oslo22:48
*** dimsum_ has quit IRC22:48
*** boden has joined #openstack-oslo22:58
*** AAzzal has joined #openstack-oslo22:58
*** AAzzal is now known as AAzza22:59
*** AAzza_afk has quit IRC22:59
*** boden has quit IRC23:02
*** dimsum_ has joined #openstack-oslo23:07
*** yamahata_ has joined #openstack-oslo23:09
*** praneshp has quit IRC23:10
*** rpodolyaka1 has quit IRC23:11
*** rpodolyaka1 has joined #openstack-oslo23:14
*** arnaud__ has quit IRC23:23
*** arnaud has quit IRC23:23
*** noelbk has quit IRC23:31
*** noelbk has joined #openstack-oslo23:31
*** zzzeek has quit IRC23:44
*** marun has quit IRC23:50
*** boden has joined #openstack-oslo23:59

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