| *** zhurong has joined #senlin | 00:36 | |
| *** xuhaiwei has joined #senlin | 00:39 | |
| *** zzxwill has joined #senlin | 00:52 | |
| *** zzxwill has quit IRC | 01:08 | |
| *** zzxwill has joined #senlin | 01:11 | |
| *** zzxwill has quit IRC | 01:13 | |
| *** zzxwill has joined #senlin | 01:17 | |
| *** yanyanhu has joined #senlin | 01:27 | |
| *** elynn has joined #senlin | 01:30 | |
| openstackgerrit | Merged openstack/senlin: Updated from global requirements https://review.openstack.org/363875 | 01:38 |
|---|---|---|
| *** elynn_ has joined #senlin | 01:39 | |
| *** elynn has quit IRC | 01:40 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Validation for zone placement policy https://review.openstack.org/364023 | 01:57 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: scaleutils.py:nodes_by_age():filter un_normal nodes https://review.openstack.org/364024 | 01:57 |
| *** zzxwill has quit IRC | 02:12 | |
| openstackgerrit | Merged openstack/senlin: Add host_cluster support in container profile https://review.openstack.org/362712 | 02:18 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Validation for region placement policy https://review.openstack.org/364030 | 02:32 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Validation for zone placement policy https://review.openstack.org/364023 | 02:34 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: EResourceOperation code comments need modify https://review.openstack.org/364036 | 02:59 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: EResourceOperation code comments need modify https://review.openstack.org/364036 | 03:00 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: Enable node filtering to filter nodes not created. https://review.openstack.org/364024 | 03:06 |
| *** zzxwill has joined #senlin | 03:16 | |
| *** zzxwill has quit IRC | 03:49 | |
| openstackgerrit | Merged openstack/senlin: Validation for zone placement policy https://review.openstack.org/364023 | 03:54 |
| *** zzxwill has joined #senlin | 04:11 | |
| *** elynn_ has quit IRC | 04:37 | |
| openstackgerrit | Merged openstack/senlin: Enable node filtering to filter nodes not created. https://review.openstack.org/364024 | 04:39 |
| *** elynn_ has joined #senlin | 05:01 | |
| *** elynn_ has quit IRC | 05:05 | |
| *** elynn_ has joined #senlin | 05:08 | |
| openstackgerrit | Merged openstack/senlin: A spec for container cluster service https://review.openstack.org/281102 | 06:39 |
| openstackgerrit | zhufl proposed openstack/senlin: Remove unused LOG and CONF https://review.openstack.org/364094 | 06:52 |
| *** zzxwill has quit IRC | 06:57 | |
| *** zzxwill has joined #senlin | 07:02 | |
| openstackgerrit | Merged openstack/senlin: EResourceOperation code comments need modify https://review.openstack.org/364036 | 07:06 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Trim container id to a suitable size https://review.openstack.org/364114 | 07:30 |
| Qiming | xuhaiwei, there? | 07:43 |
| xuhaiwei | yes, Qiming | 07:44 |
| Qiming | about triming container id | 07:44 |
| xuhaiwei | yes | 07:44 |
| Qiming | can you help paste the return value from docker-py? | 07:44 |
| xuhaiwei | ok | 07:44 |
| xuhaiwei | ee4de842244f307e21ba1697b0bd001224628f98382f794af243db026cccc325 | 07:45 |
| xuhaiwei | this is the original id | 07:45 |
| xuhaiwei | but when shown in 'docker ps', it only shows ee4de842244f | 07:45 |
| xuhaiwei | it seems docker uses short-id in the output by default | 07:46 |
| xuhaiwei | similar with senlin | 07:46 |
| *** zzxwill has quit IRC | 07:46 | |
| Qiming | the physical id column is 36 chars | 07:47 |
| Qiming | maybe we should cut the first 36 chars instead of the first 12? | 07:47 |
| xuhaiwei | I am not sure it will work in deleting the container with 36 chars id | 07:49 |
| xuhaiwei | why should we keep so long physical_id, for heat and nova, it only shows 8 chars | 07:50 |
| *** zzxwill has joined #senlin | 07:55 | |
| xuhaiwei | Qiming, 36 chars also works in deleting container, should I change it to 36 chars? | 08:00 |
| Qiming | yes, 36 is at least better than 12 | 08:00 |
| xuhaiwei | ok | 08:01 |
| Qiming | or we will need to bump the db schema version again | 08:01 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Trim container id to a suitable size https://review.openstack.org/364114 | 08:03 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Fix container profile samples and README file https://review.openstack.org/364129 | 08:04 |
| openstackgerrit | lvdongbing proposed openstack/senlin: Use openstack client in profile examples readme file uniformly https://review.openstack.org/364143 | 08:32 |
| xuhaiwei | Qiming, hi, there is an issue to discuss | 08:36 |
| Qiming | sure | 08:36 |
| xuhaiwei | when deleting a nova or heat node with container running on it, should we delete it with the container together or forbide the deletion action? | 08:37 |
| xuhaiwei | or add a 'force' delete option? | 08:38 |
| openstackgerrit | Merged openstack/senlin: Validation for region placement policy https://review.openstack.org/364030 | 08:51 |
| Qiming | wow, that is a hard question ... | 08:55 |
| Qiming | should we make a decision for the users at all? | 08:56 |
| *** elynn_ has quit IRC | 08:57 | |
| xuhaiwei | to be safe, we can raise an exception to user to tell them containers are running, and if user insists on deleting them, we can provide a --force option | 08:57 |
| Qiming | the users should know well what each vm is used for ... | 08:58 |
| xuhaiwei | so delete them without warning? | 09:00 |
| Qiming | I think so | 09:00 |
| Qiming | we don't want to get into that business | 09:00 |
| Qiming | especially when we don't have dependencies among clusters | 09:01 |
| Qiming | we can introduce dependencies later and provide this kind of warning | 09:02 |
| Qiming | telling users that the hosting cluster is still being referenced ... | 09:02 |
| xuhaiwei | I am afraid in order to deleting vm nodes with deleting container nodes together, we have to create some dependacies, for examples add container node infos into node.data | 09:03 |
| Qiming | sounds like an abuse of the data field ... | 09:04 |
| Qiming | the dependency should be persisted into database IMO | 09:04 |
| xuhaiwei | so in metadata? | 09:04 |
| Qiming | maybe as a separate column | 09:04 |
| xuhaiwei | the column's name is a little difficult, because vm type node and contianer type node seems different | 09:06 |
| Qiming | could be generalized into a 'dependents' column | 09:07 |
| xuhaiwei | ok | 09:07 |
| Qiming | for the hosting cluster, it will record the container cluster ID | 09:07 |
| Qiming | when the container cluster is removed (completely), that field will get cleared | 09:07 |
| Qiming | then the hosting (VM) cluster can be deleted | 09:08 |
| xuhaiwei | ok, I will do this | 09:08 |
| Qiming | thx | 09:15 |
| Qiming | I'm about to cut a newton-3 release | 09:16 |
| Qiming | any patches you want to get in ? | 09:16 |
| Qiming | can use some helps on reviewing this one: https://review.openstack.org/#/c/363408/ | 09:17 |
| xuhaiwei | I think the two patches about containers can be added | 09:25 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Trim container id to a suitable size https://review.openstack.org/364114 | 09:28 |
| Qiming | okay | 09:28 |
| Qiming | approved | 09:30 |
| Qiming | will see what the gate say | 09:30 |
| openstackgerrit | Shan Guo proposed openstack/python-senlinclient: senlin profile-create has no argument -t https://review.openstack.org/364195 | 09:42 |
| *** yanyanhu has quit IRC | 09:52 | |
| openstackgerrit | Merged openstack/senlin: Remove unused LOG and CONF https://review.openstack.org/364094 | 09:55 |
| openstackgerrit | Merged openstack/senlin: Protect cluster from being deleted when appropriate https://review.openstack.org/363408 | 09:55 |
| *** zzxwill has quit IRC | 09:56 | |
| *** zzxwill has joined #senlin | 09:58 | |
| openstackgerrit | Merged openstack/senlin: Fix container profile samples and README file https://review.openstack.org/364129 | 09:59 |
| openstackgerrit | Merged openstack/senlin: Use openstack client in profile examples readme file uniformly https://review.openstack.org/364143 | 10:00 |
| *** zhurong has quit IRC | 10:03 | |
| openstackgerrit | Merged openstack/senlin: Trim container id to a suitable size https://review.openstack.org/364114 | 10:35 |
| *** Qiming has quit IRC | 10:40 | |
| *** chenfei has quit IRC | 10:40 | |
| *** Qiming has joined #senlin | 10:45 | |
| *** chenfei has joined #senlin | 10:46 | |
| *** zhurong has joined #senlin | 12:34 | |
| *** zzxwill has quit IRC | 13:05 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Some more release notes for newton-3 https://review.openstack.org/364299 | 13:11 |
| *** openstackgerrit has quit IRC | 13:49 | |
| *** openstackgerrit has joined #senlin | 13:50 | |
| *** ChanServ sets mode: +v openstackgerrit | 13:50 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Refactor affinity policy https://review.openstack.org/364328 | 13:54 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Refactor zone placement policy https://review.openstack.org/364329 | 13:54 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Refactor region placement policy https://review.openstack.org/364330 | 13:54 |
| *** Drago has joined #senlin | 14:39 | |
| *** Drago has quit IRC | 14:41 | |
| *** Drago has joined #senlin | 14:41 | |
| *** zhurong has quit IRC | 14:58 | |
| *** elynn has joined #senlin | 16:55 | |
| *** elynn has quit IRC | 17:19 | |
| *** openstackgerrit has quit IRC | 18:18 | |
| *** openstackgerrit has joined #senlin | 18:19 | |
| *** ChanServ sets mode: +v openstackgerrit | 18:19 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/364506 | 18:50 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/senlin: Updated from global requirements https://review.openstack.org/364509 | 18:51 |
| *** Drago has quit IRC | 22:16 | |
| openstackgerrit | Merged openstack/senlin: Updated from global requirements https://review.openstack.org/364509 | 23:44 |
| *** zhurong has joined #senlin | 23:48 | |
| *** zhurong has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!