| openstackgerrit | xu-haiwei proposed stackforge/senlin: Check action status before lock it https://review.openstack.org/220950 | 00:19 |
|---|---|---|
| *** Qiming__ has quit IRC | 00:24 | |
| *** Qiming__ has joined #senlin | 01:25 | |
| *** zhenguo has joined #senlin | 01:35 | |
| *** Yanyanhu has joined #senlin | 01:56 | |
| *** mathspanda has joined #senlin | 02:08 | |
| *** Zhenqi has joined #senlin | 02:18 | |
| *** Qiming__ has quit IRC | 03:29 | |
| *** Qiming__ has joined #senlin | 03:29 | |
| *** Yanyan has joined #senlin | 03:30 | |
| *** Yanyanhu has quit IRC | 03:32 | |
| *** Yanyanhu has joined #senlin | 03:32 | |
| *** Yanyan has quit IRC | 03:35 | |
| *** Zhenqi has quit IRC | 03:44 | |
| *** Zhenqi has joined #senlin | 04:39 | |
| *** mathspanda has quit IRC | 05:17 | |
| *** Tennyson has joined #senlin | 05:23 | |
| *** Tennyson has quit IRC | 06:23 | |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Remove context from dispatcher and scheduler https://review.openstack.org/221166 | 06:35 |
| *** Shijia has joined #senlin | 06:36 | |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Remove context from dispatcher and scheduler https://review.openstack.org/221166 | 06:37 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Remove context from dispatcher and scheduler https://review.openstack.org/221166 | 06:39 |
| Yanyanhu | hi, Qiming, just proposed the patch for removing context from dispatcher and scheduler, could you please help to review. Thanks | 06:42 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Use versioned profile spec in functional test https://review.openstack.org/221171 | 06:57 |
| Qiming__ | Yanyanhu, I can see notifications from gerrit | 06:57 |
| Yanyanhu | ok | 06:57 |
| Qiming__ | will be in meeting with SUR team, you wanna join? | 06:58 |
| Yanyanhu | sure | 07:00 |
| Yanyanhu | a call meeting? | 07:00 |
| Qiming__ | yes | 07:00 |
| Qiming__ | conf call | 07:00 |
| Yanyanhu | ok, will go to your seat | 07:00 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Remove context from dispatcher and scheduler https://review.openstack.org/221166 | 07:10 |
| openstackgerrit | Merged stackforge/senlin: Check action status before lock it https://review.openstack.org/220950 | 07:26 |
| Qiming__ | hi, Zhenqi, no one showed up in the conf call, do you know what happened? | 07:32 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Add functional test for cluster scale in/out https://review.openstack.org/218142 | 07:36 |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Cluster action test case (4) https://review.openstack.org/221177 | 07:37 |
| *** Zhenqi has quit IRC | 07:37 | |
| *** Zhenqi has joined #senlin | 07:39 | |
| *** Shijia has quit IRC | 07:44 | |
| *** Shijia has joined #senlin | 07:45 | |
| openstackgerrit | Merged stackforge/senlin: Cluster action test case (3) https://review.openstack.org/221017 | 07:49 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Add functional test for cluster scale in/out https://review.openstack.org/218142 | 07:53 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Use versioned profile spec in functional test https://review.openstack.org/221171 | 07:53 |
| *** Shijia has quit IRC | 07:54 | |
| *** Shijia has joined #senlin | 07:54 | |
| xuhaiwei | Qiming__, Yanyanhu, in the meeting now? | 07:58 |
| Yanyanhu | yes | 07:58 |
| xuhaiwei | ok, talk to you later | 07:58 |
| Yanyanhu | :) | 07:58 |
| *** Shijia has quit IRC | 07:58 | |
| *** Shijia has joined #senlin | 07:59 | |
| *** Shijia has quit IRC | 08:03 | |
| *** Shijia has joined #senlin | 08:04 | |
| *** Shijia has quit IRC | 08:08 | |
| *** Shijia has joined #senlin | 08:09 | |
| *** Shijia has quit IRC | 08:14 | |
| *** Shijia has joined #senlin | 08:14 | |
| *** Zhenqi has quit IRC | 08:14 | |
| *** Shijia has quit IRC | 08:19 | |
| *** Shijia has joined #senlin | 08:19 | |
| *** Shijia has quit IRC | 08:27 | |
| *** Shijia has joined #senlin | 08:28 | |
| *** Shijia has quit IRC | 08:33 | |
| *** Shijia_ has joined #senlin | 08:33 | |
| *** Shijia_ has quit IRC | 08:37 | |
| *** Shijia has joined #senlin | 08:38 | |
| Yanyanhu | hi, Qiming__ , xuhaiwei, if you guys are ok with it, I will merge patch 221171 to avoid too long dependency chain. Thanks. | 08:41 |
| *** Shijia has quit IRC | 08:43 | |
| xuhaiwei | I am ok with it | 08:43 |
| *** Shijia has joined #senlin | 08:43 | |
| Qiming__ | haven't got cycle to review it | 08:43 |
| Qiming__ | just proposed this afternoon? | 08:43 |
| Qiming__ | okay, one hour ago | 08:43 |
| Qiming__ | I have patches lingering there for several days, waiting for review, then I have to approve it myself... :) | 08:44 |
| Yanyanhu | umm, need put more time on review... | 08:45 |
| Qiming__ | comments posted | 08:47 |
| Qiming__ | you can workflow+1 it if it's needed urgently | 08:47 |
| *** Shijia has quit IRC | 08:47 | |
| Yanyanhu | ok, thanks | 08:47 |
| *** Shijia has joined #senlin | 08:48 | |
| *** Shijia has quit IRC | 08:52 | |
| *** Shijia has joined #senlin | 08:53 | |
| xuhaiwei | Qiming__ | 08:54 |
| xuhaiwei | not busy now? | 08:54 |
| Qiming__ | fine | 08:55 |
| Qiming__ | doing review | 08:55 |
| Qiming__ | what's up? | 08:55 |
| xuhaiwei | found a bug about policy.json | 08:55 |
| xuhaiwei | or somewhere else | 08:55 |
| Qiming__ | okay? | 08:56 |
| xuhaiwei | in policy.json we use project_id to check the project owner | 08:56 |
| Qiming__ | no, we use project | 08:56 |
| Qiming__ | http://git.openstack.org/cgit/stackforge/senlin/tree/etc/senlin/policy.json#n3 | 08:56 |
| Qiming__ | there was a known bug before, it has been fixed | 08:57 |
| xuhaiwei | ok, my environment is old | 08:57 |
| xuhaiwei | then no problem | 08:57 |
| Qiming__ | http://git.openstack.org/cgit/stackforge/senlin/commit/etc/senlin/policy.json?id=d1ff8cba645f4e896017fd043b38e4fe81e08b04 | 08:57 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Use versioned profile spec in functional test https://review.openstack.org/221171 | 08:58 |
| xuhaiwei | got it, my policy.json is not up to date | 08:58 |
| Qiming__ | yep | 08:58 |
| Qiming__ | Yanyanhu, SDK doesn't support getting flavor by name? | 08:59 |
| Yanyanhu | yes, and you have proposed a patch for it :) | 09:00 |
| Yanyanhu | but it is still under review I think | 09:00 |
| Qiming__ | cannot recall | 09:00 |
| Yanyanhu | https://review.openstack.org/212336 | 09:00 |
| Yanyanhu | this one | 09:00 |
| Yanyanhu | there are some problems in the find method | 09:01 |
| Qiming__ | ok | 09:01 |
| Qiming__ | totally forgot it | 09:01 |
| *** Shijia has quit IRC | 09:02 | |
| *** Shijia has joined #senlin | 09:02 | |
| Yanyanhu | yes, hope they can solve it after the freezing completing | 09:04 |
| *** Shijia has quit IRC | 09:07 | |
| *** Shijia_ has joined #senlin | 09:07 | |
| openstackgerrit | Merged stackforge/senlin: Use versioned profile spec in functional test https://review.openstack.org/221171 | 09:11 |
| *** Shijia_ has quit IRC | 09:11 | |
| *** Shijia has joined #senlin | 09:12 | |
| *** Zhenqi has joined #senlin | 09:14 | |
| *** Shijia_ has joined #senlin | 09:15 | |
| openstackgerrit | Qiming Teng proposed stackforge/python-senlinclient: Shell unit test (1) https://review.openstack.org/219235 | 09:16 |
| *** Shijia has quit IRC | 09:16 | |
| *** Zhenqi has quit IRC | 09:25 | |
| *** Shijia_ has quit IRC | 09:37 | |
| openstackgerrit | Qiming Teng proposed stackforge/python-senlinclient: Shell unit test (1) https://review.openstack.org/219235 | 09:39 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Add functional test for cluster scale in/out https://review.openstack.org/218142 | 09:40 |
| *** lixinhui has joined #senlin | 09:42 | |
| openstackgerrit | Qiming Teng proposed stackforge/python-senlinclient: Shell unit test (1) https://review.openstack.org/219235 | 09:59 |
| openstackgerrit | Cindia-blue proposed stackforge/senlin: Add a placement-policy to enable vSphere DRS functions https://review.openstack.org/219212 | 10:04 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Remove context from dispatcher and scheduler https://review.openstack.org/221166 | 10:06 |
| openstackgerrit | Cindia-blue proposed stackforge/senlin: Add a placement-policy to enable vSphere DRS functions https://review.openstack.org/219212 | 10:11 |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Cluster action test case (4) https://review.openstack.org/221177 | 10:11 |
| Qiming__ | admin__, there? | 10:12 |
| openstackgerrit | Merged stackforge/python-senlinclient: Shell unit test (1) https://review.openstack.org/219235 | 10:13 |
| Yanyanhu | hi, Qiming__ | 10:17 |
| Qiming__ | yes? | 10:17 |
| Yanyanhu | about my comment in line474 https://review.openstack.org/#/c/221177/1/senlin/engine/actions/cluster_action.py | 10:17 |
| Yanyanhu | that's my fault that the count from scalingpolicy is positive | 10:17 |
| Yanyanhu | but the one from engine service is negative now | 10:18 |
| Yanyanhu | http://git.openstack.org/cgit/stackforge/senlin/tree/senlin/engine/service.py#n828 | 10:18 |
| Qiming__ | okay, then it needs a fix | 10:18 |
| Yanyanhu | yes | 10:18 |
| Yanyanhu | but after we have event support in scaling policy, I think this problem will be resolved completely | 10:19 |
| Qiming__ | my guess is that it was an attempt to make all scale_in and scale_out use the resize action | 10:19 |
| Yanyanhu | so before that, do we need an interim solution for this? | 10:19 |
| Yanyanhu | yes | 10:19 |
| Qiming__ | I don't think we are very confident about this transition yet | 10:19 |
| Qiming__ | just propose a patch | 10:19 |
| Qiming__ | it is a new bug | 10:19 |
| Yanyanhu | ok | 10:19 |
| Yanyanhu | will propose a patch for it | 10:20 |
| *** lixinhui has quit IRC | 10:20 | |
| Yanyanhu | so let's hold the patch of functional test for cluster scaling before this bug is fix | 10:21 |
| Qiming__ | ok | 10:22 |
| *** Yanyan has joined #senlin | 10:25 | |
| *** Yanyanhu has quit IRC | 10:25 | |
| *** Qiming__ has quit IRC | 10:28 | |
| *** Yanyan has quit IRC | 10:30 | |
| *** Qiming__ has joined #senlin | 11:27 | |
| openstackgerrit | Merged stackforge/senlin: Remove context from dispatcher and scheduler https://review.openstack.org/221166 | 11:33 |
| *** jdandrea has joined #senlin | 12:58 | |
| Qiming__ | meeting time | 13:02 |
| Qiming__ | no one is attending? | 13:05 |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Cluster action test case (4) https://review.openstack.org/221177 | 15:00 |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Cluster action test case (5) https://review.openstack.org/221343 | 15:00 |
| *** Qiming__ has quit IRC | 15:35 | |
| *** zhenguo has quit IRC | 16:08 | |
| *** openstackgerrit has quit IRC | 18:31 | |
| *** openstackgerrit has joined #senlin | 18:32 | |
| *** xuhaiwei has quit IRC | 22:48 | |
| *** Qiming__ has joined #senlin | 23:36 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!