Wednesday, 2016-05-18

*** heyongli has quit IRC00:02
*** heyongli has joined #senlin00:04
*** shu-mutou-AFK is now known as shu-mutou00:05
*** Qiming has quit IRC00:41
*** Qiming has joined #senlin01:38
-openstackstatus- NOTICE: Gerrit is about to be restarted to help with page timeouts01:47
*** elynn has joined #senlin01:53
*** elynn_ has joined #senlin01:58
*** elynn has quit IRC01:58
elynn_Morning02:01
openstackgerritMerged openstack/python-senlinclient: Updated from global requirements  https://review.openstack.org/31750902:04
*** zzxwill has joined #senlin02:05
Qiminghi02:14
Qimingquestion02:14
Qimingis image cirros-0.3.2-x86_64-uec available on gate?02:14
Qimingsorry cirros-0.3.4-x86_64-uec02:14
elynn_devstack will install it through internet I think.02:15
*** yuanying has joined #senlin02:17
QimingI'm asking because our functional test is using 0.3.202:23
Qimingmy local env doesn't have 0.3.202:23
elynn_Heat use 0.3.4 on gate for integrationtests02:25
elynn_https://github.com/openstack/heat/blob/master/heat_integrationtests/prepare_test_env.sh#L3802:25
elynn_So 0.3.4 should available on gate.02:25
Qimingok02:25
Qimingthx02:25
elynn_Is it work just change 0.3.2  to 0.3.4 on your local env?02:25
openstackgerritQiming Teng proposed openstack/senlin: Bump cirros version to 0.3.4  https://review.openstack.org/31780502:28
Qimingseems I have some problem setting cloud_backend02:58
Qimingswitching to openstack_test solved the problem02:58
Qimingbut anyway, bumping cirros to 0.3.4 doesn't hurt02:58
elynn_Do you want to enable gate for real driver?02:59
Qimingnot yet there03:03
Qimingwhen tempest is fully enabled, and we are working on functional tests using tempest, we can switch to use real drivers03:03
Qimingmakes sense?03:03
elynn_yes, some codes might need to be changed for real driver03:04
openstackgerritEthan Lynn proposed openstack/senlin: Add create_test_policy function to base test class  https://review.openstack.org/31782304:09
openstackgerritEthan Lynn proposed openstack/senlin: Add API test for cluster policies list/show  https://review.openstack.org/31782404:09
*** elynn_ has quit IRC04:15
*** yanyanhu has joined #senlin04:28
*** elynn_ has joined #senlin05:35
*** elynn_ has quit IRC05:39
*** elynn_ has joined #senlin05:40
openstackgerritEthan Lynn proposed openstack/senlin: Add API tests for action list/show  https://review.openstack.org/31785206:24
openstackgerritEthan Lynn proposed openstack/senlin: Add tempest tests for build-info API  https://review.openstack.org/31785606:37
openstackgerritEthan Lynn proposed openstack/senlin: Add tempest tests for build-info API  https://review.openstack.org/31785606:37
openstackgerritxu-haiwei proposed openstack/senlin: A spec for container cluster service  https://review.openstack.org/28110206:41
*** zzxwill has quit IRC07:05
openstackgerritEthan Lynn proposed openstack/senlin: Add tempest tests for events list API  https://review.openstack.org/31786407:07
openstackgerritQiming Teng proposed openstack/senlin: Tune DB API for session sync  https://review.openstack.org/31787607:23
Qimingelynn_, there?07:57
elynn_yes07:57
QimingI'm trying to debug the db lock problem07:58
Qimingwhat I found07:58
Qiming(maybe irrelevant to the problem)07:58
Qimingis that the service table contains two records07:58
Qimingeven if I'm runing just one engine07:58
elynn_Each worker will have one record07:59
Qimingshould be one worker08:00
elynn_You set num_engine_workers=1 in your senlin.conf already?08:01
Qimingno08:01
Qimingseems the other dead engine always hang there in the db08:02
elynn_ps -ef|grep senlin-engine08:03
elynn_how many do you get?08:03
elynn_then use command 'senlin-manage service list' to check service status08:03
Qimingone08:03
elynn_I think the dead one's status is 'down'08:04
Qimingsenlin-manage service list produces two records08:04
Qimingyep, the status is down08:04
Qimingone of them08:04
elynn_That is correct, When we check whether a engine is alive or not in senlin-engine, it depends on the 'status'08:05
QimingI see08:06
elynn_if the engine didn't update it's record for 2*period_time, then will consider it as dead.08:06
elynn_https://github.com/openstack/senlin/blob/master/senlin/engine/senlin_lock.py#L4208:06
Qimingweird thing is08:06
Qimingsingle functional test most of the time passes08:07
Qimingmultiple tests, however, will mostly fail08:07
Qimingleaving only one or two passes08:07
elynn_You are using real drivers?08:08
openstackgerritEthan Lynn proposed openstack/senlin: Fix functional tests  https://review.openstack.org/31789408:20
Qimingno08:20
elynn_That's strange, since gate is okay.08:22
elynn_any errors in log?08:22
Qimingdbdeadlock08:22
Qimingthere is no easy way to reproduce this08:22
Qiminghave to change some code to sanitize the logs08:23
Qimingcurrently the logs are all about messages like " ... sleep for 3 seconds"08:23
elynn_That sounds not good...08:24
elynn_Enable multi engine can solve it or not?08:24
Qimingtrying this: tox -e functional -- -r senlin.tests.functional.test_cluster_scaling -r senlin.tests.functional.test_cluster_basic08:24
elynn_let me try.08:25
Qiming... the first '-r' was ignored08:25
elynn_hmm, it indeed ignore the first one...08:28
elynn_seems ostestr didn't support two '-r'08:30
elynn_try this tox -e functional -- -r '(senlin.tests.functional.test_cluster_scaling|senlin.tests.functional.test_cluster_basic)'08:31
Qimingok, it works08:32
elynn_messages like " ... sleep for 3 seconds" might because some locks in db can't be stole...Not sure why if I can't access the env.08:34
elynn_Try to stop all senlin-engine and clean lock table in db, and run the tests again.08:34
Qimingsleep was there because some actions are waiting08:35
Qimingit is not a bug actually08:36
QimingI'm looking into this: "DBDeadlock: (pymysql.err.InternalError) (1213, u'Deadlock found when trying to get lock; try restarting transaction') "08:36
Qimingmaybe we need some oslo_db retry08:36
elynn_Hmm, yes, maybe we can add deadlock retry decorator.08:37
elynn_functional tests at my env works fine...08:38
Qimingsomething changed from oslo recently08:38
Qimingbut in my env, no08:38
Qimingif fails occasionally08:38
elynn_Wrap with retry and have a test? https://github.com/openstack/heat/blob/c9aff2fc7364ed78b31238b7c3bc035426b1788e/heat/db/sqlalchemy/api.py#L534-L53508:40
openstackgerritEthan Lynn proposed openstack/senlin: Fix functional tests  https://review.openstack.org/31789408:51
Qimingokay, working on it08:54
elynn_Hmm, indeed, deadlock happens occasionally in my env.08:55
Qimingokay, applied two wrappers09:04
Qimingtesting09:04
openstackgerritMerged openstack/senlin: Fix functional tests  https://review.openstack.org/31789409:42
openstackgerritMerged openstack/senlin: Bump cirros version to 0.3.4  https://review.openstack.org/31780509:50
*** Qiming has quit IRC10:08
*** shu-mutou is now known as shu-mutou-AFK10:17
*** yanyanhu has quit IRC10:23
*** elynn_ has quit IRC11:17
*** Qiming has joined #senlin11:18
*** jdandrea_ has quit IRC12:22
*** jdandrea has joined #senlin12:22
openstackgerritSean Dague proposed openstack/senlin: Revert "Allow parameter location to be specified in rst"  https://review.openstack.org/31802312:34
openstackgerritSean Dague proposed openstack/senlin: Migrate to os-api-ref upstream library  https://review.openstack.org/31802612:37
openstackgerritQiming Teng proposed openstack/senlin: Add DB retry decorator for DeadLock problem  https://review.openstack.org/31811514:01
*** Liuqing has joined #senlin14:04
openstackgerritSean Dague proposed openstack/senlin: Migrate to os-api-ref upstream library  https://review.openstack.org/31802614:10
openstackgerritMerged openstack/senlin: Tune DB API for session sync  https://review.openstack.org/31787614:21
*** Liuqing has quit IRC14:33
*** Drago has joined #senlin14:51
*** Drago has quit IRC14:51
*** Drago has joined #senlin14:51
*** zzxwill has joined #senlin15:05
openstackgerritQiming Teng proposed openstack/senlin: Migrate to os-api-ref upstream library  https://review.openstack.org/31802615:12
*** Qiming has quit IRC15:17
*** Qiming has joined #senlin15:17
openstackgerritMerged openstack/senlin: Updated from global requirements  https://review.openstack.org/31751216:16
*** Qiming has quit IRC16:31
*** Drago has quit IRC16:40
*** Drago has joined #senlin17:40
*** Drago1 has joined #senlin20:07
*** Drago has quit IRC20:09
*** sridhar_ram has joined #senlin21:55
*** Drago1 has quit IRC23:29
*** Qiming has joined #senlin23:32
*** sridhar_ram1 has joined #senlin23:33
*** sridhar_ram has quit IRC23:33
openstackgerritQiming Teng proposed openstack/senlin: Migrate to os-api-ref upstream library  https://review.openstack.org/31802623:41

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