| *** Qiming has quit IRC | 00:11 | |
| *** Qiming has joined #senlin | 00:11 | |
| *** rebase has quit IRC | 00:15 | |
| *** Qiming_ has joined #senlin | 00:20 | |
| *** Qiming has quit IRC | 00:20 | |
| *** openstackstatus has joined #senlin | 00:26 | |
| *** ChanServ sets mode: +v openstackstatus | 00:26 | |
| *** Qiming_ has quit IRC | 00:27 | |
| *** Qiming_ has joined #senlin | 00:27 | |
| *** Qiming__ has joined #senlin | 00:28 | |
| *** Qiming__ is now known as Qiming | 00:31 | |
| Qiming | welcome, openstackstatus | 00:31 |
|---|---|---|
| *** Qiming_ has quit IRC | 00:32 | |
| *** rebase has joined #senlin | 00:34 | |
| *** rebase has quit IRC | 00:51 | |
| *** xuhaiwei has joined #senlin | 00:53 | |
| *** zhenguo has joined #senlin | 01:19 | |
| *** openstackgerrit has quit IRC | 01:22 | |
| *** openstackgerrit has joined #senlin | 01:22 | |
| *** Yanyanhu has joined #senlin | 01:27 | |
| openstackgerrit | Merged openstack/senlin: Minor tweaks to policy-update log messages https://review.openstack.org/251113 | 01:43 |
| openstackgerrit | Merged openstack/senlin: Fix action request method and URI https://review.openstack.org/251212 | 01:43 |
| *** lixinhui has joined #senlin | 02:08 | |
| *** Yanyanhu has quit IRC | 02:17 | |
| *** elynn has joined #senlin | 02:19 | |
| elynn | morning ~ | 02:41 |
| xuhaiwei | morning | 02:42 |
| elynn | I can't connect to freenode yesterday and I don't know why... | 02:48 |
| xuhaiwei | me too | 02:48 |
| xuhaiwei | from yesterday afternoon about 15:00 | 02:49 |
| elynn | Ah, Seems freenode is not stable yesterday. | 02:49 |
| xuhaiwei | but my colleague can access to it | 02:49 |
| xuhaiwei | I am using browse not irc tools | 02:50 |
| elynn | I cant login freenode via neither web not xchat... | 02:50 |
| elynn | Always get an connection failure | 02:51 |
| xuhaiwei | yes, the server is strange yesterday | 02:54 |
| *** pm90_ has quit IRC | 02:55 | |
| Qiming | freenode was under DDOS attack yesterday | 03:04 |
| xuhaiwei | cool | 03:07 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix documentation for conformance https://review.openstack.org/251230 | 03:08 |
| openstackgerrit | Merged openstack/senlin: Decide auth_plugin type in create_connection https://review.openstack.org/251262 | 03:08 |
| elynn | OpenStack should create a project for anti-DDOS :D | 03:11 |
| Qiming | ... | 03:12 |
| openstackgerrit | Merged openstack/python-senlinclient: Fix action api calls https://review.openstack.org/251219 | 03:12 |
| Qiming | that would be a sub-project of neutron? | 03:12 |
| elynn | en, your idea sounds more reasonable | 03:14 |
| elynn | Is there any NFV project to do that? | 03:14 |
| *** yuanying has quit IRC | 03:23 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add more configurable parameters to wait_for_server https://review.openstack.org/251623 | 03:28 |
| *** Yanyanhu has joined #senlin | 03:30 | |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Add cluster lock steal https://review.openstack.org/246180 | 03:31 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Add node_steal for node lock https://review.openstack.org/246179 | 03:31 |
| *** lixinhui has quit IRC | 03:33 | |
| elynn | Yanyanhu, Qiming Does 0.1.6 of senlinclient fix sdk problem? | 03:34 |
| Qiming | better use 0.1.8 I think | 03:35 |
| Yanyanhu | yea | 03:35 |
| Yanyanhu | I think 0.1.7 is workable | 03:35 |
| Qiming | there were some other problems found later on | 03:35 |
| Qiming | "workable", :) | 03:35 |
| Qiming | no such word, Yanyanhu | 03:35 |
| Yanyanhu | ok... so just 0.1.7 works | 03:35 |
| Qiming | :P | 03:36 |
| elynn | I thought 0.1.6 is the latest one, I made a mistake.. | 03:36 |
| openstackgerrit | Merged openstack/senlin: Add node_steal for node lock https://review.openstack.org/246179 | 03:51 |
| *** elynn has quit IRC | 03:52 | |
| *** yuanying has joined #senlin | 04:07 | |
| Yanyanhu | hi, Qiming, you may need to update the raises decription in this patch https://review.openstack.org/#/c/242681/4..6/openstack/connection.py | 04:07 |
| Yanyanhu | since three types of exception could be raises now | 04:08 |
| Yanyanhu | raised | 04:08 |
| openstackgerrit | Merged openstack/senlin: Fix documentation for conformance https://review.openstack.org/251230 | 04:10 |
| Qiming | em ... there are more than 3 types | 04:11 |
| openstackgerrit | Merged openstack/senlin: Add cluster lock steal https://review.openstack.org/246180 | 04:12 |
| Yanyanhu | ah, yes, my mistake, actually http exception will be raised | 04:12 |
| openstackgerrit | junxu proposed openstack/senlin: Store the actual UUID of obj_id in Senlin database for webhook create https://review.openstack.org/251187 | 04:56 |
| *** elynn has joined #senlin | 05:00 | |
| *** elynn has quit IRC | 05:07 | |
| *** elynn has joined #senlin | 05:28 | |
| *** elynn has quit IRC | 06:03 | |
| *** elynn has joined #senlin | 06:04 | |
| xuhaiwei | .. | 06:11 |
| xuhaiwei | Anyone has the same problem? when requesting to Nova by creating/deleting a node, will get this error : public endpoint for compute service named senlin not found | 06:15 |
| Qiming | context? | 06:16 |
| xuhaiwei | no context | 06:16 |
| xuhaiwei | just creating a node | 06:16 |
| Qiming | are you using the latest senlinclient? | 06:16 |
| xuhaiwei | yes, I think | 06:17 |
| xuhaiwei | the endpoint_filter fix has been pulled | 06:17 |
| Qiming | something is wrong, there is no "compute service named senlin" | 06:18 |
| xuhaiwei | the EndpointNotFound exception is raised from keystoneauth1 | 06:19 |
| Qiming | yes | 06:19 |
| Qiming | we are requesting a "compute" service, named "senlin" | 06:19 |
| *** pm90_ has joined #senlin | 06:21 | |
| xuhaiwei | why the 'senlin' endpoint is not found | 06:21 |
| Qiming | the error says "compute service named senlin" | 06:22 |
| Qiming | "compute service named senlin" | 06:22 |
| Qiming | read it again? | 06:22 |
| xuhaiwei | should be clustering service named senlin? | 06:22 |
| Qiming | you are not creating or using webhooks? | 06:22 |
| Qiming | yes, senlin is not a compute service | 06:23 |
| xuhaiwei | no | 06:23 |
| Qiming | no compute service is named senlin | 06:23 |
| Qiming | somehow, we are doing something weird | 06:23 |
| Qiming | for example, in keystone_v3 service_get, why are we specifying both service_type and name? | 06:24 |
| Qiming | it could be the sdk side is broken as well | 06:25 |
| xuhaiwei | I have investiged that service_get method some time, it seems service_type and name are not correctly used in keystoneauth1 | 06:25 |
| Qiming | after migrating to use keystoneauth1, the identity service needs a revision? | 06:25 |
| xuhaiwei | there are several get_endpoint method in keystoneauth1, don't know whether we can use one of them directly | 06:26 |
| Qiming | well ... I don't think so | 06:27 |
| Qiming | it is sdk's problem to solve | 06:28 |
| Qiming | all we (senlin) should care about is the sdk interface | 06:28 |
| xuhaiwei | sometime just dont know who is wrong, them or us | 06:29 |
| Qiming | right, that is a huge pain | 06:29 |
| Qiming | "when requesting to Nova by creating/deleting a node," | 06:45 |
| Qiming | xuhaiwei, are you using senlin to do node creation? | 06:45 |
| xuhaiwei | yes, of course | 06:45 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Attempt to log more info on sdk exceptions https://review.openstack.org/251679 | 06:47 |
| Qiming | xuhaiwei, try this patch ^ | 06:48 |
| Qiming | see this can help give you a more detailed context | 06:48 |
| xuhaiwei | ok | 06:49 |
| xuhaiwei | thanks | 06:49 |
| Qiming | I'm moving the LOG.exception so that it is in an exception context | 06:49 |
| Qiming | the previous location where we do LOG.exception was not working, because it is already outside an exception context | 06:50 |
| xuhaiwei | yes, much more details now | 06:51 |
| Qiming | we have been logging a lot of info before, until it broke the fixture | 06:54 |
| *** xuhaiwei has quit IRC | 06:56 | |
| *** xuhaiwei has joined #senlin | 06:56 | |
| xuhaiwei | Qiming, I noticed we supported nested cluster in senlin | 07:24 |
| xuhaiwei | but in fact, this design is not well used currently, is there any use case for it? | 07:25 |
| Qiming | we supported? | 07:26 |
| xuhaiwei | we are supporting | 07:26 |
| Qiming | not sure we are supporting | 07:27 |
| Qiming | we don't have a overall design for that | 07:27 |
| xuhaiwei | we create a cluster, we can configure its parent | 07:27 |
| Qiming | the 'parent' parameter is just a placeholder | 07:27 |
| xuhaiwei | can it be used in some case? | 07:28 |
| Qiming | yes I think so | 07:28 |
| xuhaiwei | or in the future | 07:28 |
| Qiming | but I am not sure what it would be | 07:28 |
| xuhaiwei | to me, when saying a cluster has a son, I may want to caculate all the nodes to its current_size | 07:30 |
| xuhaiwei | that's wrong, right | 07:30 |
| Qiming | questions like that all need answers | 07:31 |
| Qiming | I don't think we are gonna support that in Mitaka | 07:31 |
| xuhaiwei | in fact i dont want to support it in senlin, that will totally break the current design I think | 07:32 |
| Qiming | there will be rules | 07:32 |
| *** pm90_ has quit IRC | 07:34 | |
| xuhaiwei | another question, about the senlinclient/v1/shell.py test, I am not testing the 'args', is it ok? Or I need to add them or something else | 07:35 |
| Qiming | not testing args ... what do you mean? | 07:37 |
| xuhaiwei | I mean the 'options' | 07:39 |
| xuhaiwei | which is required or not | 07:39 |
| Qiming | point me to an exact line number? | 07:50 |
| xuhaiwei | https://github.com/openstack/python-senlinclient/blob/master/senlinclient/v1/shell.py#L710 | 07:52 |
| xuhaiwei | something like this | 07:52 |
| Qiming | looking | 07:54 |
| Qiming | it depends | 07:57 |
| xuhaiwei | actually I am not clear how to test them | 07:57 |
| Qiming | maybe check this? https://github.com/openstack/python-senlinclient/blob/master/senlinclient/tests/unit/test_shell.py | 07:58 |
| Qiming | or more specifically, this function: https://github.com/openstack/python-senlinclient/blob/master/senlinclient/tests/unit/test_shell.py#L169 | 07:59 |
| xuhaiwei | I will think about it | 08:05 |
| Qiming | okay, thx | 08:07 |
| openstackgerrit | Merged openstack/senlin: Attempt to log more info on sdk exceptions https://review.openstack.org/251679 | 08:07 |
| Qiming | about your patch on 202, I've blocked it | 08:07 |
| Qiming | I mean this one: https://review.openstack.org/#/c/251669/ | 08:08 |
| xuhaiwei | ok, you mean we should check the 'action' one by one? | 08:09 |
| Qiming | let's hold it, I'm proposing a different approach, let's see my proposal is a better option | 08:09 |
| xuhaiwei | ok | 08:09 |
| Qiming | the only confusion I have is about the DELETE request | 08:11 |
| xuhaiwei | what do you mean | 08:13 |
| Qiming | see this: http://specs.openstack.org/openstack/api-wg/guidelines/http.html | 08:13 |
| Qiming | find section "2xx success codes" | 08:14 |
| Qiming | it says this: Synchronous resource deletion | 08:14 |
| Qiming | Response status code must be 204 No Content | 08:14 |
| Qiming | There is no guidance on asynchronous resource deletion | 08:15 |
| Qiming | but in real life, most resource deletion would be done asynchronously, say nova server deletion, heat stack deletion, senlin cluster deletion | 08:16 |
| Qiming | some deletions may take quite some time to complete | 08:16 |
| xuhaiwei | it says 'all others should be 200' | 08:16 |
| xuhaiwei | including Asynchronous resource deletion ? | 08:17 |
| Qiming | that is a gray area | 08:19 |
| Qiming | my understanding is that 202 might be more appropriate for asynchronous deletion | 08:26 |
| Qiming | see http://tools.ietf.org/html/rfc2616#section-10.2.3 | 08:26 |
| xuhaiwei | it's difficult to understand | 08:29 |
| Qiming | yes | 08:29 |
| Qiming | but if you compare it to 10.2.1 (200), 20.2.2 (201), 202 is more appropriate | 08:30 |
| xuhaiwei | to me, I think all asynchronous actions should return 202, no matter 'creation' or 'deletion' | 08:32 |
| Qiming | yep, agreed | 08:32 |
| Qiming | neither creation nor deletion has been completed when we return | 08:32 |
| Qiming | 201 for creation is only suitable for profile, policy | 08:33 |
| xuhaiwei | so in senlin, we should filter out 'asynchronous actions' first? | 08:33 |
| Qiming | 204 for deletion is only suitable for profile, policy, etc. not clusters or nodes | 08:33 |
| Qiming | yes | 08:34 |
| xuhaiwei | in senlin all the 'actions' are asynchronous, right? | 08:34 |
| Qiming | I'm walking through all the apis | 08:34 |
| Qiming | you cannot say that | 08:34 |
| xuhaiwei | in the api layer, it maybe not | 08:34 |
| *** pm90_ has joined #senlin | 08:35 | |
| Qiming | the 'actions' in senlin's terminology are all asynchronously executed | 08:35 |
| Qiming | regarding the 'action' defined in the 'routes' package, it is different | 08:35 |
| *** pm90_ has quit IRC | 08:39 | |
| Qiming | http://i.stack.imgur.com/whhD1.png | 08:42 |
| Qiming | o_o | 08:42 |
| xuhaiwei | what's that? I can't open the url | 08:58 |
| Yanyanhu | looks like a huge diagram of http request handling workflow... | 09:04 |
| xuhaiwei | my company's security system blocked it | 09:05 |
| openstackgerrit | xu-haiwei proposed openstack/python-senlinclient: Fix help message error for node_create https://review.openstack.org/251730 | 09:25 |
| Qiming | xuhaiwei, sent you via email | 09:25 |
| xuhaiwei | thanks | 09:26 |
| *** yuanying has quit IRC | 09:30 | |
| xuhaiwei | oh my eyes !! | 09:30 |
| Yanyanhu | :) | 09:30 |
| xuhaiwei | how is your nose today? :) | 09:31 |
| Yanyanhu | very uncomfortable | 09:35 |
| Yanyanhu | and also my throat and lung... | 09:35 |
| xuhaiwei | wearing musk? | 09:35 |
| Yanyanhu | you even can't see a building which is 100 meters away | 09:36 |
| Yanyanhu | yes, got one from the company | 09:36 |
| xuhaiwei | terrible | 09:36 |
| Yanyanhu | will wear it when getting back to home | 09:36 |
| Yanyanhu | yes... | 09:36 |
| xuhaiwei | maybe need to ware two at one time | 09:36 |
| Yanyanhu | yea :P | 09:36 |
| *** Yanyanhu has quit IRC | 09:52 | |
| *** zhenguo has quit IRC | 10:04 | |
| *** pm90_ has joined #senlin | 10:15 | |
| *** elynn has quit IRC | 10:22 | |
| *** pm90_ has quit IRC | 10:23 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix status code returned from API https://review.openstack.org/251771 | 10:41 |
| Qiming | xuhaiwei, still online? | 10:42 |
| xuhaiwei | yes | 10:42 |
| Qiming | the above patch is the one I mentioned | 10:42 |
| Qiming | please help review | 10:42 |
| Qiming | we have got to do this one by one | 10:42 |
| xuhaiwei | urgent? | 10:43 |
| Qiming | not very urgent | 10:57 |
| openstackgerrit | xu-haiwei proposed openstack/python-senlinclient: Add test case for v1/shell.py part2 https://review.openstack.org/251798 | 11:15 |
| *** lixinhui has joined #senlin | 11:41 | |
| lixinhui | johnthetubaguy | 11:41 |
| lixinhui | how for me to touch this guy | 11:46 |
| *** elynn has joined #senlin | 12:20 | |
| *** pm90_ has joined #senlin | 12:27 | |
| *** yanyanhu has joined #senlin | 12:30 | |
| *** jruano has joined #senlin | 13:04 | |
| *** pm90_ has quit IRC | 13:15 | |
| *** yanyanhu has quit IRC | 14:00 | |
| *** haiwei has joined #senlin | 14:00 | |
| *** elynn has quit IRC | 14:00 | |
| haiwei | anything else? | 14:00 |
| Qiming | nope from me sir | 14:01 |
| Qiming | it is too late for you | 14:01 |
| Qiming | time to date someone in dream | 14:01 |
| haiwei | don't be too tired | 14:01 |
| haiwei | you too | 14:01 |
| Qiming | :) | 14:01 |
| haiwei | see you | 14:02 |
| Qiming | see you | 14:02 |
| *** haiwei has quit IRC | 14:06 | |
| *** lixinhui has quit IRC | 14:11 | |
| *** zhenguo has joined #senlin | 14:18 | |
| *** Qiming has quit IRC | 14:35 | |
| *** zhenguo has quit IRC | 15:05 | |
| *** pm90_ has joined #senlin | 15:20 | |
| *** pm90__ has joined #senlin | 15:23 | |
| *** pm90_ has quit IRC | 15:25 | |
| *** jruano has quit IRC | 16:12 | |
| *** pm90__ has quit IRC | 17:43 | |
| *** pm90_ has joined #senlin | 19:15 | |
| *** pm90_ has quit IRC | 23:01 | |
| *** jruano has joined #senlin | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
