Tuesday, 2015-07-21

*** Qiming_ has quit IRC00:11
openstackgerritMerged stackforge/senlin: Revised design document for webhooks  https://review.openstack.org/19007100:13
*** Yanyanhu has joined #senlin01:20
*** Yanyan has joined #senlin01:22
*** Qiming_ has joined #senlin01:22
*** Yanyanhu has quit IRC01:25
*** jdandrea has quit IRC01:42
*** Tennyson has joined #senlin01:42
Yanyanxuhaiwei, morning :)01:45
Yanyanhow is your vacation?01:45
xuhaiweimainly stayed at home01:46
xuhaiweinot bad, anyway01:46
Yanyannice, stay with family is always good ;)01:47
*** Tennyson has quit IRC02:03
xuhaiweihi, what is the exect version we should install, it seems 0.11 doesn't work02:43
xuhaiweisome package needs 'pbr<2.0,>=1.3'02:43
xuhaiweisome package still needs 'pbr!=0.7,<1.0,>=0.6'02:44
Yanyanxuhaiwei, you reinstall your environment?02:49
Yanyanpbr 0.11 works in my local env02:50
Yanyanhi, Qiming_ , how did you solve the pbr issue?02:50
xuhaiweiI reinstalled it before the vocation, but it works at that time02:50
Yanyannow you can't start engine/api?02:51
xuhaiweiI just installed some packages by pip install -r requirement.txt02:51
xuhaiweiyes, can't start engine and api02:51
Yanyancan you check your pbr version using pip list | grep -i pbr02:51
YanyanI guess it was updated02:52
xuhaiweiyes, now 1.3 is installed by default02:52
xuhaiweibut some package still need 'pbr!=0.7,<1.0,>=0.6'02:52
Yanyanwhat I did is manually downgrade it by performing 'pip install pbr==0.11'02:53
Yanyansudo pip install pbr==0.1102:53
xuhaiweiif I reinstall pbr by pip install pbr==0.11, some package will complain 'pbr<2.0,>=1.3'02:53
xuhaiweithere is no version to satisfy both of them02:53
Yanyanwhich packages02:53
xuhaiwei Requirement.parse('pbr!=0.7,<1.0,>=0.6'), set(['sqlalchemy-migrate']))02:54
Yanyanmaybe you can check which packages were installed when you executed 'pip install -r requirement.txt'02:55
Qiming_your sqlalchemy-migrate version?02:56
xuhaiweisqlalchemy-migrate (0.9.6)02:56
Qiming_sudo pip install sqlalchemy-migrate==0.9.702:56
xuhaiweiok, works for me:)02:57
Yanyanmaybe I should also upgrade my sqlalchemy-migrate, I found it is still 0.9.6 in my env02:58
YanyanQiming_, lunch time?03:47
openstackgerritYanyan Hu proposed stackforge/senlin: Add test case for LoadBalancer driver  https://review.openstack.org/20391304:11
openstackgerritYanyan Hu proposed stackforge/senlin: Add test case for LoadBalancer driver(1)  https://review.openstack.org/20391304:11
*** Tennyson has joined #senlin05:31
*** Zhenqi has joined #senlin05:54
openstackgerritxu-haiwei proposed stackforge/senlin: Add node module test case part1 and send some events for action  https://review.openstack.org/19980306:59
openstackgerritxu-haiwei proposed stackforge/senlin: Add node module test case part1 and send some events for action  https://review.openstack.org/19980307:03
openstackgerritxu-haiwei proposed stackforge/senlin: Add node module test case part1 and send some events for action  https://review.openstack.org/19980307:43
openstackgerritYanyan Hu proposed stackforge/senlin: Add test case for LoadBalancer driver(2)  https://review.openstack.org/20395607:49
openstackgerritYanyan Hu proposed stackforge/senlin: Add test case for policy base module and fix a bug  https://review.openstack.org/19789408:16
openstackgerritYanyan Hu proposed stackforge/senlin: Add test case for policy base module and fix a bug  https://review.openstack.org/19789408:17
openstackgerritYanyan Hu proposed stackforge/senlin: Add test case for policy base module and fix a bug  https://review.openstack.org/19789408:56
Yanyanhi, Qiming_, since the issue of plugin cannot be loaded correctly is still there, I tried to provide a temporary workaround in this patch https://review.openstack.org/#/c/197894/08:56
YanyanI just mocked the get_policy method of environment.Environment to avoid the policy type NotFound error which caused by plugin loading failure08:57
YanyanI think xuhaiwei's patch of test case for node module also suffers this issue08:57
Yanyanafter plugin loading in gate can work correctly, we will remove this mock08:58
xuhaiweiyes, my patch faces the problem, I think, the test case ran correctly in my local env08:59
Yanyanem, this is caused by unstable plugin loading which sometimes could fail09:00
Qiming_it could be something wrong with the environment initialization09:10
Qiming_I'm working on a possible fix09:19
YanyanQiming_, yes, even when I manually invoke environment.initialze() before each test case, it could fail...09:19
Qiming_could be something related to test cases09:19
Yanyanyou mean python's unittest module?09:20
Qiming_possibly these lines: http://git.openstack.org/cgit/stackforge/senlin/tree/senlin/engine/environment.py#n18809:21
*** Tennyson has quit IRC09:21
Yanyanmapping_func is always None in our use cases09:24
Yanyanyou mean maybe we should provide it when doing test?09:24
Qiming_since it is related to a global env09:24
Qiming_when some test cases set it to something and never set it back, other test cases may faile09:24
Qiming_why would you provide a mapping func ...09:25
Yanyanjust not sure why there is an input parameter defined for initialize, guess we can just provide a simple mapping_func for specific policy/profile?09:27
Yanyanso you mean the global _environment could be wrong?09:27
*** Zhenqi has quit IRC09:28
Yanyansince the judgement statement in line 183?09:32
openstackgerritQiming Teng proposed stackforge/senlin: Fix environment intialization logic  https://review.openstack.org/20397809:33
Qiming_the "mapping_func" was a hack used to test inner functions09:33
Qiming_the patch above is an attempt to fix that behavior, hopefully it will help09:34
Yanyanok, let merge it first and haiwei can rebase his patch and submit it again09:35
Yanyanthen we can check whether it works09:35
Qiming_anyway, it does no harm I think09:36
Yanyanyes :)09:36
Yanyanhi, Qiming_, just noticed you remove this option when creating extention.ExtentionManager() 'verify_requirements=False'09:42
YanyanI remeber this option was added to avoid gate failure caused by package version conflict?09:44
Qiming_just checked other projects09:46
Qiming_they have all removed it09:47
Yanyanem, seems we don't need it now09:47
Yanyanhi, Qiming_ , seem the verification failed for package downloading timeout09:51
Yanyanrecheck it09:51
Yanyanwill leave, see u guys tonight10:01
xuhaiweisee you10:01
xuhaiweijenkins takes quite a long time now, since the test cases becomes more and more10:02
*** Yanyan has quit IRC10:05
openstackgerritMerged stackforge/senlin: Fix environment intialization logic  https://review.openstack.org/20397810:14
openstackgerritxu-haiwei proposed stackforge/senlin: Add node module test case part1 and send some events for action  https://review.openstack.org/19980310:18
xuhaiweiHope it will work this time10:18
xuhaiweiI will leave too, see you Qiming_10:18
openstackgerritQiming Teng proposed stackforge/senlin: Add a utility function for random name generation  https://review.openstack.org/20401410:44
*** Qiming_ has quit IRC10:49
*** lixinhui_ has joined #senlin11:22
*** Qiming_ has joined #senlin11:41
lixinhui_which one is the install dir of openstacksdk under dist-package?11:44
lixinhui_is it safe for me to remove whole dir of openstack?11:45
lixinhui_and install from source code11:46
lixinhui_is there any other module installed to the same dir11:46
*** Qiming_ has quit IRC12:08
*** yanyanhu has joined #senlin12:23
*** lkarm has joined #senlin12:31
lixinhui_has anyone use the latest code of openstacksdk?12:36
yanyanhuyes, I think I am12:36
yanyanhumaybe not the latest, I pulled the code about a week ago12:37
lixinhui_okay, yanyan, have you met this error?12:37
lixinhui_I have adopted latest code of senlin12:38
yanyanhuok, let me have a look12:38
*** haiwei has joined #senlin12:42
*** Qiming has joined #senlin12:42
Qiminglixinhui_, no other code lives in that dir12:43
Qimingsince it is just a library code12:43
lixinhui_okay, qiming12:43
Qimingyou can clone the source anywhere and do a install12:43
yanyanhumaybe need to update the configure file?12:46
yanyanhuor this should be service_project_name?12:46
Qimingit should be service_project_name I think12:50
yanyanhuem, seems an error here12:50
Qimingsomething is broken?12:51
yanyanhuI think lixinhui_ meet an error when doing profile-type-list12:52
yanyanhumaybe caused by this erroneous option name12:52
Qimingah.. that is an error12:52
*** jruano has joined #senlin12:54
Qimingproposing a fix12:55
*** mathspanda has joined #senlin12:56
openstackgerritQiming Teng proposed stackforge/senlin: Fix sdk service credentials  https://review.openstack.org/20407212:56
*** mathspanda has quit IRC12:56
yanyanhuwill need test case for this driver :)12:56
Qimingplease add agenda items here: https://wiki.openstack.org/wiki/Meetings/SenlinAgenda12:56
Qimingor we will be changing code freely ...12:57
*** jdandrea has joined #senlin13:11
openstackgerritMerged stackforge/senlin: Fix sdk service credentials  https://review.openstack.org/20407213:38
Qiminghaiwei, there?14:00
Qimingabout this one: https://bugs.launchpad.net/python-senlinclient/+bug/147255714:00
openstackLaunchpad bug 1472557 in python-senlinclient "cluster, profile and node can't be updated" [Undecided,New]14:00
yanyanhulixinhui_, since this patch has been merged https://review.openstack.org/204072, you can pull the lastest code and try it again14:00
Qimingmaybe we need to improve the code here: /usr/local/lib/python2.7/dist-packages/senlinclient/common/exc.py", line 234, in parse_exception14:00
Qimingthe exc.details sometimes is not a valid JSON14:01
Qimingsdk might have missed that part14:01
lixinhui_yanyanhu, before the error, I have adopt the patch14:01
haiweiok, I will check it out tomorrow Qiming14:01
Qiminghaiwei, please help look into the exc.details and see what we can do there14:01
Qimingmaybe it should be handled in SDK, maybe it's our job14:02
Qiminglixinhui_, you need to update your senlin.conf file, by adding a section named 'authentication'14:02
yanyanhulixinhui_, you mean the patch Qiming proposed about the option 'service_project_name'14:02
haiweiI had checked the source when I found this for the first time, but didn't resolve it14:02
haiweiwill check it again14:02
lixinhui_yes, yanyanhu14:02
Qimingthanks, haiwei14:03
yanyanhuok, lixinhui_ maybe you need update configure file as Qiming suggested14:03
lixinhui_Qiming, give me a example about uthentication14:03
lixinhui_an example14:03
lixinhui_of authetication14:03
haiweiI will quit guys, good night14:03
Qiminggood night, haiwei14:04
Qimingtoo late for you14:04
Qiminglixinhui_, http://git.openstack.org/cgit/stackforge/senlin/tree/devstack/lib/senlin#n11214:04
Qimingthe patch was about stop stealing keystonemiddleware data in our code14:04
Qimingthose options are supposed to be used by the auth_token middleware14:05
*** yanyanhu has quit IRC14:05
Qimingif we need something for even similar purposes, we should create and use our own options14:05
*** haiwei has quit IRC14:07
Qimingjruano, https://coreos.com/docs/running-coreos/platforms/openstack/14:08
*** lixinhui_ has quit IRC14:19
*** lixinhui_ has joined #senlin14:20
jruanothanks qiming. looks straightforward. i will investigate further14:34
*** Qiming has quit IRC16:09
*** lixinhui_ has quit IRC16:11
*** Qiming has joined #senlin16:44
*** Qiming has quit IRC16:59
*** xuhaiwei has quit IRC17:18
*** lkarm has quit IRC21:19
*** jruano has quit IRC23:33
*** jruano has joined #senlin23:45

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