| *** guoshan has quit IRC | 00:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/395377 | 00:21 |
|---|---|---|
| openstackgerrit | OpenStack Proposal Bot proposed openstack/senlin: Updated from global requirements https://review.openstack.org/397920 | 00:22 |
| *** shu-mutou-AWAY is now known as shu-mutou | 00:33 | |
| *** XueFeng has joined #senlin | 00:36 | |
| *** guoshan has joined #senlin | 00:53 | |
| openstackgerrit | miaohb proposed openstack/python-senlinclient: Revise the help message of policy binding list https://review.openstack.org/393981 | 00:57 |
| *** lvdongbing has joined #senlin | 00:57 | |
| *** guoshan has quit IRC | 00:58 | |
| *** yanyanhu has joined #senlin | 01:23 | |
| *** miaohb has joined #senlin | 01:24 | |
| *** miaohb has quit IRC | 01:27 | |
| *** XueFengLiu has joined #senlin | 01:29 | |
| *** root____9 has joined #senlin | 01:30 | |
| *** root____9 has left #senlin | 01:30 | |
| *** zhurong has joined #senlin | 01:30 | |
| *** yanyanhu has quit IRC | 01:30 | |
| *** miaohb has joined #senlin | 01:30 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Add container profile infor to cluster dependents https://review.openstack.org/396016 | 01:31 |
| *** yanyanhu has joined #senlin | 01:33 | |
| *** ruijie has joined #senlin | 01:35 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Add container profile infor to cluster dependents https://review.openstack.org/396016 | 01:40 |
| *** yanyanhu has quit IRC | 01:40 | |
| *** guoshan has joined #senlin | 01:42 | |
| *** elynn has joined #senlin | 01:58 | |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Revise ex_lbas.yaml cannot be found https://review.openstack.org/398027 | 02:04 |
| openstackgerrit | Merged openstack/senlin: fix policy-list2 error https://review.openstack.org/397626 | 02:10 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Update host node 'dependents' when create/delete container node https://review.openstack.org/396016 | 02:17 |
| *** elynn_ has joined #senlin | 02:21 | |
| *** elynn has quit IRC | 02:24 | |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: remove dead code in rpc https://review.openstack.org/397010 | 02:38 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Remove dead code about action list https://review.openstack.org/396914 | 02:40 |
| openstackgerrit | lvdongbing proposed openstack/senlin: Engine support for profile-update2 https://review.openstack.org/398048 | 02:44 |
| openstackgerrit | Kenji Ishii proposed openstack/senlin-dashboard: angularize cluster tables https://review.openstack.org/397730 | 02:45 |
| *** yanyanhu has joined #senlin | 02:48 | |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: add batch policy spec https://review.openstack.org/398049 | 02:48 |
| *** yuanying has quit IRC | 02:52 | |
| openstackgerrit | lvdongbing proposed openstack/senlin: API support for profile-get2 https://review.openstack.org/397493 | 02:52 |
| *** yanyanhu has quit IRC | 02:57 | |
| openstackgerrit | miaohb proposed openstack/senlin: Remove unsupported sort key "priority" https://review.openstack.org/398055 | 03:00 |
| miaohb | Qiming Please take a look at this: https://bugs.launchpad.net/senlin/+bug/1641916 | 03:06 |
| openstack | Launchpad bug 1641916 in senlin "the sort key "user" in event-list cannot work" [Undecided,In progress] - Assigned to miaohb (miao-hongbao) | 03:06 |
| miaohb | Qiming We'd better to add sort key "user", or just revise the help message? Waiting for your advice, thanks alot. | 03:09 |
| Qiming | I'd vote removing the 'user' from sort key | 03:12 |
| miaohb | ok | 03:15 |
| lvdongbing | Someone please help review this patch: https://review.openstack.org/397051 | 03:23 |
| openstackgerrit | miaohb proposed openstack/python-senlinclient: Remove unsupported sort key "user" https://review.openstack.org/398064 | 03:23 |
| *** Drago has quit IRC | 03:40 | |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: add batch policy spec https://review.openstack.org/398049 | 03:42 |
| *** yuanying has joined #senlin | 03:50 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Update host node 'dependents' when create/delete container node https://review.openstack.org/396016 | 03:52 |
| openstackgerrit | lvdongbing proposed openstack/senlin: API support for profile-update2 https://review.openstack.org/398071 | 03:58 |
| openstackgerrit | lvdongbing proposed openstack/senlin: Engine support for profile-update2 https://review.openstack.org/398048 | 04:01 |
| *** guoshan has quit IRC | 04:04 | |
| *** yanyanhu has joined #senlin | 04:04 | |
| *** guoshan has joined #senlin | 04:07 | |
| openstackgerrit | Merged openstack/python-senlinclient: Remove unsupported sort key "user" https://review.openstack.org/398064 | 04:12 |
| openstackgerrit | miaohb proposed openstack/senlin: Remove unsupported sort key "user" https://review.openstack.org/398091 | 04:22 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Revise ex_lbas.yaml cannot be found https://review.openstack.org/398027 | 04:29 |
| *** elynn_ has quit IRC | 04:35 | |
| *** guoshan has quit IRC | 04:45 | |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Revise ex_lbas.yaml cannot be found https://review.openstack.org/398027 | 04:53 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/395377 | 04:53 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/senlin: Updated from global requirements https://review.openstack.org/397920 | 04:53 |
| *** guoshan has joined #senlin | 05:15 | |
| Qiming | hi, need some reviews on the blueprint and hopefully approve it: https://blueprints.launchpad.net/senlin/+spec/generic-event | 05:18 |
| *** guoshan has quit IRC | 05:19 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Move event database driver out of engine https://review.openstack.org/398105 | 05:21 |
| Qiming | and possiblely this one as well: https://review.openstack.org/397475 | 05:22 |
| *** yanyanhu has quit IRC | 05:26 | |
| *** yanyanhu has joined #senlin | 05:28 | |
| *** yanyanhu has quit IRC | 05:34 | |
| *** yanyanhu has joined #senlin | 05:38 | |
| *** yanyanhu has quit IRC | 05:38 | |
| *** elynn has joined #senlin | 05:45 | |
| *** elynn_ has joined #senlin | 05:48 | |
| *** elynn has quit IRC | 05:49 | |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Revise ex_lbas.yaml cannot be found https://review.openstack.org/398027 | 06:06 |
| *** guoshan has joined #senlin | 06:09 | |
| *** elynn has joined #senlin | 06:11 | |
| openstackgerrit | miaohb proposed openstack/senlin: Support sort key "oid" in event list https://review.openstack.org/398116 | 06:12 |
| *** elynn_ has quit IRC | 06:13 | |
| *** guoshan has quit IRC | 06:14 | |
| *** guoshan has joined #senlin | 06:15 | |
| openstackgerrit | Merged openstack/senlin: API support for profile-list2 https://review.openstack.org/397051 | 06:17 |
| openstackgerrit | lvdongbing proposed openstack/senlin: API support for profile-update2 https://review.openstack.org/398071 | 06:19 |
| *** Eldon has joined #senlin | 06:26 | |
| openstackgerrit | lvdongbing proposed openstack/senlin: API support for profile-get2 https://review.openstack.org/397493 | 06:27 |
| *** yanyanhu has joined #senlin | 06:29 | |
| *** yanyanhu has quit IRC | 06:29 | |
| *** yanyanhu has joined #senlin | 06:33 | |
| *** yanyanhu has quit IRC | 06:44 | |
| openstackgerrit | lvdongbing proposed openstack/senlin: Engine support for profile-delete2 https://review.openstack.org/398137 | 06:58 |
| *** guoshan has quit IRC | 07:01 | |
| *** yanyanhu has joined #senlin | 07:09 | |
| *** yanyanhu has quit IRC | 07:09 | |
| openstackgerrit | lvdongbing proposed openstack/senlin: APi support for profile-delete2 https://review.openstack.org/398147 | 07:16 |
| *** guoshan has joined #senlin | 07:22 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Update host node 'dependents' when create/delete container node https://review.openstack.org/396016 | 07:32 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Add container node profile id to vm cluster's 'dependents' property https://review.openstack.org/398154 | 07:32 |
| xuhaiwei | Qiming, can you review this patch https://review.openstack.org/398154, it has some problems, I can't find a better way to resolve the dependency between container node and the vm cluster | 07:35 |
| openstackgerrit | Shan Guo proposed openstack/senlin: Update the cli in doc of policy command https://review.openstack.org/398156 | 07:37 |
| Qiming | xuhaiwei, I think some logics should be moved out of the DB layer | 07:48 |
| xuhaiwei | Qiming, I am thinking about whether should we add dependency to cluster | 07:49 |
| xuhaiwei | what about just ignoring the dependency | 07:50 |
| Qiming | then? | 07:50 |
| xuhaiwei | after a vm cluster is deleted, if user tries to create a new container node, notfound error will happen | 07:52 |
| Qiming | :) | 07:52 |
| Qiming | it is beyond that ... | 07:54 |
| Qiming | everytime you want to operate the container nodes, you may have to deal with the hosting cluster, and you will have to detect whether the cluster is there, and yell if it is not | 07:55 |
| xuhaiwei | when deleting a cluster which has vms and there are also containers running on vms, the dependency between vm node and container node will forbid the action | 07:55 |
| xuhaiwei | so ? | 07:56 |
| Qiming | right, those are "hard" dependencies between nodes ... I was hoping that we can manage "protocol" dependencies as well | 07:57 |
| Qiming | in an object-oriented world, we may want to try things in different ways | 07:58 |
| xuhaiwei | what are 'protocol' dependencies? | 07:59 |
| Qiming | a metaphor for the 'dependency' between a profile and a cluster | 07:59 |
| *** CMSS has joined #senlin | 08:00 | |
| Qiming | xuhaiwei, check this code: http://git.openstack.org/cgit/openstack/senlin/tree/senlin/engine/receivers/base.py#n100 | 08:00 |
| *** Eldon has quit IRC | 08:01 | |
| Qiming | the 'create' method defined for Receiver base class becomes the standard way to create a receiver instance | 08:02 |
| Qiming | we are not using just 'receiver = Receiver(type, cluster_id, ...)', instead we use 'Receiver.create(...)' | 08:03 |
| Qiming | then we invoke 'initialize_channel()' from the create() method | 08:04 |
| Qiming | each receiver type implementation can write their own 'intialize_channel()' method for specialization | 08:04 |
| Qiming | I'm wondering if we should do this for the Profile class as well | 08:05 |
| Qiming | so that the container profile type got a chance, during its initialization, to do some extra set up work | 08:06 |
| xuhaiwei | yes, currently from service layer then go to db layer directly, right? | 08:07 |
| Qiming | that is a "standard" way for adding another indirection layer | 08:07 |
| Qiming | right | 08:07 |
| Qiming | http://git.openstack.org/cgit/openstack/senlin/tree/senlin/engine/service.py#n431 | 08:08 |
| xuhaiwei | got this | 08:08 |
| Qiming | we invoke the constructor (__init__) there, and then do some validation, then save it to db | 08:08 |
| Qiming | if we have an indirection there, we can inject this indirection logic for container profile type | 08:09 |
| Qiming | anyway, you get the idea, :) | 08:09 |
| xuhaiwei | yes | 08:09 |
| xuhaiwei | so you still believe we should keep the dependency between vm cluster and container node? | 08:10 |
| Qiming | en | 08:15 |
| Qiming | thinking from a user's perspective, it would be very annoying that senlin tells me the VM cluster cannot be found when I scale my container cluster from 0 to 3 nodes | 08:16 |
| xuhaiwei | hmm | 08:17 |
| xuhaiwei | another problem is when storing the profile id to 'dependents' of cluster, to avoid conflict, we should do it in one db session, but the profile id is not created yet if the session is not over. | 08:20 |
| Qiming | exactly the same problem we met when creating receivers | 08:22 |
| Qiming | we need the receiver ID to generate webhook URL | 08:22 |
| xuhaiwei | how did you resolve it? | 08:23 |
| Qiming | refer to receiver base class | 08:24 |
| openstackgerrit | lvdongbing proposed openstack/senlin: Remove dead code related to profile list https://review.openstack.org/398179 | 08:25 |
| Qiming | http://git.openstack.org/cgit/openstack/senlin/tree/senlin/engine/receivers/base.py#n99 | 08:25 |
| openstackgerrit | XueFeng Liu proposed openstack/python-senlinclient: Add "cluster_id" column to `openstack cluster event list` https://review.openstack.org/397655 | 08:28 |
| xuhaiwei | I will refer to it | 08:28 |
| *** CMSS has quit IRC | 08:32 | |
| openstackgerrit | miaohb proposed openstack/python-senlinclient: Add sort key "oid" in event list https://review.openstack.org/398191 | 08:44 |
| openstackgerrit | lvdongbing proposed openstack/senlin: Revise profile-validate2 object fields https://review.openstack.org/398210 | 09:11 |
| *** yanyanhu has joined #senlin | 09:14 | |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Revise ex_lbas.yaml cannot be found https://review.openstack.org/398027 | 09:15 |
| *** ruijie has quit IRC | 09:18 | |
| *** ruijie has joined #senlin | 09:28 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Revise the DB event dumper https://review.openstack.org/398225 | 09:39 |
| *** lvdongbing has quit IRC | 09:55 | |
| *** ruijie has quit IRC | 09:55 | |
| *** zhurong has quit IRC | 10:02 | |
| *** elynn has quit IRC | 10:03 | |
| *** yanyanhu has quit IRC | 10:17 | |
| *** yanyanhu has joined #senlin | 10:17 | |
| *** yanyanhu has quit IRC | 10:18 | |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Remove dead code about action get https://review.openstack.org/396917 | 10:27 |
| *** guoshan has quit IRC | 10:51 | |
| *** shu-mutou is now known as shu-mutou-AWAY | 11:23 | |
| *** guoshan has joined #senlin | 11:35 | |
| *** guoshan_ has joined #senlin | 11:39 | |
| *** guoshan has quit IRC | 11:39 | |
| *** guoshan_ has quit IRC | 11:44 | |
| *** elynn has joined #senlin | 11:59 | |
| *** elynn_ has joined #senlin | 12:03 | |
| *** elynn has quit IRC | 12:04 | |
| *** elynn_ has quit IRC | 12:17 | |
| *** catintheroof has joined #senlin | 12:23 | |
| *** guoshan has joined #senlin | 12:33 | |
| *** zhurong has joined #senlin | 12:35 | |
| *** guoshan has quit IRC | 12:40 | |
| openstackgerrit | Qiming Teng proposed openstack/python-senlinclient: Add "cluster_id" column to `openstack cluster event list` https://review.openstack.org/397655 | 13:09 |
| *** elynn_ has joined #senlin | 13:11 | |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Add request object for event-list https://review.openstack.org/397583 | 13:16 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Add engine support for event_list2 https://review.openstack.org/398340 | 13:16 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Api support for event_list2 https://review.openstack.org/398341 | 13:17 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Add request object for event-list https://review.openstack.org/397583 | 13:19 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Add engine support for event_list2 https://review.openstack.org/398340 | 13:19 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Api support for event_list2 https://review.openstack.org/398341 | 13:19 |
| *** XueFengLiu has left #senlin | 13:22 | |
| *** guoshan has joined #senlin | 13:28 | |
| *** guoshan has quit IRC | 13:32 | |
| openstackgerrit | Merged openstack/python-senlinclient: Support xyz_update purging metadata https://review.openstack.org/391179 | 13:33 |
| openstackgerrit | Merged openstack/senlin: remove dead code of policy delete https://review.openstack.org/396566 | 13:59 |
| openstackgerrit | Merged openstack/senlin: remove dead code in rpc https://review.openstack.org/397010 | 13:59 |
| *** guoshan has joined #senlin | 14:22 | |
| *** guoshan has quit IRC | 14:26 | |
| *** Drago1 has joined #senlin | 14:28 | |
| *** Drago1 has quit IRC | 14:33 | |
| *** elynn_ has quit IRC | 14:38 | |
| *** Drago1 has joined #senlin | 14:45 | |
| *** elynn_ has joined #senlin | 14:50 | |
| *** elynn_ has quit IRC | 14:50 | |
| *** zhurong has quit IRC | 14:51 | |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Add request object for event-list https://review.openstack.org/397583 | 15:09 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Add engine support for event_list2 https://review.openstack.org/398340 | 15:14 |
| *** guoshan has joined #senlin | 15:23 | |
| *** guoshan has quit IRC | 15:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/395377 | 15:45 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/senlin: Updated from global requirements https://review.openstack.org/397920 | 15:45 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Api support for event_list2 https://review.openstack.org/398341 | 15:53 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Api support for event_list2 https://review.openstack.org/398341 | 15:54 |
| *** guoshan has joined #senlin | 16:24 | |
| *** guoshan has quit IRC | 16:28 | |
| *** guoshan has joined #senlin | 17:31 | |
| *** XueFeng has quit IRC | 17:33 | |
| *** guoshan has quit IRC | 17:36 | |
| *** catinthe_ has joined #senlin | 18:13 | |
| *** catintheroof has quit IRC | 18:15 | |
| *** catintheroof has joined #senlin | 18:15 | |
| *** catinthe_ has quit IRC | 18:18 | |
| *** guoshan has joined #senlin | 18:32 | |
| *** guoshan has quit IRC | 18:36 | |
| *** guoshan has joined #senlin | 19:26 | |
| *** guoshan has quit IRC | 19:30 | |
| *** guoshan has joined #senlin | 20:20 | |
| *** guoshan has quit IRC | 20:25 | |
| *** guoshan has joined #senlin | 21:14 | |
| *** guoshan has quit IRC | 21:19 | |
| *** catinthe_ has joined #senlin | 21:32 | |
| *** Drago1 is now known as Drago | 21:34 | |
| *** catintheroof has quit IRC | 21:36 | |
| *** catintheroof has joined #senlin | 21:51 | |
| *** catinthe_ has quit IRC | 21:51 | |
| *** guoshan has joined #senlin | 22:08 | |
| *** guoshan has quit IRC | 22:13 | |
| *** catintheroof has quit IRC | 22:54 | |
| *** guoshan has joined #senlin | 23:02 | |
| *** guoshan has quit IRC | 23:07 | |
| *** guoshan has joined #senlin | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!