| *** guoshan has joined #senlin | 00:41 | |
| *** guoshan has quit IRC | 00:45 | |
| *** catintheroof has quit IRC | 00:47 | |
| *** catintheroof has joined #senlin | 00:48 | |
| *** catintheroof has quit IRC | 00:52 | |
| *** ruijie has joined #senlin | 00:54 | |
| openstackgerrit | lvdongbing proposed openstack/senlin: API support for profile-list2 https://review.openstack.org/397051 | 00:58 |
|---|---|---|
| *** gongysh2 has quit IRC | 01:06 | |
| *** guoshan has joined #senlin | 01:21 | |
| *** yanyanhu has joined #senlin | 01:29 | |
| *** gongysh2 has joined #senlin | 01:29 | |
| *** zhurong has joined #senlin | 01:40 | |
| *** elynn has joined #senlin | 01:40 | |
| *** elynn has quit IRC | 01:45 | |
| *** elynn has joined #senlin | 01:45 | |
| Qiming | hi, check this: https://review.openstack.org/#/c/397082 | 01:46 |
| Qiming | if you are using latest version of openstacksdk and you are seeing JSONDecodeError, you may want to dump the e.response at line 77 in session.py | 01:46 |
| Qiming | we need to know what was returned and why it is not a JSON | 01:47 |
| elynn | I revert back to 0.9.7 and now it works | 01:47 |
| elynn | Just curious why neutron error will impact cluster-list. | 01:48 |
| Qiming | no, it is not neutron error ... | 01:58 |
| Qiming | it is the patch | 01:59 |
| Qiming | read the logic of the patch I'm trying to revert, any exception other than 404 will be treated as an exception that contains a JSON body, which is untru | 01:59 |
| Qiming | s/untru/untrue | 02:00 |
| Qiming | so I need your help to dump the e.response content and let us know the exact case where the reponse is not a JSON | 02:00 |
| Qiming | see brian's comment to that patch: "Do you have any specific examples of how and where it's breaking things so that it can be redone properly?" | 02:01 |
| Qiming | we need to be specific about the problem, why that change breaks senlin, and how we can improve/redo that patch | 02:01 |
| openstackgerrit | Merged openstack/senlin: versioned obj support policy type get v2 https://review.openstack.org/396976 | 02:05 |
| elynn | Qiming, I will try to use the latest version and see what's going on. | 02:05 |
| Qiming | thanks!! | 02:06 |
| *** zhurong has joined #senlin | 02:08 | |
| *** zhurong has quit IRC | 02:08 | |
| openstackgerrit | Merged openstack/senlin: engine service support policy-type-get2 https://review.openstack.org/396980 | 02:13 |
| openstackgerrit | Merged openstack/senlin: Fix unit test for policy_validate https://review.openstack.org/396995 | 02:13 |
| openstackgerrit | Merged openstack/senlin: api layer support policy-type-get2 https://review.openstack.org/396999 | 02:13 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Move approved spec into the 'approved dir https://review.openstack.org/397475 | 02:21 |
| openstackgerrit | Merged openstack/senlin: API support for receiver_get2 https://review.openstack.org/396991 | 02:25 |
| openstackgerrit | Merged openstack/senlin: Engine support for profile-get2 https://review.openstack.org/397068 | 02:25 |
| openstackgerrit | Merged openstack/senlin: remove dead code about policy-type-get https://review.openstack.org/397000 | 02:25 |
| openstackgerrit | Merged openstack/senlin: Use receiver_get2 in webhook middleware https://review.openstack.org/397009 | 02:25 |
| openstackgerrit | Merged openstack/senlin: Remove dead code about receiver_get2 https://review.openstack.org/396992 | 02:25 |
| guoshan | @XueFeng, thanks, following your temp solution, it works | 02:32 |
| elynn | Qiming, when try to run e.response.json(), and then it breaks and show the trace. | 02:34 |
| elynn | seems ipdb didn't work at my env, it breaks but didn't accept inputs.. | 02:34 |
| Qiming | elynn, a simple debugging could be just dump e.response.text ? | 02:36 |
| elynn | I will try | 02:36 |
| Qiming | using a print statement would be okay | 02:36 |
| *** Drago has quit IRC | 02:39 | |
| openstackgerrit | lvdongbing proposed openstack/senlin: Engine support for profile-list2 https://review.openstack.org/397006 | 02:39 |
| *** Drago has joined #senlin | 02:40 | |
| elynn | Qiming, I insert some print to openstacksdk/session.py | 02:41 |
| elynn | http://paste.openstack.org/show/589209/ | 02:41 |
| elynn | And I got this http://paste.openstack.org/show/589207/ Qiming | 02:42 |
| elynn | Seems it return a web html | 02:42 |
| Qiming | yes, it is apache home page | 02:44 |
| elynn | It's definitely not a json so calling json() will return decoded error. | 02:44 |
| Qiming | The message is 'Forbidden', so the text is meaningless | 02:44 |
| Qiming | it is already in an exception context ... | 02:45 |
| Qiming | what is the ex.http_status ? | 02:45 |
| elynn | let me see | 02:46 |
| Qiming | forbidden is a 403 exception ... | 02:47 |
| elynn | yes, it's a 403 | 02:47 |
| elynn | Not sure why I got this error when running cluster-list but it cause the failure. | 02:47 |
| elynn | If I comment e.response.json() and let it pass | 02:49 |
| elynn | I can list cluster. | 02:49 |
| *** yuanying has quit IRC | 02:51 | |
| Qiming | em ... | 02:51 |
| Qiming | so ... seems this e.response.json() thing is breaking the version negotiation | 02:52 |
| Qiming | elynn, I think I know the reason now | 02:53 |
| Qiming | http://paste.openstack.org/show/589129/ | 02:53 |
| Qiming | this is the log dumped ... | 02:54 |
| Qiming | when sdk session was trying to GET different path for a guess of version support, it is expecting some exceptions if the version info is not there | 02:54 |
| Qiming | however, the exception handling logic must be robust enough for this to work, it cannot naively suppose that the response contains a JSON | 02:55 |
| elynn | oh.. | 02:55 |
| elynn | I see... | 02:55 |
| Qiming | it is causing an exception inside an exception, where the original exception (403) was supposed to be handled here: http://git.openstack.org/cgit/openstack/python-openstacksdk/tree/openstack/session.py#n167 | 02:56 |
| elynn | seems they call ison() too early. | 02:58 |
| Qiming | yes | 02:58 |
| Qiming | that is a bad patch | 02:59 |
| Qiming | should be reverted | 02:59 |
| Qiming | self approved | 03:00 |
| elynn | : D | 03:01 |
| Qiming | added log trace and analysis, won't be a problem now | 03:01 |
| Qiming | however ... | 03:02 |
| Qiming | check this: http://git.openstack.org/cgit/openstack/python-openstacksdk/log/ | 03:02 |
| openstackgerrit | lvdongbing proposed openstack/senlin: API support for profile-get2 https://review.openstack.org/397493 | 03:02 |
| Qiming | it means 0.9.9 is not usable ... | 03:03 |
| Qiming | we can propose a !=0.9.9 to global requirements | 03:03 |
| Qiming | but that would mean the patches since 0.9.8 would all become not usable | 03:03 |
| elynn | maybe ping brian and release new one ? | 03:04 |
| Qiming | the best thing we can try would be 0.9.8 | 03:05 |
| Qiming | pinging | 03:05 |
| Qiming | seems he is not online | 03:05 |
| elynn | Long term we need to figure out a way to add senlin tests into openstacksdk... at least for api tests | 03:05 |
| Qiming | yes, sdk does have functional tests | 03:06 |
| Qiming | but we don't have any there | 03:06 |
| Qiming | http://git.openstack.org/cgit/openstack/python-openstacksdk/tree/openstack/tests/functional | 03:06 |
| Qiming | and ... we have something not yet completed here: http://git.openstack.org/cgit/openstack/python-openstacksdk/tree/doc/source/users/guides/cluster | 03:07 |
| elynn | total 4 tests in it : ) too few to cover all cases | 03:07 |
| elynn | Long way to go.. wish I could have more time | 03:08 |
| Qiming | 32 | 03:08 |
| Qiming | brian is online, :) | 03:12 |
| Qiming | he is helping create a new release | 03:12 |
| Qiming | \o/ | 03:12 |
| elynn | Nice! | 03:12 |
| *** zzxwill has joined #senlin | 03:22 | |
| Qiming | patch proposed to global requirements: https://review.openstack.org/397510 | 03:26 |
| Qiming | hopefully it will be approved soon | 03:26 |
| *** yuanying has joined #senlin | 03:39 | |
| *** yuanying has quit IRC | 03:45 | |
| *** yuanying has joined #senlin | 03:49 | |
| openstackgerrit | Merged openstack/senlin: Engine support for profile-list2 https://review.openstack.org/397006 | 03:50 |
| *** guoshan has quit IRC | 03:54 | |
| *** elynn_ has joined #senlin | 03:54 | |
| *** elynn has quit IRC | 03:57 | |
| *** elynn_ has quit IRC | 03:59 | |
| *** elynn_ has joined #senlin | 03:59 | |
| *** zzxwill has quit IRC | 03:59 | |
| *** Drago has quit IRC | 04:03 | |
| *** elynn__ has joined #senlin | 04:05 | |
| *** zzxwill has joined #senlin | 04:06 | |
| *** elynn_ has quit IRC | 04:06 | |
| *** elynn_ has joined #senlin | 04:10 | |
| *** zzxwill has quit IRC | 04:11 | |
| *** elynn__ has quit IRC | 04:14 | |
| *** elynn_ has quit IRC | 04:38 | |
| *** guoshan has joined #senlin | 04:45 | |
| *** guoshan has quit IRC | 04:49 | |
| *** elynn_ has joined #senlin | 05:18 | |
| *** elynn_ has quit IRC | 05:22 | |
| *** elynn_ has joined #senlin | 05:23 | |
| *** guoshan has joined #senlin | 05:39 | |
| *** elynn has joined #senlin | 05:42 | |
| *** elynn_ has quit IRC | 05:43 | |
| *** guoshan has quit IRC | 05:44 | |
| *** guoshan has joined #senlin | 06:22 | |
| *** elynn has quit IRC | 06:27 | |
| *** yanyanhu has quit IRC | 06:33 | |
| *** yanyanhu has joined #senlin | 06:35 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix environment unit test https://review.openstack.org/397555 | 07:37 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: fields support policy-type-list2 https://review.openstack.org/397565 | 08:07 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: engine service support policy-type-list2 https://review.openstack.org/397569 | 08:22 |
| *** elynn has joined #senlin | 08:24 | |
| openstackgerrit | Merged openstack/senlin: Add support to have Senlin API run under Apache https://review.openstack.org/395500 | 08:24 |
| *** shu-mutou is now known as shu-mutou-AWAY | 08:27 | |
| openstackgerrit | lvdongbing proposed openstack/senlin: Use common function 'configure_auth_token_middleware' https://review.openstack.org/397575 | 08:37 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Add request object for event-list https://review.openstack.org/397583 | 08:45 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Add request object for event-list https://review.openstack.org/397583 | 08:46 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Add container profile infor to cluster dependents https://review.openstack.org/396016 | 08:46 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: api support policy-type-list2 https://review.openstack.org/397591 | 09:00 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: remove dead code about policy-type-list https://review.openstack.org/397595 | 09:07 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: remove some useless rpc code https://review.openstack.org/397597 | 09:10 |
| guoshan | the doc http://docs.openstack.org/developer/senlin/user/scenarios/autoscaling_heat.html, there is url heat_template returns 404 | 09:16 |
| *** openstackgerrit has quit IRC | 09:18 | |
| *** openstackgerrit has joined #senlin | 09:19 | |
| *** ChanServ sets mode: +v openstackgerrit | 09:19 | |
| openstackgerrit | Merged openstack/python-senlinclient: Add filters "policy_type" and "policy_name" for policy binding list https://review.openstack.org/388509 | 09:35 |
| ruijie | I can open that page, guoshan | 09:35 |
| *** elynn has quit IRC | 09:39 | |
| *** elynn has joined #senlin | 09:39 | |
| *** elynn has quit IRC | 09:44 | |
| *** elynn has joined #senlin | 09:44 | |
| guoshan | yes， this page is reachable, in this page there is a link inside( heat template) returns 404 | 09:47 |
| guoshan | you can search for "heat template" in this page | 09:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/395377 | 09:48 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: fix policy-list2 error https://review.openstack.org/397626 | 09:50 |
| *** elynn_ has joined #senlin | 09:55 | |
| *** elynn has quit IRC | 09:56 | |
| *** yanyanhu has quit IRC | 10:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/395377 | 10:14 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/senlin: Updated from global requirements https://review.openstack.org/397650 | 10:15 |
| openstackgerrit | XueFeng Liu proposed openstack/python-senlinclient: Add "cluster_id" column to `openstack cluster event list` https://review.openstack.org/397655 | 10:24 |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Add request object for event-list https://review.openstack.org/397583 | 10:37 |
| XueFeng | @guoshan, you mean this page? http://docs.openstack.org/developer/senlin/user/scenarios/ex_lbas.yaml | 10:47 |
| XueFeng | Maybe we should change tfile name to "ex_lbas.rst" | 10:49 |
| XueFeng | s/tfile/file | 10:50 |
| *** elynn_ has quit IRC | 10:51 | |
| *** guoshan has quit IRC | 10:52 | |
| openstackgerrit | XueFeng Liu proposed openstack/python-senlinclient: Add "cluster_id" column to `openstack cluster event list` https://review.openstack.org/397655 | 11:12 |
| *** miaohb has joined #senlin | 11:12 | |
| miaohb | help | 11:15 |
| *** guoshan has joined #senlin | 11:21 | |
| *** guoshan has quit IRC | 11:26 | |
| miaohb | XueFeng hi | 11:31 |
| *** ruijie has quit IRC | 11:40 | |
| XueFeng | hi, it's ok:) | 11:43 |
| *** zhurong has joined #senlin | 11:51 | |
| *** zhurong has quit IRC | 11:59 | |
| miaohb | +1 | 12:01 |
| *** miaohb has quit IRC | 12:03 | |
| *** zhurong has joined #senlin | 12:04 | |
| *** zhurong has quit IRC | 12:14 | |
| *** zhurong has joined #senlin | 12:15 | |
| *** guoshan has joined #senlin | 12:16 | |
| *** guoshan has quit IRC | 12:20 | |
| *** catintheroof has joined #senlin | 12:28 | |
| *** XueFeng has quit IRC | 12:44 | |
| *** lvdongbing has joined #senlin | 12:52 | |
| openstackgerrit | Merged openstack/senlin: Updated from global requirements https://review.openstack.org/397650 | 12:54 |
| *** yanyanhu has joined #senlin | 12:55 | |
| *** lixinhui has joined #senlin | 12:57 | |
| openstackgerrit | Kenji Ishii proposed openstack/senlin-dashboard: angularize cluster tables https://review.openstack.org/397730 | 12:59 |
| yanyanhu | hi, guys, meeting will start soon | 12:59 |
| *** elynn has joined #senlin | 13:01 | |
| *** XueFeng has joined #senlin | 13:02 | |
| *** lvdongbing has quit IRC | 13:07 | |
| *** guoshan has joined #senlin | 13:10 | |
| *** zhurong has quit IRC | 13:14 | |
| *** guoshan has quit IRC | 13:14 | |
| *** lixinhui has quit IRC | 13:28 | |
| *** Ruijie has joined #senlin | 13:58 | |
| *** guoshan has joined #senlin | 14:04 | |
| *** elynn has quit IRC | 14:05 | |
| *** guoshan has quit IRC | 14:08 | |
| *** yanyanhu has quit IRC | 14:14 | |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Add request object for event-list https://review.openstack.org/397583 | 14:50 |
| *** Drago1 has joined #senlin | 14:51 | |
| *** Drago1 has quit IRC | 14:55 | |
| *** guoshan has joined #senlin | 14:58 | |
| *** Drago1 has joined #senlin | 15:01 | |
| *** Ruijie has quit IRC | 15:01 | |
| *** guoshan has quit IRC | 15:02 | |
| *** openstackgerrit has quit IRC | 15:48 | |
| *** openstackgerrit has joined #senlin | 15:49 | |
| *** ChanServ sets mode: +v openstackgerrit | 15:49 | |
| *** guoshan has joined #senlin | 15:52 | |
| *** guoshan has quit IRC | 15:57 | |
| *** Drago1 is now known as Drago | 16:02 | |
| openstackgerrit | XueFeng Liu proposed openstack/python-senlinclient: Support "global_project" arguments for action-list https://review.openstack.org/397805 | 16:12 |
| *** XueFeng has quit IRC | 16:30 | |
| *** guoshan has joined #senlin | 16:46 | |
| *** guoshan has quit IRC | 16:50 | |
| *** guoshan has joined #senlin | 17:40 | |
| *** guoshan has quit IRC | 17:45 | |
| *** guoshan has joined #senlin | 18:34 | |
| *** guoshan has quit IRC | 18:38 | |
| *** guoshan has joined #senlin | 19:28 | |
| *** guoshan has quit IRC | 19:33 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/senlin: Updated from global requirements https://review.openstack.org/397920 | 19:59 |
| *** Drago has quit IRC | 20:17 | |
| *** Drago has joined #senlin | 20:18 | |
| *** Drago has quit IRC | 20:22 | |
| *** guoshan has joined #senlin | 20:23 | |
| *** guoshan has quit IRC | 20:27 | |
| *** Drago has joined #senlin | 20:31 | |
| *** guoshan has joined #senlin | 21:17 | |
| *** guoshan has quit IRC | 21:21 | |
| *** guoshan has joined #senlin | 22:11 | |
| *** guoshan has quit IRC | 22:15 | |
| *** catinthe_ has joined #senlin | 22:18 | |
| *** catintheroof has quit IRC | 22:21 | |
| *** catintheroof has joined #senlin | 22:35 | |
| *** catinthe_ has quit IRC | 22:36 | |
| *** guoshan has joined #senlin | 23:05 | |
| *** guoshan has quit IRC | 23:09 | |
| *** catintheroof has quit IRC | 23:52 | |
| *** catintheroof has joined #senlin | 23:54 | |
| *** catintheroof has quit IRC | 23:58 | |
| *** guoshan has joined #senlin | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!