Thursday, 2015-03-05

*** pmathews has quit IRC00:07
*** pmathews has joined #openstack-oslo00:09
openstackgerritJoshua Harlow proposed openstack/taskflow: Add warning about transient arguments and worker-based-engines  https://review.openstack.org/16149300:15
*** prad has quit IRC00:15
openstackgerritJoshua Harlow proposed openstack/taskflow: Add warning about transient arguments and worker-based-engines  https://review.openstack.org/16149300:17
*** daniel3_ has quit IRC00:17
openstackgerritJoshua Harlow proposed openstack/taskflow: Add warning about transient arguments and worker-based-engines  https://review.openstack.org/16149300:19
openstackgerritJoshua Harlow proposed openstack/taskflow: Add warning about transient arguments and worker-based-engines  https://review.openstack.org/16149300:20
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module  https://review.openstack.org/13821500:21
*** pmathews has quit IRC00:21
*** pmathews has joined #openstack-oslo00:22
*** pmathews has quit IRC00:30
openstackgerritJoshua Harlow proposed openstack/taskflow: Ensure ordered set is pickleable  https://review.openstack.org/16151700:39
*** stevemar has quit IRC00:39
*** devlaps has quit IRC00:41
*** crc32 has quit IRC00:41
openstackgerritJoshua Harlow proposed openstack/taskflow: Ensure ordered set is pickleable  https://review.openstack.org/16151700:42
*** salv-orlando has quit IRC01:16
*** zzzeek has quit IRC01:17
*** tsekiyam_ has joined #openstack-oslo01:21
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a bookshelf developer section  https://review.openstack.org/16153401:22
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a bookshelf developer section  https://review.openstack.org/16153401:22
*** tsekiyama has quit IRC01:25
*** tsekiyam_ has quit IRC01:26
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a bookshelf developer section  https://review.openstack.org/16153401:27
*** mtanino has quit IRC01:30
openstackgerritMerged openstack/oslo.config: fix bug link in readme  https://review.openstack.org/16125301:33
*** amotoki has joined #openstack-oslo01:35
*** liusheng has quit IRC01:38
*** dims has quit IRC01:43
*** sputnik13 has quit IRC01:44
*** jaosorior has quit IRC01:52
*** stevemar has joined #openstack-oslo02:16
*** jgrimmn has joined #openstack-oslo02:20
*** fesp has joined #openstack-oslo02:20
*** flaper87 has quit IRC02:21
*** zz_jgrimm has quit IRC02:21
*** jgrimmn has quit IRC02:21
*** jgrimmn has joined #openstack-oslo02:21
*** fesp has quit IRC02:21
*** fesp has joined #openstack-oslo02:21
*** fesp is now known as flaper8702:21
*** amrith is now known as _amrith_02:26
*** ChuckC has joined #openstack-oslo02:26
*** dims has joined #openstack-oslo02:39
*** achanda has quit IRC02:55
*** yamahata has quit IRC03:00
*** ChuckC has quit IRC03:01
*** ChuckC has joined #openstack-oslo03:06
*** ccrouch is now known as ccrouch-afk03:10
*** haypo has quit IRC03:15
*** haypo has joined #openstack-oslo03:16
gusIs it important to keep py26 compatibility with oslo_rootwrap, or are we dropping that with Kilo?03:16
gus(I see tox envlist includes py26, but we're dropping it elsewhere in openstack)03:16
*** pmathews has joined #openstack-oslo03:21
*** takedakn has joined #openstack-oslo03:24
*** takedakn has quit IRC03:29
*** achanda has joined #openstack-oslo03:29
openstackgerritJoshua Harlow proposed openstack/taskflow: Add ability to use tooz to find and advertise workers  https://review.openstack.org/15149503:31
*** jungleboyj has joined #openstack-oslo03:41
*** sigmavirus24 is now known as sigmavirus24_awa03:47
openstackgerritJoshua Harlow proposed openstack/taskflow: Ensure the thread bundle stops in last to first order  https://review.openstack.org/16156103:53
*** salv-orlando has joined #openstack-oslo04:16
*** salv-orlando has quit IRC04:21
*** yamahata has joined #openstack-oslo04:22
*** achanda has quit IRC04:22
*** dims has quit IRC04:22
openstackgerritMerged openstack/taskflow: Add node removal/disassociate functions  https://review.openstack.org/16137604:37
*** vishy has quit IRC04:47
openstackgerritOpenStack Proposal Bot proposed openstack/oslo-incubator: Updated from global requirements  https://review.openstack.org/16158104:57
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.concurrency: Updated from global requirements  https://review.openstack.org/16158204:57
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.db: Updated from global requirements  https://review.openstack.org/16139404:57
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.log: Updated from global requirements  https://review.openstack.org/16158304:57
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements  https://review.openstack.org/15778304:57
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.middleware: Updated from global requirements  https://review.openstack.org/16158404:57
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.policy: Updated from global requirements  https://review.openstack.org/16158504:57
*** harlowja is now known as harlowja_away04:57
*** harlowja_at_home has joined #openstack-oslo05:10
*** stevemar has quit IRC05:13
openstackgerritJoshua Harlow proposed openstack/taskflow: Ensure the thread bundle stops in last to first order  https://review.openstack.org/16156105:15
openstackgerritJoshua Harlow proposed openstack/taskflow: Ensure the thread bundle stops in last to first order  https://review.openstack.org/16156105:17
openstackgerritJoshua Harlow proposed openstack/taskflow: Add warning about transient arguments and worker-based-engines  https://review.openstack.org/16149305:21
*** achanda has joined #openstack-oslo05:23
*** dims has joined #openstack-oslo05:23
*** achanda has quit IRC05:23
*** achanda has joined #openstack-oslo05:23
*** dims has quit IRC05:28
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a bookshelf developer section  https://review.openstack.org/16153405:33
*** harlowja_at_home has quit IRC05:35
openstackgerritJoshua Harlow proposed openstack/taskflow: Ensure the thread bundle stops in last to first order  https://review.openstack.org/16156105:41
openstackgerritSachi King proposed openstack/oslo-specs: graduate-oslo-service  https://review.openstack.org/14265906:04
*** ttx has quit IRC06:07
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.utils: Imported Translations from Transifex  https://review.openstack.org/16064906:14
openstackgerritLi Ma proposed openstack/oslo.messaging: Add pluggability for matchmakers  https://review.openstack.org/16161506:15
*** ttx has joined #openstack-oslo06:19
openstackgerritLi Ma proposed openstack/oslo.messaging: ZeroMQ deployment guide  https://review.openstack.org/13094306:30
*** e0ne has joined #openstack-oslo06:36
*** pmathews has quit IRC06:36
*** e0ne has quit IRC06:38
*** e0ne has joined #openstack-oslo06:42
*** e0ne has quit IRC06:43
openstackgerritJoshua Harlow proposed openstack/taskflow: Add ability to use tooz to find and advertise workers  https://review.openstack.org/15149506:44
*** Krast has joined #openstack-oslo06:51
*** stevemar has joined #openstack-oslo06:55
openstackgerritAbhijeet Malawade proposed openstack/tooz: fix mysql driver url parsing  https://review.openstack.org/15983507:05
*** exploreshaifali has joined #openstack-oslo07:07
openstackgerritSachi King proposed openstack/oslo-specs: graduate-oslo-service  https://review.openstack.org/14265907:07
*** vigneshvar has joined #openstack-oslo07:13
*** salv-orlando has joined #openstack-oslo07:14
*** exploreshaifali has quit IRC07:15
*** gcb has joined #openstack-oslo07:16
*** salv-orlando has quit IRC07:19
*** inc0 has joined #openstack-oslo07:20
*** salv-orlando has joined #openstack-oslo07:27
*** liusheng has joined #openstack-oslo07:32
openstackgerritAbhijeet Malawade proposed openstack/tooz: fix mysql driver url parsing  https://review.openstack.org/15983507:32
*** inc0 has quit IRC07:33
*** openstackgerrit has quit IRC07:36
*** openstackgerrit has joined #openstack-oslo07:36
*** YorikSar has joined #openstack-oslo07:36
*** exploreshaifali has joined #openstack-oslo07:45
*** ajo has quit IRC07:52
*** ajo has joined #openstack-oslo07:53
*** inc0 has joined #openstack-oslo07:54
*** inc0 has quit IRC07:59
*** inc0_ has joined #openstack-oslo07:59
*** flwang has joined #openstack-oslo08:01
flwanggcb: ping08:01
*** inc0_ has quit IRC08:10
*** gcb has quit IRC08:10
*** ajo has quit IRC08:14
*** achanda has quit IRC08:16
*** dulek has joined #openstack-oslo08:23
*** inc0 has joined #openstack-oslo08:26
*** Krast has quit IRC08:33
*** achanda has joined #openstack-oslo08:33
*** salv-orlando has quit IRC08:33
*** ttx has quit IRC08:34
*** ttx has joined #openstack-oslo08:34
openstackgerritMerged openstack/oslo.policy: Updated from global requirements  https://review.openstack.org/16158508:35
openstackgerritBogdan Dobrelya proposed openstack/oslo.log: Use syslog for logging in an RFC5424 compliant format by default.  https://review.openstack.org/14608208:41
*** jaosorior has joined #openstack-oslo08:46
*** dulek has quit IRC08:53
*** dulek has joined #openstack-oslo08:53
flwangsileht: ping08:58
flwanghttps://bugs.launchpad.net/oslo.messaging/+bug/1338732 i'm looking for this bug08:58
openstackLaunchpad bug 1338732 in oslo.messaging "Timed out waiting for a reply via rabbit" [High,Fix released] - Assigned to Mehdi Abaakouk (sileht)08:58
*** dtantsur|afk is now known as dtantsur08:58
silehtflwang, which version of oslo.messaging ?08:59
flwangwe run into the rpc timeout issue on icehouse08:59
flwang1.308:59
flwangbut we are not really sure if it's the same problem08:59
flwangso i'm trying to run the test script08:59
silehtflwang, Timeout issue always finish with the same error message but they can be a lot of different root cause09:00
flwangyep09:00
silehtflwang, the test script don't work with all oslo.msg version09:00
flwangwhy? i'm confused09:01
*** dims has joined #openstack-oslo09:01
*** ajo has joined #openstack-oslo09:01
silehtflwang, https://bugs.launchpad.net/oslo.messaging/+bug/1338732/comments/1309:02
openstackLaunchpad bug 1338732 in oslo.messaging "Timed out waiting for a reply via rabbit" [High,Fix released] - Assigned to Mehdi Abaakouk (sileht)09:02
*** pblaho has joined #openstack-oslo09:02
flwangso it should work after adding msg.acknowledge() under msg = listener.poll(),  is it?09:03
*** stevemar has quit IRC09:03
silehtflwang, yes09:03
flwangbut I got the same result, http://paste.openstack.org/show/188813/09:04
silehtflwang, the most important fixes that results of timeout issue, that I want to be backported are: https://review.openstack.org/#/q/status:open+project:openstack/oslo.messaging+branch:stable/juno+topic:backport_fix,n,z09:04
*** stevemar has joined #openstack-oslo09:04
silehtoutch coredump09:04
silehtflwang, it can be applied on icehouse version too09:04
flwangsileht: ok, I will back port them to icehouse09:05
*** salv-orlando has joined #openstack-oslo09:05
flwangbut now, I need to confirm if we're running into the same issue09:05
flwangmy test env is the latest devstack09:05
flwangand there is always a coredump09:05
*** dims has quit IRC09:06
*** yamahata has quit IRC09:07
*** dulek_ has joined #openstack-oslo09:08
*** dulek has quit IRC09:08
*** achanda has quit IRC09:12
*** inc0 has quit IRC09:16
*** ihrachyshka has joined #openstack-oslo09:22
*** inc0 has joined #openstack-oslo09:29
openstackgerritJulien Danjou proposed openstack/oslo.log: Replace RFCSysLogHandler by a syslog() based one  https://review.openstack.org/13684209:31
*** inc0 has quit IRC09:32
*** akhivin has quit IRC09:32
*** khivin has quit IRC09:32
*** khivin has joined #openstack-oslo09:32
*** inc0 has joined #openstack-oslo09:32
flwangsileht: any idea?09:34
silehtflwang, for the core dump I have no idea09:38
flwangok, thanks09:39
khivinsileht, dhellmann Hello. Can we speak a liitle about https://bugs.launchpad.net/oslo.messaging/+bug/141467409:42
openstackLaunchpad bug 1414674 in heat "Number of Rabbitmq queues is growing from failover to failover" [High,Fix committed]09:42
khivinsileht, dhellmann bug is closed, but the problem still remains09:43
silehtkhivin, I have reopen the bug for oslo.messaging, it was only half fixed09:45
khivinsileht, I am ready to make a patch or any investigation  but can we discuss what kind of changes can be accepted09:46
silehtkhivin, the oslo.messaging part is to add x-expire on the queue, but I don't known how to migrate an already created queue09:47
khivinsileht, add x-expire is not a problem, but  maybe additional parameter  Target can be added?09:48
khivinsileht, add x-expire is not a problem, but  maybe additional parameter  to a Target constructor can be added?09:49
silehtkhivin, nothing in target have to be added09:50
silehtkhivin, they are no raise to keep unused stuff on the broker09:50
silehtkhivin, the application doesn't have to deal with this kind of stuffs09:50
silehtkhivin, this issue is only rabbitmq related, not qpid or zeromq09:50
khivinsileht, do you think so? will other brokers delete unused queues by default?09:52
silehtkhivin, zeromq doesn't really have 'queues'09:52
silehtkhivin, it seems that qpid delete queues09:53
*** inc0 has quit IRC09:53
openstackgerritBogdan Dobrelya proposed openstack/oslo.log: Use syslog for logging in an RFC5424 compliant format by default.  https://review.openstack.org/14608209:53
openstackgerritBogdan Dobrelya proposed openstack/oslo.log: Deprecate use-syslog-rfc-format for removal  https://review.openstack.org/14608209:54
khivinsileht, I understood. thanks.09:55
*** inc0 has joined #openstack-oslo09:57
*** salv-orlando has quit IRC09:59
*** liusheng has quit IRC10:00
*** liusheng has joined #openstack-oslo10:01
*** dulek_ has quit IRC10:06
*** wendar has quit IRC10:06
openstackgerritEric Brown proposed openstack/oslo.config: Generate help text indicating possible values  https://review.openstack.org/16165510:09
*** koesik has quit IRC10:10
*** wendar has joined #openstack-oslo10:16
*** kiseok7_ has joined #openstack-oslo10:16
*** inc0 has quit IRC10:22
*** e0ne has joined #openstack-oslo10:26
*** e0ne is now known as e0ne_10:26
*** e0ne_ has quit IRC10:27
*** e0ne has joined #openstack-oslo10:27
*** AAzza has joined #openstack-oslo10:29
*** AAzza is now known as AAzza_afk10:32
*** inc0 has joined #openstack-oslo10:32
*** dstanek has quit IRC10:33
*** alexpilotti has joined #openstack-oslo10:35
*** inc0 has quit IRC10:38
openstackgerritMerged openstack/oslo.concurrency: Updated from global requirements  https://review.openstack.org/16158210:39
*** inc0 has joined #openstack-oslo10:46
*** inc0 has quit IRC10:51
*** dims has joined #openstack-oslo10:55
*** salv-orlando has joined #openstack-oslo11:00
*** e0ne is now known as e0ne_11:00
*** e0ne_ has quit IRC11:11
*** e0ne has joined #openstack-oslo11:14
*** stevemar has quit IRC11:28
*** inc0 has joined #openstack-oslo11:39
*** vigneshvar has quit IRC11:40
*** salv-orlando has quit IRC11:42
sdagueso, I think https://review.openstack.org/#/c/160407/ is caught in the Depends-On stuck bit, where it probably needs another +A to kick through11:42
*** cdent has joined #openstack-oslo11:55
*** inc0_ has joined #openstack-oslo12:03
*** inc0 has quit IRC12:04
*** dtantsur is now known as dtantsur|brb12:13
*** e0ne is now known as e0ne_12:20
*** inc0_ has quit IRC12:23
*** _amrith_ is now known as amrith12:25
khivinsileht, I think there should be some recommendation or naming conventions for using in oslo.messaging for queues, topic and etc. It is too hard to understand who uses particular queue in the broker. Something like prefix for each component. heat.* or nova.*. Do you think it is good idea?12:30
*** e0ne_ has quit IRC12:30
*** inc0 has joined #openstack-oslo12:31
*** salv-orlando has joined #openstack-oslo12:56
*** exploreshaifali has quit IRC13:01
silehtkhivin, I agree, but this is hard to change because we should not break alreadu deployed cloud13:05
khivinsilent, yes, I understand, but maybe it is possible to make recommendations for a new code in the openstack13:09
silehtkhivin, yes13:10
*** kgiusti has joined #openstack-oslo13:16
*** ihrachyshka has quit IRC13:18
*** exploreshaifali has joined #openstack-oslo13:21
*** amotoki has quit IRC13:21
*** ihrachyshka has joined #openstack-oslo13:22
*** inc0 has quit IRC13:29
*** inc0 has joined #openstack-oslo13:29
*** e0ne has joined #openstack-oslo13:29
*** dtantsur|brb is now known as dtantsur13:32
*** gordc has joined #openstack-oslo13:33
*** inc0 has quit IRC13:40
*** dstanek has joined #openstack-oslo13:45
*** inc0 has joined #openstack-oslo13:49
*** takedakn has joined #openstack-oslo13:51
*** dims has quit IRC13:51
*** takedakn has quit IRC13:51
*** dims has joined #openstack-oslo13:52
*** inc0 has quit IRC13:54
*** inc0 has joined #openstack-oslo13:55
*** inc0 has quit IRC14:05
*** inc0 has joined #openstack-oslo14:06
openstackgerritMatthew Booth proposed openstack/oslo.db: Implement generic update-on-match feature  https://review.openstack.org/14622814:07
haypoi hate eventlet14:11
haypoi expected https://github.com/eventlet/eventlet/pull/187 to be the latest required change in eventlet to support python 314:11
haypobut there are still issues, and monkey patchs makes me mad14:12
*** stpierre has joined #openstack-oslo14:12
YorikSarttx: Hi.14:18
YorikSarttx: You didn't leave a mark in my CR (about env in rootwrap's daemon mode). I guess the only thing that's not clear there is if we should care about compatibility with this.14:20
*** inc0 has quit IRC14:20
YorikSardhellmann: Could you please get back to https://review.openstack.org/161183 ? It looks like both question in your -1 comment have been answered.14:22
*** amotoki has joined #openstack-oslo14:23
*** jungleboyj has quit IRC14:24
*** zzzeek has joined #openstack-oslo14:29
*** mriedem-away is now known as mriedem14:29
*** inc0 has joined #openstack-oslo14:31
*** exploreshaifali has quit IRC14:36
*** jgrimmn has quit IRC14:38
*** jgrimm has joined #openstack-oslo14:39
*** jgrimm has quit IRC14:44
*** jgrimm has joined #openstack-oslo14:45
*** e0ne is now known as e0ne_14:47
ttxYorikSar: will review, wanted to address Terry's comment first14:47
*** inc0 has quit IRC14:51
*** zzzeek has quit IRC14:52
*** inc0 has joined #openstack-oslo14:57
*** sigmavirus24_awa is now known as sigmavirus2414:59
*** jungleboyj has joined #openstack-oslo15:04
*** inc0 has quit IRC15:06
*** inc0_ has joined #openstack-oslo15:06
openstackgerritDan Krause proposed openstack/taskflow: Persistence backend refactor  https://review.openstack.org/16088915:07
*** e0ne_ is now known as e0ne15:08
*** jecarey has joined #openstack-oslo15:08
*** prad has joined #openstack-oslo15:09
*** exploreshaifali has joined #openstack-oslo15:12
*** mtanino has joined #openstack-oslo15:13
*** daniel3_ has joined #openstack-oslo15:13
*** inc0_ has quit IRC15:14
*** ChuckC has quit IRC15:19
*** pmathews has joined #openstack-oslo15:20
*** yamahata has joined #openstack-oslo15:20
*** inc0 has joined #openstack-oslo15:20
*** pmathews has quit IRC15:25
*** inc0 has quit IRC15:26
*** hockeynut has quit IRC15:31
*** hockeynut has joined #openstack-oslo15:32
*** ccrouch-afk is now known as ccrouch15:36
bnemecgus: We're keeping py26 compatibility in the oslo libs until all of our consumers (including client libs) have dropped it.15:42
bnemecIt's probably not necessary for rootwrap, but for the sake of sanity we made it a blanket policy.15:43
YorikSarttx: Ok, thanks.15:43
ttxYorikSar: done15:44
sdaguedhellmann: because of Depends-On - https://review.openstack.org/#/c/160407/ needs another +A or a +A toggle to move forward16:13
*** YorikSar has quit IRC16:15
*** ChuckC has joined #openstack-oslo16:17
*** ChuckC has quit IRC16:18
*** ChuckC has joined #openstack-oslo16:18
*** stevemar has joined #openstack-oslo16:21
otherwiseguyttx: responded to your comment on https://review.openstack.org/#/c/161183/16:23
openstackgerritOleksii Chuprykov proposed openstack/oslo.db: WIP Handle DBReferenceError for pypy  https://review.openstack.org/13764416:25
*** ihrachyshka has quit IRC16:25
openstackgerritJoshua Harlow proposed openstack/taskflow: Use links instead of raw block quotes  https://review.openstack.org/16176616:26
otherwiseguyttx: I almost removed my -1 just because no code in neutron actually passes environment variables to rootwrap so none of it really matters that much, but I just wanted to make sure everyone was aware of the behavior change between daemon/non-daemon.16:26
*** tsekiyama has joined #openstack-oslo16:28
*** salv-orlando has quit IRC16:29
openstackgerritSteve Martinelli proposed openstack/oslo.policy: deprecate policy_dirs option  https://review.openstack.org/16040716:31
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a bookshelf developer section  https://review.openstack.org/16153416:31
sdagueanyone up for an oslo_context question16:32
sdague?16:32
sdaguesince dims change nova to use it, one of our unit tests fails randomly16:32
sdaguehttp://logs.openstack.org/53/155853/16/check/gate-nova-python27/1f928bf/console.html#_2015-03-05_15_33_13_11616:32
sdaguehttps://github.com/openstack/nova/blob/master/nova/tests/unit/test_context.py#L141-L14716:33
sdagueand I'm trying to figure out if it's a bug in nova tests, or a bug in the oslo lib16:34
mriedemsdague: dims: https://bugs.launchpad.net/nova/+bug/142872816:34
openstackLaunchpad bug 1428728 in OpenStack Compute (nova) "nova.tests.unit.test_context.ContextTestCase.test_store_when_no_overwrite intermittently fails with MismatchError" [Undecided,New]16:34
dimssdague: logstash shows just 3 hits in 7 days?16:34
mriedemyes, on gate, 2 different changes16:34
sdaguedims: remove the gate restriction16:34
mriedemmore in check16:34
sdagueit's a bunch more16:35
dimsah ok16:35
sdagueand it's all since the oslo context code change16:35
dimsgive me a few sdague16:36
sdaguedims: ok, no prob16:36
*** salv-orlando has joined #openstack-oslo16:39
mriedemsdague: dims: comparing nova and oslo_context, the only difference i really see is the store16:42
mriedembut they might be the same underneath, o.c uses: _request_store = threading.local()16:42
mriedemnova used: from nova.openstack.common import local16:42
mriedemthe test is new16:43
mriedemso it might just be the test is buggy16:43
dimsmriedem: are you looking at latest code? should not have that import16:43
dimsmriedem: yes, i added that test16:43
mriedemdims: i'm comparing the old nova code to the current oslo_context RequestContext code16:43
mriedemthat code looks basically identical16:43
mriedemso it could just be a racy test16:43
sdaguemriedem: the test is really small16:44
sdaguethe only thing I can think is buggy is the fixture setup16:44
mriedemdoes the same test exist in o.c?16:44
sdaguethat's a good question16:44
dimsmriedem: yes16:44
mriedemk, i see it16:45
sdaguedims: yeh, but there are so few tests in context, they actually each run in their own worker for me16:45
mriedemsdague: if i do an e-r query for this, i'd have to restrict on gate given how generic it is16:46
mriedemmessage:"nova.tests.unit.test_context.ContextTestCase.test_store_when_no_overwrite" AND message:"FAILED" AND tags:"console"16:46
sdaguemriedem: it's such a small test, I think it's fine on check16:46
sdagueplus it's unit tests16:47
mriedemfamous last words :) we've had plenty of racy "unit" tests in nova16:47
mriedembut yeah, this is small, not a compute manager full run through16:47
*** pmathews has joined #openstack-oslo16:48
sdagueyeh, but unit tests are never supposed to be racey16:48
openstackgerritMatthew Booth proposed openstack/oslo.db: Writer -> mode  https://review.openstack.org/16177616:49
*** yamahata has quit IRC16:50
*** achanda has joined #openstack-oslo16:52
sdaguedims: so, honestly, I expect that this is untestable in oslo_context because there are too few tests to expose races here16:53
*** amotoki has quit IRC16:53
sdaguedims: oh, I know what the bug is. The fixture only does cleanup16:56
sdagueit doesn't ensure initial reset state16:56
sdaguedims: working up a patch16:58
dimssdague: aha, thanks17:00
dimssorry still doing something else, should be free in 5 mins17:00
*** zzzeek has joined #openstack-oslo17:03
*** pmathews has quit IRC17:05
*** achanda has quit IRC17:05
openstackgerritSean Dague proposed openstack/oslo.context: ensure we reset contexts when fixture is used  https://review.openstack.org/16178417:06
sdaguedims: that's the fix17:06
openstackgerritDan Krause proposed openstack/taskflow: add start and stop time to the timing listener  https://review.openstack.org/16178517:06
dimssdague: +217:08
dimsdhellmann: jd__: bnemec: one of you please bless this? https://review.openstack.org/#/c/161784/17:08
sdagueit would be nice to get a release on that soonish, so that we can fix that bug, as it's hitting us a decent amount17:08
dimssdague: ack, will do17:09
*** jaypipes has joined #openstack-oslo17:09
dhellmannhmm, I wonder if that will break anything that assumes the current behavior. I guess we'll find out.17:09
sdaguedhellmann: I hope not17:10
dhellmannyeah, I'm thinking of maybe a test that creates a context and uses this to clean up afterwards -- which is fixable, obviously17:10
dhellmannsdague: whitespace weirdness on line 3717:11
sdaguedhellmann: ok, let me look17:11
dhellmannthat's likely to fail pep8; looks like an errant carriage return17:11
sdagueyeh, duh17:12
sdaguefixing...17:12
openstackgerritSean Dague proposed openstack/oslo.context: ensure we reset contexts when fixture is used  https://review.openstack.org/16178417:12
*** yamahata has joined #openstack-oslo17:12
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module  https://review.openstack.org/13821517:13
*** dtantsur is now known as dtantsur|afk17:16
*** yamahata has quit IRC17:18
*** yamahata has joined #openstack-oslo17:19
*** pmathews has joined #openstack-oslo17:26
openstackgerritDan Krause proposed openstack/taskflow: Persistence backend refactor  https://review.openstack.org/16088917:27
*** pmathews has quit IRC17:27
*** pmathews has joined #openstack-oslo17:29
*** flwang has quit IRC17:50
*** harlowja_away is now known as harlowja17:58
*** sreshetn1ak has joined #openstack-oslo17:59
*** YorikSar has joined #openstack-oslo18:07
*** achanda has joined #openstack-oslo18:08
*** e0ne has quit IRC18:15
*** breton_ is now known as breton18:19
*** prad has quit IRC18:24
*** prad has joined #openstack-oslo18:24
*** cdent has quit IRC18:26
*** exploreshaifali has quit IRC18:29
*** pmathews1 has joined #openstack-oslo18:29
*** pmathews has quit IRC18:29
*** zzzeek has quit IRC18:33
*** pmathews1 has quit IRC18:34
openstackgerritJoshua Harlow proposed openstack/taskflow: Rename the timing listeners to duration listeners  https://review.openstack.org/16182918:36
openstackgerritMerged openstack/oslo.policy: deprecate policy_dirs option  https://review.openstack.org/16040718:37
*** pmathews has joined #openstack-oslo18:41
stevemardhellmann, are we going to release a new version of policy today? this guy ^ finally merged18:41
*** rharwood has left #openstack-oslo18:41
stevemartoday might be ambitious18:41
dhellmannstevemar: yeah, I can do that18:41
stevemaryippie18:42
*** dtantsur|afk has quit IRC18:43
dhellmannstevemar: oslo.policy 0.4.0 tagged18:43
dhellmannstevemar: do you want to send the email about the first public release?18:43
stevemarwe skipped 0.3.0 ? :)18:43
dhellmannapparently :-)18:44
dhellmannoh, no, I tagged it right18:44
dhellmann0.3.018:44
stevemarah, great18:44
stevemaroh sure!18:44
dhellmannI added 1 again between that window and my irc window :-)18:44
stevemardo you have a template?18:44
openstackgerritMerged openstack/oslo.utils: Allow providing a logger to save_and_reraise_exception  https://review.openstack.org/15815818:44
dhellmannfor the first release, I usually just write a little paragraph explaining what it is. We don't want the full release notes history thing.18:45
*** dtantsur has joined #openstack-oslo18:45
stevemari suppose i could just look at an old one on the ML18:45
stevemarokay cool18:45
dhellmannfor intermediate releases we have a script to produce the message, but for the first release it's nice to have the human touch :-)(18:46
*** amrith is now known as _amrith_18:46
*** _amrith_ is now known as amrith18:49
*** crc32 has joined #openstack-oslo18:50
*** pmathews has quit IRC18:56
*** pmathews has joined #openstack-oslo18:57
*** pmathews has quit IRC18:59
*** dims_ has joined #openstack-oslo19:01
*** pmathews has joined #openstack-oslo19:02
*** achanda has quit IRC19:04
*** dims has quit IRC19:04
openstackgerritMerged openstack/oslo.context: ensure we reset contexts when fixture is used  https://review.openstack.org/16178419:08
*** crc32 has quit IRC19:13
*** sreshetn1ak has quit IRC19:14
*** pmathews has quit IRC19:20
*** exploreshaifali has joined #openstack-oslo19:21
*** jamespage has quit IRC19:23
*** e0ne has joined #openstack-oslo19:24
*** pmathews has joined #openstack-oslo19:24
*** achanda has joined #openstack-oslo19:24
stevemardhellmann, sent a note :)19:31
dhellmannstevemar: thanks!19:33
*** exploreshaifali has quit IRC19:44
*** dims_ has quit IRC19:44
*** dims has joined #openstack-oslo19:45
openstackgerritJoshua Harlow proposed openstack/taskflow: Add FETCHING notification state (not persisted)  https://review.openstack.org/16185819:45
dimsdhellmann: can we please cut a fresh oslo.context release?19:46
dhellmanndims: I was waffling on that. How bad is the nova instability?19:46
dimsbad hits on check jobs and personal builds19:47
dhellmannok19:47
dhellmanncutting release now19:47
dimsthanks!19:47
dimsstevemar: "thrilled" :)19:49
dhellmanndims: was there a bug for this change?19:50
dimsdhellmann: no we did not file one19:50
dhellmanndims: we're public with context, right?19:52
dimsyes19:52
dhellmanntagged and release notes sent19:52
openstackgerritDoug Hellmann proposed openstack/oslo.context: fix bug tracker link  https://review.openstack.org/16186419:53
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo-incubator: Inject a bit more emotions to our releases  https://review.openstack.org/16186519:55
sabaridims: dhellmann: One of my glance bp for k3 needs a new release of oslo.vmware. Can you guys help me with it ?19:58
sabariI merged some new api's to oslo.vmware for https://blueprints.launchpad.net/glance/+spec/vmware-store-multiple-datastores19:58
dhellmannsabari: we can, but that lib has a lot of changes so can we wait until monday in case it introduces issues?19:59
sabaridhellmann: Thanks. Let me check with the glance guys on the cut-off :).20:00
sabaridhellmann: I think monday should be fine. If not, I will let you know.20:02
dhellmannsabari: I'll make a note to do it early monday20:03
*** flwang has joined #openstack-oslo20:03
sabaridhellmann: Thanks, btw I just see my patches since 0.10.0 release. Are you talking about changes in-review ?20:03
dhellmannsabari: oops, I was looking at 0.9.0 as the latest release20:04
dimsyay "chuffed" dhellmann!20:04
dhellmanndims: :-)20:05
dhellmanndims: talking with sabari about an oslo.vmware release, the only change that gives me reason to wait is https://review.openstack.org/#/c/156889/ -- what can you tell me there?20:06
dimslooking dhellmann20:07
dhellmann"not available on all systems" in the python docs makes me a little concerned20:07
sabaridhellmann: I remember the change came in after the bandit workgroup meetup ;)20:07
sabarilet me check on it.20:08
dimsdhellmann: i could not figure out where it is not available...20:10
dhellmannyeah, it sounds like there's an implementation for windows and unix. Maybe it's not there on some of the older UNIX or embedded platforms.20:10
dhellmanndims, sabari: ok, I think we're probably safe to release today.20:10
dimsdhellmann: let's roll the dice.20:10
* dhellmann rolls his eyes at dims' random joke20:11
dims:)20:11
sabarihaha20:11
sabaridhellmann: I was looking at a reference and it was your book :)20:13
dhellmannoslo.vmware 0.11.0 released20:13
dimssabari: dhellmann: "from Crypto.Random import random" apparently cool too20:13
dhellmannsabari: :-)20:13
sabaridhellmann: Thanks a lot !20:13
jungleboyjdhellmann: Let me know if you have a minute for a couple of questions about the config-generator stuff again.20:13
dhellmannjungleboyj: sure, what's up?20:13
jungleboyjSeeing a couple of things as I am trying to set up the cinder.opts file.20:14
jungleboyjdhellmann: First, it seems to not like if I am trying to add a single config item, seems it needs to be a list.  Otherwise it complains I am trying to add a Bool, etc .20:15
jungleboyjAm I doing something wrong trying to add a single config item?20:16
sabaridims: thanks for looking into it too :)20:16
dhellmannjungleboyj: have a patch I can look at?20:16
jungleboyjdhellmann: Give me a minute.20:17
dhellmannjungleboyj: or pastebin?20:17
jungleboyjdhellmann: Yeah, that was what I was going to do.20:17
jungleboyjdhellmann: Ok, I have something less invasive set up.  Let me put together a paste.20:19
jungleboyjdhellmann: http://paste.openstack.org/show/189566/20:22
dhellmannjungleboyj: so api_common.api_common_opts is one item?20:23
jungleboyjOther thing I don't like this is we have lost is the config file telling where the option is coming from.  All comes out as 'From Cinder' instead of 'From cinder.api.common' and 'From cinder.api.middleware.auth'20:24
jungleboyjdhellmann: Yes.20:24
dhellmannjungleboyj: ok, so you're adding an Opt to a list there. you can just make that copy.deepcopy([ api_common.api_common_opts, auth.use_forwarded_for_opt ])20:24
*** flwang has quit IRC20:24
dhellmannbut if the variable name ends in "opts" I would expect it to be plural, so making that variable a list might make more sense20:25
dhellmannwhy do deployers care what file an option is defined in?20:25
*** openstackgerrit has quit IRC20:25
jungleboyjSorry, api.common_opts is multiple items, auth.use.forwarded_for_opt is the single option.20:25
*** openstackgerrit has joined #openstack-oslo20:26
* dhellmann is thinking20:27
jungleboyjdhellmann: I guess it is fine to do what I did in the code and make the single options a list of one.  Just wanted to make sure I wasn't missing something.  :-)20:27
dhellmannI don't see anything obviously wrong with what you haev here, can you push a review that I can download to try this locally?20:28
jungleboyjSure.  Let me try one other thing here.20:28
dhellmannjungleboyj: is that version of list_opts the version that works, or the version that produces the traceback?20:29
dhellmannyou say "Otherwise use_forwarded_for fails to get added to the list..."20:29
*** pmathews has quit IRC20:30
jungleboyjThe paste?20:30
dhellmannyeah20:30
jungleboyjdhellmann: That is the one that works.20:30
dhellmannoh, ok, good20:31
* dhellmann was trying to figure out why a list was being interpreted as not a list20:31
jungleboyj:-)20:31
jungleboyjFear not.  I figured that out.20:31
dhellmannI think making the list of one item is what you want there20:31
jungleboyjdhellmann: Ok, that was what I had determined as well.20:31
dhellmannas far as including the module names in the output, I think that was a deliberate choice. The namespace is the important info there, deployers shouldn't have to worry about module names.20:32
jungleboyjOk, maybe I am too developer centered and do care.20:33
dhellmannyeah, trade-offs :-/20:33
jungleboyjThe other option is to do this: http://paste.openstack.org/show/189586/20:34
jungleboyjdhellmann: Check the bottom of the paste.20:35
dhellmannjungleboyj: that first value is the group name, so the it needs to reflect the correct section in the config file where the deployer should set the option20:36
jungleboyjdhellmann: Ok, so that is not the right solution. :-)20:36
dhellmannyeah20:36
jungleboyjOk, let me work up the patch doing the concatenated list of options under [cinder].20:37
dhellmannwe could talk about adding making it possible to return a 3-tuple where the last item is the module name, but that's going to make constructing the list of return values more complex20:37
dhellmannjungleboyj: do you use "cinder" instead of "DEFAULT" for the main group name?20:37
*** achanda has quit IRC20:38
jungleboyjdhellmann: Good point.  All of these should be under DEFAULT.20:39
jungleboyjhttps://github.com/openstack/cinder/blob/stable/icehouse/etc/cinder/cinder.conf.sample20:40
jungleboyjIn the old config file these all went under [DEFAULT] with the module it came from in the comment lest.20:40
jungleboyj*list20:40
dhellmannyeah20:41
*** achanda has joined #openstack-oslo20:41
jungleboyjdhellmann: Ok, there we go.  I will go forward with this.  If the rest of the cinder guys try to lynch me we may need to talk about changing this so we can indicate the module as before.20:43
* jungleboyj is starting to worry that I won't be able to get this into Kilo.20:43
dhellmannAt this point that change will have to wait until liberty, since we're coming up on feature freeze next week.20:43
jungleboyjdhellmann: Yeah, I am getting push back on getting more changes for Oslo into Cinder.20:43
jungleboyj:-(20:44
jungleboyjdhellmann: Thanks for answering my questions.  You rock!20:44
*** david-lyle_afk has joined #openstack-oslo20:45
dhellmannjungleboyj: any time, and let me know if I can help with reviews on those updates20:45
jungleboyjdhellmann: Will do.20:46
*** david-lyle_afk has quit IRC20:47
*** david-lyle_afk has joined #openstack-oslo20:47
*** salv-orlando is now known as ihate-salv-orlan20:59
*** ihate-salv-orlan is now known as salv-orlando21:00
*** boris-42 has quit IRC21:02
*** david-lyle_afk is now known as david-lyle21:03
*** crc32 has joined #openstack-oslo21:05
*** ajo has quit IRC21:05
*** achanda has quit IRC21:09
*** flwang has joined #openstack-oslo21:15
*** boris-42 has joined #openstack-oslo21:19
*** jamespage has joined #openstack-oslo21:20
*** jamespage has quit IRC21:20
*** jamespage has joined #openstack-oslo21:20
*** kgiusti has left #openstack-oslo21:20
openstackgerritMerged openstack/oslo.utils: Add a stopwatch + split for duration(s)  https://review.openstack.org/15431121:21
*** pmathews has joined #openstack-oslo21:27
*** achanda has joined #openstack-oslo21:30
*** miguelgrinberg has joined #openstack-oslo21:32
*** achanda has quit IRC21:32
sigmavirus24dhellmann: quick pbr question if you have a second21:34
*** achanda has joined #openstack-oslo21:34
sigmavirus24actually, one sec21:34
sigmavirus24miguelgrinberg: did you try (from heat/) doing pip install contrib/{plugin-name} ?21:34
miguelgrinberglet me try that, one sec21:34
sigmavirus24(instead of pushd contrib/{plugin-name}; pip install . ; popd)21:35
miguelgrinbergfails in the same way21:35
sigmavirus24weird21:35
sigmavirus24miguelgrinberg: is there a stacktrace you can put on paste.openstack.org ?21:36
miguelgrinbergyes, but it's not very exciting, it's a standard "pbr cannot find a version" stack trace21:36
*** jungleboyj has quit IRC21:37
*** achanda has quit IRC21:37
miguelgrinbergsigmavirus24: here you go: http://paste.openstack.org/show/189635/21:38
*** achanda has joined #openstack-oslo21:39
*** jecarey has quit IRC21:41
sigmavirus24dhellmann: ^21:43
dhellmannsigmavirus24, miguelgrinberg : looking21:48
miguelgrinbergdhellmann: you are going to need some background I think21:48
dhellmannmiguelgrinberg: likely. Is there a git repo I can checkout?21:48
miguelgrinbergthis particular package is a sub-package of heat, and does not have a version21:48
miguelgrinbergdhellmann: https://github.com/openstack/heat/tree/master/contrib/extraroute21:49
miguelgrinbergif you clone heat, then go into contrib/extraroute and do a "pip install ." you can repro the problem21:49
dhellmannmiguelgrinberg: why does it not have a version and why is it a subpackage?21:51
miguelgrinbergdhellmann: it's an optional plugin for heat. Don't know why it does not have a version, none of the plugins there have an explicit version.21:51
dhellmannmiguelgrinberg: ok, pbr wants them to have a version somewhere. If you're not using the git tag as the version, you could try adding a version to their setup.cfg21:52
miguelgrinbergI want to use the git tag, but because of the way the installation goes it is not accessible21:52
dhellmannin general though, we try to keep it to one python package per git repository because we're using the tags as the version and to trigger publishing releases21:52
dhellmannmiguelgrinberg: you'll fight less with the tools if you create a separate git repository for each installable thing21:53
miguelgrinbergdhellmann: I agree with that view of one package per repo21:53
miguelgrinbergbut I don't control heat :)21:53
dhellmannmiguelgrinberg: but as I said, you could also try adding a version line to the setup.cfg21:53
miguelgrinbergI will propose adding versions though21:53
miguelgrinbergI tried that, btw, with an explicit version everything works21:55
dhellmannok, well, that's good at least21:55
dhellmannhow are these things published? are they?21:56
miguelgrinbergthey are "contrib" type things, most people don't use them I guess21:56
miguelgrinbergI can also get it installed using "python setup.py install", without pip everything works also21:57
dhellmannI don't think our packaging jobs will know how to deal with them in subdirectories, so unless someone has done custom work or they're just being included in heat's package all the time I don't know how they're being used21:57
miguelgrinbergbecause there is no move to /tmp21:57
miguelgrinbergdhellmann: yeah, I agree, it's hard to pinpoint a bug here21:58
dhellmannmiguelgrinberg: as far as I can tell, pbr is working as designed in this case21:58
openstackgerritDoug Hellmann proposed openstack/oslo-incubator: Update projects ignored for unreleased changes  https://review.openstack.org/16127522:00
*** jaypipes has quit IRC22:08
dimse0ne: sigmavirus24: saw the oslo.log reviews filed for cinder and glance. yay!22:09
sigmavirus24dims: heh, thanks for reminding me about it22:11
e0nedims, sigmavirus24: we are going to merge oslo.log o cinder on march 11th after our feature freaze to not force people to rebase22:11
sigmavirus24I need to annoy kragniz22:11
sigmavirus24we lost track of the graduated modules22:11
sigmavirus24(actually kragniz did 90% of the work)22:11
dimse0ne: cool22:16
dimsthanks kragniz22:16
*** jecarey has joined #openstack-oslo22:17
*** jungleboyj has joined #openstack-oslo22:17
*** stevemar has quit IRC22:20
*** stevemar has joined #openstack-oslo22:20
openstackgerritDoug Hellmann proposed openstack/oslo.config: Expand and clean up documentation for the config generator  https://review.openstack.org/16142522:35
openstackgerritDoug Hellmann proposed openstack/oslo.config: Add FAQ section to documentation  https://review.openstack.org/15754622:35
openstackgerritJoshua Harlow proposed openstack/taskflow: Add FETCHING notification state (not persisted)  https://review.openstack.org/16185822:35
*** achanda has quit IRC22:36
openstackgerritDoug Hellmann proposed openstack/oslo.config: Expand and clean up documentation for the config generator  https://review.openstack.org/16142522:38
openstackgerritDoug Hellmann proposed openstack/oslo.config: Add FAQ section to documentation  https://review.openstack.org/15754622:38
*** gordc has quit IRC22:38
*** achanda has joined #openstack-oslo22:40
*** e0ne has quit IRC22:44
openstackgerritJoshua Harlow proposed openstack/taskflow: Use links instead of raw block quotes  https://review.openstack.org/16176622:44
*** pmathews has quit IRC22:51
*** pmathews has joined #openstack-oslo22:52
*** mriedem is now known as mriedem_away22:55
*** takedakn has joined #openstack-oslo22:55
*** jaypipes has joined #openstack-oslo22:57
morganfainberghmmm no zzzeeeek huh?22:57
*** prad has quit IRC23:03
*** achanda has quit IRC23:05
*** achanda has joined #openstack-oslo23:08
openstackgerritEric Brown proposed openstack/oslo.config: Generate help text indicating possible values  https://review.openstack.org/16165523:18
*** ChuckC is now known as ChuckC_23:19
*** pmathews has quit IRC23:19
*** ChuckC_ is now known as ChuckC23:26
*** stpierre has quit IRC23:26
*** jaypipes has quit IRC23:28
*** takedakn has quit IRC23:31
gusbnemec: ack, thanks (re py26).  I figured oslo might be more conservative than others in this regard.23:32
*** sigmavirus24 is now known as sigmavirus24_awa23:32
openstackgerritJoshua Harlow proposed openstack/debtcollector: By default mutate the docstring(s) for deprecations  https://review.openstack.org/15598823:33
openstackgerritJoshua Harlow proposed openstack/taskflow: Add ability to use tooz to find and advertise workers  https://review.openstack.org/15149523:43
openstackgerritSteve Martinelli proposed openstack/oslo-incubator: Remove policy from oslo-incubator  https://review.openstack.org/15281223:51
*** jungleboyj has quit IRC23:51
openstackgerritSteve Martinelli proposed openstack/oslo-incubator: Prevent update.py from updating policy  https://review.openstack.org/15281323:51
*** jungleboyj has joined #openstack-oslo23:52
*** openstack has joined #openstack-oslo23:53

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