Thursday, 2015-02-12

*** jecarey has quit IRC00:00
*** dims__ has joined #openstack-oslo00:04
openstackgerritIan Wienand proposed openstack/oslo.config: Clear up MultiStrOpt and related documentation  https://review.openstack.org/13261300:07
*** gordc has quit IRC00:07
*** mriedem has joined #openstack-oslo00:15
*** achanda has quit IRC00:16
openstackgerritJoshua Harlow proposed openstack/debtcollector: Add examples of using the new removals decorator  https://review.openstack.org/15512100:21
harlowjamtreinish ^00:21
openstackgerritJoshua Harlow proposed openstack/debtcollector: Format the method/class removals messages like the others  https://review.openstack.org/15512400:29
*** vigneshvar has quit IRC00:34
openstackgerritJoshua Harlow proposed openstack/taskflow: Use the new table length constants  https://review.openstack.org/12616600:40
openstackgerritJoshua Harlow proposed openstack/oslo.serialization: Correctly load and dump items with datetime.date(s)  https://review.openstack.org/15513300:53
openstackgerritJoshua Harlow proposed openstack/oslo.serialization: Avoid using strtime for serializing datetimes  https://review.openstack.org/15454700:56
openstackgerritJoshua Harlow proposed openstack/oslo.serialization: Correctly load and dump items with datetime.date(s)  https://review.openstack.org/15513300:56
*** dims__ has quit IRC00:58
*** ChuckC has quit IRC01:02
*** ChuckC has joined #openstack-oslo01:03
*** daniel3__ has quit IRC01:10
openstackgerritJoshua Harlow proposed openstack/debtcollector: Match the updated openstack-manuals description  https://review.openstack.org/15514701:12
*** achanda has joined #openstack-oslo01:16
*** hemna is now known as hemnaf01:17
*** hemnaf is now known as hemnafk01:17
openstackgerritCsaba Henk proposed openstack/oslo.concurrency: processutils: execute(): fix option incompatibility  https://review.openstack.org/14703301:18
*** achanda has quit IRC01:18
*** achanda has joined #openstack-oslo01:19
*** tsekiyam_ has joined #openstack-oslo01:26
*** amotoki has joined #openstack-oslo01:27
*** amotoki has quit IRC01:28
*** tsekiyama has quit IRC01:28
*** tsekiyam_ has quit IRC01:30
*** ViswaV has quit IRC01:37
*** sigmavirus24 is now known as sigmavirus24_awa01:43
*** ChuckC_ has joined #openstack-oslo01:46
*** noelbk has quit IRC01:48
*** ChuckC has quit IRC01:49
dstanekdhellmann: you around?01:59
*** mtanino has quit IRC02:04
*** achanda has quit IRC02:06
*** zzzeek has quit IRC02:08
*** achanda has joined #openstack-oslo02:12
*** salv-orlando has quit IRC02:15
*** MasterPiece has quit IRC02:19
*** MasterPiece has joined #openstack-oslo02:28
*** MasterPiece| has joined #openstack-oslo02:29
*** zzzeek has joined #openstack-oslo02:30
*** dims__ has joined #openstack-oslo02:32
*** MasterPiece has quit IRC02:33
*** tsekiyama has joined #openstack-oslo02:36
openstackgerritJoshua Harlow proposed openstack/taskflow: Chain exceptions correctly on py3.x (WIP)  https://review.openstack.org/15516802:39
*** zzzeek has quit IRC02:40
*** tsekiyama has quit IRC02:41
*** xek_ has joined #openstack-oslo02:45
*** rodrigod` has joined #openstack-oslo02:51
*** xek has quit IRC02:52
*** rodrigods has quit IRC02:52
*** david-lyle is now known as david-lyle_afk02:59
openstackgerritJoshua Harlow proposed openstack/taskflow: Chain exceptions correctly on py3.x (WIP)  https://review.openstack.org/15516803:01
*** zzzeek has joined #openstack-oslo03:04
openstackgerritJoshua Harlow proposed openstack/taskflow: Chain exceptions correctly on py3.x (WIP)  https://review.openstack.org/15516803:05
*** yamahata has quit IRC03:06
*** takedakn has joined #openstack-oslo03:14
*** salv-orlando has joined #openstack-oslo03:16
*** zzzeek has quit IRC03:17
*** salv-orlando has quit IRC03:21
*** MasterPiece| has quit IRC03:21
*** salv-orlando has joined #openstack-oslo03:36
*** dims__ has quit IRC03:37
*** salv-orlando has quit IRC03:40
*** salv-orlando has joined #openstack-oslo03:41
*** MasterPiece has joined #openstack-oslo03:51
*** yamahata has joined #openstack-oslo03:52
*** takedakn has quit IRC03:53
*** mriedem has quit IRC03:55
*** salv-orlando has quit IRC03:57
*** achanda has quit IRC03:59
*** salv-orlando has joined #openstack-oslo04:01
*** achanda has joined #openstack-oslo04:02
*** salv-orlando has quit IRC04:05
*** rushiagr_away is now known as rushiagr04:06
*** harlowja is now known as harlowja_away04:08
*** achanda has quit IRC04:10
*** salv-orlando has joined #openstack-oslo04:37
*** MasterPiece| has joined #openstack-oslo04:38
*** sileht has quit IRC04:40
*** MasterPiece has quit IRC04:40
*** salv-orlando has quit IRC04:41
*** rushiagr is now known as rushiagr_away04:48
*** harlowja_at_home has joined #openstack-oslo04:49
openstackgerritJoshua Harlow proposed openstack/oslo.serialization: Avoid using strtime for serializing datetimes  https://review.openstack.org/15454704:55
openstackgerritJoshua Harlow proposed openstack/oslo.serialization: Correctly load and dump items with datetime.date(s)  https://review.openstack.org/15513304:55
*** salv-orlando has joined #openstack-oslo04:59
*** salv-orlando has quit IRC05:02
*** salv-orlando has joined #openstack-oslo05:05
*** ViswaV has joined #openstack-oslo05:07
*** salv-orlando has quit IRC05:12
openstackgerritJoshua Harlow proposed openstack/taskflow: Chain exceptions correctly on py3.x  https://review.openstack.org/15516805:14
*** salv-orlando has joined #openstack-oslo05:15
openstackgerritJoshua Harlow proposed openstack/taskflow: Use the ability to chain exceptions correctly (WIP)  https://review.openstack.org/15518705:17
*** harlowja_at_home has quit IRC05:18
*** ajo has joined #openstack-oslo05:24
*** amotoki has joined #openstack-oslo05:25
*** ankit_ag has joined #openstack-oslo05:25
*** ankit_ag has joined #openstack-oslo05:27
*** salv-orlando has quit IRC05:30
*** salv-orlando has joined #openstack-oslo05:40
*** dtroyer has left #openstack-oslo05:41
openstackgerritSyed Ismail Faizan Barmawer proposed openstack/oslo-incubator: Add common Ironic modules as libironic  https://review.openstack.org/15519005:49
*** MasterPiece| has quit IRC05:51
*** salv-orlando has quit IRC05:52
*** achanda has joined #openstack-oslo05:52
*** achanda has quit IRC05:56
*** sileht has joined #openstack-oslo06:04
*** achanda has joined #openstack-oslo06:08
*** salv-orlando has joined #openstack-oslo06:08
*** ViswaV has quit IRC06:09
*** crc32 has joined #openstack-oslo06:15
*** salv-orlando has quit IRC06:15
*** salv-orlando has joined #openstack-oslo06:16
*** salv-orlando has quit IRC06:18
*** salv-orlando has joined #openstack-oslo06:21
*** salv-orlando has quit IRC06:25
*** boris-42 has joined #openstack-oslo06:33
*** dims__ has joined #openstack-oslo06:38
*** dims__ has quit IRC06:42
*** salv-orlando has joined #openstack-oslo06:49
*** vigneshvar has joined #openstack-oslo06:52
*** salv-orlando has quit IRC06:56
*** inc0 has joined #openstack-oslo06:57
inc0good morning06:57
*** daniel3_ has joined #openstack-oslo06:58
openstackgerritMerged openstack/taskflow: Modify stop and add wait on conductor to prevent lockups  https://review.openstack.org/15428107:08
openstackgerritMerged openstack/taskflow: Let the multi-lock convert the provided value to a tuple  https://review.openstack.org/15477007:08
openstackgerritMerged openstack/taskflow: Use the new table length constants  https://review.openstack.org/12616607:08
openstackgerritMerged openstack/taskflow: DFS in right order when not starting at the provided node  https://review.openstack.org/15470707:08
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: NotifyPublisher need handle amqp_auto_delete  https://review.openstack.org/13872307:10
*** burgobianco has quit IRC07:12
*** takedakn has joined #openstack-oslo07:29
*** achanda has quit IRC07:31
*** achanda has joined #openstack-oslo07:37
*** burgobianco has joined #openstack-oslo07:38
*** e0ne has joined #openstack-oslo07:40
*** noelbk has joined #openstack-oslo07:41
*** flaper87 has quit IRC07:41
*** flaper87 has joined #openstack-oslo07:41
openstackgerritMerged openstack/oslo.messaging: amqp1: fix functional tests deps  https://review.openstack.org/15375407:45
*** jaosorior has joined #openstack-oslo07:47
*** xek_ has quit IRC07:48
*** xek_ has joined #openstack-oslo07:48
*** boris-42 has quit IRC07:49
*** boris-42 has joined #openstack-oslo07:49
*** inc0 has quit IRC07:49
*** inc0 has joined #openstack-oslo07:49
*** jaosorior has quit IRC07:49
*** jaosorior has joined #openstack-oslo07:49
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: rabbit: Fix behavior of rabbit_use_ssl  https://review.openstack.org/15444607:56
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: rabbit: allow to configure heartbeat rate  https://review.openstack.org/15220107:56
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: rabbit: heartbeat implementation  https://review.openstack.org/14604707:56
*** dtantsur|afk is now known as dtantsur08:02
*** stevemar has quit IRC08:04
*** salv-orlando has joined #openstack-oslo08:11
*** e0ne has quit IRC08:17
*** openstackgerrit has quit IRC08:21
*** openstackgerrit has joined #openstack-oslo08:21
*** dulek has joined #openstack-oslo08:29
*** achanda has quit IRC08:35
*** andreykurilin_ has joined #openstack-oslo08:50
*** andreykurilin__ has joined #openstack-oslo09:01
*** andreykurilin_ has quit IRC09:01
*** i159 has joined #openstack-oslo09:21
*** vipul has quit IRC09:22
*** amotoki has quit IRC09:23
*** takedakn has quit IRC09:24
*** vipul has joined #openstack-oslo09:25
*** amotoki has joined #openstack-oslo09:28
*** e0ne has joined #openstack-oslo09:31
*** dtantsur is now known as dtantsur|bbl10:09
*** e0ne is now known as e0ne_10:21
openstackgerritMerged openstack/oslo.db: Organize provisioning to use testresources  https://review.openstack.org/12087010:25
*** e0ne_ is now known as e0ne10:30
openstackgerritMerged openstack/oslo.versionedobjects: Rename tests/fixtures to test/obj_fixtures to avoid import collisions  https://review.openstack.org/15497110:36
*** cdent has joined #openstack-oslo10:44
*** ihrachyshka has joined #openstack-oslo10:51
*** salv-orlando has quit IRC10:53
*** yamahata has quit IRC10:59
*** dtantsur|bbl is now known as dtantsur10:59
*** dims__ has joined #openstack-oslo11:15
*** inc0_ has joined #openstack-oslo11:17
*** inc0 has quit IRC11:21
*** salv-orlando has joined #openstack-oslo11:27
*** inc0_ has quit IRC11:28
*** khivin has quit IRC11:31
*** khivin has joined #openstack-oslo11:31
*** e0ne is now known as e0ne_11:48
*** rodrigod` is now known as rodrigods11:49
*** e0ne_ has quit IRC11:58
*** amotoki has quit IRC11:59
*** MasterPiece has joined #openstack-oslo12:05
*** MasterPiece has quit IRC12:12
*** MasterPiece has joined #openstack-oslo12:22
*** salv-orlando has quit IRC12:29
*** dims_ has joined #openstack-oslo12:37
*** dims__ has quit IRC12:38
*** inc0 has joined #openstack-oslo12:51
*** dims_ has quit IRC12:57
openstackgerritMichal Jastrzebski (inc0) proposed openstack/oslo.versionedobjects: Remove references to Nova  https://review.openstack.org/15440113:00
*** _amrith_ is now known as amrith13:01
*** salv-orlando has joined #openstack-oslo13:03
openstackgerritMichal Jastrzebski (inc0) proposed openstack/oslo.versionedobjects: New field type - Json  https://review.openstack.org/15482013:05
*** kgiusti has joined #openstack-oslo13:05
*** dims__ has joined #openstack-oslo13:05
*** e0ne has joined #openstack-oslo13:05
*** alexpilotti has joined #openstack-oslo13:10
*** dims__ has quit IRC13:13
openstackgerritMerged openstack/oslo.versionedobjects: Replace metaclass-based registry with a decorator-based one  https://review.openstack.org/15497213:16
openstackgerritMerged openstack/oslo.versionedobjects: Remove the ambiguous context argument on remotable methods  https://review.openstack.org/15504813:18
*** dims__ has joined #openstack-oslo13:19
openstackgerritMehdi Abaakouk proposed openstack/tooz: Add an in-memory driver  https://review.openstack.org/15530213:30
openstackgerritMehdi Abaakouk proposed openstack/tooz: TODO: don't use weakref.WeakRefValueDict  https://review.openstack.org/15530313:30
openstackgerritMehdi Abaakouk proposed openstack/tooz: TODO: don't use weakref.WeakRefValueDict  https://review.openstack.org/15530313:30
silehtjd__, I have written a lock driver with threading.Lock()13:31
silehtjd__, the issue is always triggred13:31
jd__makes sense13:31
jd__you mean the Gnocchi issue right?13:31
silehtjd__, it's perhaps related :p13:32
silehtjd__, with the memory driver in gnocchi, I got a ton of other error like this13:32
jd__memory driver?13:33
silehtjd__, the threading.Lock one13:33
jd__ah13:33
jd__but that's not surprise13:33
silehtjd__, now we have to remove the WeakRefValueDict, but I don't known how to track the lock13:33
jd__yeah that's the issue :/13:33
silehtjd__, I can add a memory leak :p that will fix the issue13:34
silehtjd__, perhaps something like using Weakref only on released lock ?13:34
*** e0ne is now known as e0ne_13:35
silehtjd__, lock acquired go into a normal dict, and released lock into a WeakRefValueDict13:35
* sileht will try13:35
jd__ok :)13:38
*** amotoki has joined #openstack-oslo13:48
*** e0ne_ is now known as e0ne13:48
openstackgerritMehdi Abaakouk proposed openstack/tooz: TODO: don't use weakref.WeakRefValueDict  https://review.openstack.org/15530313:50
openstackgerritMehdi Abaakouk proposed openstack/tooz: Don't always use weakref.WeakRefValueDict to track lock  https://review.openstack.org/15530313:52
silehtjd__, WDYT ? ^13:52
*** amotoki_ has joined #openstack-oslo14:00
*** sigmavirus24_awa is now known as sigmavirus2414:01
*** amotoki__ has joined #openstack-oslo14:02
*** amotoki has quit IRC14:02
*** amotok___ has joined #openstack-oslo14:02
*** vigneshvar has quit IRC14:05
*** amotoki_ has quit IRC14:05
*** miqui has joined #openstack-oslo14:06
*** amotoki__ has quit IRC14:06
*** denis_makogon has joined #openstack-oslo14:08
*** salv-orlando has quit IRC14:08
*** salv-orlando has joined #openstack-oslo14:08
*** salv-orlando has quit IRC14:13
openstackgerritMerged openstack/oslo.serialization: Avoid using strtime for serializing datetimes  https://review.openstack.org/15454714:13
openstackgerritMerged openstack/oslo.serialization: Correctly load and dump items with datetime.date(s)  https://review.openstack.org/15513314:13
*** miqui_ has joined #openstack-oslo14:13
*** gordc has joined #openstack-oslo14:24
*** david-lyle_afk is now known as david-lyle14:25
*** jecarey has joined #openstack-oslo14:25
*** ankit8188 has joined #openstack-oslo14:27
*** alexpilotti has quit IRC14:28
*** alexpilotti has joined #openstack-oslo14:29
*** ankit_ag has quit IRC14:29
*** boris-42 has quit IRC14:32
*** aagrawal has joined #openstack-oslo14:35
*** aagrawal has quit IRC14:36
*** cdent has quit IRC14:38
*** cdent has joined #openstack-oslo14:39
*** ankit8188 has quit IRC14:39
*** mriedem has joined #openstack-oslo14:39
*** boris-42 has joined #openstack-oslo14:42
andrearosaam I correct if I say that in oslo_messaging 'rpc_notifier' and 'messaging' as notification_driver are the same thing?14:47
*** mtanino has joined #openstack-oslo14:50
openstackgerritMerged openstack/oslo.messaging: rabbit: Fix behavior of rabbit_use_ssl  https://review.openstack.org/15444614:56
*** inc0 has quit IRC15:01
*** prad has joined #openstack-oslo15:06
*** bryansd has joined #openstack-oslo15:07
*** bryansd has left #openstack-oslo15:07
*** zzzeek has joined #openstack-oslo15:13
*** salv-orlando has joined #openstack-oslo15:14
*** stevemar has joined #openstack-oslo15:15
*** pblaho_ has joined #openstack-oslo15:16
*** pblaho has quit IRC15:17
*** salv-orlando has quit IRC15:17
*** pblaho_ is now known as pblaho15:23
*** e0ne is now known as e0ne_15:23
*** jgrimm is now known as zz_jgrimm15:24
*** e0ne_ is now known as e0ne15:35
*** achanda has joined #openstack-oslo15:36
openstackgerritMehdi Abaakouk proposed openstack/tooz: Add an in-memory driver  https://review.openstack.org/15530215:39
openstackgerritMehdi Abaakouk proposed openstack/tooz: Don't always use weakref.WeakRefValueDict to track lock  https://review.openstack.org/15530315:39
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: rabbit: allow to configure heartbeat rate  https://review.openstack.org/15220115:42
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: rabbit: heartbeat implementation  https://review.openstack.org/14604715:42
openstackgerritVictor Sergeyev proposed openstack/oslo.db: Use PyMySQL as DB driver in py3 environment  https://review.openstack.org/13396215:43
*** achanda has quit IRC15:44
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: rabbit: heartbeat implementation  https://review.openstack.org/14604715:44
*** crc32 has quit IRC15:45
openstackgerritMehdi Abaakouk proposed openstack/tooz: Don't always use weakref.WeakRefValueDict to track lock  https://review.openstack.org/15530315:47
*** amotok___ has quit IRC15:48
*** salv-orlando has joined #openstack-oslo15:54
*** jecarey_ has joined #openstack-oslo15:58
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: rabbit: heartbeat implementation  https://review.openstack.org/14604715:58
*** jecarey has quit IRC16:01
*** jaosorior has quit IRC16:01
*** zz_jgrimm is now known as jgrimm16:05
*** salv-orlando has quit IRC16:08
*** daniel3_ has quit IRC16:09
*** salv-orlando has joined #openstack-oslo16:14
*** dulek has quit IRC16:20
*** ihrachyshka has quit IRC16:22
*** SlickNik has quit IRC16:24
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Remove references to Nova  https://review.openstack.org/15440116:25
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Compatibility with Python 3  https://review.openstack.org/15440716:25
*** yamahata has joined #openstack-oslo16:26
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Remove more Nova-isms  https://review.openstack.org/15539316:30
*** SlickNik has joined #openstack-oslo16:30
silehtjd__, in my new test case break all drivers except redis/zookeeper : https://review.openstack.org/155303 \o/16:30
jd__sileht: nice16:31
*** tsekiyama has joined #openstack-oslo16:33
silehtjd__, I have added a check that a second call of acquire(blocking=False) should return False and it returns True for most drivers16:33
dansmithdims__: rebased inc0's patches ^16:33
dims__dansmith: ack, will look in a little bit16:33
*** tsekiyama has quit IRC16:37
*** tsekiyama has joined #openstack-oslo16:38
jd__sileht: are you working on fixing it then?16:42
silehtjd__, I try16:42
jd__k16:42
silehtjd__, looks like mysql GET_LOCK is a RLock()16:43
sileht"If you have a lock obtained with GET_LOCK(), it is released when you execute RELEASE_LOCK(), execute a new GET_LOCK(), or your connection terminates (either normally or abnormally)."16:43
jd__err16:43
jd__well it's not even a real RLock16:43
jd__it's a weird lock16:43
silehtthat the name of this one :)16:44
silehtjd__, Are you ok on the fact that my test case should work with all drivers ?16:45
*** yamahata has quit IRC16:45
*** noelbk has quit IRC16:46
*** yamahata has joined #openstack-oslo16:46
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Remove more Nova-isms  https://review.openstack.org/15539316:46
silehtjd__, what kind of lock the tooz one is ? I guess a normal Lock()16:47
jd__sileht: yes16:47
jd__a RLock should be release only if you call release() the same number of times that you call acquire() IIRC16:48
jd__which is not the case with MySQL AFAIU16:48
silehtjd__, yes16:48
jd__so yes your test should pass16:48
silehtjd__, ok, so pgsql and mysql are broken :(16:48
silehtjd__, does it resonable that each Lock instance have this one sql connection ?16:49
*** ChuckC_ has quit IRC16:51
*** ChuckC_ has joined #openstack-oslo16:51
*** ChuckC_ is now known as ChuckC16:51
*** ihrachyshka has joined #openstack-oslo16:53
*** e0ne has quit IRC16:54
jd__sileht: don't know, what's your alternative?16:54
silehtjd__, to get GET_LOCK() working as we want we need to do it into a new sql connection16:54
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Remove more Nova-isms  https://review.openstack.org/15539316:55
*** e0ne has joined #openstack-oslo16:55
*** dims__ has quit IRC16:56
*** daniel3_ has joined #openstack-oslo16:56
jd__sileht: go ahead :)16:57
*** dims__ has joined #openstack-oslo16:57
*** i159 has quit IRC17:03
*** e0ne has quit IRC17:05
*** e0ne has joined #openstack-oslo17:06
*** noelbk has joined #openstack-oslo17:07
*** pblaho_ has joined #openstack-oslo17:10
*** pblaho has quit IRC17:13
*** ViswaV has joined #openstack-oslo17:14
*** jecarey has joined #openstack-oslo17:17
dims__dansmith: the py34 seems to fail for me - http://paste.openstack.org/show/172210/17:19
dims__(https://review.openstack.org/#/c/154407/)17:19
*** hemnafk is now known as hemna17:19
*** jecarey_ has quit IRC17:20
dansmithdims__: hmm, seems clearly missing the module17:21
dims__dansmith: add something like this? ./oslo.vmware/requirements-py3.txt:eventlet>=0.16.117:22
dansmithoh, is there a different requirements file?17:23
dansmithdims__: if that works, let me know and I can amend that patch17:23
*** vks has joined #openstack-oslo17:24
*** dtantsur is now known as dtantsur|afk17:26
*** salv-orlando has quit IRC17:29
*** salv-orlando has joined #openstack-oslo17:29
vkshi , i am facing one issue . I have my configuration file http://pastebin.com/j6XWaiXh   but I am not getting the value of "NODE_NAME" , i am using -> CONF.register_opt(cfg.StrOpt('NODE_NAME'), group='HA')17:30
vkswhere am i doing wrong???17:31
dims__dansmith: i copied requirements.txt into requirements-py3.txt and added that line, then patched tox.ini - http://paste.openstack.org/show/172227/ then the tests pass fine17:35
dansmithdims__: sweet, lemme amend17:36
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Compatibility with Python 3  https://review.openstack.org/15440717:38
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Remove more Nova-isms  https://review.openstack.org/15539317:38
dansmithdims__: ^17:38
vksdansmith, i am having one issue , can u suggest anything?17:38
*** pblaho__ has joined #openstack-oslo17:39
dansmithvks: I dunno, sorry17:40
dims__dansmith: need to add "eventlet>=0.16.1" to the py3.txt17:40
dansmithdims__: doh17:40
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Compatibility with Python 3  https://review.openstack.org/15440717:41
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Remove more Nova-isms  https://review.openstack.org/15539317:41
*** ChuckC has quit IRC17:42
*** ChuckC has joined #openstack-oslo17:42
dims__vks: looks right to me, compare with the test case - http://git.openstack.org/cgit/openstack/oslo.config/tree/tests/test_cfg.py#n160417:42
*** pblaho_ has quit IRC17:43
khivinHello17:47
*** bknudson has quit IRC17:47
vksdims__, i changed that to cfg.CONF.register_opt(cfg.StrOpt('NODE_NAME'), cfg.OptGroup('HA')) ----- i am getting same value "OPENSTACK"17:50
dims__vks paste me both your code and the config file17:51
*** andreykurilin__ has quit IRC17:52
vksdims__, http://pastebin.com/LH0Wuija17:59
*** ViswaV has quit IRC18:00
dims__vks: http://paste.openstack.org/show/172249/18:06
dims__vks: that code prints me NODE118:07
morganfainbergdhellmann, when / if you have a chance I'd love some feedback on https://review.openstack.org/#/c/148354/18:08
vksdims__, let me chk18:08
khivinI am sorry, but can someone help me with the rpc api issue?18:09
khivinIs it possible in oslo.messaging to create rpc server like this https://github.com/openstack/heat/blob/master/heat/engine/service.py#L238 but using temporary queue18:09
*** salv-orlando has quit IRC18:10
*** salv-orlando has joined #openstack-oslo18:10
*** salv-orl_ has joined #openstack-oslo18:12
vksdims__, thnx, default_config_file solved that :)18:12
vksdims__, thnk u very much18:12
*** salv-orl_ has quit IRC18:12
*** salv-orl_ has joined #openstack-oslo18:13
*** bknudson has joined #openstack-oslo18:13
*** vigneshvar has joined #openstack-oslo18:18
*** salv-orlando has quit IRC18:20
*** harlowja_away is now known as harlowja18:21
dhellmannmorganfainberg: added to my review list18:21
*** MasterPiece has quit IRC18:23
*** achanda has joined #openstack-oslo18:25
*** ViswaV has joined #openstack-oslo18:28
dansmithdims__: did that last patch work on py34?18:29
dims__dansmith: yep18:29
dansmithdims__: sweet18:29
*** boris-42 has quit IRC18:32
*** ViswaV has quit IRC18:33
*** e0ne is now known as e0ne_18:33
*** e0ne_ is now known as e0ne18:35
*** jaypipes has quit IRC18:36
*** ViswaV has joined #openstack-oslo18:36
*** mriedem has quit IRC18:40
*** e0ne has quit IRC18:44
*** MasterPiece has joined #openstack-oslo18:49
*** ihrachyshka has quit IRC18:50
*** vks has quit IRC18:53
*** ihrachyshka has joined #openstack-oslo18:54
dhellmannstevemar, morganfainberg, bnemec: this issue looks like something we should work out before releasing the policy lib: https://review.openstack.org/#/c/154742/119:09
*** achanda has quit IRC19:11
*** alexpilotti has quit IRC19:11
bnemecdhellmann: Agreed.  I think we should also look into the config_dir dependency, even if we don't change the default.  We shouldn't be referencing external config opts from a library.19:14
bnemecOh, maybe that's not an actual opt.  It looks like we search for files using a method of the conf object itself.19:17
*** ViswaV has quit IRC19:20
dansmithdims__: is there a py34 job we can turn on for this once we merge this fix?19:24
dims__yep, i can do that19:24
*** ViswaV has joined #openstack-oslo19:25
dims__dansmith: https://review.openstack.org/15545519:31
dansmithdims__: orly19:31
*** achanda has joined #openstack-oslo19:33
sdaguedims__: I don't think it actually works across those 2 changes19:33
sdaguethey have to share a test job19:33
dims__sdague: can i use Depends-On: as a documentation of this patch would only work after that patch went in...19:35
sdagueyeh, sure19:35
sdagueand it provides nice hot linking19:36
sdagueit just doesn't trigger the magic19:36
dims__sdague: works for me for at least this case19:36
sdagueyeh19:37
*** e0ne has joined #openstack-oslo19:41
*** dims__ has quit IRC19:48
*** dims__ has joined #openstack-oslo19:50
openstackgerritMichal Jastrzebski (inc0) proposed openstack/oslo.versionedobjects: New field type - Json  https://review.openstack.org/15482019:50
dhellmannbnemec: yeah, that search does use an option defined by oslo.config but it does it through a public method19:51
dhellmanndims__: see my comment on that py34 job change19:54
dims__dhellmann: good point, should run it on gate too19:56
*** achanda has quit IRC19:58
*** bogdando has quit IRC19:58
*** achanda has joined #openstack-oslo19:58
*** zhiyan has quit IRC20:08
*** jraim has quit IRC20:09
*** jraim has joined #openstack-oslo20:15
*** zhiyan has joined #openstack-oslo20:18
*** boden has joined #openstack-oslo20:36
bodenoslo.messaging question -- can you have multiple notification listeners using different executors on the same targets?? pls see: http://goo.gl/ayRyDu20:38
*** salv-orlando has joined #openstack-oslo20:38
*** prad has quit IRC20:38
*** bogdando has joined #openstack-oslo20:39
*** salv-orl_ has quit IRC20:41
dhellmannboden: for the same "executor" or transport?20:41
dhellmannsorry, different executors20:42
*** cdent has quit IRC20:42
dhellmannboden: why would you want different executors in the same application?20:43
openstackgerritKen Giusti proposed openstack/oslo.messaging: Create a unique transport for each server in the functional tests  https://review.openstack.org/15547620:45
bodendhellmann: the application supports defining pluggable / configurable "actions" for event notifications... the definition allows the specification of the executor20:45
*** yamahata has quit IRC20:46
*** mriedem has joined #openstack-oslo20:46
dhellmannboden: ok, but that's not really what executors are for. It's assumed that an application will be architected around an executor, because it implies runtime characteristics like using thread vs. greenlets20:47
dhellmannboden: which application is it?20:47
bodendhellmann: home grown20:47
bodendellmann: perhaps I need to revisit the "drawing board"20:48
dhellmannok. like I said, I think you want to pick one executor and build the app around that, then let the user specify some of the other settings20:48
bodendhellmann: not what I was hoping for, but I'll make something work... appreciate your time20:49
dhellmannboden: well, what's the use case for different executors? what are you trying to do there? maybe there's another way to accomplish it20:50
dhellmannboden: fwiw, if you choose to use the eventlet executor then you need to do a bunch of monkey patching that you wouldn't want to do if you were using the blocking or threading executors, so choosing an executor has implications for the way you write the rest of your code and that's why it's not something we usually recommend putting in a configuration file20:51
bodendhellmann: so if eventlet monkey_patch(thread=True) is used the blocking executor wouldn't function properly?20:53
dhellmannWe don't expect that configuration to be used, so I have no idea how it would behave. I wouldn't be surprised to find it exhibiting unusual deadlocking behavior, for example, or not actually blocking.20:54
bodendhellmann: in that case I'll go with a single executor per app model20:54
bnemecAvoid eventlet if at all possible.20:54
bodenmany thanks20:54
bodenlol20:54
bnemecThat way lies madness.20:54
dhellmannboden: sounds good20:55
jogoso http://logs.openstack.org/54/155454/1/check/check-requirements-integration-dsvm/2e38791/console.html#_2015-02-12_20_20_20_70020:56
jogonow oslo_policy is breaking20:56
sigmavirus24jogo: what is oslo_policy breaking?20:57
jogorequirements-integration-dsvm20:57
jogocan't copy 'oslo_policy/tests/var/policy.d/fake_link': doesn't exist or not a regular file20:57
sigmavirus24o_O20:57
stevemarjogo, but oslo_policy isn't out yet...20:58
stevemarunless this change broke it: https://review.openstack.org/#/c/154615/20:58
jogohmm maybe yeah ...20:59
dhellmannstevemar: why do we have a symlink checked into that repository? can we rewrite that test to create the link in a temporary directory?21:00
sdagueummm... yeh, this tests are super crazy bad21:00
dhellmannsdague: that is not helpful language21:00
stevemardhellmann, the tests are pretty awful in oslo.policy, something i was hoping to look at post release21:01
dhellmannstevemar: sounds like we need to move that up in the schedule :-)21:01
stevemardhellmann, let me take a deeper look into the tests then21:01
stevemarjogo, would you like me to issue a revert for the devstack-gate change?21:01
sdagueso carrying around an exploded unix filesystem in a git tree is not a thing that should be done21:01
openstackgerritMerged openstack/taskflow: Docstrings should document parameters return values  https://review.openstack.org/15237521:02
dhellmannsdague: yep, I agree21:02
dhellmannstevemar: would it be faster to jam a patch through oslo_policy to skip the test and remove the files, then rewrite the test?21:02
*** e0ne has quit IRC21:03
jogodhellmann stevemar: whatever fixes it works for me21:03
stevemardhellmann, probably not, let me just revert it, it's only a one liner21:03
dhellmannstevemar: well, I'm thinking of lining up reviewers21:03
dhellmannstevemar: I can +2a your patch as an emergency fix, but I don't have rights on devstack-gate21:04
stevemardhellmann, okay, let me poke around21:04
*** mriedem has quit IRC21:07
*** mriedem has joined #openstack-oslo21:07
stevemardhellmann, what the frack, i just deleted the symlink and re-ran tests, they passed. not being used?21:07
harlowjaboden bnemec or just accept eventlet into your heart (saw executor conversation, lol)21:07
dhellmannstevemar: I haven't had a chance to study the tests yet. Where did that link come from?21:07
stevemardhellmann, looks like https://github.com/openstack/oslo.policy/commit/14c680b807a90d23959de56951cff0a8b098665221:08
stevemarbknudson, do you know the origins of the symlink file ^21:08
bknudsonstevemar: fake_link? I think it's used in a test.21:09
stevemartheres also a '/tests/var/policy.d/.fake_hide_file'21:09
*** kgiusti has left #openstack-oslo21:09
bknudsonI copied things from oslo-incubator to make the tests pass.21:09
openstackgerritDan Krause proposed openstack/taskflow: catch NotFound errors when consuming or abandoning  https://review.openstack.org/15548321:10
*** boris-42 has joined #openstack-oslo21:10
sdaguebknudson: so... carrying around a filesystem like that is problematic21:10
bnemecharlowja: :-)21:10
sdaguethat should be constructed by a fixture on demand21:10
bknudsonsure... I didn't write the tests.21:10
bknudsonI was just trying to get oslo.policy passing jenkins again.21:11
bknudsonactually, it didn't pass jenkins up until that commit21:11
bnemecYeah, I recall the policy tests being problematic for the parallel testing too.21:11
bnemecI know some of the problems have already been cleaned up in oslo.policy, but apparently not all.21:12
bknudsonI don't think oslo.policy has even been released yet so would be happy to see it changed...21:12
dhellmannbknudson: yeah, stevemar is looking at it now21:12
stevemarbknudson, correct21:12
bknudsonhttps://github.com/openstack/oslo-incubator/tree/stable/juno/tests/var/policy.d21:12
bknudsonhere's the files in stable/juno ^21:13
dhellmannyeah, we missed those when we exported the code from the incubator the first time21:13
bknudsonstevemar: thanks for looking at it!21:14
stevemari think the author was a bit zealous in his testing21:14
bknudsonstevemar: you going to change it to create files as fixtures or something?21:15
stevemarbknudson, i deleted it the symlinked file and re-ran the tests, expecting some failures, so i would skip them21:15
stevemarbknudson, but everything is passing still21:15
bknudsonalso, will note there's plenty of other places in keystone where we carry around filesystems.... actually, I think all the keystone repos have something.21:15
sdaguebknudson: with symlinks?21:16
bknudsonno, no symlinks.21:16
stevemari think the symlink is the issue21:16
sdaguethat's what I meant by carrying filesystems21:16
stevemarsdague, do you know if the same issue will be caused by a hidden file?21:16
sdagueit's not just a directory of files, it's actually semantically a filesystem21:16
sdagueI don't, but realistically for repeatability that should just be created in a tempdir somewhere21:17
sdaguevia a fixture I would think21:17
dhellmann++21:17
stevemarlets try this first, i'm just going to propose we delete the symlink for now, the tests are still good21:17
dhellmannsounds good21:17
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow loading conductors via entrypoints  https://review.openstack.org/12461721:18
openstackgerritSteve Martinelli proposed openstack/oslo.policy: Remove symlinked file from tests  https://review.openstack.org/15548521:19
stevemar^21:19
stevemardhellmann, ^21:19
*** MasterPiece has quit IRC21:20
stevemarfwiw, copying the hidden file was fine: http://logs.openstack.org/54/155454/1/check/check-requirements-integration-dsvm/2e38791/console.html#_2015-02-12_20_20_20_69921:20
bknudsonwho's having problems cloning oslo.policy?21:21
dhellmannstevemar: +2a, so as long as the tests actually pass it should be fine21:21
dhellmannbknudson: jogo or clarkb reported it in openstack-infra21:21
bknudsonsecurity issue?21:22
bknudsonthey build on windows?21:22
stevemarbknudson, hehe, discovered when someone tried to submit patch for requirements i think21:22
bknudsonmaybe need a hacking rule.21:23
openstackgerritJoshua Harlow proposed openstack/taskflow: Rename the string constants to better suite there usage  https://review.openstack.org/15548821:24
bknudsonok, it used to be in oslo-policy which was never installed by itself... and tests aren't copied to other projects.21:24
dhellmannright21:25
stevemardhellmann, zuul is happy with the py27 and py34 tests, just waiting on dsvm neutron now21:27
dhellmannstevemar: cool21:27
dhellmannand thanks!21:28
stevemarnot sure how, but my day has become full of oslo related work lately :P21:28
stevemarbetween policy and the move to oslo.log, fun times21:28
dhellmannstevemar: it's good to have you on the team :-)21:29
*** jaypipes has joined #openstack-oslo21:39
*** ViswaV has quit IRC21:40
*** jaypipes has quit IRC21:40
*** e0ne has joined #openstack-oslo21:45
*** ViswaV has joined #openstack-oslo21:45
*** jaypipes has joined #openstack-oslo21:49
*** ihrachyshka has quit IRC21:55
*** sigmavirus24 is now known as sigmavirus24_awa21:55
*** bogdando has quit IRC21:55
dims__stevemar: woot :)21:56
stevemari'll see if i can secure a block of time to get the oslo.service / oslo.systemd stuff off the ground, that would be the last library keystone needs to be fully decoupled from the incubator21:58
stevemarit's a stretch goal, but it would be nice21:59
*** denis_makogon has quit IRC22:00
bnemecstevemar: https://review.openstack.org/#/c/142659/ for oslo.service22:00
stevemarbnemec, yep, i saw that a few weeks ago, was going to comment asking whats up with it, but figured i should finish a few more things before starting others22:02
stevemaroh right i did make a few comments on there at least :)22:02
bnemecstevemar: Yeah, I think Sachi was underwhelmed by the response when she asked for volunteers to own the new lib. ;-)22:06
*** denis_makogon has joined #openstack-oslo22:13
*** harlowja_ has joined #openstack-oslo22:18
*** e0ne has quit IRC22:18
*** salv-orlando has quit IRC22:19
*** andreykurilin has quit IRC22:19
*** salv-orlando has joined #openstack-oslo22:19
*** andreykurilin has joined #openstack-oslo22:19
*** harlowja has quit IRC22:21
openstackgerritJoshua Harlow proposed openstack/debtcollector: Add a 'removed_kwarg' function/method decorator  https://review.openstack.org/15550622:21
*** ViswaV has quit IRC22:21
openstackgerritJoshua Harlow proposed openstack/debtcollector: Add a 'removed_kwarg' function/method decorator  https://review.openstack.org/15550622:24
*** yamahata has joined #openstack-oslo22:25
*** vigneshvar has quit IRC22:27
*** e0ne has joined #openstack-oslo22:27
*** ViswaV has joined #openstack-oslo22:33
*** miqui is now known as miqui_away22:37
*** bogdando has joined #openstack-oslo22:37
harlowja_hmmm, is the oslo.policy adjustment getting fixed (seems like requirements jobs are dying due to it?)22:38
harlowja_'error: can't copy 'oslo_policy/tests/var/policy.d/fake_link': doesn't exist or not a regular file' i guess22:38
*** e0ne has quit IRC22:43
*** vigneshvar has joined #openstack-oslo22:44
openstackgerritJoe Gordon proposed openstack/debtcollector: Move to hacking 0.10  https://review.openstack.org/15551722:48
*** zzzeek_ has joined #openstack-oslo22:49
openstackgerritMerged openstack/oslo.policy: Remove symlinked file from tests  https://review.openstack.org/15548522:49
*** zzzeek has quit IRC22:51
*** zzzeek_ is now known as zzzeek22:51
stevemarjogo, ^ everything should be good now22:52
jogostevemar: thanks22:52
* jogo rechecks a bunch of patches22:53
stevemarharlowja_, the fix above should resolve the error you saw22:53
harlowja_kk22:53
harlowja_thx22:53
stevemarat least, i hope it does :D22:53
harlowja_:)22:53
*** ViswaV has quit IRC22:55
openstackgerritJoe Gordon proposed openstack/taskflow: Move to hacking 0.10  https://review.openstack.org/15552123:04
*** jgrimm is now known as zz_jgrimm23:06
*** boden has quit IRC23:07
*** jecarey has quit IRC23:12
*** ViswaV has joined #openstack-oslo23:15
*** vigneshvar has quit IRC23:18
*** ViswaV has quit IRC23:20
*** ViswaV has joined #openstack-oslo23:20
*** zzzeek_ has joined #openstack-oslo23:22
openstackgerritJoshua Harlow proposed openstack/taskflow: Use the ability to chain exceptions correctly  https://review.openstack.org/15518723:24
*** zzzeek has quit IRC23:24
*** zzzeek_ is now known as zzzeek23:24
openstackgerritMerged openstack/oslo.config: Clear up MultiStrOpt and related documentation  https://review.openstack.org/13261323:26
openstackgerritMerged openstack/oslo.versionedobjects: Remove references to Nova  https://review.openstack.org/15440123:32
harlowja_btw jogo on a fresh run, the pippin scan takes like 15 minutes (to download alot of things, ha); once cached it takes like 2.5 to create the following analysis (dependency graph + tree of dependencies) ; http://paste.ubuntu.com/10195246/ the resolution phase still needs to be added (aka the traversal of those things part to find a set that works for all)23:32
jogoharlowja_: well the second part is sorta the important part ;)23:33
jogobut awesome23:33
jogoharlowja_: we can use that to find a list of all deps23:33
jogoand install with --no-deps23:33
harlowja_ya, '20088 solution paths found for 'django-openstack-auth==1.1.7'' is the weird part, haha23:33
harlowja_which is basically all the leaf node requirements of ''django-openstack-auth' paths up to ''django-openstack-auth'23:34
harlowja_the possible paths23:34
harlowja_*if u imagine the requirements as a tree (with leaves having no requirements)23:34
harlowja_lots of permuations23:35
harlowja_anyways23:36
harlowja_so get all the paths (check, got that); then try to find a set of paths that works (and there u go the full requirement list)23:37
*** yamahata has quit IRC23:39
*** yamahata has joined #openstack-oslo23:39
*** openstack has joined #openstack-oslo23:41
harlowja_ex,23:41
harlowja_DEBUG: @pippin : Solution path 528:23:41
harlowja_DEBUG: @pippin :  - sphinxcontrib-seqdiag==0.7.123:41
harlowja_DEBUG: @pippin :  - seqdiag==0.9.323:41
harlowja_DEBUG: @pippin :  - blockdiag==1.4.723:41
harlowja_DEBUG: @pippin :  - setuptools==12.0.223:41
harlowja_DEBUG: @pippin : Solution path 529:23:41
harlowja_DEBUG: @pippin :  - sphinxcontrib-seqdiag==0.7.123:41
harlowja_DEBUG: @pippin :  - seqdiag==0.9.323:41
harlowja_DEBUG: @pippin :  - blockdiag==1.4.723:41
harlowja_DEBUG: @pippin :  - setuptools==12.0.323:41
harlowja_*difference is the leaf 12.0.3 vs 12.0.223:41
harlowja_*for sphinxcontrib-seqdiag (for top level requirement sphinxcontrib-seqdiag )23:42
*** bknudson has quit IRC23:42
*** salv-orlando has quit IRC23:54

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