| openstackgerrit | Qiming Teng proposed openstack/senlin: Add missing options registration for webhook https://review.openstack.org/273586 | 00:03 |
|---|---|---|
| *** bdrich has quit IRC | 00:57 | |
| *** bdrich has joined #senlin | 01:09 | |
| *** xuhaiwei has quit IRC | 01:19 | |
| *** Qiming has quit IRC | 01:24 | |
| *** Qiming has joined #senlin | 01:31 | |
| *** Qiming has quit IRC | 01:38 | |
| *** Yanyanhu has joined #senlin | 01:45 | |
| *** elynn has joined #senlin | 02:00 | |
| elynn | Morning | 02:09 |
| *** xuhaiwei has joined #senlin | 02:11 | |
| *** Qiming has joined #senlin | 02:15 | |
| *** bdrich has quit IRC | 02:24 | |
| Yanyanhu | morning | 02:27 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Support property update for os.heat.stack profile https://review.openstack.org/273385 | 02:31 |
| xuhaiwei | wa! http://osdir.com/ml/openstack-dev/2016-01/msg01986.html | 02:34 |
| openstackgerrit | Merged openstack/senlin: Fix Error in Base Recover https://review.openstack.org/273535 | 02:37 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Support property update for os.heat.stack profile https://review.openstack.org/273385 | 02:54 |
| openstackgerrit | Merged openstack/senlin: Trivial: Make parse_exception cover all cases https://review.openstack.org/273554 | 03:09 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Use server_metadata_delete to update meatadata value in do_leave https://review.openstack.org/273410 | 03:16 |
| Yanyanhu | xuhaiwei, yep, this is a great work from zigo :) | 03:33 |
| Qiming | maybe too early to tell | 03:34 |
| Qiming | https://review.openstack.org/273586 | 03:34 |
| Qiming | just got complaints that receivers could not be created | 03:34 |
| Qiming | because the webhook.host option is invalid | 03:34 |
| zigo | Yanyanhu: xuhaiwei: Please do test the package and let me know if there's any isue! | 03:34 |
| zigo | issue | 03:34 |
| Qiming | so the patch above has to be merged to make things work | 03:34 |
| Qiming | we got things messed up so ... it is not a problem of packaging | 03:35 |
| Yanyanhu | yes | 03:36 |
| Yanyanhu | will spend time on receiver test in coming week | 03:36 |
| Qiming | so this bug is not fixed https://bugs.launchpad.net/senlin/+bug/1531074 | 03:36 |
| openstack | Launchpad bug 1531074 in senlin "Functional test for receiver is missing" [Medium,Triaged] - Assigned to Yanyan Hu (yanyanhu) | 03:36 |
| Yanyanhu | yes, should work on this earlier... | 03:36 |
| *** Tiancheng has joined #senlin | 03:37 | |
| Yanyanhu | so this means we need to back port this fix? | 03:37 |
| Yanyanhu | 273586 | 03:37 |
| Qiming | it is not a stable branch | 03:37 |
| Qiming | so it is okay I think | 03:37 |
| Yanyanhu | I see | 03:37 |
| Yanyanhu | hope can finish the functional test of receiver next week | 03:38 |
| Yanyanhu | at least a basic wone | 03:38 |
| Yanyanhu | s/wone/one | 03:38 |
| Qiming | it is pretty straightfoward one than the webhook interface | 03:38 |
| *** yuanying_ has joined #senlin | 03:38 | |
| Qiming | should be just one hour's work or so | 03:39 |
| Qiming | let me check if we still have the webhook test around | 03:39 |
| Yanyanhu | ok, will work on it | 03:39 |
| *** yuanying has quit IRC | 03:40 | |
| zigo | Is there some tempest tests for Senlin yet? | 03:43 |
| Qiming | zigo, not yet | 03:43 |
| Qiming | we touched that issue during our mid-cycle meetup early this month | 03:44 |
| Qiming | we will work on that soon | 03:44 |
| openstackgerrit | Cindia-blue proposed openstack/senlin: Add Node Check and Recover into API https://review.openstack.org/273866 | 03:47 |
| Qiming | lixinhui, you need to rebase your patch on the master | 03:53 |
| Yanyanhu | can't report bug in launchpad, always met timeout error | 03:58 |
| Yanyanhu | did you guys meet the same issue? | 03:58 |
| Qiming | no | 03:58 |
| Yanyanhu | let me try again | 03:58 |
| *** Tiancheng_ has joined #senlin | 04:00 | |
| *** Tiancheng has quit IRC | 04:00 | |
| *** yuanying has joined #senlin | 04:06 | |
| *** yuanying_ has quit IRC | 04:06 | |
| *** yuanying has quit IRC | 04:11 | |
| xuhaiwei | thanks zigo, just be back | 04:11 |
| zigo | xuhaiwei: Will you test the package? :) | 04:12 |
| xuhaiwei | I am not sure how to test it, zigo | 04:12 |
| *** elynn has quit IRC | 04:12 | |
| xuhaiwei | install it and try to use it? | 04:13 |
| zigo | xuhaiwei: apt-get install, see if it works? :) | 04:13 |
| zigo | Yup. | 04:13 |
| xuhaiwei | ok | 04:13 |
| zigo | Maybe, on a devstack setup, bring senlin daemons down, and use the packaged version? (note: this *will* install many python system packages which may disrupt your current devstack setup, so don't complain if your devstack is fucked after that...) | 04:14 |
| xuhaiwei | the package name is Senlin? | 04:15 |
| *** yuanying has joined #senlin | 04:17 | |
| *** Ayush has joined #senlin | 04:19 | |
| *** lixinhui_ has joined #senlin | 04:20 | |
| *** elynn has joined #senlin | 04:33 | |
| *** elynn has quit IRC | 04:38 | |
| *** elynn has joined #senlin | 04:38 | |
| xuhaiwei | zigo, can't find the package, is the package name Senlin or senlin? | 04:41 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Use server_metadata_delete to update meatadata value in do_leave https://review.openstack.org/273410 | 04:42 |
| *** Ayush has quit IRC | 04:47 | |
| *** Tiancheng_ has quit IRC | 04:48 | |
| *** Tiancheng_ has joined #senlin | 04:49 | |
| *** Ayush has joined #senlin | 04:56 | |
| *** elynn has quit IRC | 05:00 | |
| *** Ayush_ has joined #senlin | 05:00 | |
| *** Ayush_ has left #senlin | 05:01 | |
| *** Ayush_ has joined #senlin | 05:04 | |
| *** Ayush has left #senlin | 05:04 | |
| *** Ayush has joined #senlin | 05:05 | |
| *** Ayush_ has quit IRC | 05:05 | |
| xuhaiwei | hi, I want to have a discussion about cluster's nodes, as far as I know, in current design a cluster can have nodes which are created from different profile(not different profile type), that is ok in most cases, but if a cluster is attached by a LB policy, there will be conflicts, lb policy desires nodes all in the same network, but in fact may not(different pofile may define different networks) | 05:27 |
| zigo | xuhaiwei: There's never upper case in package names in Debian or Ubuntu. | 05:28 |
| xuhaiwei | so should we block adding nodes with different profiles into one cluster | 05:28 |
| zigo | xuhaiwei: There's python-senlin, senlin-common, senlin-api and senlin-engine. | 05:28 |
| xuhaiwei | I should install them one by one? | 05:29 |
| Qiming | xuhaiwei, I'd like to keep this open to users | 05:35 |
| Qiming | users will know what they are doing | 05:35 |
| xuhaiwei | ok | 05:36 |
| Qiming | for example, you may have one heat template containing a nova server, another template containing a cinder volume | 05:36 |
| Qiming | it is fine | 05:36 |
| Qiming | the reason behind this is about flexibility, some use cases users (or we) can work out | 05:38 |
| xuhaiwei | yes, reasonable | 05:38 |
| Qiming | for example, you may want to create a cluster of "compute" | 05:38 |
| Qiming | but you want to intelligently scale your cluster by adding a container or a VM or a baremetal | 05:38 |
| zigo | xuhaiwei: They depend on each other, so just apt-get install senlin-api and senlin-engine, python-senlin and senlin-common will be automatically pulled. | 05:40 |
| zigo | xuhaiwei: Do you use Debian Sid, Debian Jessie, or Ubuntu Trusty (worst choice... :P) ? | 05:41 |
| xuhaiwei | thanks zigo | 05:41 |
| xuhaiwei | the worst one | 05:41 |
| zigo | LOL | 05:41 |
| zigo | I'm not sure if Canonical will even make a Senlin pakage by themselves, probably they will just sync with an old version in Debian! :) | 05:42 |
| zigo | deb http://mitaka-trusty.pkgs.mirantis.com/debian trusty-mitaka-backports main | 05:42 |
| zigo | deb-src http://mitaka-trusty.pkgs.mirantis.com/debian trusty-mitaka-backports main | 05:42 |
| zigo | deb http://mitaka-trusty.pkgs.mirantis.com/debian trusty-mitaka-backports-nochange main | 05:42 |
| zigo | deb-src http://mitaka-trusty.pkgs.mirantis.com/debian trusty-mitaka-backports-nochange main | 05:42 |
| zigo | I'm currenlty doing the validation of the packages (ie: installing all, then checking tempest runs correctly). | 05:43 |
| zigo | So far, there's (to my point of view, minor) issues with Neutron, but I'm on my way to have it fixed. | 05:43 |
| zigo | xuhaiwei: Do you work at 99Cloud? | 05:44 |
| xuhaiwei | no, zigo | 05:45 |
| xuhaiwei | I think we met once in Tokyo summit | 05:45 |
| zigo | xuhaiwei: huawei, sorry, I'm being stupid ! :) | 05:45 |
| xuhaiwei | never mind | 05:45 |
| zigo | xuhaiwei: In Shenzhen ? | 05:46 |
| xuhaiwei | I am in Tokyo | 05:46 |
| zigo | Ok. | 05:46 |
| xuhaiwei | I am from NEC | 05:46 |
| zigo | Ok. | 05:46 |
| zigo | I was pleased to see Senlin seem mostly made in Asia. | 05:47 |
| zigo | xuhaiwei: Your name sounds Chinese though. | 05:47 |
| Yanyanhu | zigo, he is chinese :) | 05:47 |
| Yanyanhu | just worked in Japan | 05:47 |
| zigo | OK. | 05:47 |
| xuhaiwei | yes, are you from Germany? | 05:47 |
| zigo | FYI, I'm in Zhengzhou, Henan, until after Chinese new year, at which point I'll go to Shanghai until begining of March. | 05:47 |
| zigo | Both wife and kids are Chinese (kids born in Shanghai and Zhengzhou respectively), though I'm French. | 05:48 |
| zigo | We now live in Grenoble, next to Mirantis office there. | 05:48 |
| zigo | (Grenoble is in the French Alpes) | 05:48 |
| zigo | It's been a year we moved back, though I stayed 8 years in China, so I feel like half Chinese myself. :) | 05:49 |
| xuhaiwei | interesting | 05:49 |
| Yanyanhu | zigo, cool. You have been very familiar with China I believe :) | 05:49 |
| Qiming | it's a clever choice, marrying a foreigner makes it difficult to have disputes at home | 05:50 |
| Qiming | you speak different languages | 05:50 |
| zigo | LOL | 05:50 |
| zigo | I can speak Chinese and know most of the swear words of my wife! :) | 05:50 |
| zigo | (and same for her with French) | 05:50 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Re-enable lock breaker https://review.openstack.org/262151 | 05:50 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Trival fix to service report https://review.openstack.org/273409 | 05:50 |
| Qiming | pitty you, :) | 05:50 |
| zigo | Though argues have to be in English, so it's fair enough for everyone... :D | 05:50 |
| Qiming | next time, you can try argue in Japanese or some other language | 05:51 |
| Yanyanhu | sometimes, you have to learn new language if you want to argue with your wife | 05:51 |
| Yanyanhu | efficiently | 05:51 |
| zigo | Exatly ! :) | 05:52 |
| Qiming | yeah, senlin team is helpful in many ways, you see? | 05:52 |
| xuhaiwei | I know a girl whose husband is Japanese, his husband said your Japanese is good only when quarrals with me | 05:52 |
| Qiming | LOL | 05:53 |
| Yanyanhu | :P | 05:53 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Fix check_size_params method https://review.openstack.org/273894 | 06:02 |
| Qiming | Yanyanhu, posted comments to your patch | 06:10 |
| Yanyanhu | ok | 06:10 |
| Qiming | the revised logic is not correct, please think through | 06:10 |
| Yanyanhu | ok, let me think about it | 06:11 |
| openstackgerrit | EldonZhao proposed openstack/python-senlinclient: Fix cluster_update_policy parameter passing error https://review.openstack.org/273897 | 06:15 |
| *** gongysh has joined #senlin | 06:18 | |
| Qiming | Yanyanhu, when we are checking size params, strict should only mean what it was designed to be | 06:20 |
| Yanyanhu | yes | 06:20 |
| Qiming | it is only about checking the request against the current attributes | 06:21 |
| Qiming | so we need to go through all the tests and see if strict should play a role in that test | 06:21 |
| Yanyanhu | right | 06:22 |
| Qiming | it is not about whether the checking as a whole should be 'strict' or not, :) | 06:22 |
| Yanyanhu | so the answer to the 4th question should be yes | 06:23 |
| Yanyanhu | understand | 06:23 |
| Yanyanhu | sorry, 3rd | 06:23 |
| Qiming | it is about sanity of the provided parameters | 06:23 |
| Yanyanhu | I see | 06:23 |
| Yanyanhu | sanity check is not related to 'strict' param | 06:23 |
| Qiming | exactly | 06:24 |
| Qiming | we will return an error no matter 'strict' or not | 06:24 |
| Yanyanhu | got it | 06:24 |
| openstackgerrit | EldonZhao proposed openstack/python-senlinclient: Fix cluster_update_policy parameter passing error https://review.openstack.org/273897 | 06:25 |
| openstackgerrit | Qiming Teng proposed openstack/python-senlinclient: Fix cluster_update_policy parameter passing error https://review.openstack.org/273897 | 06:29 |
| openstackgerrit | EldonZhao proposed openstack/python-senlinclient: Fix cluster_update_policy parameter passing error https://review.openstack.org/273897 | 06:40 |
| openstackgerrit | Merged openstack/senlin: Add missing options registration for webhook https://review.openstack.org/273586 | 06:52 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Fix check_size_params method https://review.openstack.org/273894 | 06:59 |
| *** elynn has joined #senlin | 07:12 | |
| *** lixinhui_ has quit IRC | 07:17 | |
| xuhaiwei | Qiming, how is the Senlin summit proposal? it seems next Monday is the deadline | 07:30 |
| Qiming | busy on code review atm | 07:32 |
| Qiming | elynn, checked your sdk patch, please review | 07:33 |
| Qiming | Since we have a very different way of handling DELETE requests, we should do it in our own way anyhow. | 07:33 |
| Qiming | I wasn't aware that the Resource.delete method is not copying the 'location' header out of the response | 07:34 |
| Qiming | let's do our own hack. | 07:34 |
| elynn | Qiming, | 07:34 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Fix check_size_params method https://review.openstack.org/273894 | 07:34 |
| elynn | you mean back to patchiest? | 07:35 |
| elynn | patchset 1? | 07:35 |
| Qiming | not exactly | 07:35 |
| elynn | I think return {'action': 'action_id'} is enough | 07:35 |
| elynn | what do you think about it? | 07:35 |
| Qiming | yes | 07:35 |
| Qiming | return that action dict is the only non-disruptive change | 07:36 |
| elynn | or directly return location ? | 07:36 |
| Qiming | em ... I think returning a dict is more generic | 07:37 |
| Qiming | in that dict, we have {'location': 'pointer to location'}, right? | 07:37 |
| elynn | yes | 07:38 |
| Qiming | in other cases, say a cluster action or a node action (being added), we are returning {'action': 'action_id'} | 07:38 |
| *** Liuqing has joined #senlin | 07:38 | |
| Qiming | while at the same time, we are also returing a {'location': 'pointer to location'} header | 07:38 |
| elynn | yes | 07:39 |
| Qiming | so .. a unified way from client's perspective would be a dict | 07:39 |
| elynn | so a dict containing 'action' and 'location' ? | 07:39 |
| Qiming | We do have some resistence from SDK about handling response headers | 07:40 |
| Qiming | hopefully, we can unify that | 07:40 |
| Qiming | the current situation is that there are consistencies | 07:40 |
| Qiming | the only way to unify, as I see it, is to return just {'location': 'pointer'} | 07:41 |
| elynn | ok, got it. | 07:41 |
| Qiming | but I'm not encouraging to inflate this patch to handle all return values | 07:41 |
| Qiming | let's make delete work as expected, i.e. get DELETE handling there | 07:42 |
| Qiming | then we recheck the cluster-action, node-action, cluster-update ... hopefully, migrate all them to the same format | 07:42 |
| Qiming | I'm a little bit afraid that you will evolve the current patch to a 'Trivial' one that covers all cases, ;) | 07:43 |
| elynn | ok, so in this patch, I will only handle delete :) | 07:43 |
| Qiming | great | 07:43 |
| elynn | then let's how to handle the rest of them | 07:44 |
| elynn | I have to leave for a while, will update this patch later. | 07:44 |
| Qiming | okay | 07:44 |
| *** elynn has quit IRC | 07:46 | |
| Qiming | Yanyanhu, about the two questions | 07:51 |
| Yanyanhu | yes? | 07:51 |
| Qiming | raising min_size and lowering max_size | 07:51 |
| Qiming | are we able to handle those requests already? | 07:51 |
| Qiming | I just read your comments | 07:51 |
| Yanyanhu | yes, they have been coverred in current implementation | 07:51 |
| Yanyanhu | of cluster_resize | 07:52 |
| Qiming | okay, cool | 07:52 |
| Qiming | I cannot remember things clearly | 07:52 |
| Yanyanhu | actually, me neither... I checked it again | 07:52 |
| Qiming | and ... if the request comes from users, 'strict' has default to True ... | 07:54 |
| Yanyanhu | yes | 07:54 |
| Qiming | okay, I'm fine with it now | 07:54 |
| Yanyanhu | ok, so users can do that kind of adjusting if they explicitly specify strict to False | 07:55 |
| Qiming | yes, let's kick it in | 07:57 |
| Qiming | xuhaiwei, I think I'm aware of two talk proposals | 07:59 |
| xuhaiwei | yes | 08:00 |
| Qiming | one is from Ethan, it is about an overall introduction of Senlin | 08:00 |
| xuhaiwei | not propose it yet? | 08:00 |
| Qiming | another is from Xinhui, it is about a comprehensive usage scenario -- creating a cluster with LB, autoscaling, auto-healing easily | 08:01 |
| Qiming | got notification from the first proposal | 08:01 |
| xuhaiwei | there will be a demo? | 08:01 |
| Qiming | not yet notified by the second one | 08:01 |
| xuhaiwei | the notification is from IBM? | 08:02 |
| Qiming | no, from community | 08:02 |
| xuhaiwei | ok, who will be the speaker of it? | 08:03 |
| Qiming | I don't have the complete list | 08:04 |
| Qiming | topic wise, I believe both are good candidates | 08:05 |
| xuhaiwei | yes | 08:05 |
| Qiming | since there are still two days at least open for proposal | 08:05 |
| Qiming | if you have any ideas, we can discuss | 08:05 |
| xuhaiwei | there is one proposal from our team, which will introduce Senlin briefly | 08:06 |
| Qiming | cool | 08:07 |
| xuhaiwei | hope it will be passed | 08:07 |
| Qiming | blessing | 08:07 |
| Qiming | it would be very very interesting if we have something to show related to container clusters | 08:08 |
| xuhaiwei | yuanying will have some discussion with Magnum guys about 'container as a service', it is worked, that may be one way for Senlin to do 'container cluster' | 08:16 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Add missing unit tests for nova driver https://review.openstack.org/273939 | 08:21 |
| Qiming | cool, better with something working | 08:21 |
| xuhaiwei | just reviewed Yanyanhu's patch | 08:22 |
| xuhaiwei | we will force to create/delete nodes when 'strict' is False? | 08:23 |
| xuhaiwei | that is a little confusing | 08:23 |
| Yanyanhu | hi, xuhaiwei, yes | 08:23 |
| Yanyanhu | if user specifies strict to False, we will try to adjust the size of cluster to match the new given size limitation | 08:24 |
| Yanyanhu | the new given upper/lower size limit | 08:24 |
| xuhaiwei | yes, but 'strict=False' usually mean this option is not taking effect | 08:25 |
| xuhaiwei | from user's point, it's confusing for me | 08:25 |
| Yanyanhu | ah, we need a clarrification here :) | 08:25 |
| Yanyanhu | strict means whether the parameter size check will be strictly done | 08:26 |
| openstackgerrit | Merged openstack/senlin: Fix check_size_params method https://review.openstack.org/273894 | 08:26 |
| Yanyanhu | it's different from 'enforced' :) | 08:26 |
| xuhaiwei | I misunderstood it | 08:27 |
| Yanyanhu | really a little confusing :) | 08:28 |
| Qiming | confusing ? | 08:29 |
| Qiming | we in cluster_resize function, we have a very detailed docstring there | 08:29 |
| Yanyanhu | the client help msg has a explanation as well :) | 08:29 |
| xuhaiwei | ok, I often go with feel | 08:30 |
| Qiming | just curious, has anyone tried enabling multi-engine support and see if everything works? | 08:41 |
| Yanyanhu | yes, it is always enabled in my local env | 08:42 |
| Qiming | \o/ | 08:42 |
| Qiming | don't change your setting then, ;) | 08:42 |
| Yanyanhu | my engine number is 2 :) | 08:42 |
| Yanyanhu | sure | 08:42 |
| *** Ayush has quit IRC | 08:45 | |
| *** Ayush has joined #senlin | 08:48 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Add unit tests for node check/recover https://review.openstack.org/273947 | 08:49 |
| *** Ayush has quit IRC | 08:52 | |
| *** Ayush has joined #senlin | 08:52 | |
| *** Ayush has quit IRC | 09:03 | |
| *** Ayush has joined #senlin | 09:04 | |
| openstackgerrit | Merged openstack/senlin: Use server_metadata_delete to update meatadata value in do_leave https://review.openstack.org/273410 | 09:04 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Add missing unit tests for cluster module https://review.openstack.org/273953 | 09:08 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Fix cluster name update error https://review.openstack.org/273959 | 09:22 |
| *** lixinhui_ has joined #senlin | 09:25 | |
| *** Liuqing has quit IRC | 09:34 | |
| Yanyanhu | hi, Qiming, around? | 09:40 |
| Yanyanhu | just reported a bug about size param check in cluster_scale_in/out service routines https://bugs.launchpad.net/senlin/+bug/1539478 | 09:40 |
| openstack | Launchpad bug 1539478 in senlin "Incorrect param size check in cluster_scale_in/out service routines" [Undecided,New] - Assigned to Yanyan Hu (yanyanhu) | 09:41 |
| Yanyanhu | but not sure which kind of fix is better | 09:41 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Fix cluster name update error https://review.openstack.org/273959 | 09:41 |
| Yanyanhu | one possible solution is removing size parameter check in cluster_scale_in/out in service layer as I described here https://bugs.launchpad.net/senlin/+bug/1539478 | 09:42 |
| Qiming | Yanyanhu, an easy fix is to make the strict parameter default to False | 09:42 |
| openstack | Launchpad bug 1539478 in senlin "Incorrect param size check in cluster_scale_in/out service routines" [Undecided,New] - Assigned to Yanyan Hu (yanyanhu) | 09:42 |
| Yanyanhu | Qiming, ok | 09:42 |
| Yanyanhu | it has the same effect | 09:42 |
| Qiming | the purpose of this fix is about desired_capacity | 09:43 |
| Yanyanhu | I see | 09:43 |
| Qiming | we should reject requests early and even set desired_capacity to a correct number no matter the following actions succeed or not | 09:43 |
| Qiming | this is a prerequisite to that end | 09:43 |
| Qiming | when I tried to fix this, I found there is a redundant, new checking procedure in engine | 09:44 |
| Qiming | that is where I started this effort | 09:44 |
| Yanyanhu | this makes sense | 09:45 |
| xuhaiwei | don't remember it clearly, during the meetup, have we got an agreement about deprecate nested cluster support? Or just leave it there | 09:48 |
| openstackgerrit | Cindia-blue proposed openstack/senlin: Add Node Check and Recover into API https://review.openstack.org/273866 | 09:49 |
| Qiming | xuhaiwei, hide it if possible | 09:52 |
| Qiming | it would cause confusion | 09:52 |
| xuhaiwei | you mean delete it from client side? | 09:53 |
| Qiming | from client, api, even engine | 09:53 |
| Qiming | leave it in db | 09:53 |
| Qiming | create an impression that we never thought of it, :) | 09:53 |
| xuhaiwei | ok, I will do it | 09:53 |
| Qiming | thanks, that would be very helpful | 09:54 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Add Node Check and Recover into API https://review.openstack.org/273866 | 10:03 |
| *** Qiming has quit IRC | 10:08 | |
| *** gongysh has quit IRC | 10:12 | |
| *** gongysh has joined #senlin | 10:13 | |
| *** shu-mutou has joined #senlin | 10:17 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Set strict param to False by default for check_size_params https://review.openstack.org/273987 | 10:31 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Fix a bug in cluster_resize service routine https://review.openstack.org/273988 | 10:31 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Fix functional test https://review.openstack.org/273989 | 10:31 |
| *** lixinhui_ has quit IRC | 10:36 | |
| openstackgerrit | xu-haiwei proposed openstack/python-senlinclient: Remove nested cluster support https://review.openstack.org/273994 | 10:37 |
| *** Tiancheng_ has quit IRC | 10:40 | |
| xuhaiwei | still there Yanyanhu? | 10:41 |
| *** Yanyanhu has quit IRC | 10:41 | |
| *** Tiancheng has joined #senlin | 10:43 | |
| *** Liuqing has joined #senlin | 10:48 | |
| *** Tiancheng has quit IRC | 10:49 | |
| *** Tiancheng has joined #senlin | 10:58 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Remove nested cluster support https://review.openstack.org/274005 | 10:58 |
| *** Tiancheng has quit IRC | 11:03 | |
| *** Qiming has joined #senlin | 11:13 | |
| *** Liuqing has quit IRC | 11:15 | |
| *** lixinhui_ has joined #senlin | 11:39 | |
| openstackgerrit | EldonZhao proposed openstack/python-senlinclient: Fix cluster_update_policy parameter passing error https://review.openstack.org/273897 | 11:49 |
| openstackgerrit | Merged openstack/senlin: Set strict param to False by default for check_size_params https://review.openstack.org/273987 | 11:51 |
| openstackgerrit | Merged openstack/senlin: Add Node Check and Recover into API https://review.openstack.org/273866 | 12:17 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Trival fix to service report https://review.openstack.org/273409 | 12:17 |
| openstackgerrit | Merged openstack/senlin: Support property update for os.heat.stack profile https://review.openstack.org/273385 | 12:21 |
| *** dixiaoli has quit IRC | 12:33 | |
| openstackgerrit | Merged openstack/senlin: Fix a bug in cluster_resize service routine https://review.openstack.org/273988 | 12:35 |
| openstackgerrit | Shu Muto proposed openstack/senlin-dashboard: Add receivers panel https://review.openstack.org/269528 | 12:43 |
| openstackgerrit | Shu Muto proposed openstack/senlin-dashboard: Update URLs to Django 1.8 style https://review.openstack.org/274049 | 12:43 |
| *** shu-mutou has quit IRC | 12:58 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Relax checking on server metadata https://review.openstack.org/274065 | 13:32 |
| openstackgerrit | Merged openstack/senlin: Trival fix to service report https://review.openstack.org/273409 | 13:32 |
| openstackgerrit | Merged openstack/senlin: Fix functional test https://review.openstack.org/273989 | 13:33 |
| openstackgerrit | Merged openstack/senlin: Fix cluster name update error https://review.openstack.org/273959 | 13:34 |
| openstackgerrit | Merged openstack/senlin: Add missing unit tests for cluster module https://review.openstack.org/273953 | 13:36 |
| openstackgerrit | Merged openstack/senlin: Add unit tests for node check/recover https://review.openstack.org/273947 | 13:36 |
| openstackgerrit | Merged openstack/senlin: Add missing unit tests for nova driver https://review.openstack.org/273939 | 13:36 |
| openstackgerrit | Merged openstack/senlin: Add unit test for cluster_resize in engine https://review.openstack.org/273403 | 13:37 |
| openstackgerrit | Merged openstack/senlin: One more test case for deletion policy https://review.openstack.org/273523 | 13:37 |
| openstackgerrit | Merged openstack/senlin: Fix service_get call in keystone driver https://review.openstack.org/273560 | 13:38 |
| openstackgerrit | Merged openstack/senlin: Remove useless function from db model https://review.openstack.org/273569 | 13:38 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Re-enable lock breaker https://review.openstack.org/262151 | 13:39 |
| *** elynn has joined #senlin | 13:41 | |
| *** elynn has quit IRC | 13:47 | |
| *** Ayush has quit IRC | 13:53 | |
| openstackgerrit | Merged openstack/senlin: Re-enable lock breaker https://review.openstack.org/262151 | 13:55 |
| openstackgerrit | Merged openstack/senlin: Relax checking on server metadata https://review.openstack.org/274065 | 13:56 |
| *** lixinhui_ has quit IRC | 14:21 | |
| *** Qiming has quit IRC | 15:06 | |
| *** bdrich has joined #senlin | 15:17 | |
| *** pratikmallya has joined #senlin | 16:19 | |
| *** pratikma_ has joined #senlin | 16:35 | |
| *** pratikmallya has quit IRC | 16:37 | |
| *** pratikma_ has quit IRC | 16:58 | |
| *** pratikmallya has joined #senlin | 17:05 | |
| *** pratikmallya has quit IRC | 18:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/senlin: Updated from global requirements https://review.openstack.org/273545 | 20:50 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/senlin-dashboard: Updated from global requirements https://review.openstack.org/268539 | 20:50 |
| *** Qiming has joined #senlin | 22:22 | |
| *** bdrich has quit IRC | 22:26 | |
| *** Qiming has quit IRC | 22:26 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!