| *** Qiming has quit IRC | 00:34 | |
| *** zzxwill has quit IRC | 00:50 | |
| *** zzxwill has joined #senlin | 00:55 | |
| *** Qiming has joined #senlin | 01:23 | |
| *** dixiaoli_ has joined #senlin | 01:35 | |
| *** Yanyanhu has joined #senlin | 01:47 | |
| Yanyanhu | hi, Qiming, just noticed that brian workflow + 1 at the patch about path_args to make it as workaround | 02:11 |
|---|---|---|
| Yanyanhu | but seems the -2 from Terry still blocks the gerrit to merge it | 02:11 |
| Qiming | yep | 02:12 |
| Yanyanhu | so we should talk with Terry to remove the -2 to make it work? | 02:12 |
| Qiming | then ping terry | 02:12 |
| Qiming | -2 is now sticky | 02:12 |
| Yanyanhu | ok | 02:12 |
| Yanyanhu | what his irc name in sdk channel | 02:12 |
| Qiming | unless the reviewer removes it | 02:12 |
| Qiming | terrylhowe ? | 02:12 |
| Yanyanhu | just two people whose name starts with letter 't' | 02:12 |
| Yanyanhu | ok, looks like he is offline | 02:13 |
| Yanyanhu | BTW, I saw brian's patch for refactoring resource | 02:14 |
| Yanyanhu | it's really huge... | 02:14 |
| Yanyanhu | not sure whether he plan to land it in Mitaka release | 02:14 |
| Qiming | he is offline | 02:23 |
| Qiming | no | 02:24 |
| Qiming | we won't | 02:24 |
| Qiming | we will lock the dependency to 0.8.1 | 02:24 |
| Qiming | not further changes on requirements | 02:24 |
| Yanyanhu | ok | 02:24 |
| Qiming | when RC1 is out, we create a branch for the 1.0 release | 02:24 |
| Yanyanhu | if so, maybe we should start to rework our driver | 02:24 |
| Yanyanhu | maybe not merge it | 02:25 |
| Qiming | yep | 02:25 |
| Qiming | that would be great | 02:25 |
| Yanyanhu | will start this job this week | 02:25 |
| Qiming | I'm more concerned with the client side | 02:25 |
| Qiming | because we are finalizing senlinclient this week | 02:25 |
| Yanyanhu | right, that's a big deal as well | 02:25 |
| Qiming | the schedue is we will do a final release for senlinclient this week | 02:26 |
| Yanyanhu | so the client release has to be done by the end of this week> | 02:26 |
| Yanyanhu | I see | 02:26 |
| Qiming | http://releases.openstack.org/mitaka/schedule.html | 02:26 |
| Yanyanhu | no much time left | 02:26 |
| Yanyanhu | hi, Qiming, I'm now testing the cluster update API. But I found 'location' field is not included in resp body. Just 'cluster' field is there. | 02:29 |
| Yanyanhu | so is location handled in special way thus it is not included in response content? | 02:30 |
| Yanyanhu | let me check it manually | 02:31 |
| Yanyanhu | it is really not included in resp body... using debug mode to perform cluster-update operation | 02:32 |
| Yanyanhu | I see. I need to revise the request handling to get it from resp.location | 02:33 |
| Yanyanhu | not in resp.content | 02:33 |
| Qiming | should be part of resource.location | 02:35 |
| Yanyanhu | yes, I misunderstood it | 02:35 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: (WIP)Revise DRS policy to be a generic affinity policy https://review.openstack.org/286461 | 02:57 |
| *** zzxwill has quit IRC | 03:30 | |
| *** yuanying has quit IRC | 03:44 | |
| *** shu-mutou-AFK is now known as shu-mutou | 03:53 | |
| *** dixiaoli_ has quit IRC | 03:53 | |
| *** yuanying has joined #senlin | 04:07 | |
| *** zzxwill has joined #senlin | 04:50 | |
| openstackgerrit | Merged openstack/senlin: Add devstack support for senlin dashboard https://review.openstack.org/282138 | 05:01 |
| openstackgerrit | Merged openstack/senlin: Optimize the registry claim function https://review.openstack.org/286043 | 05:03 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add functional test for node get details https://review.openstack.org/286417 | 05:27 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add functional test for cluster update https://review.openstack.org/286987 | 05:27 |
| openstackgerrit | Merged openstack/senlin: Add functional test for node get details https://review.openstack.org/286417 | 05:39 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Revise the logic of cluster/profile metadata update https://review.openstack.org/286998 | 05:45 |
| Qiming | Yanyanhu, lixinhui, .. | 05:48 |
| Yanyanhu | hi | 05:48 |
| Qiming | could you please help compile a list of patches we need from sdk? | 05:49 |
| Qiming | we need to push sdk to version 0.8.1 | 05:49 |
| Yanyanhu | you mean the patches in sdk we need them to be merged? | 05:49 |
| Qiming | then we need work hard to adapt both senlinclient and senlin | 05:49 |
| Qiming | yes | 05:49 |
| Qiming | still any patches hanging? | 05:49 |
| Yanyanhu | ok, will try to look through the patch list | 05:49 |
| Qiming | thanks | 05:50 |
| Qiming | we need to freeze senlinclient this week | 05:50 |
| Yanyanhu | no problem | 05:50 |
| Yanyanhu | ok | 05:50 |
| zzxwill | Is there a way to pull a fix pack into my local environment for code-review and test which has NOT been merged? | 05:52 |
| Qiming | git review -d ? | 05:55 |
| zzxwill | I'd like to run Yanyan's fix in my local, but I find it hard to merge his fix. | 05:55 |
| zzxwill | I will try, thanks:) | 05:55 |
| Yanyanhu | you can also find the link from gerrit | 05:55 |
| Yanyanhu | under the download catalog | 05:56 |
| Yanyanhu | the first one is checkout | 05:56 |
| zzxwill | Not got it. Thank you:) | 05:57 |
| *** dixiaoli has quit IRC | 05:58 | |
| Yanyanhu | you're welcome :) | 06:05 |
| Yanyanhu | hi, Qiming, the following is the patch list we may rely on | 06:05 |
| Yanyanhu | at sdk side | 06:05 |
| Yanyanhu | http://paste.openstack.org/show/488902/ | 06:05 |
| Yanyanhu | only three ones I think | 06:05 |
| Yanyanhu | all others have been merged | 06:06 |
| *** zzxwill has quit IRC | 06:08 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: (WIP)Revise DRS policy to be a generic affinity policy https://review.openstack.org/286461 | 06:08 |
| Qiming | https://review.openstack.org/286131 is still not good | 06:09 |
| Yanyanhu | yes | 06:10 |
| *** dixiaoli has joined #senlin | 06:14 | |
| *** zzxwill has joined #senlin | 06:26 | |
| Qiming | dixiaoli, hi | 06:27 |
| lixinhui | Qiming | 06:29 |
| lixinhui | I am just back | 06:29 |
| lixinhui | different things these days | 06:29 |
| lixinhui | https://review.openstack.org/286131 | 06:30 |
| lixinhui | The patch about hypervisor | 06:30 |
| lixinhui | I have forward it to Bran to commit into new patch based on the comments there | 06:30 |
| lixinhui | who will commmit in very soon | 06:30 |
| Qiming | great | 06:31 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/senlin-dashboard: Imported Translations from Zanata https://review.openstack.org/286397 | 06:41 |
| openstackgerrit | Cindia-blue proposed openstack/senlin: Revise DB API to Ensure Delete Success https://review.openstack.org/286575 | 06:44 |
| zzxwill | FYI. Just in case you core guys missed this message. (The message occurred to me after I saw the language pack from Zanata.) | 06:47 |
| zzxwill | I especially need liaisons in below projects, which are in Mitaka translation plan: | 06:48 |
| zzxwill | nova, glance, keystone, cinder, swift, neutron, heat, horizon, ceilometer. | 06:48 |
| zzxwill | I also need liaisons from Horizon plugin projects, which are ready in translation website: | 06:48 |
| zzxwill | trove-dashboard, sahara-dashboard,designate-dasbhard, magnum-ui, | 06:48 |
| zzxwill | monasca-ui, murano-dashboard and senlin-dashboard. | 06:48 |
| zzxwill | I need liaisons tell us whether they are ready for translation from project view. | 06:48 |
| zzxwill | From Daisy. (The message above is in a bad style, sorry) | 06:49 |
| zzxwill | **In all, a liaison for senlin i18n is required, and the role should be a core member.** | 06:50 |
| zzxwill | For detailed info, please refer to https://etherpad.openstack.org/p/zzxwill | 06:51 |
| Qiming | zzxwill, I called for a liaison for this among senlin cores | 06:51 |
| Qiming | no one has volunteered | 06:51 |
| Qiming | so it is now on me | 06:52 |
| Qiming | I will work with Daisy on this | 06:52 |
| Qiming | the rough plan is to follow the community's practice | 06:52 |
| zzxwill | Sure. Just a kind reminder:) | 06:52 |
| Qiming | which means we will need to do soft string freeze this week | 06:52 |
| Qiming | and we will do hard string freeze later | 06:53 |
| Qiming | after rc1, no string modification will be merged on that branch | 06:53 |
| Qiming | Yanyanhu, I have got a better idea to solve the db issue | 07:13 |
| Qiming | will work on it today, after the loooooooong meeting | 07:13 |
| Qiming | lixinhui, how to reproduce the bug you are solving in patch #286131 ? | 07:14 |
| lixinhui | Just attach the health policy then detach it after a while, maybe 5 mins | 07:15 |
| Qiming | okay, I'm more inclined to make sure registry_create is wrapped into a db session | 07:18 |
| Qiming | and see if that solves the problem | 07:18 |
| lixinhui | okay, I will try it | 07:20 |
| openstackgerrit | Merged openstack/senlin-dashboard: Use scheduled deletion msg while deleting node/cluster https://review.openstack.org/282598 | 08:24 |
| openstackgerrit | Merged openstack/senlin-dashboard: Delete useless return statement https://review.openstack.org/286645 | 08:25 |
| lixinhui | Qiming, you are genius | 08:40 |
| lixinhui | Your suggestion works well | 08:40 |
| lixinhui | I will resubmit the fix patch for your review | 08:41 |
| Qiming | lixinhui, patch https://review.openstack.org/#/c/287029 still needs revision | 08:42 |
| Qiming | bran should be a little careful before committing the changes | 08:42 |
| lixinhui | I am just discussed with him | 08:45 |
| lixinhui | delete 'am' | 08:45 |
| openstackgerrit | Cindia-blue proposed openstack/senlin: Revise DB API to Ensure Delete Success https://review.openstack.org/286575 | 08:50 |
| lixinhui | Qiming, Yanyanhu, and all | 09:01 |
| lixinhui | I have finished designed tests to health_manager functions including long run | 09:01 |
| Qiming | long run | 09:02 |
| lixinhui | It would be very nice if all of you can help test more and review all the fix patches | 09:02 |
| lixinhui | start the health_manager periodic_task for one night | 09:02 |
| lixinhui | trigger check action every 60 | 09:03 |
| lixinhui | Just now, I also trigger deattach policy operation long after attach, the bug 1552093 does not jump out any more | 09:04 |
| openstack | bug 1552093 in senlin "Registry Deletion can not succeed every time" [Undecided,New] https://launchpad.net/bugs/1552093 | 09:04 |
| lixinhui | s/deattach/detach | 09:05 |
| *** zzxwill has quit IRC | 09:15 | |
| Yanyanhu | hi, lixinhui, so you tested it using a cluster of nova server? | 09:18 |
| Yanyanhu | with lbaasv2 service | 09:18 |
| lixinhui | Yes, Yanyanhu. I tested it with nova, without LBAAS triggered. I am testing it now on another machine | 09:21 |
| Yanyanhu | ok | 09:22 |
| Yanyanhu | I'm wondering whether we can test it using fake driver in functional test :) | 09:22 |
| lixinhui | Good idea | 09:22 |
| lixinhui | I can try | 09:22 |
| Yanyanhu | that might be able to increase the test efficiency :) | 09:22 |
| lixinhui | :) | 09:23 |
| openstackgerrit | Cindia-blue proposed openstack/senlin: Revise DB API to Ensure Delete Success https://review.openstack.org/286575 | 09:29 |
| *** Qiming has quit IRC | 09:45 | |
| *** zzxwill has joined #senlin | 09:56 | |
| *** shu-mutou is now known as shu-mutou-AFK | 10:18 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add functional test for cluster update with new profile https://review.openstack.org/287091 | 10:33 |
| *** Yanyanhu has quit IRC | 10:39 | |
| *** Qiming has joined #senlin | 10:49 | |
| *** zzxwill has quit IRC | 11:28 | |
| *** lixinhui_ has joined #senlin | 11:34 | |
| *** Liuqing has joined #senlin | 12:27 | |
| *** Qiming has quit IRC | 12:59 | |
| *** lixinhui_ has quit IRC | 13:17 | |
| *** Liuqing has quit IRC | 14:44 | |
| *** zzxwill has joined #senlin | 15:20 | |
| *** haiwei_ has quit IRC | 15:31 | |
| *** zzxwill has quit IRC | 16:33 | |
| *** Qiming has joined #senlin | 23:05 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!