Thursday, 2016-11-10

*** zhurong has joined #senlin00:15
*** zhurong has quit IRC00:22
*** hanwei has joined #senlin01:05
*** ruijie has joined #senlin01:08
*** yanyanhu has joined #senlin01:32
*** zhurong has joined #senlin01:33
*** XueFeng has quit IRC01:49
*** XueFeng has joined #senlin01:49
*** XueFeng has left #senlin01:52
*** XueFeng has joined #senlin01:52
*** XueFeng has quit IRC02:00
*** XueFeng has joined #senlin02:00
openstackgerritMerged openstack/senlin-dashboard: Updated from global requirements  https://review.openstack.org/39538702:20
*** Drago has quit IRC02:42
*** yuanying has quit IRC02:51
*** Drago has joined #senlin02:52
*** shu-mutou-AWAY is now known as shu-mutou02:55
openstackgerritQiming Teng proposed openstack/senlin: A spec for generic event/notification support  https://review.openstack.org/39487402:59
*** lvdongbing has joined #senlin03:30
openstackgerritRUIJIE YUAN proposed openstack/senlin: engine service support policy-create v2  https://review.openstack.org/39592303:33
*** yuanying has joined #senlin03:49
*** elynn has joined #senlin03:49
xuhaiweihi Qiming, are you around?04:38
*** XueFeng has quit IRC04:56
*** XueFeng has joined #senlin04:57
*** yuanying has quit IRC05:34
*** XueFeng has quit IRC05:37
*** XueFeng has joined #senlin05:37
openstackgerritRUIJIE YUAN proposed openstack/senlin: engine service support policy-create v2  https://review.openstack.org/39592305:40
*** XueFeng has quit IRC05:48
*** XueFeng has joined #senlin05:49
ruijiehi, Qiming, around?05:57
Qimingyes06:01
ruijiehttps://review.openstack.org/#/c/395679/06:01
ruijiefor this patch06:01
ruijiehttps://review.openstack.org/gitweb?p=openstack/senlin.git;a=blob;f=senlin/objects/base.py;h=0e1292d9bc6b0de2fba814f8e8a17b78b1500a2e;hb=e2d81eaf7619c64a6afc9930e53aee18e8639e63#l7306:02
ruijieI checked this method.06:02
ruijieI found it is a 2 level check and will construct the request obj.06:02
Qimingyes?06:02
ruijielike request: {body: ''}06:03
openstackgerritYanyan Hu proposed openstack/senlin: Remove dead code about receiver list  https://review.openstack.org/39530006:03
openstackgerritYanyan Hu proposed openstack/senlin: API support for receiver_list2  https://review.openstack.org/39529906:03
ruijiebut the policy-validate and policy-create request has 3 level06:03
Qimingwe don't check the policy spec, right?06:03
ruijiebut we need to construct it06:04
Qimingit is {"policy": {"name": "foo", "spec": { a json here}, "version": "1.0"}} ?06:05
Qimingoh, no, no version key06:05
Qimingshould be just  {"policy": {"name": "foo", "spec": { a json here}}06:06
ruijieactually, it is {"policy": {"spec": {"name": "", "version": "", "properties": {}}}}06:06
ruijieit's a 3 level construction06:07
Qimingat the api layer, we didn't define the format of the 'spec' value, right?06:07
ruijiehm..  but the request sample shows that..06:08
Qimingthen that sample is wrong06:08
Qimingread the code06:08
Qimingfrom senlinclient06:08
ruijieand for the policy-create, the request format should be the same as policy-validate06:09
Qimingyes, that was the design I think06:10
Qiminga real world example, I just tried, is this:06:10
Qimingcurl -g -i -X POST http://192.168.42.11:8778/v1/policies -H "openstack-api-version: clustering 1.2" -H "User-Agent: python-senlinclient openstacksdk/0.9.10 keystoneauth1/2.14.0 python-requests/2.11.1 CPython/2.7.12" -H "Content-Type: application/json" -H "X-Auth-Token: {SHA1}298a84a39e22b794ae6fa4a0ced465a2173ff803" -d '{"policy": {"name": "pr01", "spec": {"version": 1.0, "type": "senlin.policy.region_placement", "description": "A policy for node placement ac06:10
Qimingross regions", "properties": {"regions": [{"cap": 150, "name": "RegionOne", "weight": 100}, {"cap": 200, "name": "RegionTwo", "weight": 100}]}}}}'06:10
ruijieyes, the 3 level one is strange...06:10
Qimingthe data part is this:06:11
Qiming'{"policy": {"name": "pr01", "spec": {"version": 1.0, "type": "senlin.policy.region_placement", "description": "A policy for node placement across regions", "properties": {"regions": [{"cap": 150, "name": "RegionOne", "weight": 100}, {"cap": 200, "name": "RegionTwo", "weight": 100}]}}}}'06:11
Qimingyes, in the 'spec' key, we are expecting a dict that contains 'type', 'version', 'description' and 'properties' values, but those are details the API layer should not care06:12
Qimingactually, the 'spec' value comes from a spec file content06:12
Qimingthe content and format of the spec is a policy format specification, not part of the API specification06:13
ruijieI got it, like 'policy': fields.Obj....field('PolicyRequestBody') and in the body definition it will be 'spec': fields.Json...Field()06:14
ruijiewe don't need to define the content of spec.06:15
*** XueFeng has quit IRC06:16
*** XueFeng has joined #senlin06:16
*** XueFeng has quit IRC06:26
*** XueFeng has joined #senlin06:26
openstackgerritYanyan Hu proposed openstack/senlin: Fix unstable test  https://review.openstack.org/39597006:39
Qimingyes, ruijie06:40
ruijieokay, thanks, Qiming.06:44
openstackgerritShu Muto proposed openstack/senlin-dashboard: Add delete action for angular nodes panel  https://review.openstack.org/39597606:55
openstackgerritShu Muto proposed openstack/senlin-dashboard: Add JS tests for profile delete service  https://review.openstack.org/39597807:02
openstackgerritRUIJIE YUAN proposed openstack/senlin: fields support policy-validate2  https://review.openstack.org/39541207:07
openstackgerritRUIJIE YUAN proposed openstack/senlin: engine service support policy validate v2  https://review.openstack.org/39549507:18
*** XueFeng has quit IRC07:25
*** XueFeng has joined #senlin07:25
*** XueFeng has quit IRC07:34
*** XueFeng has joined #senlin07:34
*** XueFeng has quit IRC07:37
*** XueFeng has joined #senlin07:37
*** openstackgerrit has quit IRC07:48
*** openstackgerrit has joined #senlin07:49
*** ChanServ sets mode: +v openstackgerrit07:49
*** yuanying has joined #senlin07:55
*** Drago has quit IRC08:31
openstackgerritxu-haiwei proposed openstack/senlin: Add container profile infor to cluster dependents  https://review.openstack.org/39601608:37
*** shu-mutou is now known as shu-mutou-AWAY08:37
openstackgerritRUIJIE YUAN proposed openstack/senlin: api support policy validate v2  https://review.openstack.org/39567909:06
openstackgerritRUIJIE YUAN proposed openstack/senlin: remove dead code in policy validate  https://review.openstack.org/39568009:11
*** elynn has quit IRC09:23
*** elynn has joined #senlin09:24
openstackgerritRUIJIE YUAN proposed openstack/senlin: api support policy validate v2  https://review.openstack.org/39567909:39
openstackgerritRUIJIE YUAN proposed openstack/senlin: remove dead code in policy validate  https://review.openstack.org/39568009:40
*** lvdongbing has quit IRC09:43
*** yanyanhu has quit IRC09:59
*** zhurong has quit IRC10:01
*** hanwei has quit IRC10:17
openstackgerritMerged openstack/senlin-dashboard: Add JS tests for profile delete service  https://review.openstack.org/39597810:27
*** hanwei has joined #senlin10:47
openstackgerritKenji Ishii proposed openstack/senlin-dashboard: Add update profile action for angular profile panel  https://review.openstack.org/39133210:57
openstackgerritKenji Ishii proposed openstack/senlin-dashboard: Add filterFacets for angular profile table  https://review.openstack.org/38954411:04
*** gongysh has joined #senlin11:05
*** gongysh has quit IRC11:07
openstackgerritKenji Ishii proposed openstack/senlin-dashboard: Add update profile action for angular profile panel  https://review.openstack.org/39133211:14
*** gongysh2 has joined #senlin11:17
*** hanwei has quit IRC12:43
*** hanwei has joined #senlin12:44
*** zhurong has joined #senlin12:46
*** catintheroof has joined #senlin13:08
openstackgerritXueFeng Liu proposed openstack/senlin: Api support for action_list2  https://review.openstack.org/39618813:21
*** Drago has joined #senlin14:15
*** Drago has quit IRC14:19
*** Drago has joined #senlin14:25
*** elynn has quit IRC14:41
*** elynn_ has joined #senlin14:41
*** elynn_ has quit IRC14:46
*** zhurong has quit IRC15:24
*** Drago has quit IRC16:40
*** Drago has joined #senlin16:41
*** catinthe_ has joined #senlin16:45
*** catintheroof has quit IRC16:46
*** elynn has joined #senlin16:59
*** elynn has quit IRC17:02
*** Drago has quit IRC17:05
*** Drago has joined #senlin17:11
*** Drago has quit IRC17:21
*** hanwei has quit IRC18:15
*** Drago has joined #senlin19:33
*** openstackgerrit has quit IRC19:48
*** openstackgerrit has joined #senlin19:49
*** ChanServ sets mode: +v openstackgerrit19:49
*** Drago has quit IRC21:49
*** catinthe_ has quit IRC21:59
*** Drago has joined #senlin22:27
*** catintheroof has joined #senlin23:05
*** catintheroof has quit IRC23:09
*** catintheroof has joined #senlin23:17
*** yuanying has quit IRC23:34
*** yuanying has joined #senlin23:44
*** elynn has joined #senlin23:58

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