| openstackgerrit | xu-haiwei proposed openstack/senlin: Store node relationship into 'dependents' https://review.openstack.org/364846 | 00:16 |
|---|---|---|
| openstackgerrit | xu-haiwei proposed openstack/senlin: Store node relationship into 'dependents' https://review.openstack.org/364846 | 00:50 |
| *** yanyanhu has joined #senlin | 01:23 | |
| *** yuanying has joined #senlin | 01:24 | |
| *** yuanying has quit IRC | 01:24 | |
| *** zhurong has joined #senlin | 01:25 | |
| xuhaiwei | Hi Qiming | 01:48 |
| Qiming | yes | 01:49 |
| xuhaiwei | about the 'dependents' patch, do you think we need to add different 'key' name for the 'dependents' property? Qiming | 01:50 |
| Qiming | well, as I said in the comment just posted | 01:51 |
| xuhaiwei | I was think like this: container node has the format of {'host':'xxx'}, while vm node is {'containers': []} | 01:51 |
| Qiming | I don't see a clear value in doing that | 01:51 |
| Qiming | why should we differentiate it at the abstract node layer? | 01:51 |
| xuhaiwei | if we just add some UUID into it, it's not very readable | 01:51 |
| Qiming | at the abstract node layer, we just care about the dependencies would be good | 01:51 |
| Qiming | it seems that the engine.node module will be hijacked by different profile type implementations in this way | 01:52 |
| xuhaiwei | so you mean for the add_dependents method, just do node.update, leave all the data format to container layer? | 01:53 |
| Qiming | the rough idea of the engline.cluster and engine.node layer was to make them capable of handling whatever nodes ... e.g. physical machine, virtual machine, container, webapplication, or even micro-service etc | 01:53 |
| xuhaiwei | Qiming, so you mean for the add_dependents method, just do node.update, leave all the data format to container layer? | 01:55 |
| Qiming | if a container node C is hosted on a specific node A (i.e. a VM or BM node), we are about to record into A's 'dependents' property that it is depended by a node C | 01:55 |
| xuhaiwei | I think we need to add 'dependents' to both container node and vm node | 01:56 |
| Qiming | if a container node C is hosted on a specific cluster B (could be a VM cluster or something), we record C into the B's 'dependents' property, that is enough | 01:56 |
| Qiming | oh, no | 01:56 |
| Qiming | you will get the links very difficult to manage | 01:57 |
| xuhaiwei | if we don't store it into vm node, when deleting a vm, we can't judge whether it has some containers running on it | 01:57 |
| Qiming | we may need that link later on, not sure yet, but managing a dual link is not trivial task | 01:57 |
| Qiming | if a VM node is depended by a container node, we can check the vm_node.dependents property and reject the deletion request with 409 | 01:58 |
| Qiming | the 'dependency' we are modeling is more than just "container hosted on a VM" | 01:59 |
| Qiming | it could be used later for "web vm needs the db vm for being operational" | 01:59 |
| xuhaiwei | ok, I think I got your idea | 02:00 |
| Qiming | cool | 02:00 |
| Qiming | just trying to avoid blocking our own path moving forward, ;) | 02:01 |
| *** elynn has joined #senlin | 02:04 | |
| *** elynn has quit IRC | 02:10 | |
| *** elynn has joined #senlin | 02:11 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Fix receiver create https://review.openstack.org/365404 | 02:44 |
| openstackgerrit | Nguyen Phuong An proposed openstack/senlin: [api-ref] Correcting parameter's types https://review.openstack.org/365824 | 02:49 |
| *** yuanying has joined #senlin | 02:52 | |
| *** shu-mutou-AFK is now known as shu-mutou | 02:55 | |
| *** yuanying has quit IRC | 02:55 | |
| *** yuanying has joined #senlin | 02:57 | |
| *** yuanying has quit IRC | 03:00 | |
| *** yuanying has joined #senlin | 03:00 | |
| *** yuanying has quit IRC | 03:18 | |
| *** yuanying has joined #senlin | 03:20 | |
| openstackgerrit | Merged openstack/senlin: cluster-check results the status error https://review.openstack.org/365199 | 03:26 |
| openstackgerrit | xiaozhuangqing proposed openstack/senlin: Don't import unused logging https://review.openstack.org/365830 | 03:27 |
| yanyanhu | hi, Qiming around? | 03:39 |
| Qiming | yes | 03:40 |
| yanyanhu | currently in senlinclient, the shell client will print '-' if value is None, However, for openstackclient, it will print None if value is None | 03:41 |
| yanyanhu | so there is inconsistency here | 03:41 |
| yanyanhu | which one you suggest to apply here? | 03:41 |
| Qiming | follow osc's convention | 03:42 |
| yanyanhu | I see | 03:42 |
| Qiming | though I personally prefer '-' | 03:42 |
| yanyanhu | will change local shell to make them consistent | 03:42 |
| yanyanhu | :) | 03:42 |
| Qiming | thx | 03:42 |
| yanyanhu | my pleasure | 03:42 |
| yanyanhu | hi, Qiming, recalled this patch https://review.openstack.org/#/c/340108/ | 03:47 |
| yanyanhu | actually its purpose also makes sense | 03:47 |
| Qiming | okay, then keep it, :) | 03:48 |
| Qiming | just be consistent would be okay | 03:48 |
| yanyanhu | yes, maybe we just keep it and see what will happen in openstackclient in future | 03:48 |
| yanyanhu | whether they will apply the same change for similar reason | 03:48 |
| openstackgerrit | Yanyan Hu proposed openstack/python-senlinclient: Fix receiver list https://review.openstack.org/365843 | 03:55 |
| *** yuanying has quit IRC | 04:00 | |
| *** elynn has quit IRC | 04:01 | |
| *** yuanying has joined #senlin | 04:02 | |
| *** shu-mutou is now known as shu-mutou-AFK | 04:12 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Store node relationship into 'dependents' https://review.openstack.org/364846 | 04:32 |
| openstackgerrit | xu-haiwei proposed openstack/python-senlinclient: Show 'dependents' property in node details https://review.openstack.org/365853 | 04:37 |
| *** elynn has joined #senlin | 05:05 | |
| *** elynn has quit IRC | 05:10 | |
| *** elynn has joined #senlin | 05:11 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Fix receiver create https://review.openstack.org/365404 | 06:04 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Build trust for receiver creation https://review.openstack.org/365463 | 06:04 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Rework rally test jobs https://review.openstack.org/365878 | 06:17 |
| openstackgerrit | Merged openstack/senlin: [api-ref] Correcting parameter's types https://review.openstack.org/365824 | 06:32 |
| openstackgerrit | Merged openstack/senlin: Don't import unused logging https://review.openstack.org/365830 | 06:32 |
| openstackgerrit | Merged openstack/senlin: Store node relationship into 'dependents' https://review.openstack.org/364846 | 06:32 |
| openstackgerrit | Merged openstack/senlin: Modify parameters' type https://review.openstack.org/365535 | 06:35 |
| openstackgerrit | Merged openstack/senlin: Misspelling in the comment of cluster and node https://review.openstack.org/365515 | 06:38 |
| *** elynn has quit IRC | 06:47 | |
| *** elynn has joined #senlin | 06:52 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Handle exceptions of docker driver https://review.openstack.org/365890 | 06:56 |
| *** elynn has quit IRC | 06:57 | |
| *** elynn has joined #senlin | 06:57 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Build trust for receiver creation https://review.openstack.org/365463 | 07:06 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Rework rally test jobs https://review.openstack.org/365878 | 07:09 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Guard against driver error for nova validation https://review.openstack.org/365912 | 07:36 |
| openstackgerrit | Nguyen Phuong An proposed openstack/senlin: [api-ref] Remove parameters unused https://review.openstack.org/365927 | 08:13 |
| openstackgerrit | Merged openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/364506 | 08:44 |
| openstackgerrit | Merged openstack/python-senlinclient: senlin profile-create has no argument -t https://review.openstack.org/364195 | 08:44 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Guard against driver error for nova validation https://review.openstack.org/365912 | 08:45 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/senlin: Updated from global requirements https://review.openstack.org/365989 | 09:22 |
| openstackgerrit | Merged openstack/senlin: [api-ref] Remove parameters unused https://review.openstack.org/365927 | 09:40 |
| openstackgerrit | Merged openstack/senlin: Updated from global requirements https://review.openstack.org/365989 | 09:50 |
| *** zhurong has quit IRC | 09:58 | |
| *** yanyanhu has quit IRC | 10:01 | |
| openstackgerrit | Merged openstack/python-senlinclient: Fix receiver list https://review.openstack.org/365843 | 10:04 |
| *** elynn has quit IRC | 10:15 | |
| *** zhurong has joined #senlin | 12:10 | |
| *** lulei has quit IRC | 12:27 | |
| *** yanyanhu has joined #senlin | 12:38 | |
| *** elynn has joined #senlin | 12:52 | |
| *** zzxwill has joined #senlin | 13:00 | |
| Qiming | #startmeeting senlin | 13:00 |
| openstack | Meeting started Tue Sep 6 13:00:44 2016 UTC and is due to finish in 60 minutes. The chair is Qiming. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
| openstack | The meeting name has been set to 'senlin' | 13:00 |
| Qiming | #endmeeting | 13:00 |
| openstack | Meeting ended Tue Sep 6 13:00:58 2016 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:01 |
| Qiming | wrong channel | 13:01 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/senlin/2016/senlin.2016-09-06-13.00.html | 13:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/senlin/2016/senlin.2016-09-06-13.00.txt | 13:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/senlin/2016/senlin.2016-09-06-13.00.log.html | 13:01 |
| *** zhurong has quit IRC | 13:26 | |
| openstackgerrit | Merged openstack/senlin: Handle exceptions of docker driver https://review.openstack.org/365890 | 13:27 |
| *** yanyanhu has quit IRC | 13:51 | |
| *** zzxwill has quit IRC | 14:08 | |
| *** elynn has quit IRC | 14:25 | |
| *** zigo has quit IRC | 14:25 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Guard against driver error for nova validation https://review.openstack.org/365912 | 14:27 |
| *** zzxwill has joined #senlin | 14:28 | |
| *** zzxwill has quit IRC | 14:30 | |
| *** zzxwill has joined #senlin | 14:31 | |
| *** zigo has joined #senlin | 14:32 | |
| *** zigo is now known as Guest39147 | 14:33 | |
| *** elynn has joined #senlin | 14:35 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Guard against driver error for nova validation https://review.openstack.org/365912 | 14:37 |
| *** Guest39147 has quit IRC | 14:39 | |
| *** zigo_ has joined #senlin | 14:42 | |
| *** zigo_ has quit IRC | 14:47 | |
| *** zigo_ has joined #senlin | 14:48 | |
| *** Drago has joined #senlin | 15:05 | |
| *** Drago has quit IRC | 15:05 | |
| *** Drago has joined #senlin | 15:05 | |
| *** zigo_ is now known as zigo | 15:20 | |
| *** zzxwill has quit IRC | 15:21 | |
| *** zzxwill has joined #senlin | 15:22 | |
| *** zzxwill has quit IRC | 15:24 | |
| *** zzxwill has joined #senlin | 15:25 | |
| *** zzxwill has quit IRC | 15:28 | |
| *** zzxwill has joined #senlin | 15:36 | |
| *** zzxwill has quit IRC | 15:51 | |
| *** zzxwill has joined #senlin | 15:51 | |
| *** Drago has quit IRC | 15:51 | |
| *** zzxwill has quit IRC | 16:06 | |
| *** elynn has quit IRC | 16:08 | |
| *** Drago has joined #senlin | 16:08 | |
| *** Drago has quit IRC | 17:00 | |
| *** Drago1 has joined #senlin | 17:00 | |
| *** Drago1 has quit IRC | 17:00 | |
| *** Drago has joined #senlin | 17:01 | |
| *** Drago has quit IRC | 17:01 | |
| *** Drago has joined #senlin | 17:01 | |
| *** Drago has quit IRC | 22:34 | |
| *** zzxwill has joined #senlin | 23:52 | |
| *** zzxwill has quit IRC | 23:52 | |
| *** zzxwill has joined #senlin | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!