Tuesday, 2015-11-03

*** pratikma_ has quit IRC00:06
*** lkarm has joined #senlin01:03
*** zhenguo has joined #senlin01:09
*** lvdongbing has joined #senlin01:20
*** Liuqing has joined #senlin01:35
*** Qiming has joined #senlin01:36
*** Yanyanhu has joined #senlin01:36
*** elynn has joined #senlin01:52
openstackgerritlvdongbing proposed openstack/senlin: Put py34 first in the env order of tox  https://review.openstack.org/24107002:09
*** lkarm_ has joined #senlin02:20
openstackgerritQiming Teng proposed openstack/python-senlinclient: Revised client interface  https://review.openstack.org/24080502:22
*** lkarm has quit IRC02:22
Qiminghi, guys, patch 240805 is a more complete revision to the client interface02:23
Qimingplease help test each and every of them02:23
QimingI'm trapped by something else at the moment, will get back to your review comments later this week02:23
Yanyanhugot it02:24
elynnLooking at it02:27
Qimingthe primary problem I have is about the passing of dicts, currently, we have a mix of passing dict as a whole and passing them as keyword arguments02:29
Yanyanhuyes, that's true02:31
Yanyanhumaybe we can unify the usage and pass dict as kwargs for all cases?02:32
YanyanhuAFK for a while, back soon02:33
Qimingat interface layer, this patch is trying to propose a unified scheme, which is using keyword arguments02:33
Qimingunder that layer, we still some times pass dict as a whole, I think that can be another patch though02:33
*** lkarm_ has quit IRC02:37
openstackgerritQiming Teng proposed openstack/python-senlinclient: Revised client interface  https://review.openstack.org/24080502:39
Qimingsorry about the pep8 errors02:39
Yanyanhujust back02:44
YanyanhuYes, that can be resolved in another patch02:45
Qimingactually there are other methods we can remove from client.py03:00
openstackgerritlvdongbing proposed openstack/senlin: API resource names should not include underscores  https://review.openstack.org/24074003:27
*** Qiming has quit IRC03:36
*** gongysh has joined #senlin04:33
openstackgerritOpenStack Proposal Bot proposed openstack/senlin-dashboard: Updated from global requirements  https://review.openstack.org/24109604:35
*** lkarm has joined #senlin04:48
*** lkarm has quit IRC04:53
*** Liuqing has quit IRC05:03
*** Liuqing has joined #senlin05:04
*** Qiming has joined #senlin05:56
Yanyanhuhi, Qiming, I'm thinking whether 'cluster-node-add' cmd is functional duplicated with 'node-join' cmd?06:13
Yanyanhuand also the 'cluster-node-del' and 'node-leave'06:13
Qimingfuntionally, it is an overlap06:14
Qimingyes, it is for convenience06:14
*** Liuqing has quit IRC06:35
*** Liuqing has joined #senlin06:35
*** gongysh has quit IRC07:18
*** gongysh has joined #senlin07:19
*** lkarm has joined #senlin07:38
*** lkarm has quit IRC07:42
*** Liuqing has quit IRC07:47
*** Liuqing has joined #senlin07:48
openstackgerritQiming Teng proposed openstack/python-senlinclient: Adapt resource path to server revision  https://review.openstack.org/24112808:03
Qiminglooks like patch 240740 is ready, thanks, lvdongbing08:06
*** Liuqing has quit IRC08:08
*** Liuqing has joined #senlin08:09
lvdongbingThanks, Qiming :)08:15
QimingYanyanhu, you have helped tested all the calls?08:18
Yanyanhuyes, almost all of them. And still have a issue that I'm not very sure about the reason happened when creating webhook with count parameter08:19
openstackQiming: Error: "!!" is not a valid command.08:19
Qimingso nice of you08:19
Yanyanhuno problem :) this is an urgent requirement08:20
YanyanhuBTW, if we specify the parameter using '-P count=1' when creating a webhook, the count value 1 will be parsed as an unicode string '1' rather than integer 1 which will finally cause error when calculating count value in cluster_scale_in/out action08:21
Yanyanhuwill try to fix this08:21
Qimingdata type conversion problem?08:29
Yanyanhuyes, I think we forget to make data type conversion in scaling policy.08:30
Yanyanhuthis conversion is done correctly in service module and thus prevent this error happening in cluster action progress08:31
Yanyanhuso I guess we need to add the same conversion in pre_op of scaling policy08:32
Qimingspeaking of the default value for 'count', in the context of scale_in and scale_out08:33
QimingI think it is okay to have a default value of 108:33
Qimingthe default value is consistent with server side logic08:34
Qimingthe default value will make some of the logic at server side not that useful, but no inconsistency08:34
Qimingthe default value is only impacting command line interface, not from other services that talk to senlinclient08:35
Qimingthe reason I added the 'default=1' for both operations is that I saw inconsistency in existing shell logic (before the current patch)08:36
Yanyanhuyes, I agree with this change. Currently, the default value=1 is implemented in engine side.08:37
YanyanhuJust because the 'count' value provided in action input will always override the count result calculated by scaling policy, the scaling policy actually doesn't get chance to take effect if 'count' input is provided for cluster_scale_in/out action.08:37
Yanyanhulike this: http://git.openstack.org/cgit/openstack/senlin/tree/senlin/policies/scaling_policy.py#n14408:37
*** Liuqing has quit IRC08:38
Qimingokay, that makes sense08:38
Qimingso I'm leaving them as unset and see what happens?08:38
*** Liuqing has joined #senlin08:39
openstackgerritQiming Teng proposed openstack/python-senlinclient: Revised client interface  https://review.openstack.org/24080508:42
Yanyanhuhi, Qiming, the new patchset works well08:50
Qiminggood to know!08:50
Yanyanhuhi, Qiming, just made a test and didn't reproduce the error Avi mentioned.09:13
lvdongbingHi, Qiming, I proposed a patch before: Update node status when fail to create nova instance   https://review.openstack.org/#/c/238753 , it failed in py34 test. I found it might be a bug of python3.4: http://bugs.python.org/issue2300309:14
Qiminglvdongbing, yes, it is only solved in py3.509:17
Qimingso the current workaround, sadly, is to avoid dumping stack trace outside of a exception context09:17
QimingYanyanhu, thanks. Will check with Avi09:17
lvdongbingOK, I'll fix it09:19
openstackgerritMerged openstack/senlin: API resource names should not include underscores  https://review.openstack.org/24074009:21
*** jdandrea has quit IRC09:22
Qimingprofile resource merged into sdk09:25
Yanyanhunice! much helpful for elynn's work :)09:28
elynncool, thanks a lot.09:28
elynnSeems stackalytics didn't cover senlin ?09:30
Qimingit is there09:31
Qimingbut it is ... openstack-others09:31
Qimingall non-official projects fall into that category09:31
Qimingelynn, http://stackalytics.com/?project_type=openstack-others&module=senlin09:35
elynnOh, I see that.09:35
elynnHope we can change project type to openstack after tonight :D09:37
openstackgerritlvdongbing proposed openstack/senlin: Update node status when fail to create nova instance  https://review.openstack.org/23875309:38
*** lvdongbing has quit IRC09:40
openstackgerritYanyan Hu proposed openstack/senlin: Convert count value into integer in scaling policy  https://review.openstack.org/24114509:44
Qiming https://review.openstack.org/225560 stack update patch merged in sdk09:47
*** Liuqing has quit IRC09:55
*** Yanyanhu has quit IRC09:55
*** Qiming has quit IRC09:56
*** zhenguo has quit IRC10:03
*** lkarm has joined #senlin10:09
*** lkarm has quit IRC10:14
*** elynn has quit IRC10:17
*** Liuqing has joined #senlin10:31
*** Qiming has joined #senlin10:54
*** gongysh has quit IRC10:55
*** gongysh has joined #senlin10:57
*** Liuqing has quit IRC11:17
*** Liuqing has joined #senlin11:19
*** lixinhui has joined #senlin11:50
*** lkarm has joined #senlin12:26
*** lkarm has quit IRC12:30
*** elynn has joined #senlin12:50
*** haiwei_ has joined #senlin12:52
*** yanyanhu has joined #senlin12:55
openstackgerritMerged openstack/senlin: Move policy detach from action to cluster  https://review.openstack.org/24068513:01
yanyanhuhi, Qiming, I found ttx in the etherpad of senlin's meetup13:19
*** pratikmallya has joined #senlin13:25
haiwei_yanyanhu, maybe ttx is reviewing senlin joining the big tent patch13:28
yanyanhuI guess so :)13:30
openstackgerritMerged openstack/senlin: Convert count value into integer in scaling policy  https://review.openstack.org/24114513:37
openstackgerritMerged openstack/senlin: Put py34 first in the env order of tox  https://review.openstack.org/24107013:40
Qimingyanyanhu, haiwei_ still on?14:00
Qimingfor the mitaka workitems page14:00
haiwei_Qiming, I have some additional jobs for the summit recently, after that, I will start client test14:00
*** jruano has joined #senlin14:00
QimingI'll start tune it14:01
Qimingthere are things we cannot finish in a month14:01
QimingI'm moving them into a backlog section14:01
Qimingyanyanhu, please help consolidate them into the rst files later14:01
Qimingneed to sleep a little earlier today14:02
Qiming4am morning call14:02
haiwei_anything else?14:02
haiwei_ok, me too14:02
Qimingnothing now14:03
yanyanhuin the meeting channel?14:03
haiwei_though I had a holiday today14:03
yanyanhuthe tc discussion14:03
jruanoare we meeting this morning?14:03
yanyanhuok, will check the irc14:03
Qimingyes jruano14:03
Qimingjust finished14:03
haiwei_see you tomorrow guys14:03
*** lixinhui has quit IRC14:03
Qimingbye haiwei_14:03
jruanomy apologies14:04
Qimingno worry, jruano14:04
Qiminghow was your trip back? still suffering from jet lag?14:04
jruanoah, i got messed up with the time change for us14:04
jruanoon sunday14:04
jruanomy fault14:04
jruanoim great today. yesterday and sunday were bad14:04
Qimingnext week, I'll be tortured :)14:05
jruanowe observe daylight savings and move the clock back an hour on sunday14:05
Qiming11.8 -> San Francisco, 11.12 -> Austin, 11.15 -> New York, 11.18 -> Beijing14:05
jruanowhat is happening in austin?14:06
QimingChina tried daylight savings a few decades ago, it didn't work14:06
*** lkarm has joined #senlin14:06
QimingI was a child, cannot remember the reason, :)14:06
QimingI'm meeting Sandy and trying to figure out how to get close to product/service team14:07
*** haiwei_ has quit IRC14:07
jruanocool, well ping me when you get in. we can do lunch or something14:08
Qimingsurely I will14:08
jruanolet me know if you need any help as well14:08
jruanoill be around14:08
*** elynn has quit IRC14:10
*** yanyanhu has quit IRC14:12
*** gongysh has quit IRC14:12
*** gongysh has joined #senlin14:13
*** jruano has quit IRC14:21
*** elynn has joined #senlin14:32
*** Qiming has quit IRC14:32
*** elynn has quit IRC14:37
*** Liuqing has quit IRC15:24
*** jdandrea has joined #senlin15:29
*** pratikmallya has quit IRC16:02
*** gongysh has quit IRC16:06
*** gongysh has joined #senlin16:07
*** jruano has joined #senlin16:24
*** pratikmallya has joined #senlin16:24
*** pratikmallya has quit IRC16:30
*** pratikmallya has joined #senlin16:30
*** pratikma_ has joined #senlin16:34
*** jruano has quit IRC16:35
*** pratikmallya has quit IRC16:37
*** pratikma_ is now known as pratikmallya16:40
*** pratikmallya has quit IRC17:26
*** pratikmallya has joined #senlin17:29
*** lkarm has quit IRC18:02
*** lkarm has joined #senlin18:03
*** lkarm has quit IRC18:07
*** lkarm has joined #senlin18:17
*** rebase has joined #senlin18:49
*** jruano has joined #senlin19:15
*** Qiming has joined #senlin19:33
*** pratikmallya has quit IRC19:46
*** pratikmallya has joined #senlin20:15
*** rebase has quit IRC20:50
*** Qiming has quit IRC21:07
*** jruano has quit IRC21:12
*** pratikmallya has quit IRC21:38
*** pratikmallya has joined #senlin22:15
*** pratikma_ has joined #senlin22:18
*** pratikmallya has quit IRC22:21
*** lkarm has quit IRC22:23
*** lkarm has joined #senlin22:24
*** lkarm has quit IRC22:28
*** yuanying has quit IRC23:16
*** yuanying has joined #senlin23:43

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