| *** ChrisSen has joined #senlin | 01:29 | |
| *** Qiming has joined #senlin | 01:33 | |
| *** Yanyanhu has joined #senlin | 01:38 | |
| openstackgerrit | Merged stackforge/python-senlinclient: Fix cluster-resize operation param checking https://review.openstack.org/206954 | 02:11 |
|---|---|---|
| *** mathspanda has joined #senlin | 02:16 | |
| *** mathspanda has quit IRC | 02:18 | |
| openstackgerrit | Yanyan Hu proposed stackforge/python-senlinclient: Use correct default domain name in senlinrc https://review.openstack.org/208269 | 02:23 |
| *** mathspanda has joined #senlin | 02:44 | |
| *** Tennyson has joined #senlin | 02:44 | |
| *** Qiming_ has joined #senlin | 02:57 | |
| *** Qiming has quit IRC | 02:59 | |
| *** mathspanda has quit IRC | 03:00 | |
| *** lixinhui_ has joined #senlin | 03:23 | |
| *** xuhaiwei has quit IRC | 03:25 | |
| *** Tennyson has quit IRC | 03:33 | |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Add config reload capability to wsgi https://review.openstack.org/207968 | 03:35 |
| *** mathspanda has joined #senlin | 03:35 | |
| *** Tennyson has joined #senlin | 03:36 | |
| *** mathspanda has quit IRC | 03:46 | |
| *** mathspanda has joined #senlin | 04:16 | |
| *** xuhaiwei has joined #senlin | 04:27 | |
| *** Tennyson has quit IRC | 04:28 | |
| *** Tennyson has joined #senlin | 04:30 | |
| *** Zhenqi has joined #senlin | 04:36 | |
| *** mathspanda has quit IRC | 04:38 | |
| *** mathspanda has joined #senlin | 04:41 | |
| openstackgerrit | xu-haiwei proposed stackforge/senlin: Fix some spelling misses https://review.openstack.org/208306 | 05:03 |
| *** elynn has joined #senlin | 05:04 | |
| ChrisSen | http://paste.openstack.org/show/406694/ | 05:08 |
| ChrisSen | No bug, but documentation and help for creating a profile with metadata is misleading. | 05:09 |
| xuhaiwei | ChrisSen, I think '-d' is not used in a right way in your command | 05:14 |
| xuhaiwei | for all the client command, '-d' means DEBUG, which should be used between 'senlin' and COMMAND_NAME, in the profile creation's case, like "senlin -d profile-create -t xxx ..." | 05:15 |
| Qiming_ | okay, this is a documentation defect I think | 05:17 |
| ChrisSen | Yes, that's my point, there is a mistake in the documentation. | 05:18 |
| xuhaiwei | yes, the -M option? | 05:18 |
| Qiming_ | the documentation is still talking about the '-d' option | 05:18 |
| Qiming_ | though the code has switched to use '-M' | 05:18 |
| Qiming_ | http://git.openstack.org/cgit/stackforge/senlin/tree/doc/source/getting_started/profiles.rst#n184 | 05:18 |
| Qiming_ | http://git.openstack.org/cgit/stackforge/senlin/tree/doc/source/getting_started/profiles.rst#n257 | 05:19 |
| xuhaiwei | yes, that should be fixed | 05:20 |
| xuhaiwei | and about the metadata, it seems we can't set more than one key-value pair using only one -M option | 05:21 |
| Qiming_ | ... | 05:22 |
| Qiming_ | -M can be used more then once | 05:22 |
| ChrisSen | but the help for the "senlin profile-create" command says, you could use multiple key-value pairs for a single -M | 05:22 |
| Qiming_ | it doesn't work? | 05:23 |
| xuhaiwei | seems not | 05:23 |
| xuhaiwei | I just tried it too | 05:23 |
| Qiming_ | okay, then file a bug | 05:23 |
| xuhaiwei | ChrisSen, will you report it and fix it? | 05:25 |
| xuhaiwei | it seems other resources like node also have this problem | 05:26 |
| ChrisSen | Ok, we will report and fix it. | 05:28 |
| Qiming_ | ok | 05:28 |
| *** Qiming_ is now known as Qiming | 05:30 | |
| openstackgerrit | Merged stackforge/python-senlinclient: Use correct default domain name in senlinrc https://review.openstack.org/208269 | 05:34 |
| Qiming | hi | 05:47 |
| Qiming | in the senlin-liberty-workitems list | 05:47 |
| Qiming | we have "splitting cluster_action" module | 05:47 |
| Qiming | I just spent some time revisiting the code | 05:47 |
| Qiming | it seems to me not a big module to be splitted, refactor this module to several modules may hurt the code readability at the moment | 05:48 |
| Qiming | most of the action logics are not very complicated | 05:48 |
| Qiming | so ... maybe we can just split the test cases when they come in | 05:48 |
| Yanyanhu | Qiming, yes, actually it is not a very huge module now | 05:48 |
| Yanyanhu | agree | 05:49 |
| Qiming | the current code is only 730 lines | 05:49 |
| Yanyanhu | yep | 05:49 |
| Qiming | a complete unit test case coverage may lead to a big module | 05:49 |
| Qiming | but we can split the unit test | 05:49 |
| Yanyanhu | ok | 05:50 |
| Qiming | so ... I'm removing that item from the list | 05:50 |
| Yanyanhu | ok | 05:50 |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Cleanse health policy stubs in cluster https://review.openstack.org/208324 | 06:02 |
| openstackgerrit | LinPeiyu proposed stackforge/senlin: Check profile type matching earlier https://review.openstack.org/208329 | 06:21 |
| openstackgerrit | LinPeiyu proposed stackforge/senlin: Check profile type matching earlier https://review.openstack.org/208329 | 06:24 |
| openstackgerrit | LinPeiyu proposed stackforge/senlin: Check profile type matching earlier https://review.openstack.org/208329 | 06:46 |
| ChrisSen | Qiming, concerning the profile creation bug: we corrected the documentation, but should we also change the help statement for the profile-create command? Currently, it says [-M <KEY1=VALUE1;KEY2=VALUE2...>] which doesn't work. | 06:49 |
| Qiming | ChrisSen, if it doesn't work, something is wrong at the client side | 07:01 |
| Qiming | ChrisSen, http://git.openstack.org/cgit/stackforge/python-senlinclient/tree/senlinclient/common/utils.py#n110 | 07:02 |
| Qiming | the format_parameters function is suppoed to parse the KEY1=VALUE1;KEY2=VALUE2 string | 07:03 |
| Qiming | s/suppoed/supposed/ | 07:03 |
| ChrisSen | Ok, we'll have a look on it. | 07:10 |
| Qiming | thanks | 07:14 |
| *** Zhenqi has quit IRC | 07:18 | |
| *** Zhenqi has joined #senlin | 07:18 | |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: [WIP]Start point of Senlin functional tests https://review.openstack.org/208355 | 08:01 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: [WIP]Start point of Senlin functional tests https://review.openstack.org/208355 | 08:04 |
| openstackgerrit | LinPeiyu proposed stackforge/senlin: Check profile type matching earlier https://review.openstack.org/208329 | 08:52 |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Make node-update support more parameters https://review.openstack.org/208377 | 08:59 |
| *** Tennyson has quit IRC | 09:07 | |
| *** Tennyson has joined #senlin | 09:09 | |
| *** Tennyson has quit IRC | 09:13 | |
| *** lixinhui_ has quit IRC | 09:13 | |
| *** Tennyson has joined #senlin | 09:15 | |
| *** lixinhui_ has joined #senlin | 09:17 | |
| *** mathspanda has quit IRC | 09:32 | |
| *** Tennyson has quit IRC | 09:33 | |
| openstackgerrit | xu-haiwei proposed stackforge/senlin: Fix node join/leave cluster error https://review.openstack.org/208392 | 09:33 |
| *** lixinhui_ has quit IRC | 09:34 | |
| *** Zhenqi has quit IRC | 09:46 | |
| *** ChrisSen has quit IRC | 09:57 | |
| *** Qiming has quit IRC | 10:05 | |
| openstackgerrit | Merged stackforge/senlin: Check profile type matching earlier https://review.openstack.org/208329 | 10:12 |
| *** elynn has quit IRC | 10:13 | |
| *** Yanyanhu has quit IRC | 10:13 | |
| *** Yanyanhu has joined #senlin | 10:13 | |
| *** Yanyanhu has quit IRC | 10:18 | |
| *** lixinhui_ has joined #senlin | 10:22 | |
| openstackgerrit | Cindia-blue proposed stackforge/senlin: Resolve TypeError of cooldown_inprogress when cluster_scale_out https://review.openstack.org/208408 | 10:22 |
| *** lixinhui has quit IRC | 11:15 | |
| *** Qiming has joined #senlin | 11:25 | |
| *** lixinhui_ has quit IRC | 11:28 | |
| openstackgerrit | Merged stackforge/senlin: DB support for triggers https://review.openstack.org/207733 | 12:02 |
| *** Qiming has quit IRC | 12:17 | |
| openstackgerrit | Merged stackforge/senlin: Resolve TypeError of cooldown_inprogress when cluster_scale_out https://review.openstack.org/208408 | 13:00 |
| openstackgerrit | Merged stackforge/senlin: Fix some spelling misses https://review.openstack.org/208306 | 13:01 |
| openstackgerrit | Merged stackforge/senlin: Add test case for Nova Server Profile https://review.openstack.org/205845 | 13:05 |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Utility function for type and version checking https://review.openstack.org/208476 | 13:24 |
| *** Qiming has joined #senlin | 13:25 | |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Make node-update support more parameters https://review.openstack.org/208377 | 13:48 |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Environment support for trigger types https://review.openstack.org/208497 | 14:24 |
| openstackgerrit | Merged stackforge/senlin: Utility function for type and version checking https://review.openstack.org/208476 | 15:38 |
| *** Qiming has quit IRC | 15:41 | |
| *** openstackgerrit_ has joined #senlin | 15:54 | |
| *** jdandrea has joined #senlin | 16:29 | |
| *** openstackgerrit_ has quit IRC | 18:07 | |
| *** Qiming has joined #senlin | 19:45 | |
| *** Qiming has quit IRC | 20:15 | |
| *** lixinhui has joined #senlin | 23:22 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!