Friday, 2016-12-09

*** ruijie has joined #senlin00:58
*** zhurong has joined #senlin01:13
*** Drago1 has quit IRC01:13
*** Drago1 has joined #senlin01:14
*** guoshan has joined #senlin01:21
*** EldonZhao has joined #senlin01:28
*** yanyanhu has joined #senlin01:42
*** elynn has joined #senlin01:43
*** chohoor has joined #senlin01:46
*** elynn has quit IRC01:48
*** elynn has joined #senlin01:50
openstackgerritMerged openstack/senlin: Remove 'validate_sort_params"  https://review.openstack.org/40748301:55
EldonZhaoDo we support creating a node using an existed nova instance?01:58
QimingEldonZhao, we discussed that02:03
Qiming[16:00] <Qiming> a draft idea on solving this:02:04
Qiming[16:01] <Qiming> 1. senlin profile-spec-create -t os.nova.server -o <spec_file> <nova_instance>02:04
Qiming[16:01] <Qiming> 2. edit the spec_file created, fix parameters like volume and network settings02:04
Qiming[16:02] <Qiming> 3. senlin profile-create -s <spec_file> profile_for_existing_instance02:04
Qiming[16:02] <Qiming> 4. senlin node-adopt -p profile_for_existing_instance <nova_instance>02:04
Qiming[16:03] <Qiming> 5. senlin cluster-node-add -n <node_id_from_step_4> <cluster_identity>02:04
Qiming[16:06] <Qiming> it is a lengthy flow, some steps can be merged/combined I think02:04
Qimingthe first step would be creating a spec file from an existing nova instance02:05
Qimingthen users are expected to do some tuning/modifications to the spec file, regarding the network and storage parameters02:05
Qimingthen he/she creates a new profile instance using this spec02:06
Qimingthe next step (regarding senlin's support), would be adding a node-adopt API, treating the nova instance as created from senlin02:06
*** elynn_ has joined #senlin02:19
*** elynn has quit IRC02:20
EldonZhaoQiming, Thanks:)02:20
openstackgerritAaron Ding proposed openstack/python-senlinclient: Add destroy_after_deletion param in`cluster members del`.  https://review.openstack.org/40889802:20
openstackgerritAaron Ding proposed openstack/senlin: Add destroy_after_deletion param when cluster del nodes.  https://review.openstack.org/40889902:21
QimingEldonZhao, do you have a strong requirement on this?02:40
Qimingwe can put it to a higher priority if so02:40
Qimingjust talk to yanyan02:40
QimingI'm afraid we won't be able to land it before Ocata-2, the window is too short, Ocata-3 would be a possibility02:41
EldonZhaoWe are discussing with users now, and I think they need us support it just like Aliyun.02:41
Qimingokay, understood02:42
Qimingdo you understand why senlin needs to create a spec out of a nova server and then have the user to modify it?02:43
*** yuanying has quit IRC02:50
*** yuanying has joined #senlin03:50
*** yuanying has quit IRC03:50
*** yuanying has joined #senlin03:51
*** guoshan has quit IRC03:55
*** elynn_ has quit IRC04:23
*** yuanying has quit IRC04:23
*** yuanying has joined #senlin04:24
*** yuanying has quit IRC04:24
*** yuanying has joined #senlin04:25
*** yuanying has quit IRC04:25
*** guoshan has joined #senlin04:56
*** guoshan has quit IRC05:00
*** yuanying has joined #senlin05:26
*** yuanying has quit IRC05:35
*** yuanying has joined #senlin05:36
*** yuanying has quit IRC05:49
*** elynn_ has joined #senlin05:52
*** elynn__ has joined #senlin05:56
*** elynn_ has quit IRC05:56
openstackgerritAaron Ding proposed openstack/python-senlinclient: Get project_id and user_id when show profile  https://review.openstack.org/40742006:07
*** guoshan has joined #senlin06:09
*** guoshan has quit IRC06:13
openstackgerritMerged openstack/senlin-dashboard: Replace mox with mock  https://review.openstack.org/40860506:14
*** guoshan has joined #senlin06:15
openstackgerritQiming Teng proposed openstack/senlin: Augment request context with api microversion  https://review.openstack.org/40897006:19
*** yuanying has joined #senlin06:28
openstackgerritRUIJIE YUAN proposed openstack/senlin: engine service support cluster-policy-list2  https://review.openstack.org/40850906:39
openstackgerritRUIJIE YUAN proposed openstack/senlin: api layer support cluster-policy-list2  https://review.openstack.org/40897606:39
openstackgerritRUIJIE YUAN proposed openstack/senlin: api layer support cluster-policy-list2  https://review.openstack.org/40897606:42
*** Drago1 has quit IRC06:42
openstackgerritRUIJIE YUAN proposed openstack/senlin: engine service support cluster-policy-list2  https://review.openstack.org/40850906:53
openstackgerritRUIJIE YUAN proposed openstack/senlin: api layer support cluster-policy-list2  https://review.openstack.org/40897606:53
openstackgerritRUIJIE YUAN proposed openstack/senlin: remove dead code of cluster-policy-list  https://review.openstack.org/40897906:53
*** yuanying_ has joined #senlin06:55
openstackgerritMerged openstack/senlin: Updated from global requirements  https://review.openstack.org/40811406:56
openstackgerritMerged openstack/senlin: Temporarily disable failed receiver integration test  https://review.openstack.org/40727406:56
*** zhurong has quit IRC06:57
*** zhurong has joined #senlin06:58
*** yuanying has quit IRC06:58
openstackgerritRUIJIE YUAN proposed openstack/senlin: remove dead code of cluster-policy-list  https://review.openstack.org/40897907:01
openstackgerritMerged openstack/senlin: Remove 'ProfileTypeNotMatch' exception type  https://review.openstack.org/40793807:02
*** yuanying_ has quit IRC07:14
*** guoshan has quit IRC07:22
ruijiehi, Qiming07:22
Qiminghi07:27
openstackgerritYanyan Hu proposed openstack/senlin: Versioned request object for receiver_nofity  https://review.openstack.org/40899007:27
openstackgerritYanyan Hu proposed openstack/senlin: Engine support for receiver_notify2  https://review.openstack.org/40899107:27
ruijieI am thinking about a scenario07:28
ruijiee.g. we have a cron task to do cluster_scale_out periodly07:28
ruijiesay every 11:00 am07:28
ruijieif the task failed to scale out the cluster, we need to redo it.07:29
ruijiecount=5 succeeded=4, failed=1. then the problem will be we create 9 node if the 2nd task succeeded07:30
ruijiedo the recover action can cover this case?07:30
ruijiedoes07:30
*** guoshan has joined #senlin07:30
Qimingit should07:33
Qimingso, the design philosophy is like this07:33
Qimingwe can never ensure that all cluster operations are completely successful07:34
Qimingespecially when you do scaling07:34
Qimingafter each cluster operation, especially the cluster scaling operations, we update the cluster's desired_capacity07:35
Qimingand you can count the actual capacity by counting cluster nodes07:35
Qimingsay you have a cluster of 3 nodes, you want to add 4 more to it, by cluster-add-nodes or cluster-scale-out07:36
Qimingthen senlin knows the 'desired_capacity' is 7 now07:36
Qimingit can never ensure that the cloud can eventually give you 7 nodes to use07:37
Qimingyou can detect that from the cluster status, by checking count of cluster-node-list07:37
ruijieso this working flow it totally controlled by the user07:38
ruijieit/is07:39
Qimingyes, we wanted to make some decisions for users07:42
Qimingbut it doesn't seem the right approach07:42
Qimingby the way, cluster-check and cluster-recover workflows need some more hammers07:43
ruijieem, infact we are trying to avoid repeatedly creating resources07:43
ruijiethe recover action is the way we are finding :)07:44
Qimingcurrently, take cluster-recover as an example, we only recover the list of nodes that are not ACTIVE07:44
Qimingand I'm not 100% confident that workflow is free of bugs07:45
ruijiewe need to monitor the real status of the server?07:45
Qimingwe have cluster-check API to do that07:46
openstackgerritYanyan Hu proposed openstack/senlin: API support for receiver_notify2  https://review.openstack.org/40899707:46
openstackgerritYanyan Hu proposed openstack/senlin: Remove dead code about receiver-notify  https://review.openstack.org/40899807:46
Qimingwe rejected the idea to to automatic periodical checking07:46
Qimingwe only expose such APIs for users to invoke on demand07:46
Qimingif you want to achieve a high availability of your cluster, you may want to create and attach a health policy07:47
ruijieyes, that makes sense07:48
ruijieor that will be heavy work07:48
Qimingit doesn't matter if it is a heavy work or not07:48
openstackgerritQiming Teng proposed openstack/senlin: Add version map search to base object  https://review.openstack.org/40900107:48
Qimingthe real question is whether we are doing too much to make users hate us07:49
ruijieI see, Qiming, let the user themselves to build their workflow07:50
Qimingyes07:50
Qimingif they trust us, we do offer some built in workflows07:51
Qimingfor example, the ha policy is doing a lot of that07:51
Qimingthe policy does periodical health check on your cluster nodes and attempt a recovery if node failures are found07:51
Qimingit can also listen to VM failure events and then react to those events (properly)07:52
ruijiethat's cool07:54
ruijiethat makes the cluster more reliable07:54
ruijieQiming, thanks for your time, will check the HA part07:55
Qimingthere are still a lot of things to do in that space07:55
Qimingthe health policy is unable to treat different events in different ways07:55
Qimingwe are also looking into extending the workflow to execute jobs defined by users07:56
ruijieah, only the active and Non-active?07:56
Qimingin the case of a VM failure, senlin will try reboot, rebuild, recreate that VM, and after that, we want enable senlin to invoke a mistral workflow for further recovery07:57
Qiminglook here, ruijie08:00
Qiminghttp://git.openstack.org/cgit/openstack/senlin/tree/senlin/engine/health_manager.py#n5708:00
ruijiesince we already could recreate a server, what do we want to to with mistral?08:00
Qimingbecause creating VMs is not the goal08:01
Qimingyour goal is always about getting some applications/services configured and started in those VMs08:01
Qimingthose are part of the recovery operation08:02
ruijieright08:02
Qimingand we don't know what users want to do after the VM is recovered08:02
Qimingif they model their operations using mistral already, we can invoke those workflows08:02
ruijiegreat, then the user could use that plugin/driver .. to run init scripts or something to let their app or service up08:03
Qimingyes08:05
Qimingthe whole workflow would be something like this:08:05
Qimingvm failed -> detected by nova -> nova sends notification -> senlin heard the notification -> senlin decides whether fencing is needed and does it if so -> senlin initiates a vm recover operation by providing the needed info --> senlin invoke external logics to get apps resurrected ...08:07
openstackgerritRUIJIE YUAN proposed openstack/senlin: engine service support cluster-policy-list2  https://review.openstack.org/40850908:08
ruijiegreat08:10
ruijieQiming, I will check this 2 APIs and the HA part08:10
Qimingokay08:12
ruijiethanks, Qiming :)08:14
openstackgerritRUIJIE YUAN proposed openstack/senlin: api layer support cluster-policy-list2  https://review.openstack.org/40897608:15
openstackgerritMerged openstack/senlin: Request object for action-create  https://review.openstack.org/40856708:17
openstackgerritRUIJIE YUAN proposed openstack/senlin: remove dead code of cluster-policy-list  https://review.openstack.org/40897908:18
openstackgerritRUIJIE YUAN proposed openstack/senlin: engine service support cluster-policy-list2  https://review.openstack.org/40850908:18
openstackgerritRUIJIE YUAN proposed openstack/senlin: api layer support cluster-policy-list2  https://review.openstack.org/40897608:18
openstackgerritYanyan Hu proposed openstack/senlin: Correct the output of cluster-update  https://review.openstack.org/40902108:36
openstackgerritYanyan Hu proposed openstack/senlin: Correct the output of cluster-update  https://review.openstack.org/40902108:39
openstackgerritYanyan Hu proposed openstack/senlin: Correct the output of cluster-update  https://review.openstack.org/40902108:42
openstackgerritYanyan Hu proposed openstack/senlin: Correct the output of node-update  https://review.openstack.org/40902908:50
openstackgerritRUIJIE YUAN proposed openstack/senlin: fields obj support cluster-policy-get2  https://review.openstack.org/40903909:12
yanyanhuhi, elynn__, around?09:13
elynn__Hi yanyanhu09:14
yanyanhuhi09:14
yanyanhuI noticed integration test failed for heat service was not setup correctly during devstack-gate installation09:14
elynn__any link?09:14
yanyanhuand seems heat is not enabled as plugin in gate job template of some other services09:15
yanyanhuactually heat service was not installed09:15
elynn__heat?09:15
yanyanhuyes09:15
yanyanhudo you know any recent change about it?09:15
yanyanhubefore, we only need to add heat service to enabled service list09:16
yanyanhuthen devstack-gate will set it up09:16
yanyanhulike this: ENABLED_SERVICES=heat,h-api,h-api-cfn,h-api-cw,h-eng,09:16
elynn__https://review.openstack.org/#/c/361542/09:16
yanyanhuah09:16
yanyanhugot it09:16
yanyanhuexport DEVSTACK_LOCAL_CONFIG+=$'\n'"enable_plugin heat git://git.openstack.org/openstack/heat"09:16
yanyanhuneed this line09:16
elynn__might need to use heat devstack plugin09:17
yanyanhuyes, seems so09:17
yanyanhujust a little concern once they change it back someday, we need to change our gate job again...09:18
yanyanhusigh09:18
yanyanhuanyway, it only influences the integration test09:18
elynn__devstack plugin is the future09:19
elynn__I don't think they will change it back..09:20
yanyanhuI see09:20
yanyanhuwill propose change to gate09:20
elynn__And I don't think you need to add "ENABLED_SERVICES=heat,h-api,h-api-cfn,h-api-cw,h-eng,"09:20
elynn__These service will set by https://github.com/openstack/heat/blob/master/devstack/settings09:21
yanyanhuok09:21
yanyanhujust noticed that those services are added to list in heat's gate job09:22
elynn__Is there any variables like "enable_heat yes" to enable heat on gate?09:23
elynn__So that we don't need to follow the change.09:23
yanyanhunope, didn't see such variables09:24
elynn__export DEVSTACK_GATE_HEAT=1 ?09:25
elynn__Seems most of the projects use "enable_plugin heat"09:25
yanyanhuyou mean just adding this line?09:26
yanyanhua little projects use export devstack_gate_heat=109:26
elynn__I mean I saw  this line but I don't know if it can work or not....09:26
elynn__"enable_plugin heat" is the safer way.09:27
yanyanhubut that was used together with "DEVSTACK_LOCAL_CONFIG+=$'\n'"enable_plugin heat git://git.openstack.org/openstack/heat""09:27
yanyanhuI see09:27
yanyanhumay I should try enable_plugin heat first09:27
yanyanhuthanks, elynn__  :)09:28
openstackgerritRUIJIE YUAN proposed openstack/senlin: fields obj support cluster-policy-get2  https://review.openstack.org/40903909:28
*** elynn__ has quit IRC09:29
*** yanyanhu has quit IRC09:39
openstackgerritRUIJIE YUAN proposed openstack/senlin: engine service support cluster-policy-get2  https://review.openstack.org/40904809:45
*** zhurong has quit IRC09:46
*** EldonZhao has quit IRC10:14
*** guoshan has quit IRC10:24
*** chohoor has quit IRC10:53
fabian4hi, i have a Problem, when i use senlin cluster-policy-attach  lb-policy  appear ERROR senlin.drivers.openstack.sdk [req-cd565cb4-33f4-44ab-aa14-fe1decac8b03 8874cf3479ad4f92ade25715d2a0f591 79797a227f1445e0aa847b93026cc81e - - -] NotFoundException: Not Found, 404 Not Found11:37
fabian4the other policy not appear error11:38
*** guoshan has joined #senlin12:10
*** elynn__ has joined #senlin13:04
*** elynn__ has quit IRC13:40
Qimingfabian4, have you installed lbaas service?13:41
Qimingsenlin only support lbaasv2 api13:42
openstackgerritMerged openstack/senlin: Versioned request object for receiver_nofity  https://review.openstack.org/40899013:59
openstackgerritMerged openstack/senlin: Move 'parse_bool_param' to api common  https://review.openstack.org/40751514:18
openstackgerritMerged openstack/senlin: Remove 'InvalidParameter' exception type  https://review.openstack.org/40793214:26
*** guoshan has quit IRC15:26
*** catintheroof has joined #senlin15:54
*** Drago1 has joined #senlin16:24
*** Drago1 has quit IRC17:25
*** openstack has joined #senlin19:48
*** Drago1 has joined #senlin22:00

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