| openstackgerrit | Kenji Ishii proposed openstack/senlin-dashboard: Add create profile action for angular profile panel https://review.openstack.org/390416 | 00:54 |
|---|---|---|
| *** ruijie has joined #senlin | 01:03 | |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: action handler support batch policy https://review.openstack.org/385741 | 01:11 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: functional test for batch policy https://review.openstack.org/390234 | 01:13 |
| *** guoshan has joined #senlin | 01:22 | |
| *** yanyanhu has joined #senlin | 01:29 | |
| ruijie | Hi, I have a question about the 'cluster_replace_nodes' action. | 02:06 |
| ruijie | The action is now consists of 'node_leave' and 'node_join' | 02:07 |
| yanyanhu | hi, ruijie | 02:07 |
| ruijie | Both of these two actions will check the desired_capacity before doing the job | 02:08 |
| ruijie | So, in some case, this action may fail due to this reason | 02:08 |
| ruijie | like desired_Capacity:5 max:5 | 02:08 |
| yanyanhu | so, there should be dependency relationship between node join and node leave actions | 02:09 |
| yanyanhu | to ensure they won't touch the desired_capacity concurrently | 02:09 |
| ruijie | hi, yanyan. e.g. node_join depends on node_leave | 02:09 |
| yanyanhu | e.g. performing node leave first which will reduce the desired_capacity | 02:09 |
| yanyanhu | then performing node join | 02:10 |
| ruijie | however, the desired_capacity:1 and min:1 | 02:10 |
| ruijie | no matter how the dependency is, that may happen... | 02:10 |
| yanyanhu | I see | 02:11 |
| yanyanhu | unless we put them into the same action | 02:11 |
| yanyanhu | but that will cause other issues | 02:11 |
| yanyanhu | let me see | 02:11 |
| ruijie | the replace_nodes action may not be able to change the desired_capacity and check it | 02:11 |
| yanyanhu | yes, that is another possible workaround | 02:12 |
| ruijie | yes yanyan. I thinking if we could modify the 'do_leave' and 'do_join' in node_action layer | 02:12 |
| ruijie | am | 02:12 |
| ruijie | or we just add another two methods to implement it... | 02:12 |
| yanyanhu | ruijie, it is possible to add another two methods. The problem is it will be difficult to handle the lock if we mixed both removing node operation and adding node operation in the same action | 02:15 |
| yanyanhu | s/mixed/mix | 02:15 |
| ruijie | sure yanyan. And then it will be much more easier to handle the index and role. | 02:16 |
| yanyanhu | so I think maybe we can add an input parameter for node_leave and node_join action. If that parameter is set to true, we skip size limitation check | 02:16 |
| ruijie | eh.. in fact, I think a 'replace' action do not need to check the size limitation | 02:17 |
| yanyanhu | ruijie, yes, there is no requirement to check size limitation for "replace" action | 02:17 |
| yanyanhu | just those two sub actions "node leave" and "node join" have in other cases | 02:18 |
| ruijie | yes, yanyanhu. If we still use 'node_leave/join', we do need that param. | 02:19 |
| ruijie | So, we keep using the old methods to implement it or we create another two methods to handle it? | 02:19 |
| yanyanhu | ruijie, let me think about it | 02:22 |
| ruijie | Thanks yanyanhu. :) | 02:22 |
| yanyanhu | ruijie, currently, action target object(e.g. node for node_join/leave action) will be locked before it is operated(e.g. change its cluster_id). However, current design of action execution is only for single target for each action | 02:25 |
| yanyanhu | e.g. a cluster action has only one target object which is a cluster | 02:25 |
| yanyanhu | and a node action has only one target object which is a node | 02:25 |
| ruijie | yes, yanyan | 02:26 |
| yanyanhu | so if we want to handle both removing and adding node in the same action(either cluster_node_replace or corresponding node action), we need redesign here | 02:26 |
| yanyanhu | this will be a big change | 02:26 |
| ruijie | so, it should be divided to two parts | 02:27 |
| yanyanhu | ruijie, yes, that is the current design of cluster_replace_nodes | 02:28 |
| yanyanhu | node leave and node join | 02:28 |
| ruijie | yes yanyan. that makes sense. | 02:28 |
| yanyanhu | so, maybe we start from adding parameter for node_join and node_leave action | 02:29 |
| yanyanhu | to see whether it can resolve this issue | 02:29 |
| yanyanhu | and won't cause negative impact | 02:29 |
| ruijie | so, we add parameters to tell 'replace' or 'add' and to see if we need to check the size limitation | 02:31 |
| yanyanhu | ruijie, yes | 02:31 |
| ruijie | Okay, it makes sense. | 02:32 |
| ruijie | thanks yanyanhu. Thanks you for answering. | 02:33 |
| yanyanhu | my pleasure :) | 02:33 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add API support for node_create2 https://review.openstack.org/390254 | 02:38 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add engine support for node_create2 https://review.openstack.org/390152 | 02:38 |
| *** heyongli has joined #senlin | 03:24 | |
| openstackgerrit | Shan Guo proposed openstack/senlin: log.exception should use _LE of i18n https://review.openstack.org/390438 | 03:47 |
| *** heyongli has quit IRC | 03:50 | |
| openstackgerrit | Shan Guo proposed openstack/senlin: log.exception should use _LE of i18n https://review.openstack.org/390438 | 03:55 |
| *** guoshan has quit IRC | 04:45 | |
| openstackgerrit | Merged openstack/senlin-dashboard: Add delete action for angular profile panel https://review.openstack.org/387827 | 05:02 |
| *** Drago has quit IRC | 05:14 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Remove dead code about node_create https://review.openstack.org/390442 | 05:43 |
| openstackgerrit | Merged openstack/senlin: action handler support batch policy https://review.openstack.org/385741 | 05:56 |
| openstackgerrit | Merged openstack/senlin: Request object for node-create https://review.openstack.org/390138 | 05:57 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/senlin-dashboard: Imported Translations from Zanata https://review.openstack.org/389288 | 06:17 |
| openstackgerrit | Allen proposed openstack/senlin: Use 'openstack' command-client to replace 'senlin receiver-create' https://review.openstack.org/390001 | 06:35 |
| openstackgerrit | Allen proposed openstack/senlin: Use 'openstack' command-client to replace 'senlin receiver-create' https://review.openstack.org/390001 | 06:37 |
| *** yuanying_ has joined #senlin | 06:38 | |
| openstackgerrit | Merged openstack/senlin: Add engine support for node_create2 https://review.openstack.org/390152 | 06:39 |
| *** yuanying has quit IRC | 06:42 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add NODE_STATUSES definition into consts https://review.openstack.org/390457 | 07:07 |
| openstackgerrit | Merged openstack/senlin: log.exception should use _LE of i18n https://review.openstack.org/390438 | 07:21 |
| *** zzxwill has joined #senlin | 07:42 | |
| *** zzxwill has quit IRC | 08:08 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add NODE_STATUSES definition into consts https://review.openstack.org/390457 | 08:24 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add request object for node-list https://review.openstack.org/390472 | 08:24 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add engine support for node_list2 https://review.openstack.org/390473 | 08:24 |
| openstackgerrit | Merged openstack/senlin-dashboard: Updated from global requirements https://review.openstack.org/389948 | 08:39 |
| openstackgerrit | Merged openstack/senlin-dashboard: Add the detail link for the cluster in receiver detail page https://review.openstack.org/390009 | 08:40 |
| openstackgerrit | Allen proposed openstack/senlin-dashboard: Add angular policies table https://review.openstack.org/389076 | 08:55 |
| *** guoshan has joined #senlin | 09:11 | |
| *** Drago1 has joined #senlin | 09:12 | |
| *** Drago1 has quit IRC | 09:13 | |
| *** Drago1 has joined #senlin | 09:13 | |
| *** zzxwill has joined #senlin | 09:22 | |
| *** zzxwill has quit IRC | 09:26 | |
| *** heyongli has joined #senlin | 09:30 | |
| *** heyongli has quit IRC | 09:42 | |
| *** zzxwill has joined #senlin | 09:43 | |
| *** Drago1 has quit IRC | 09:58 | |
| *** zzxwill has quit IRC | 10:00 | |
| *** Drago1 has joined #senlin | 10:09 | |
| *** Drago1 has quit IRC | 10:10 | |
| *** Drago1 has joined #senlin | 10:10 | |
| *** yanyanhu has quit IRC | 10:23 | |
| *** zzxwill has joined #senlin | 10:25 | |
| Qiming | ruijie, welcome | 10:29 |
| Qiming | are you at Barcelona at the moment? | 10:29 |
| ruijie | Qiming, you are asking me? | 10:30 |
| Qiming | yes | 10:30 |
| ruijie | No, I am not... | 10:30 |
| Qiming | okay, I thought I can meet you in person here | 10:31 |
| ruijie | Sorry Qiming. I am still in working overtime :) | 10:31 |
| Qiming | cool, :) | 10:31 |
| Qiming | a lot of people here at the summit are working "over time" now | 10:32 |
| ruijie | haha, yeah. And still have several hours. | 10:32 |
| *** zzxwill has quit IRC | 10:36 | |
| *** zzxwill has joined #senlin | 10:36 | |
| *** Drago1 has quit IRC | 10:39 | |
| *** zzxwill has quit IRC | 10:41 | |
| *** guoshan has quit IRC | 10:44 | |
| *** Drago1 has joined #senlin | 10:45 | |
| *** Drago1 has quit IRC | 10:45 | |
| *** Drago1 has joined #senlin | 10:45 | |
| *** Drago1 has quit IRC | 11:44 | |
| *** guoshan has joined #senlin | 11:45 | |
| *** guoshan has quit IRC | 11:49 | |
| *** Drago has joined #senlin | 12:15 | |
| *** Drago has quit IRC | 12:15 | |
| *** Drago has joined #senlin | 12:17 | |
| *** guoshan has joined #senlin | 12:22 | |
| *** guoshan has quit IRC | 12:27 | |
| *** Drago has quit IRC | 12:32 | |
| *** Drago has joined #senlin | 12:33 | |
| *** ruijie has quit IRC | 13:10 | |
| *** guoshan has joined #senlin | 13:23 | |
| *** guoshan has quit IRC | 13:27 | |
| openstackgerrit | Jeremy Liu proposed openstack/senlin: Standardize log translation https://review.openstack.org/390561 | 13:35 |
| *** miaohb has joined #senlin | 13:47 | |
| *** Drago has quit IRC | 13:49 | |
| *** openstackstatus has joined #senlin | 13:51 | |
| *** ChanServ sets mode: +v openstackstatus | 13:51 | |
| *** miaohb has quit IRC | 13:54 | |
| *** XueFengLiu has joined #senlin | 14:19 | |
| openstackgerrit | Jeremy Liu proposed openstack/senlin: Standardize log translation https://review.openstack.org/390561 | 14:23 |
| *** guoshan has joined #senlin | 14:24 | |
| *** guoshan has quit IRC | 14:28 | |
| *** Drago has joined #senlin | 14:49 | |
| *** Drago has quit IRC | 14:49 | |
| *** Drago has joined #senlin | 14:49 | |
| *** XueFengLiu has left #senlin | 15:09 | |
| *** guoshan has joined #senlin | 15:24 | |
| *** guoshan has quit IRC | 15:29 | |
| *** guoshan has joined #senlin | 16:25 | |
| *** guoshan has quit IRC | 16:30 | |
| *** Drago has quit IRC | 16:40 | |
| *** Drago has joined #senlin | 16:47 | |
| *** Drago has quit IRC | 16:51 | |
| *** Drago has joined #senlin | 17:07 | |
| *** Drago has quit IRC | 17:07 | |
| *** Drago has joined #senlin | 17:07 | |
| *** guoshan has joined #senlin | 17:26 | |
| *** guoshan has quit IRC | 17:30 | |
| *** guoshan has joined #senlin | 18:00 | |
| *** guoshan has quit IRC | 18:05 | |
| *** guoshan has joined #senlin | 19:02 | |
| *** yuanying_ has quit IRC | 19:04 | |
| *** guoshan has quit IRC | 19:06 | |
| *** Drago has quit IRC | 19:18 | |
| *** Drago has joined #senlin | 19:43 | |
| *** Drago has quit IRC | 19:45 | |
| *** Drago has joined #senlin | 19:45 | |
| *** guoshan has joined #senlin | 20:03 | |
| *** guoshan has quit IRC | 20:07 | |
| *** guoshan has joined #senlin | 21:03 | |
| *** xuhaiwei has quit IRC | 21:08 | |
| *** guoshan has quit IRC | 21:08 | |
| *** guoshan has joined #senlin | 22:04 | |
| *** guoshan has quit IRC | 22:08 | |
| *** guoshan has joined #senlin | 23:05 | |
| *** guoshan has quit IRC | 23:09 | |
| *** Drago has quit IRC | 23:15 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!