| *** xuhaiwei has quit IRC | 00:06 | |
| *** pratikmallya has quit IRC | 00:10 | |
| *** lvdongbing has joined #senlin | 00:37 | |
| *** pratikmallya has joined #senlin | 00:50 | |
| *** xuhaiwei has joined #senlin | 00:55 | |
| *** Qiming has joined #senlin | 01:07 | |
| *** Yanyanhu has joined #senlin | 01:38 | |
| *** zhenguo has joined #senlin | 01:42 | |
| *** elynn__ has joined #senlin | 02:01 | |
| *** pratikmallya has quit IRC | 02:04 | |
| xuhaiwei | why openstackbot is not invited to Senlin channel? | 02:05 |
|---|---|---|
| Qiming | seems they are unstable at the moment | 02:05 |
| Qiming | they may need rests as human | 02:06 |
| Qiming | :) | 02:06 |
| *** elynn__ has quit IRC | 02:06 | |
| xuhaiwei | yes, not only senlin | 02:06 |
| *** elynn has joined #senlin | 02:06 | |
| xuhaiwei | everyone does not want to work on Sunday | 02:07 |
| xuhaiwei | on Monday | 02:07 |
| elynn | Morning | 02:09 |
| xuhaiwei | morning | 02:13 |
| *** cschulz has quit IRC | 02:16 | |
| *** branw has joined #senlin | 02:47 | |
| *** R_lixh has joined #senlin | 03:26 | |
| *** Liuqing has joined #senlin | 03:39 | |
| *** elynn has quit IRC | 04:02 | |
| *** Qiming has quit IRC | 04:29 | |
| *** elynn has joined #senlin | 05:01 | |
| *** elynn_ has joined #senlin | 05:03 | |
| *** elynn has quit IRC | 05:06 | |
| *** elynn_ has quit IRC | 05:15 | |
| *** elynn has joined #senlin | 05:15 | |
| *** Qiming has joined #senlin | 05:21 | |
| elynn | Hi Qiming, Yanyanhu, I'm working on adding lock breaker functional tests to senlin, but I don't know how to kill a senlin-engine during tests, do you know how ? | 05:37 |
| elynn | Or can I operate db_api directly ? | 05:37 |
| Yanyanhu | you mean kill the engine process? | 05:37 |
| Yanyanhu | if so you can use 'kill' cmd I think :) | 05:37 |
| *** Liuqing has quit IRC | 05:37 | |
| elynn | Yanyanhu: Yes | 05:37 |
| Yanyanhu | or 'kill -1' if you want to restart an engine | 05:37 |
| elynn | Can I use kill cmd in functional tests? | 05:37 |
| Yanyanhu | yes I think | 05:38 |
| elynn | I'm afraid that will affect other tests in parallels. Is there any way to directly operate db_api> | 05:38 |
| Yanyanhu | just like what we did for reloading some configurations before starting the functional test | 05:38 |
| elynn | ? | 05:38 |
| Yanyanhu | oh, this is a problem | 05:38 |
| Yanyanhu | elynn, I'm afraid not... unless there is an interface defined for this purpose | 05:39 |
| Yanyanhu | hi, elynn, can restarting engine meets your requirement? | 05:40 |
| Yanyanhu | it won't influence other functional test cases I think | 05:40 |
| elynn | Yanyanhu: How to restart engine? | 05:41 |
| Yanyanhu | just kill -1 will restarted I think | 05:41 |
| Yanyanhu | but I'm not sure whether the engine id will be changed | 05:41 |
| elynn | Why it won't affect other functional tests? | 05:41 |
| Yanyanhu | hmm, it is based on the assumption that functional test is not run in parallel | 05:42 |
| Yanyanhu | or can we split functional test about lock breaker from other cases? | 05:43 |
| elynn | Yanyanhu: split out means put it in a new class? | 05:43 |
| Yanyanhu | let me see | 05:44 |
| Yanyanhu | hmm, maybe this is infeasible | 05:45 |
| Yanyanhu | unless we split them in tox | 05:45 |
| elynn | Yes, It's hard to simulate this situlation from functional tests if I can't operate on db api... | 05:46 |
| Yanyanhu | yes, understand | 05:47 |
| *** Qiming has quit IRC | 05:47 | |
| *** Qiming has joined #senlin | 05:47 | |
| Yanyanhu | because senlin is a process running as service in devstack gate VM, not sure how to touch those interfaces inside the process | 05:48 |
| Yanyanhu | if there is no API for it | 05:48 |
| Yanyanhu | can we control the sequence of test cases? | 05:49 |
| *** Liuqing has joined #senlin | 05:49 | |
| elynn | Yanyanhu: I'm not sure about it, still investigating... | 05:50 |
| Yanyanhu | ok | 05:50 |
| Yanyanhu | BTW, just made a test, using kill -1 to restart senlin service will change the engine id | 05:51 |
| Yanyanhu | the old engine process will be killed I think | 05:51 |
| elynn | Yanyanhu: That's cool! | 05:51 |
| elynn | functional test on my local env failed, I'm not sure what's going on. | 05:52 |
| Yanyanhu | yep, so if we can figure out how to avoid influencing other test cases, you can use this way to generate engine failure | 05:52 |
| Yanyanhu | any error log? | 05:52 |
| elynn | Yanyanhu: http://paste.openstack.org/show/481767/ | 05:54 |
| elynn | Something like this | 05:54 |
| Yanyanhu | oh, you may need to apply this patch if you did the test in multi-engine env https://review.openstack.org/#/c/256949/ | 05:54 |
| Yanyanhu | ok, let me check it | 05:54 |
| Yanyanhu | ah, this is a place we may need to promote in future | 05:55 |
| Yanyanhu | the current exception log in functional test just gave the information where the error happen | 05:55 |
| Yanyanhu | but more useful information is actully in engine log | 05:56 |
| Yanyanhu | e.g. why the action failed during test | 05:56 |
| elynn | Let me try to set num_engine to 1 | 05:56 |
| Yanyanhu | elynn, I think you can check the engine log and locate the error based on the timestamp of exception | 05:56 |
| Yanyanhu | elynn, ok | 05:57 |
| Yanyanhu | also suggest you to apply this patch https://review.openstack.org/#/c/256949/ | 05:57 |
| elynn | Yanyanhu: ok, let me try... | 05:57 |
| *** Qiming_ has joined #senlin | 06:02 | |
| *** Qiming has quit IRC | 06:03 | |
| *** yuanying has joined #senlin | 06:11 | |
| *** elynn has quit IRC | 06:24 | |
| *** elynn has joined #senlin | 06:31 | |
| *** shu-mutou has joined #senlin | 06:36 | |
| *** lvdongbing has quit IRC | 06:52 | |
| *** openstackstatus has joined #senlin | 07:06 | |
| *** ChanServ sets mode: +v openstackstatus | 07:07 | |
| *** openstackgerrit has joined #senlin | 07:10 | |
| *** shu-mutou has quit IRC | 07:16 | |
| openstackgerrit | Merged openstack/senlin: Engine support for receivers https://review.openstack.org/257046 | 07:32 |
| openstackgerrit | Merged openstack/senlin: Update TODO list https://review.openstack.org/257089 | 07:32 |
| openstackgerrit | junxu proposed openstack/senlin: Fix cluster node updating failed with new image https://review.openstack.org/256937 | 07:36 |
| openstackgerrit | Merged openstack/senlin: Remove `return None` for functions in file senlin/engine/service.py https://review.openstack.org/257019 | 07:43 |
| Qiming_ | elynn, there? | 08:06 |
| elynn | Hi Qiming_ | 08:06 |
| Qiming_ | https://bugs.launchpad.net/senlin/+bug/1514742 | 08:06 |
| openstack | Launchpad bug 1514742 in senlin "Failed to acquire lock if engine restarts during a action" [Undecided,New] | 08:07 |
| Qiming_ | are you linking back your fix to the bug report? | 08:07 |
| Qiming_ | or should I just close it? | 08:07 |
| elynn | No.. | 08:07 |
| elynn | Just close it would be ok. | 08:07 |
| Qiming_ | got it, thx | 08:08 |
| elynn | np | 08:08 |
| openstackgerrit | Merged openstack/senlin: Fix cluster node updating failed with new image https://review.openstack.org/256937 | 08:20 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix list data type implementation https://review.openstack.org/257223 | 08:52 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix list data type implementation https://review.openstack.org/257223 | 09:04 |
| *** Qiming_ has quit IRC | 09:14 | |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Use jsonutils from oslo in engine/parser.py https://review.openstack.org/257238 | 09:32 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Refactor YamlLoader https://review.openstack.org/257239 | 09:32 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Add functional tests to engine.parser https://review.openstack.org/257240 | 09:33 |
| elynn | Unittests for engine/parser ^ | 09:34 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Add unit tests to engine.parser https://review.openstack.org/257240 | 09:43 |
| *** Liuqing has quit IRC | 09:57 | |
| *** zhenguo has quit IRC | 10:04 | |
| *** Yanyanhu has quit IRC | 10:14 | |
| *** Qiming has joined #senlin | 10:27 | |
| *** elynn has quit IRC | 10:33 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Add unit tests to engine.parser https://review.openstack.org/257240 | 10:34 |
| openstackgerrit | Merged openstack/senlin: Use jsonutils from oslo in engine/parser.py https://review.openstack.org/257238 | 11:15 |
| openstackgerrit | Merged openstack/senlin: Revise list field read/write https://review.openstack.org/256949 | 11:24 |
| openstackgerrit | Merged openstack/senlin: Fix behavior of deletion policy with 'RANDOM' constrain not in random way for selecting candidates. https://review.openstack.org/256923 | 11:24 |
| openstackgerrit | Merged openstack/senlin: Refactor YamlLoader https://review.openstack.org/257239 | 11:27 |
| openstackgerrit | zhangguoqing proposed openstack/senlin: delete __context__ that unused https://review.openstack.org/257283 | 11:29 |
| *** pratikmallya has joined #senlin | 13:02 | |
| *** elynn has joined #senlin | 13:03 | |
| *** pratikmallya has quit IRC | 13:11 | |
| *** cschulz has joined #senlin | 14:18 | |
| cschulz | Qiming I'm having trouble with nova clusters. | 14:20 |
| cschulz | Using nova_stack example | 14:20 |
| cschulz | Senlin is getting 400 responses back from Nova API | 14:21 |
| cschulz | Nova api log says "Invalid keyname" | 14:21 |
| cschulz | Is this a known problem? | 14:22 |
| Qiming | hi, cschulz, never encountered that before | 14:22 |
| cschulz | Hi | 14:22 |
| Qiming | are you referring to this one? https://bugs.launchpad.net/senlin/+bug/1517353 | 14:22 |
| openstack | Launchpad bug 1517353 in senlin "Node can't be created for KeyError: 'server'" [Undecided,New] - Assigned to Haiwei Xu (xu-haiwei) | 14:23 |
| cschulz | No that is different. In our case Nova API is catching an error and giving 400 status code. | 14:25 |
| cschulz | I'll open bug report with details | 14:25 |
| Qiming | okay | 14:25 |
| Qiming | btw, cschulz, what do you mean by 'nova_stack' example? | 14:26 |
| Qiming | are you referring to this line? http://git.openstack.org/cgit/openstack/senlin/tree/examples/profiles/nova_server.yaml#n7 | 14:27 |
| cschulz | yes | 14:28 |
| Qiming | it means you will have to create a nova keypair named 'oskey' | 14:28 |
| Qiming | or if you are using other nova keypair, you will need to revise that line | 14:29 |
| cschulz | Got it. Sorry I missed that detail. | 14:29 |
| Qiming | senlin is not returning useful information on this? | 14:30 |
| cschulz | That is true unfortunately, but it is probably also a fault of nova as it is just responding with 400 bad request. | 14:38 |
| Qiming | okay | 14:39 |
| cschulz | works now thanks for the help. | 14:39 |
| Qiming | great | 14:40 |
| Qiming | about returning 400, I think nova is following the api-wg guidelines here: http://git.openstack.org/cgit/openstack/api-wg/tree/guidelines/http.rst#n157 | 14:41 |
| Qiming | if you are creating a server in your request, and the server is referencing a resource (keypair here) that cannot be found | 14:42 |
| Qiming | it should return 400 bad request | 14:42 |
| cschulz | Yes but for Senlin to be able to issue a helpful error message it need to have some indication of what was wrong with the nova request. | 14:43 |
| Qiming | right | 14:43 |
| cschulz | I seem to remember running into some thing similar with Heat. I think that Heat actually asks nova if the keypair exists. Than it can issue an informed error message. | 14:46 |
| Qiming | right, when it is doing a stack validation | 14:47 |
| cschulz | Yes your correct. | 14:47 |
| *** zhenguo has joined #senlin | 15:17 | |
| *** zhenguo has quit IRC | 15:56 | |
| *** pratikmallya has joined #senlin | 16:30 | |
| *** Qiming has quit IRC | 16:43 | |
| *** pratikma_ has joined #senlin | 17:06 | |
| *** pratikmallya has quit IRC | 17:09 | |
| *** elynn has quit IRC | 17:10 | |
| *** openstackstatus has quit IRC | 17:20 | |
| *** openstackstatus has joined #senlin | 17:23 | |
| *** ChanServ sets mode: +v openstackstatus | 17:23 | |
| *** cschulz has quit IRC | 21:47 | |
| *** pratikma_ has quit IRC | 22:43 | |
| *** pratikmallya has joined #senlin | 23:28 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!