| *** Qiming has quit IRC | 00:13 | |
| *** lvdongbing has joined #senlin | 00:20 | |
| *** shu-mutou has joined #senlin | 01:03 | |
| *** pratikmallya has joined #senlin | 01:33 | |
| *** lixinhui has joined #senlin | 01:45 | |
| *** pratikmallya has quit IRC | 02:03 | |
| *** Qiming has joined #senlin | 02:04 | |
| Qiming | morning | 02:12 |
|---|---|---|
| *** Yanyanhu has joined #senlin | 02:21 | |
| *** elynn has joined #senlin | 02:24 | |
| *** elynn has quit IRC | 02:24 | |
| *** elynn_ has joined #senlin | 02:24 | |
| Qiming | xuhaiwei, https://review.openstack.org/#/c/256014/ | 02:25 |
| Qiming | seems the failure is not related to the patch | 02:25 |
| elynn_ | moring | 02:30 |
| elynn_ | Hi Qiming, got time to review https://review.openstack.org/#/c/247429/ ? | 02:30 |
| Qiming | hi, elynn_ | 02:32 |
| Qiming | looking | 02:33 |
| Qiming | a little confused why we have desired_capacity as attribute | 02:33 |
| Qiming | but min_size and max_size are not attributes | 02:34 |
| elynn_ | if we do a scale-out/in in senlin command | 02:34 |
| elynn_ | then desired_capacity will change | 02:34 |
| Qiming | when we do a resize operation, we also allow a change to min_size and max_size | 02:34 |
| elynn_ | it represent the current_size | 02:35 |
| elynn_ | hmm, haven't thought of that, you are right. | 02:35 |
| Qiming | also cluster status is incomplete | 02:35 |
| elynn_ | incomplete? | 02:36 |
| Qiming | yes, at least we should add UPDATING | 02:36 |
| elynn_ | I should define all the status here? | 02:36 |
| elynn_ | Oh, I got your point, I plan to add it in handle_update patch, but since you mention it , will add it in next patchset. | 02:37 |
| Qiming | WARNING is also a status to be exposed | 02:37 |
| elynn_ | I don't use warning, stll need to define it here? | 02:37 |
| Qiming | warning means a cluster is operational, but its status is not stable, something was wrong | 02:38 |
| Qiming | ERROR means a cluster is not operational, needs some human intervention | 02:38 |
| Qiming | better add these definitions there although we are still trying to figure out how/when to label a cluster as 'WARNING' status | 02:39 |
| Qiming | e.g. 10% nodes are down, should this status be treated as 'WARNING' | 02:39 |
| Qiming | we are adding a health threshold to a cluster | 02:39 |
| elynn_ | L149 check the status != CREATING will raise a resourceunknowerror. | 02:40 |
| Qiming | just want to raise an alarm | 02:40 |
| elynn_ | if cluster in warning, what kind of error should I raise? | 02:40 |
| Qiming | no error | 02:40 |
| Qiming | it is a warning only | 02:41 |
| elynn_ | so warning means create success? | 02:41 |
| Qiming | yes | 02:41 |
| *** Tiancheng has joined #senlin | 02:41 | |
| Qiming | we don't handle it | 02:41 |
| Qiming | we will allow a cluster to stay at WARNING status | 02:41 |
| Qiming | leave the decisions to users | 02:41 |
| elynn_ | ok, then I will adjust my patch for that. | 02:42 |
| Qiming | thx | 02:42 |
| *** Tiancheng_ has joined #senlin | 02:42 | |
| *** Tiancheng has quit IRC | 02:42 | |
| elynn_ | Is there any case that when we create a cluster, then the cluster status will directly go into WARNING? | 02:43 |
| Qiming | currently, no | 02:43 |
| Qiming | we are setting a cluster to WARNING when update or delete | 02:45 |
| elynn_ | When we doing update, then cluster change its status to warning, we still mark it as update_success? | 02:46 |
| *** pratikmallya has joined #senlin | 02:47 | |
| Qiming | em .... that is a good question | 02:47 |
| Qiming | in heat we only have SUCCESS, FAILED, IN_PROGRESS ... | 02:48 |
| elynn_ | Yes... | 02:48 |
| *** Tiancheng_ has quit IRC | 02:48 | |
| Qiming | this is a bad design | 02:48 |
| Qiming | you can never guarantee that something as a whole can be completed | 02:48 |
| Qiming | there will always be partial success | 02:48 |
| Qiming | it is one of the dilemma in heat, sometimes you cannot proceed to complete the failed operation, and you cannot delete the stack | 02:49 |
| Qiming | my suggestion is that we treat WARNING as a success | 02:52 |
| *** elynn_ has quit IRC | 02:52 | |
| *** elynn_ has joined #senlin | 02:53 | |
| elynn_ | Am I offline again? | 02:53 |
| elynn_ | ok, will follow your suggestion to treat warning as success for now. | 02:54 |
| Qiming | yes | 02:58 |
| Qiming | the definition of WARNING is a partial SUCCESS | 02:59 |
| openstackgerrit | Liuqing Jing proposed openstack/senlin-dashboard: [Test] Don't merge https://review.openstack.org/256187 | 03:04 |
| *** Yanyan has joined #senlin | 03:07 | |
| *** Yanyanhu has quit IRC | 03:07 | |
| *** elynn_ has quit IRC | 03:09 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix SDK test case https://review.openstack.org/256191 | 03:11 |
| Qiming | Yanyan, xuhaiwei, please review ^ | 03:12 |
| Qiming | we have test case broken at gate | 03:12 |
| *** elynn_ has joined #senlin | 03:13 | |
| Yanyan | Qiming, ok | 03:17 |
| *** Tiancheng has joined #senlin | 03:19 | |
| openstackgerrit | Merged openstack/senlin: Fix SDK test case https://review.openstack.org/256191 | 03:38 |
| openstackgerrit | Liuqing Jing proposed openstack/senlin-dashboard: [Test] Don't merge https://review.openstack.org/256187 | 03:48 |
| openstackgerrit | Merged openstack/senlin: Fix action status setting logic https://review.openstack.org/255925 | 04:02 |
| openstackgerrit | Merged openstack/senlin: Fix typo https://review.openstack.org/256014 | 04:03 |
| *** elynn_ has quit IRC | 04:06 | |
| openstackgerrit | Liuqing Jing proposed openstack/senlin-dashboard: [Test] Don't merge https://review.openstack.org/256187 | 04:07 |
| *** pratikmallya has quit IRC | 04:07 | |
| openstackgerrit | Liuqing Jing proposed openstack/senlin-dashboard: [Test] Don't merge https://review.openstack.org/256187 | 04:28 |
| openstackgerrit | Liuqing Jing proposed openstack/senlin-dashboard: Fix broken unit test aganist the latest horizon https://review.openstack.org/256187 | 04:42 |
| *** elynn_ has joined #senlin | 05:06 | |
| elynn_ | Hi Qiming | 05:06 |
| *** elynn__ has joined #senlin | 05:10 | |
| *** elynn_ has quit IRC | 05:10 | |
| openstackgerrit | Liuqing Jing proposed openstack/senlin-dashboard: Fix broken unit test aganist the latest horizon https://review.openstack.org/256187 | 05:37 |
| Qiming | hi | 06:10 |
| Qiming | elynn__, I was in meeting (ring) | 06:10 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix MutableList implementation https://review.openstack.org/255267 | 06:20 |
| *** zhenguo has joined #senlin | 06:24 | |
| *** Qiming has quit IRC | 06:44 | |
| *** pratikmallya has joined #senlin | 07:25 | |
| *** elynn__ has quit IRC | 07:29 | |
| openstackgerrit | junxu proposed openstack/senlin: Add block_device_mapping_v2 support for 'os.nova.server' profile. https://review.openstack.org/255755 | 07:51 |
| openstackgerrit | junxu proposed openstack/senlin: Add block_device_mapping_v2 support for 'os.nova.server' profile. https://review.openstack.org/255755 | 08:04 |
| *** pratikmallya has quit IRC | 08:31 | |
| openstackgerrit | Liuqing Jing proposed openstack/senlin-dashboard: Use page_title attr in Cluster/Profile/Polices/Nodes IndexView https://review.openstack.org/256286 | 08:50 |
| *** branw has joined #senlin | 09:14 | |
| *** gongysh_ has quit IRC | 09:30 | |
| branw | i have a node-create error, it says keystoneauth.session 500 request, don't know how to fix it. http://paste.openstack.org/show/481595/ | 09:33 |
| Yanyan | hi, branw, could you please help to report a bug for it. Will look at it later. Thanks. | 09:43 |
| Yanyan | I guess there are still some problems need to fix after openstacksdk applies keystoneauth for authentication | 09:44 |
| *** Yanyan has quit IRC | 09:50 | |
| *** lvdongbing has quit IRC | 09:51 | |
| branw | okay, thanks | 09:59 |
| *** zhenguo has quit IRC | 10:01 | |
| *** branw has quit IRC | 10:05 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Remove oslosphinx https://review.openstack.org/256360 | 11:22 |
| openstackgerrit | Merged openstack/senlin: Fix MutableList implementation https://review.openstack.org/255267 | 11:26 |
| *** lvdongbing has joined #senlin | 11:34 | |
| *** Tiancheng has quit IRC | 11:42 | |
| *** lixinhui has quit IRC | 11:50 | |
| *** lvdongbing has quit IRC | 12:24 | |
| *** lvdongbing has joined #senlin | 12:51 | |
| openstackgerrit | Merged openstack/senlin-dashboard: Fix broken unit test aganist the latest horizon https://review.openstack.org/256187 | 13:23 |
| openstackgerrit | Merged openstack/senlin-dashboard: Use page_title attr in Cluster/Profile/Polices/Nodes IndexView https://review.openstack.org/256286 | 13:24 |
| openstackgerrit | Merged openstack/python-senlinclient: Fix pep8 error about D300 https://review.openstack.org/255046 | 13:39 |
| openstackgerrit | xu-haiwei proposed openstack/python-senlinclient: Make cluster-policy-update help message clear https://review.openstack.org/255177 | 13:55 |
| openstackgerrit | Merged openstack/python-senlinclient: Make cluster-policy-update help message clear https://review.openstack.org/255177 | 14:00 |
| openstackgerrit | Liuqing Jing proposed openstack/senlin-dashboard: Update profile detail page and use breadcrumb https://review.openstack.org/256432 | 14:28 |
| openstackgerrit | Liuqing Jing proposed openstack/senlin-dashboard: Update node detail page and use breadcrumb https://review.openstack.org/256442 | 14:43 |
| openstackgerrit | Liuqing Jing proposed openstack/python-senlinclient: `Print` here is more reasonable than `Prints` https://review.openstack.org/256451 | 14:59 |
| openstackgerrit | Liuqing Jing proposed openstack/senlin-dashboard: Use more correct column name https://review.openstack.org/256456 | 15:04 |
| *** Liuqing has joined #senlin | 15:05 | |
| *** lvdongbing has quit IRC | 15:28 | |
| *** Liuqing has quit IRC | 15:42 | |
| *** pratikmallya has joined #senlin | 16:16 | |
| *** pratikma_ has joined #senlin | 16:18 | |
| openstackgerrit | Andreas Jaeger proposed openstack/senlin: Fix "report a bug" launchpad project https://review.openstack.org/256565 | 16:20 |
| *** pratikmallya has quit IRC | 16:21 | |
| *** pratikma_ has quit IRC | 19:00 | |
| *** yuanying has quit IRC | 19:03 | |
| *** pratikmallya has joined #senlin | 19:25 | |
| openstackgerrit | Ondřej Nový proposed openstack/python-senlinclient: Deprecated tox -downloadcache option removed https://review.openstack.org/256714 | 20:26 |
| *** pratikmallya has quit IRC | 20:46 | |
| *** pratikmallya has joined #senlin | 22:10 | |
| *** pratikma_ has joined #senlin | 22:13 | |
| *** pratikmallya has quit IRC | 22:16 | |
| *** pratikmallya has joined #senlin | 22:28 | |
| *** pratikma_ has quit IRC | 22:28 | |
| *** openstackgerrit has quit IRC | 22:37 | |
| *** openstackgerrit has joined #senlin | 22:38 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!