| openstackgerrit | Merged openstack/senlin master: Validating adjustment properties for scaling policy https://review.openstack.org/434718 | 00:47 |
|---|---|---|
| *** catintheroof has quit IRC | 01:13 | |
| *** edisonxiang has quit IRC | 01:19 | |
| *** edisonxiang has joined #senlin | 01:19 | |
| *** guoshan has joined #senlin | 01:19 | |
| *** yanyanhu has joined #senlin | 01:28 | |
| XueFeng | hi, all | 01:39 |
| XueFeng | Morning | 01:39 |
| yanyanhu | hi, morning | 01:46 |
| yuanbin | hi, morining | 01:51 |
| ruijie | morning :) | 01:53 |
| openstackgerrit | Qiming Teng proposed openstack/senlin master: More release notes for the Ocata RC https://review.openstack.org/435199 | 02:25 |
| openstackgerrit | Merged openstack/senlin master: More release notes for the Ocata RC https://review.openstack.org/435199 | 02:51 |
| openstackgerrit | Qiming Teng proposed openstack/senlin master: Fix node get bug in docker profile https://review.openstack.org/429285 | 02:57 |
| chohoor | Senlin have a feature: Support to Suspend/Resume Operations. When suspend a cluster, which operation will be refused? Only auto-scale or all operations? | 03:29 |
| Qiming | chohoor, not yet decided | 03:30 |
| Qiming | the general idea is to suspend a cluster so no operation actions will be accepted | 03:31 |
| Qiming | it is still an open question | 03:31 |
| openstackgerrit | Qiming Teng proposed openstack/senlin master: Add tags to newly created heat stacks https://review.openstack.org/435207 | 03:33 |
| chohoor | alright,my company have a similar feature, when suspend a cluster, cluster can't be auto-scale but could dmanual-scale... | 03:35 |
| chohoor | s/dmanual/manual/ | 03:36 |
| XueFeng | Qiming, when I edit senlin wiki.I get a error "Incorrect or missing CAPTCHA." | 03:43 |
| *** guoshan has quit IRC | 04:06 | |
| wllabs | Qiming | 04:06 |
| wllabs | 我发现你们现在目前这个r 版很不稳定，有bug | 04:06 |
| wllabs | NOTIFICATION_PHASES = ( | 04:06 |
| wllabs | PHASE_START, PHASE_END, PHASE_ERROR, | 04:06 |
| wllabs | ) = ( | 04:06 |
| wllabs | 'start', 'end', 'error', | 04:06 |
| wllabs | ) | 04:06 |
| wllabs | 这里应该加个check | 04:06 |
| wllabs | EVENT.debug(self, 'check', reason) | 04:07 |
| wllabs | 不然这个里面加上 check 就会报错 | 04:07 |
| openstackgerrit | Aaron Ding proposed openstack/senlin master: Revise cluster get with admin user https://review.openstack.org/435223 | 04:09 |
| wllabs | Qiming | 04:13 |
| wllabs | 我发现的bug我能自己修吗 | 04:13 |
| wllabs | 我发现我昨天提的问题，你们就改了一行，就修了 | 04:13 |
| wllabs | 我也想做做开源 | 04:13 |
| wllabs | 而且昨天你们的处理方式，和我是一样的 | 04:14 |
| wllabs | https://review.openstack.org/434700 | 04:14 |
| yanyanhu | hi, wllabs, you're welcome to claim bugs reported by anyone, including yourself | 04:15 |
| yanyanhu | actually, people are encourage to claim the bug after they reported it | 04:15 |
| openstackgerrit | Merged openstack/senlin master: Fix node get bug in docker profile https://review.openstack.org/429285 | 04:23 |
| *** deepbook5broo has joined #senlin | 04:53 | |
| *** deepbook5broo has left #senlin | 04:53 | |
| *** guoshan has joined #senlin | 05:06 | |
| *** guoshan has quit IRC | 05:10 | |
| ruijie | hi wllabs | 05:21 |
| ruijie | http://paste.openstack.org/show/599328/ | 05:21 |
| ruijie | check this out | 05:21 |
| Qiming | XueFeng, you will need a proxy to edit senlin wiki | 05:25 |
| Qiming | that have been many spams reported | 05:25 |
| Qiming | so after your edit, you will be prompted for a captcha | 05:26 |
| openstackgerrit | Qiming Teng proposed openstack/senlin master: Enable heat stack event listener https://review.openstack.org/435243 | 05:26 |
| Qiming | the problem for Chinese developer is that the capcha website is blocked by the damn firewall | 05:26 |
| Qiming | wllabs, 'check' is not supposed to be one of the phases | 05:27 |
| wllabs | 但是代码里有 EVENT.debug(self, 'check', reason) 发送 check 啊 | 05:28 |
| wllabs | 代码里有 EVENT.debug(self, 'check', reason) 然后就报错了 | 05:28 |
| wllabs | Qiming | 05:38 |
| Qiming | alright, that is a bug | 05:38 |
| Qiming | it was introduced in fa78792e758fcc2ca56bb0b6ba16e3ceafe3d8ce | 05:40 |
| Qiming | let me try a quick fix | 05:40 |
| openstackgerrit | Qiming Teng proposed openstack/senlin master: Fixed inappropriate event generation in action base https://review.openstack.org/435249 | 05:45 |
| Qiming | wllabs, ^ | 05:46 |
| wllabs | 可以我来改吗？ | 05:47 |
| wllabs | 我想尝试向社区提下代码 | 05:47 |
| wllabs | 走出第一步 | 05:47 |
| Qiming | oh, it is done | 05:47 |
| Qiming | pls try something else | 05:47 |
| wllabs | 给新人一个机会嘛 | 05:48 |
| wllabs | 真是quick fix， | 05:48 |
| wllabs | 发现个一个bug好难的 | 05:48 |
| Qiming | we are releasing senlin ocata RC2 today, so I didn't leave that to others | 05:48 |
| wllabs | 而且这个bug我还追了好长时间 | 05:48 |
| wllabs | 还有昨天我提的那个也是 | 05:49 |
| Qiming | that means ... senlin is pretty stable, right? | 05:49 |
| wllabs | 以前稳定 | 05:49 |
| wllabs | 这几天，我发现这二个bug，感觉一点也不稳定 | 05:49 |
| Qiming | stability is always a relative assessment | 05:50 |
| Qiming | is linux kernel stable enough? | 05:50 |
| wllabs | Qiming 你们还招人吗？ | 05:50 |
| wllabs | 我可以去学习吗 | 05:50 |
| Qiming | several hundreds of bugs are still found and fixed during each cycle | 05:50 |
| Qiming | team is recruiting experts in deep learning | 05:51 |
| Qiming | yanyanhu, I believe we are pretty there to cut a release today | 05:52 |
| wllabs | 我想去学习 | 05:53 |
| wllabs | 我会openstack | 05:53 |
| Qiming | if we can get https://review.openstack.org/435243 in, we don't have any high priority bugs in the queue | 05:53 |
| Qiming | em ... openstack is just a tool | 05:54 |
| Qiming | as far as I know (I'm not a manager), we don't hire openstack developers | 05:54 |
| Qiming | we have desire to hire cloud researchers instead | 05:55 |
| wllabs | 谁要我 | 05:56 |
| wllabs | 我要工作 | 05:56 |
| Qiming | you may want to build a shining profile by producing some great things first | 05:57 |
| openstackgerrit | Merged openstack/senlin master: Add tags to newly created heat stacks https://review.openstack.org/435207 | 06:01 |
| *** guoshan has joined #senlin | 06:07 | |
| *** guoshan has quit IRC | 06:12 | |
| *** guoshan has joined #senlin | 06:19 | |
| *** yuanbin has quit IRC | 06:31 | |
| *** yuanbin has joined #senlin | 06:32 | |
| yanyanhu | Qiming, hi, sorry just noticed your message, I will make a cut based on this patch https://review.openstack.org/435207 | 06:32 |
| yanyanhu | or we include this as well? https://review.openstack.org/435249 | 06:32 |
| *** yuanbin has left #senlin | 06:34 | |
| *** yuanying has quit IRC | 06:37 | |
| ruijie | hi yanyanhu | 06:37 |
| yanyanhu | hi, ruijie | 06:37 |
| ruijie | is that better to proposal new patches after today or next week? | 06:38 |
| ruijie | I saw today is the deadline for rc3 | 06:38 |
| ruijie | final rc | 06:38 |
| yanyanhu | ruijie, you can keep proposing patch after rc2, but that will not be included in final release unless that is a critical fi | 06:38 |
| yanyanhu | fix | 06:39 |
| yanyanhu | since that required release team's help to backport it to final release | 06:39 |
| yanyanhu | so you may need to convince them | 06:39 |
| yanyanhu | so if that is an important, better finish it today :) | 06:39 |
| ruijie | sure, that will be fine :) | 06:40 |
| ruijie | thanks yanyanhu | 06:40 |
| yanyanhu | my pleasure, should I wait for your patch since I'm now writing the release patch | 06:40 |
| ruijie | no, I just want to make sure that, still thinking about the logic :) its not a bug | 06:41 |
| ruijie | Will try to propose it after final rc | 06:42 |
| yanyanhu | ok :) | 06:42 |
| *** yuanbin has joined #senlin | 06:47 | |
| XueFeng | hi,yanyan | 06:52 |
| yanyanhu | hi, XueFeng | 06:55 |
| wllabs | Is there any job opportunity here? | 06:58 |
| openstackgerrit | xurong00037997 proposed openstack/senlin master: Remove unused pylintrc https://review.openstack.org/435286 | 07:48 |
| Qiming | yanyanhu, I believe all current patches should be merged before release | 07:48 |
| ruijie | Hi Qiming | 07:53 |
| Qiming | hi | 07:53 |
| ruijie | I m investigating event notifications these days | 07:53 |
| ruijie | have some questions about it | 07:54 |
| ruijie | let me find the code | 07:54 |
| ruijie | https://review.openstack.org/gitweb?p=openstack/senlin.git;a=blob;f=senlin/engine/actions/base.py;h=6af18827a1bb9686922cb5c3a10affb2323fb093;hb=373cf4cf8aa0c75501af1cb35556fa034a5fda02#l472 | 07:54 |
| ruijie | The fisrt event is dumped here | 07:55 |
| Qiming | yes | 07:55 |
| Qiming | that is where the event starts | 07:55 |
| ruijie | sorry the network is unstable .. | 07:56 |
| ruijie | https://review.openstack.org/gitweb?p=openstack/senlin.git;a=blob;f=senlin/engine/actions/cluster_action.py;h=2a7481fb138842bb83d5c3e356755812f67c1e15;hb=373cf4cf8aa0c75501af1cb35556fa034a5fda02#l949 | 07:57 |
| ruijie | then we do policy_check | 07:57 |
| *** yuanying has joined #senlin | 07:57 | |
| yanyanhu | Qiming, you mean all opened patch? | 07:57 |
| Qiming | no, the health manager patch | 07:58 |
| Qiming | the event patch | 07:58 |
| Qiming | one is about closing the heat listener bug | 07:58 |
| yanyanhu | ok | 07:58 |
| Qiming | the other is obviously a critical bug for event generation | 07:58 |
| yanyanhu | will wait for those patches | 07:58 |
| Qiming | ruijie, yes | 07:59 |
| ruijie | then we will dump Event.error in policy_check() logic if the result is false | 07:59 |
| Qiming | what's your question, ruijie | 07:59 |
| Qiming | yes | 07:59 |
| ruijie | again in the execute() another Event.error will be dumped | 07:59 |
| ruijie | so there will be 3 events dumped here? | 07:59 |
| Qiming | where is the "another Event.error" ? | 08:00 |
| ruijie | https://review.openstack.org/gitweb?p=openstack/senlin.git;a=blob;f=senlin/engine/actions/cluster_action.py;h=2a7481fb138842bb83d5c3e356755812f67c1e15;hb=373cf4cf8aa0c75501af1cb35556fa034a5fda02#l952 | 08:01 |
| Qiming | 952 dumps the error and then RETURN | 08:02 |
| ruijie | line 949, inside policy_check() there's one | 08:02 |
| Qiming | 961 dump another error then RETURN | 08:02 |
| ruijie | here https://review.openstack.org/gitweb?p=openstack/senlin.git;a=blob;f=senlin/engine/actions/base.py;h=6af18827a1bb9686922cb5c3a10affb2323fb093;hb=373cf4cf8aa0c75501af1cb35556fa034a5fda02#l374 | 08:02 |
| Qiming | okay, I see | 08:03 |
| Qiming | the one dumped inside policy_check can be removed | 08:04 |
| Qiming | looks like a duplication | 08:04 |
| ruijie | agreed | 08:04 |
| ruijie | And another question is about the cluster obj we dumped | 08:04 |
| ruijie | we used action.entity to get the cluster object | 08:05 |
| Qiming | yes | 08:05 |
| Qiming | it is a cache | 08:05 |
| Qiming | have you found any inconsistency? | 08:06 |
| ruijie | https://review.openstack.org/gitweb?p=openstack/senlin.git;a=blob;f=senlin/events/message.py;h=352422b9ee02f1243dd7a72c650ed9f34db485c4;hb=373cf4cf8aa0c75501af1cb35556fa034a5fda02#l64 | 08:06 |
| ruijie | the problem is after finish executing the action | 08:06 |
| wllabs | Qiming | 08:06 |
| wllabs | https://review.openstack.org/#/c/434700/1/senlin/profiles/os/nova/server.py | 08:06 |
| ruijie | the status and status reason will be updated | 08:06 |
| wllabs | this fix has a problem | 08:06 |
| ruijie | however the entity is still the cached object | 08:06 |
| Qiming | em, yes | 08:07 |
| ruijie | http://paste.openstack.org/show/599341/ | 08:07 |
| Qiming | if you really care about the most recent status, you may have to refresh it | 08:07 |
| ruijie | so, actually, the message we dumped to mq contains an old cluster object | 08:07 |
| Qiming | no so old | 08:08 |
| wllabs | this commit don't pass the test | 08:08 |
| ruijie | Yes, I can do another API call to check the cluster obj | 08:08 |
| Qiming | it is retrieved from database at the begining of that particular action | 08:08 |
| Qiming | it is not very old, but I agree it may not reflect each and every field you have changed during the action execution | 08:09 |
| ruijie | yes Qiming, this is not a big problem | 08:09 |
| Qiming | surely you can patch the logic so that the cluster object is retrieved from database when generating events | 08:10 |
| wllabs | availability_zone attribute don't exists in the server | 08:10 |
| wllabs | Qiming 是不是 | 08:11 |
| ruijie | just thought, okay, we are using MQ to uncouple the service and senlin | 08:11 |
| wllabs | 我又发现你们这个bug, 这下可以我来修了吧 | 08:12 |
| ruijie | so we hope to make it as simple as possible :) | 08:12 |
| ruijie | wllabs, you are welcome to draft bug and propose patch :) we will review it and check it :) | 08:12 |
| wllabs | thank you | 08:13 |
| Qiming | ruijie, http://git.openstack.org/cgit/openstack/senlin/tree/senlin/events/message.py#n66 | 08:14 |
| Qiming | a possible workaround is to retrieve the cluster or node object in before calling cls._notify_xyz there | 08:15 |
| Qiming | I'm not sure about the overhead of an additional db transaction | 08:15 |
| Qiming | wllabs, what to you mean availability_zone attribute doesn't exist? | 08:17 |
| *** xuao has joined #senlin | 08:18 | |
| wllabs | https://review.openstack.org/#/c/434700/1/senlin/profiles/os/nova/server.py | 08:18 |
| Qiming | yes? | 08:18 |
| wllabs | 获取到的 server 会被 _update_zone_info 这个函数做为参数 | 08:19 |
| wllabs | 这个函数里是去更新zone的 | 08:19 |
| wllabs | 但是现在直接是 if server.availability_zone: | 08:19 |
| wllabs | server并没有这个属性 | 08:19 |
| wllabs | zone = getattr(server, 'OS-EXT-AZ:availability_zone', None) | 08:19 |
| wllabs | 而应该是这样获取zone | 08:19 |
| Qiming | it is set here: http://git.openstack.org/cgit/openstack/python-openstacksdk/tree/openstack/compute/v2/server.py#n95 | 08:20 |
| wllabs | heat里就是这样做的 | 08:20 |
| Qiming | who cares about how heat does things? | 08:20 |
| wllabs | 但是我昨天试了，如果不改，会报错 | 08:20 |
| Qiming | feel free to file a bug, with the trace log attached | 08:21 |
| wllabs | 好，我找找log | 08:22 |
| ruijie | a query may not be that heavy I guess? | 08:22 |
| Qiming | it won't be, ruijie, in such a case | 08:22 |
| ruijie | okay, Qiming. Last question :) | 08:22 |
| Qiming | but ... I'm more inclined to save db queries whenever possible | 08:22 |
| Qiming | consider managing a cluster of 1000 vms | 08:23 |
| Qiming | everything trivial may become a huge problem | 08:23 |
| ruijie | maybe we can update action.entity directly? | 08:23 |
| Qiming | :) | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base File "/usr/lib/python2.7/site-packages/senlin/engine/actions/base.py", line 478, in ActionProc | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base result, reason = action.execute() | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base File "/usr/lib/python2.7/site-packages/senlin/engine/actions/node_action.py", line 256, in execute | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base res, reason = self._execute() | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base File "/usr/lib/python2.7/site-packages/senlin/engine/actions/node_action.py", line 222, in _execute | 08:24 |
| Qiming | it is a hack | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base return method() | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base File "/usr/lib/python2.7/site-packages/osprofiler/profiler.py", line 117, in wrapper | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base return f(*args, **kwargs) | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base File "/usr/lib/python2.7/site-packages/senlin/engine/actions/node_action.py", line 67, in do_create | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base res = self.entity.do_create(self.context) | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base File "/usr/lib/python2.7/site-packages/senlin/engine/node.py", line 218, in do_create | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base physical_id = pb.Profile.create_object(context, self) | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base File "/usr/lib/python2.7/site-packages/osprofiler/profiler.py", line 117, in wrapper | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base return f(*args, **kwargs) | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base File "/usr/lib/python2.7/site-packages/senlin/profiles/base.py", line 223, in create_object | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base return profile.do_create(obj) | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base File "/usr/lib/python2.7/site-packages/senlin/profiles/os/nova/server.py", line 610, in do_create | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base self._update_zone_info(obj, server) | 08:24 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base File "/usr/lib/python2.7/site-packages/senlin/profiles/os/nova/server.py", line 518, in _update_zone_info | 08:25 |
| Qiming | wllabs, please avoid pasting traces in the IRC channel | 08:25 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base if server.availability_zone: | 08:25 |
| wllabs | 2017-02-15 18:04:22.697 17522 ERROR senlin.engine.actions.base AttributeError: 'Server' object has no attribute 'availability_zone' | 08:25 |
| ruijie | http://paste.openstack.org/ | 08:25 |
| ruijie | actually, wllabs, you can paste it here next time | 08:25 |
| wllabs | 我还以为你要看 | 08:25 |
| Qiming | it is like painting on a public wall, very annoying, wllabs | 08:25 |
| openstackgerrit | huangtianhua proposed openstack/senlin master: Correct a config option name https://review.openstack.org/435300 | 08:25 |
| *** XueFeng has quit IRC | 08:26 | |
| wllabs | i know | 08:26 |
| Qiming | you can paste it elsewhere, you can paste it into your bug report | 08:26 |
| Qiming | then tell use the link | 08:26 |
| wllabs | ok | 08:26 |
| wllabs | 新手，第一次做社区，不太懂。我会改的，谢谢各位的宽容 | 08:26 |
| ruijie | Qiming, the last question is about the "reason" field. | 08:27 |
| ruijie | https://review.openstack.org/gitweb?p=openstack/senlin.git;a=blob;f=senlin/engine/actions/cluster_action.py;h=2a7481fb138842bb83d5c3e356755812f67c1e15;hb=373cf4cf8aa0c75501af1cb35556fa034a5fda02#l961 | 08:27 |
| ruijie | I didn't find the code that we used the "reason" field to generate the event | 08:28 |
| Qiming | wllabs, looks very strange, the 'availability_zone' is defined as a property of Server resource: http://git.openstack.org/cgit/openstack/python-openstacksdk/tree/openstack/compute/v2/server.py#n95 | 08:28 |
| openstackgerrit | huangtianhua proposed openstack/senlin master: Correct a const name https://review.openstack.org/435300 | 08:28 |
| openstackgerrit | Merged openstack/senlin master: Fixed inappropriate event generation in action base https://review.openstack.org/435249 | 08:28 |
| openstackgerrit | Merged openstack/senlin master: Enable heat stack event listener https://review.openstack.org/435243 | 08:28 |
| Qiming | ruijie, http://git.openstack.org/cgit/openstack/senlin/tree/senlin/engine/event.py | 08:29 |
| ruijie | yes Qiming, I see | 08:31 |
| ruijie | https://review.openstack.org/gitweb?p=openstack/senlin.git;a=blob;f=senlin/events/message.py;h=352422b9ee02f1243dd7a72c650ed9f34db485c4;hb=373cf4cf8aa0c75501af1cb35556fa034a5fda02#l24 | 08:31 |
| ruijie | but in this method, we only use "phase" from kwargs | 08:32 |
| openstackgerrit | Qiming Teng proposed openstack/senlin master: backport candidate https://review.openstack.org/435302 | 08:33 |
| Qiming | yes.ruijie, do you have a suggestion where should we put it? | 08:34 |
| ruijie | sorry Qiming, It is done in ActionProc(..) | 08:34 |
| Qiming | np | 08:36 |
| ruijie | https://review.openstack.org/gitweb?p=openstack/senlin.git;a=blob;f=senlin/engine/actions/base.py;h=6af18827a1bb9686922cb5c3a10affb2323fb093;hb=373cf4cf8aa0c75501af1cb35556fa034a5fda02#l478 | 08:36 |
| ruijie | https://review.openstack.org/gitweb?p=openstack/senlin.git;a=blob;f=senlin/engine/actions/base.py;h=6af18827a1bb9686922cb5c3a10affb2323fb093;hb=373cf4cf8aa0c75501af1cb35556fa034a5fda02#l327 | 08:37 |
| ruijie | but again, there are 2 dumplicated events | 08:37 |
| ruijie | duplicate | 08:37 |
| wllabs | Qiming 那个问题我又复现了，要不你教我怎么修，我来修了。或者按我的方式修了 | 08:39 |
| ruijie | I think we can remove all other event dumping and just keep these 2 in ActionProc(...) | 08:39 |
| Qiming | ruijie, it is possible | 08:39 |
| Qiming | I don't like the way to read code in the review history ... | 08:40 |
| Qiming | hard to find a thing | 08:40 |
| Qiming | this link is a lot better I think, http://git.openstack.org/cgit/openstack/senlin/tree/senlin/engine/actions/base.py | 08:41 |
| ruijie | okay Qiming :) | 08:42 |
| ruijie | I mean we can just keep line 471 and line 492 | 08:42 |
| ruijie | then we get all the events we want | 08:42 |
| Qiming | yes, the question then become to which fine granularity we want to dump "useful" events | 08:43 |
| Qiming | I'm okay with this simplification though | 08:43 |
| ruijie | great. thanks Qiming | 08:45 |
| ruijie | thanks for your time :) | 08:45 |
| Qiming | welcome | 08:46 |
| wllabs | http://git.openstack.org/cgit/openstack/python-openstacksdk/tree/openstack/compute/v2/server.py#n95 | 08:57 |
| wllabs | Qiming 看他这根本就没有实现 | 08:58 |
| wllabs | 那个Body返回的也只是个key，不是 value | 08:58 |
| Qiming | ... | 09:01 |
| Qiming | please don't make such conclusions until you are sure you know all the details | 09:06 |
| wllabs | 嗯，我正在细看 | 09:06 |
| Qiming | this script works perfectly for me | 09:06 |
| Qiming | http://paste.openstack.org/show/599347/ | 09:06 |
| Qiming | and it shows me the availability_zone value as 'nova' | 09:06 |
| elynn | wllabs, you could try to update openstacksdk and try again :) | 09:09 |
| Qiming | availability_zone property has been there for centuries | 09:10 |
| *** XueFeng has joined #senlin | 09:23 | |
| wllabs | 我错了，版本不对 | 09:24 |
| Qiming | yanyanhu, I think we are in good shape to cut a release based on this: https://review.openstack.org/435300 | 09:28 |
| yanyanhu | yes, will wait for this patch to propose a new release cut | 09:31 |
| Qiming | it has been a little scurry during the last hours, but, eventually, I believe we are there | 09:31 |
| yanyanhu | just I found there are three patches in post gate queue... | 09:31 |
| yanyanhu | not sure how long it will take to finish... | 09:31 |
| yanyanhu | that's why the last release cut failed in verification... | 09:31 |
| yanyanhu | https://review.openstack.org/#/c/435282/ | 09:31 |
| Qiming | noticed that | 09:32 |
| Qiming | I think you can use the commit id from the git log | 09:32 |
| yanyanhu | will cut release on 435300 and recheck the release patch if needed | 09:32 |
| Qiming | http://git.openstack.org/cgit/openstack/senlin/log/ | 09:32 |
| Qiming | you will use the commit with Merge in commit message | 09:33 |
| yanyanhu | understand | 09:33 |
| Qiming | i.e. patch commited by Jenkins | 09:33 |
| Qiming | just something worth trying, not 100% sure though | 09:33 |
| yanyanhu | I see | 09:34 |
| *** Jeffrey4l has quit IRC | 09:42 | |
| *** guoshan has quit IRC | 09:53 | |
| *** Jeffrey4l has joined #senlin | 09:55 | |
| *** Jeffrey4l has quit IRC | 09:56 | |
| openstackgerrit | Merged openstack/senlin master: Correct a const name https://review.openstack.org/435300 | 09:57 |
| *** Jeffrey4l has joined #senlin | 09:57 | |
| *** yanyanhu has quit IRC | 10:01 | |
| openstackgerrit | Hongbin Li proposed openstack/senlin master: Remove param that never be used https://review.openstack.org/435340 | 10:01 |
| *** ruijie has quit IRC | 10:10 | |
| openstackgerrit | Aaron Ding proposed openstack/senlin master: Revise cluster get request https://review.openstack.org/435223 | 10:14 |
| *** Jeffrey4l has quit IRC | 10:18 | |
| *** Jeffrey4l has joined #senlin | 10:19 | |
| *** xuao has quit IRC | 10:50 | |
| openstackgerrit | Hongbin Li proposed openstack/senlin master: Remove param that never be used https://review.openstack.org/435340 | 10:51 |
| wllabs | 发现一个问题 | 11:14 |
| wllabs | 一个cluster怎么可以只有一个 profile | 11:14 |
| wllabs | cluster下面有 db, 有web, 有services, 有mq 配置肯定不一样 | 11:15 |
| wllabs | 这样才正常吧 | 11:16 |
| wllabs | @all | 11:20 |
| XueFeng | hi,wllabs | 11:46 |
| XueFeng | If we have db, web , services. we can define 3 clusters | 11:46 |
| wllabs | yes | 11:47 |
| wllabs | i just think so | 11:48 |
| wllabs | thank you | 11:48 |
| XueFeng | But if you want you can define one cluster .It can contins these three function type nodes, because there are all the nova server type.And the profile to the cluster will be the default profile for one function type node | 11:50 |
| XueFeng | Which city are you in? | 11:50 |
| wllabs | beijing | 11:52 |
| XueFeng | Oh | 11:53 |
| wllabs | 你是中兴的？ | 11:53 |
| wllabs | 你还会做android? | 11:53 |
| XueFeng | 19:53 < wllabs> 你是中兴的？ | 11:56 |
| XueFeng | 19:53 < wllabs> 你还会åandroid? | 11:56 |
| XueFeng | Can you write in English | 11:57 |
| XueFeng | I can't see | 11:57 |
| wllabs | Are you working for ZTE ? | 11:58 |
| XueFeng | Yes | 11:58 |
| wllabs | do you master android ? | 11:59 |
| XueFeng | No | 11:59 |
| XueFeng | Why ask about android? | 12:00 |
| XueFeng | I have been involved in writing a book of android | 12:03 |
| XueFeng | I will leave for a moment | 12:04 |
| wllabs | you are great | 12:05 |
| wllabs | Take me to learn technology | 12:06 |
| *** catintheroof has joined #senlin | 12:23 | |
| *** chenyb has joined #senlin | 12:26 | |
| *** yuanying has quit IRC | 13:18 | |
| *** wllabs has quit IRC | 13:46 | |
| *** chenyb has left #senlin | 14:33 | |
| *** yanyanhu has joined #senlin | 14:44 | |
| yanyanhu | hi, Qiming, I'm at home now. Thanks a lot for helping backport those patches! So every patches after rc1(marked as stable/ocata) will be part of pike release by default? | 14:59 |
| *** test_test_ has joined #senlin | 15:08 | |
| *** test_test_ has quit IRC | 15:09 | |
| yanyanhu | hi, Qiming, looks all backport have been done, will try to update the release patch with commit id "3278a7fc8777a6153a11987a76b1ba498ecf20e3" | 15:49 |
| *** Drago1 has joined #senlin | 16:01 | |
| *** yanyanhu has quit IRC | 16:03 | |
| -openstackstatus- NOTICE: Restarting gerrit due to performance problems | 19:05 | |
| *** xiangxinyong has joined #senlin | 20:23 | |
| *** edisonxiang has quit IRC | 20:25 | |
| *** Jeffrey4l_ has joined #senlin | 21:34 | |
| *** Jeffrey4l has quit IRC | 21:34 | |
| *** Drago1 has quit IRC | 21:52 | |
| *** catintheroof has quit IRC | 23:21 | |
| *** catintheroof has joined #senlin | 23:22 | |
| *** catintheroof has quit IRC | 23:26 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!