| *** openstackgerrit has quit IRC | 00:16 | |
| *** openstackgerrit has joined #senlin | 00:17 | |
| *** rebase has quit IRC | 00:39 | |
| *** rebase has joined #senlin | 01:15 | |
| *** pratikmallya has joined #senlin | 01:20 | |
| *** Yanyanhu has joined #senlin | 01:28 | |
| *** shu has quit IRC | 01:37 | |
| *** shu-mutou has joined #senlin | 01:37 | |
| *** zhenguo has joined #senlin | 01:38 | |
| *** Qiming has joined #senlin | 01:56 | |
| *** Qiming has quit IRC | 02:01 | |
| *** Yanyanhu has quit IRC | 02:03 | |
| openstackgerrit | Merged openstack/senlin-dashboard: Return node.physical_id while node is creating https://review.openstack.org/248711 | 02:19 |
|---|---|---|
| *** jdandrea has quit IRC | 02:20 | |
| *** Yanyanhu has joined #senlin | 02:47 | |
| *** rebase has quit IRC | 02:56 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Enable update of image property for nova server profile https://review.openstack.org/248555 | 03:11 |
| *** Qiming has joined #senlin | 03:15 | |
| Yanyanhu | hi, Qiming, I found the 'authenticate' method in sdk driver doesn't work for this patch was merged https://review.openstack.org/#/c/186140/ | 03:16 |
| Yanyanhu | thus unit test also failed | 03:17 |
| Qiming | yes, I'm trying to fix | 03:17 |
| Yanyanhu | ok | 03:17 |
| Qiming | to diagnose I have to do tox -epy27 -r | 03:17 |
| Yanyanhu | ... | 03:17 |
| Qiming | and network is broken | 03:17 |
| Qiming | cannot recreate test environment | 03:17 |
| Yanyanhu | crap network | 03:17 |
| Yanyanhu | it's terrible, especially in recent days | 03:18 |
| Qiming | huangtianhua is also complaining | 03:18 |
| Qiming | senlinclient is broken by sdk change | 03:18 |
| Yanyanhu | ok | 03:19 |
| Qiming | thus senlinclient plugin is not working, it is starting to block heat gate | 03:19 |
| Yanyanhu | https://review.openstack.org/#/c/242681/1/openstack/connection.py is this the new method we should use? | 03:19 |
| Yanyanhu | ... | 03:19 |
| Yanyanhu | sigh | 03:19 |
| Yanyanhu | if so, IMHO, this method should return some auth info rather than just succeed silently | 03:20 |
| Qiming | cannot recall why we are using 'authenticate' | 03:20 |
| Yanyanhu | we use it for token and user_id | 03:20 |
| Qiming | okay, then we use it | 03:21 |
| Qiming | make it return an auth_info | 03:21 |
| Yanyanhu | yes, I guess so | 03:21 |
| Yanyanhu | but this patch is still in progress | 03:21 |
| Qiming | it needs a rebase | 03:22 |
| Qiming | my tox environment is completely broken too | 03:22 |
| Qiming | cannot do a thing | 03:22 |
| Yanyanhu | seems so | 03:22 |
| Qiming | damn China | 03:22 |
| Yanyanhu | really need a proxy. I set up a socket5 proxy in my local env for yum | 03:23 |
| *** yuanying has quit IRC | 03:23 | |
| Yanyanhu | but for pip, still need a http proxy | 03:23 |
| Yanyanhu | http://git.openstack.org/cgit/openstack/senlin/tree/senlin/drivers/openstack/keystone_v3.py#n140 | 03:23 |
| Yanyanhu | get_token and get_user_id are two methods using authenticate function | 03:24 |
| Qiming | so we can use get_token and get_user_id directly? | 03:34 |
| Yanyanhu | no, they still depend on authenticate method | 03:34 |
| Yanyanhu | http://paste.openstack.org/show/479809/ | 03:35 |
| Yanyanhu | can we rewrite authenticate method like this? before brain's patch is merged | 03:35 |
| Yanyanhu | currently, sdk still doesn't support get_token in identity proxy layer I think | 03:36 |
| Qiming | a minute | 03:36 |
| Yanyanhu | ok | 03:36 |
| *** elynn has joined #senlin | 03:36 | |
| Qiming | check this: https://review.openstack.org/#/c/242681/ | 03:36 |
| Qiming | I'm taking over that patch | 03:36 |
| Qiming | but this is orthogonal to your proposal | 03:37 |
| Qiming | you can still avoid 'import transport' directly | 03:37 |
| Yanyanhu | yes, but if this patch is better I think | 03:38 |
| Yanyanhu | if we can merge it | 03:38 |
| Qiming | I'm waiting for the gate to do a quick approval of 242681 | 03:38 |
| Qiming | the solution you posted is also needed | 03:38 |
| Yanyanhu | we should invoke conn.authorize after this patch is merged | 03:38 |
| Qiming | we won't be able to import transport directly | 03:38 |
| Yanyanhu | ok | 03:38 |
| Yanyanhu | yes | 03:38 |
| Qiming | it is gone | 03:39 |
| Yanyanhu | yes, I will help to fix sdk driver in senlin service using this way | 03:40 |
| Qiming | elynn | 03:40 |
| Yanyanhu | oh, Qiming, I think before sdk gives a new release, we may still need a workaround? | 03:41 |
| Qiming | yes | 03:42 |
| elynn | Hi Qiming | 03:42 |
| Yanyanhu | in client side | 03:42 |
| Qiming | elynn, huangtianhua pinged me | 03:42 |
| Qiming | complaining that senlin client plugin is breaking heat gate | 03:42 |
| elynn | for what? | 03:42 |
| Qiming | we are diagnosing the problem | 03:42 |
| Qiming | the root cause is that sdk changed something which may be causing problems in both senlin server and senlinclient | 03:43 |
| Qiming | this is the damn change: https://review.openstack.org/#/c/186140/ | 03:43 |
| elynn | sigh! | 03:43 |
| Qiming | elynn, please help diagnose it from heat side | 03:43 |
| elynn | I will check the log | 03:43 |
| Qiming | thanks | 03:45 |
| elynn | Is it ok to just skip senlin_plugin tests for now? | 03:46 |
| Qiming | it is that the senlinclient is broken | 03:47 |
| Qiming | skip it is okay I think | 03:47 |
| openstackgerrit | Qiming Teng proposed openstack/python-senlinclient: Fix bug introduced by openstacksdk https://review.openstack.org/248988 | 03:54 |
| *** pratikmallya has quit IRC | 03:57 | |
| *** yuanying has joined #senlin | 04:08 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Fix authenticate method in sdk driver https://review.openstack.org/248994 | 04:09 |
| elynn | also fire a bug in senlin https://bugs.launchpad.net/senlin/+bug/1519192 | 04:11 |
| openstack | Launchpad bug 1519192 in senlin "New version of openstacksdk is breaking jenkins" [Undecided,New] | 04:11 |
| elynn | So that we can have a place to track it. | 04:11 |
| *** elynn has quit IRC | 04:15 | |
| xuhaiwei | Hi, everyone, I am just back to office just now ;( | 04:18 |
| *** elynn has joined #senlin | 04:23 | |
| *** pratikmallya has joined #senlin | 04:33 | |
| openstackgerrit | Merged openstack/senlin: Fix authenticate method in sdk driver https://review.openstack.org/248994 | 04:46 |
| openstackgerrit | Merged openstack/senlin: API doc change for profile-update operation https://review.openstack.org/246720 | 04:52 |
| Qiming | Yanyanhu, https://review.openstack.org/#/c/242681/ | 06:41 |
| Qiming | reworked Brian's patch | 06:41 |
| Yanyanhu | ok, will check it | 06:42 |
| Qiming | not sure just returning a token would suffice | 06:42 |
| Yanyanhu | last patchset's unit test failed | 06:42 |
| Yanyanhu | hmm, currently, we also rely on it to get user_id based on credential given | 06:42 |
| Qiming | it was wrong because it still assuming there is a transport module | 06:42 |
| Yanyanhu | since keystone's policy will prevent user performing user_get operation by default | 06:43 |
| Yanyanhu | so, token is the only information that can be got using this way? | 06:44 |
| Qiming | but as you can see from the patch, we can always do a conn.session.get_user_id()? | 06:44 |
| Yanyanhu | ok | 06:45 |
| Yanyanhu | I didn't see get_token method in session.Session class? | 06:45 |
| Yanyanhu | only get_endpoint method there | 06:46 |
| Qiming | damn it, get_user_id() demands a auth | 06:46 |
| Qiming | it is not in session.Session | 06:46 |
| Qiming | that Session is a wrapper of the keystoneauth1.session | 06:46 |
| Yanyanhu | ok | 06:46 |
| Yanyanhu | I think their rework about authentication is not clean, still some problems need to fix | 06:48 |
| Qiming | it is much cleaner if you spend some time on the openstack/keystoneauth project | 06:50 |
| Qiming | I just quickly browsed thru the code | 06:50 |
| Qiming | it is actually helpful | 06:50 |
| Yanyanhu | oh, didn't notice this project before | 06:50 |
| Qiming | it is making sdk code a lot cleaner | 06:50 |
| Yanyanhu | will look at it | 06:50 |
| Qiming | right, we were not looking closely at keystone's development | 06:50 |
| Qiming | actually, the current 'authorize()' function added is optional | 06:51 |
| Qiming | we can avoid using it | 06:51 |
| Yanyanhu | ok | 06:51 |
| Qiming | just do a conn.session.get_token() | 06:52 |
| Yanyanhu | yes, that will be much clearer | 06:52 |
| Qiming | I'm still studying the AccessInfo structure and see whether we can make authorize() return such a struct | 06:53 |
| Qiming | that would be much helpful | 06:53 |
| Yanyanhu | yes, for both verification and getting useful authencation info | 06:53 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Enable update of network properties for nova server profile https://review.openstack.org/241561 | 06:54 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Enable update of image property for nova server profile https://review.openstack.org/248555 | 06:54 |
| Yanyanhu | oh, BTW, Qiming, I think you may need to rebase all those patches which failed unit test | 06:54 |
| Qiming | yes | 06:57 |
| Qiming | will do that later | 06:57 |
| Yanyanhu | and the latest two patches in client side hang there although the workflow has been triggered in gerrit | 06:58 |
| Yanyanhu | weird, I didn't see related jobs were added in zuul status page | 06:58 |
| Qiming | maybe need to retrigger again | 06:59 |
| Qiming | global_project wasn't reviewed so ... it forms a chain now | 06:59 |
| Qiming | a pain for solving it | 06:59 |
| Yanyanhu | yes | 07:00 |
| openstackgerrit | Qiming Teng proposed openstack/python-senlinclient: Fix bug introduced by openstacksdk https://review.openstack.org/248988 | 07:01 |
| Qiming | retriggering | 07:01 |
| Yanyanhu | ok | 07:01 |
| Qiming | hold the approval | 07:02 |
| Qiming | till jenkins say 'yes' | 07:02 |
| Yanyanhu | on check? | 07:02 |
| Qiming | yep | 07:02 |
| Yanyanhu | ok | 07:02 |
| Qiming | don't know if some protocol has changed | 07:02 |
| Yanyanhu | ok | 07:02 |
| Qiming | didn't realize this was created on the master branch: https://review.openstack.org/#/c/248733/ | 07:03 |
| Qiming | oh I see, there is a dependency: https://review.openstack.org/#/c/248517/ | 07:03 |
| Qiming | this latter one wasn't reviewed | 07:04 |
| Yanyanhu | ah | 07:04 |
| Yanyanhu | didn't notice this | 07:04 |
| Qiming | in commit message, I introduced a dependency: Change-Id: I9fac64f1e81a6d4e61a93d57a1272d175e8b61e0 | 07:04 |
| Qiming | Depends-On: Icab38c970de7704c3134afd1b96a03b3670bd2ad | 07:04 |
| Qiming | we really need more cores to review | 07:05 |
| Yanyanhu | let me check it | 07:05 |
| Yanyanhu | right | 07:05 |
| openstackgerrit | lvdongbing proposed openstack/senlin: Support single test https://review.openstack.org/248511 | 07:05 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Reimplement authenticate method in sdk driver https://review.openstack.org/249043 | 07:11 |
| Qiming | Yanyanhu, about 249043 | 07:15 |
| Qiming | I'd suggest we go with conn.session.get_token() or conn.session.get_user_id() directly | 07:15 |
| Qiming | the conn.authorize() method is no good | 07:16 |
| Yanyanhu | ok, so we don't rely on connection.authorize() | 07:16 |
| Qiming | yep | 07:16 |
| Yanyanhu | ok, understand | 07:16 |
| Qiming | that thing is okay returning a token | 07:16 |
| Qiming | I'm expecting some objections from brian and terry | 07:16 |
| Qiming | also, I tried to return some AccessInfo but it seems not that straightforward | 07:17 |
| Yanyanhu | ok, will rework on it | 07:18 |
| Qiming | thx | 07:18 |
| Yanyanhu | no problem | 07:19 |
| Qiming | by the way | 07:19 |
| Qiming | the sdk.authenticate() method is invoked from keystone_v3 | 07:19 |
| Yanyanhu | yes | 07:20 |
| Qiming | each time we invoke kv3.get_token(), we are providing a dict for a new connection, is that always the case? | 07:20 |
| Yanyanhu | I think so | 07:20 |
| Qiming | okay | 07:21 |
| Qiming | I see, they are 'classmethod' | 07:21 |
| Qiming | got it | 07:21 |
| Yanyanhu | yep | 07:22 |
| openstackgerrit | Merged openstack/senlin: Fix cluster-list api inconsistency https://review.openstack.org/248517 | 07:23 |
| openstackgerrit | Merged openstack/python-senlinclient: Add 'global_project' option for cluster-list https://review.openstack.org/248733 | 07:37 |
| Qiming | it has just become a huge mess now | 07:38 |
| Qiming | git review | 07:38 |
| Qiming | You are about to submit multiple commits. This is expected if you are | 07:38 |
| Qiming | submitting a commit that is dependent on one or more in-review | 07:38 |
| Qiming | commits. Otherwise you should consider squashing your changes into one | 07:38 |
| Qiming | commit before submitting. | 07:38 |
| Qiming | The outstanding commits are: | 07:38 |
| Qiming | 095a60c (HEAD, fix-cluster-update) Let cluster-update return 400 if profile not found | 07:38 |
| Qiming | 0f85d60 Let cluster-create return 400 if profile not found | 07:38 |
| Qiming | 6348c25 (master) Merge branch 'master' of git://git.openstack.org/openstack/senlin | 07:39 |
| Qiming | 6dd053a Merge branch 'master' of git://git.openstack.org/openstack/senlin | 07:39 |
| Qiming | 214fc8d Add parameter sanitization for policy-attach | 07:39 |
| Qiming | Do you really want to submit the above commits? | 07:39 |
| Qiming | Type 'yes' to confirm, other to cancel: yes | 07:39 |
| Qiming | remote: Resolving deltas: 100% (22/22) | 07:39 |
| Qiming | remote: Processing changes: refs: 1, done | 07:39 |
| Qiming | To https://tengqm:ZcK4qjpWkuIY@review.openstack.org/openstack/senlin.git | 07:39 |
| Qiming | ! [remote rejected] HEAD -> refs/publish/master/bug/1518209 (you are not allowed to upload merges) | 07:39 |
| Qiming | error: failed to push some refs to 'https://tengqm:ZcK4qjpWkuIY@review.openstack.org/openstack/senlin.git' | 07:39 |
| Qiming | How am I supposed to solve this ... sigh | 07:39 |
| Yanyanhu | ... | 07:39 |
| Qiming | no single patch can be resubmitted for review now | 07:40 |
| elynn | do a rebase first | 07:40 |
| Qiming | yes, I did | 07:40 |
| elynn | ? | 07:40 |
| Qiming | I was trying to avoid dependencies among these patches | 07:40 |
| Qiming | I don't understand how there they come in again | 07:41 |
| Qiming | git checkout master | 07:41 |
| Qiming | Switched to branch 'master' | 07:41 |
| Qiming | Your branch is ahead of 'origin/master' by 3 commits. | 07:41 |
| Qiming | (use "git push" to publish your local commits) | 07:41 |
| elynn | 6dd053a Merge branch 'master' of git://git.openstack.org/openstack/senlin | 07:41 |
| Qiming | I cannot recover my master branch now | 07:41 |
| Yanyanhu | it is a nightmare when some unrelated patches were merged and appeared in the chain when you're doing rebase... | 07:41 |
| elynn | 6348c25 (master) Merge branch 'master' of git://git.openstack.org/openstack/senlin | 07:41 |
| Qiming | I don't know how to recover now | 07:43 |
| elynn | yes, rebase is always a nightmare.. | 07:43 |
| elynn | git checkout master | 07:43 |
| elynn | git branch -m somename | 07:43 |
| elynn | git checkout -B master origin/master | 07:43 |
| Qiming | [tengqm@node1 senlin]$ git checkout master | 07:43 |
| Qiming | Already on 'master' | 07:43 |
| Qiming | Your branch is ahead of 'origin/master' by 3 commits. | 07:43 |
| Qiming | (use "git push" to publish your local commits) | 07:43 |
| elynn | Then you have a clean master now... | 07:43 |
| elynn | Then switch to your branch, do 'git rebase -i master' | 07:44 |
| Yanyanhu | elynn, oh, nice :) | 07:44 |
| elynn | hope that can help... | 07:45 |
| xuhaiwei | I think you can use 'git reset ~HEAD' | 07:45 |
| Qiming | okay trying | 07:45 |
| Qiming | xuhaiwei, I have got 3 commits ahead of master | 07:45 |
| Qiming | am I supposed to do a reset? | 07:45 |
| xuhaiwei | Qiming, you can do it for three times | 07:45 |
| Qiming | then? | 07:46 |
| Qiming | start all over again? | 07:46 |
| xuhaiwei | you are in which branch now? | 07:46 |
| Qiming | I don't know | 07:46 |
| Qiming | I think I'm in master | 07:47 |
| Qiming | and I'm trying to rebase my 7-8 patches that were breaking | 07:47 |
| xuhaiwei | then do that command to have a try | 07:47 |
| Qiming | they were not reviewed and accumulated and they have some dependencies | 07:47 |
| Qiming | then sdk is breaking us | 07:47 |
| Qiming | then we got a new master | 07:47 |
| Qiming | somehow, I managed to get a state 'ahead of master by 3 commits' | 07:48 |
| xuhaiwei | the new master contains some patches that are not merged? | 07:48 |
| Qiming | I don't know | 07:48 |
| Qiming | I have local branch names | 07:48 |
| Qiming | but if I'm fixing a bug | 07:48 |
| xuhaiwei | git log shows what? | 07:48 |
| Qiming | the branch name changed to somthing like 'bug/234252' | 07:48 |
| Qiming | have to check one by one again ... | 07:48 |
| Qiming | sigh | 07:48 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Add jobs related to gating/publishing (API) docs https://review.openstack.org/246710 | 07:49 |
| Qiming | let me try rework the whole thing piece by piece | 07:50 |
| *** gongysh_ has joined #senlin | 07:54 | |
| *** shu-mutou has quit IRC | 07:57 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Change pre_test_hook.sh mode to executable https://review.openstack.org/249056 | 07:59 |
| openstackgerrit | Merged openstack/senlin: Add jobs related to gating/publishing (API) docs https://review.openstack.org/246710 | 08:02 |
| *** pratikmallya has quit IRC | 08:04 | |
| openstackgerrit | Merged openstack/senlin: Support single test https://review.openstack.org/248511 | 08:11 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Reimplement authenticate method in sdk driver https://review.openstack.org/249043 | 08:26 |
| *** shu-mutou has joined #senlin | 08:27 | |
| *** lvdongbing has joined #senlin | 08:28 | |
| *** shu-mutou has quit IRC | 08:31 | |
| Yanyanhu | hi, Qiming, I guess there are still some issues in senlin client or serice side since I got the following exception when using senlin cli to create nova server node sometimes | 08:46 |
| Yanyanhu | Unknown exception: Could not find requested endpoint in Service Catalog. | 08:46 |
| Yanyanhu | 'NoneType' object has no attribute 'to_dict' | 08:46 |
| Qiming | okay | 08:46 |
| Qiming | pinpoint it? | 08:47 |
| Yanyanhu | didn't see related error log. | 08:47 |
| Yanyanhu | yes, will check it | 08:47 |
| Qiming | senlin -d | 08:47 |
| Yanyanhu | ok | 08:47 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Disallow profile update for spec https://review.openstack.org/246694 | 08:56 |
| openstackgerrit | Liuqing Jing proposed openstack/senlin-dashboard: Add event-list api https://review.openstack.org/249077 | 08:57 |
| *** openstackgerrit has quit IRC | 09:16 | |
| *** openstackgerrit has joined #senlin | 09:17 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Change developer/user docs about profile-update https://review.openstack.org/246717 | 09:22 |
| Qiming | reminder: team meeting on #openstack-meeting at 1300 UTC | 09:23 |
| Yanyanhu | ok | 09:25 |
| xuhaiwei | ok | 09:28 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Claim fixing 404/400 usage in engine service https://review.openstack.org/247963 | 09:28 |
| elynn | got it. | 09:29 |
| *** lvdongbing has quit IRC | 09:38 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Reimplement authenticate method in sdk driver https://review.openstack.org/249043 | 09:40 |
| Yanyanhu | Qiming, still didn't find the exact reason of the error I mentioned before... I doubt it is related to recent change on session module in openstack | 09:59 |
| Yanyanhu | plan to leave, will keep looking at it | 09:59 |
| xuhaiwei | have you reported it already Yanyanhu? | 10:00 |
| Yanyanhu | nope, since I'm not sure whose error it is | 10:00 |
| Yanyanhu | senlinclient, senlin service, sdk or even keystoneauth... | 10:00 |
| xuhaiwei | just can't create a node in senlin? | 10:01 |
| Yanyanhu | I can, but sometimes, it will fail | 10:01 |
| Yanyanhu | for this exception: Unknown exception: Could not find requested endpoint in Service Catalog. | 10:01 |
| Yanyanhu | which given by senlinclient | 10:01 |
| Yanyanhu | this exception happened in execution progress of 'session.get()' | 10:01 |
| xuhaiwei | it happens when creating a node? | 10:02 |
| Yanyanhu | but haven't find the exact reason | 10:02 |
| *** zhenguo has quit IRC | 10:02 | |
| Yanyanhu | yes, actually, it happened when getting a node detail | 10:02 |
| xuhaiwei | ok | 10:02 |
| Yanyanhu | not sure this is the original place that error happened | 10:02 |
| Yanyanhu | need keep debuging | 10:03 |
| elynn | Yanyanhu, sounds like a bug in senlin service. | 10:03 |
| Yanyanhu | elynn, ok, just didn't saw error info in service log | 10:03 |
| elynn | ok... | 10:05 |
| Qiming | could be sdk bug | 10:05 |
| Yanyanhu | I also doubted it happened in endpoint_get method in keystone_v3 driver | 10:05 |
| Qiming | sometimes sdk just assume you have swift installed | 10:05 |
| elynn | will off work now | 10:05 |
| Yanyanhu | but looks like it wasn't | 10:05 |
| Qiming | for eample | 10:05 |
| Yanyanhu | Qiming, ok | 10:05 |
| elynn | ttyl | 10:05 |
| Yanyanhu | ttyl | 10:05 |
| xuhaiwei | see u | 10:05 |
| Qiming | bye | 10:05 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Add reno for release notes generation https://review.openstack.org/249104 | 10:06 |
| *** elynn has quit IRC | 10:06 | |
| Yanyanhu | Qiming, I also guess so since it looks like the error is directly given by some sdk lib which are invoked in client side | 10:07 |
| Yanyanhu | anyway, need further debug | 10:07 |
| Yanyanhu | will leave, ttyl | 10:08 |
| Qiming | sdk lib? | 10:08 |
| Yanyanhu | yes | 10:08 |
| Qiming | okay | 10:08 |
| Yanyanhu | I think this error is not returned from senlin service | 10:08 |
| Qiming | don't understand you | 10:09 |
| Yanyanhu | I mean it's not a user-visible exception that happened in senlin service side and returned to client | 10:09 |
| Qiming | okay | 10:10 |
| Qiming | the "in client side" confused me | 10:10 |
| Qiming | basically, we need some LOG.exception() call | 10:10 |
| Qiming | could have added that if not for python version problem | 10:10 |
| xuhaiwei | I can't run any senlin command now, got this exception: keystoneauth1.exceptions.auth_plugins.NoMatchingPlugin | 10:11 |
| xuhaiwei | SDK is up to date, and reinstalled | 10:11 |
| Qiming | xuhaiwei, fixing that: https://review.openstack.org/#/c/248988/ | 10:12 |
| Qiming | if you can help do a review, it would be great | 10:12 |
| xuhaiwei | I will learn it first | 10:13 |
| Qiming | ok | 10:17 |
| *** Yanyanhu has quit IRC | 10:17 | |
| *** Qiming has quit IRC | 10:26 | |
| openstackgerrit | Merged openstack/python-senlinclient: Fix bug introduced by openstacksdk https://review.openstack.org/248988 | 10:42 |
| openstackgerrit | Merged openstack/senlin: Disallow profile update for spec https://review.openstack.org/246694 | 10:43 |
| *** Liuqing has joined #senlin | 10:50 | |
| *** Qiming has joined #senlin | 11:26 | |
| openstackgerrit | Merged openstack/senlin: Add parameter sanitization for policy-attach https://review.openstack.org/248529 | 11:32 |
| openstackgerrit | Merged openstack/senlin: Fix CONF.set_override usage https://review.openstack.org/247918 | 11:42 |
| Liuqing | will we have meetings today? | 12:05 |
| openstackgerrit | Merged openstack/senlin: Claim fixing 404/400 usage in engine service https://review.openstack.org/247963 | 12:06 |
| Qiming | Liuqing, yes | 12:10 |
| Qiming | 9:00 pm | 12:10 |
| Qiming | openstack-meeting | 12:10 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Let cluster-create return 400 if profile not found https://review.openstack.org/247966 | 12:11 |
| Liuqing | yes, but i couldn't found today's meeting agenda. | 12:11 |
| Qiming | while, not yet added, :) | 12:11 |
| Qiming | everyone is welcomed to add items | 12:11 |
| Liuqing | O | 12:12 |
| *** Qiming has quit IRC | 12:22 | |
| *** Qiming has joined #senlin | 12:23 | |
| openstackgerrit | Merged openstack/senlin: Let cluster-create return 400 if profile not found https://review.openstack.org/247966 | 12:29 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Let cluster-update return 400 if profile not found https://review.openstack.org/247982 | 12:30 |
| *** yanyanhu has joined #senlin | 12:38 | |
| openstackgerrit | Merged openstack/senlin: Let cluster-update return 400 if profile not found https://review.openstack.org/247982 | 12:40 |
| *** elynn has joined #senlin | 12:40 | |
| openstackgerrit | Merged openstack/senlin: Change pre_test_hook.sh mode to executable https://review.openstack.org/249056 | 12:42 |
| *** lvdongbing has joined #senlin | 13:01 | |
| elynn | meeting? | 13:01 |
| *** zhenguo has joined #senlin | 13:01 | |
| yanyanhu | meeting time | 13:03 |
| *** lvdongbing has quit IRC | 13:04 | |
| *** lvdongbing has joined #senlin | 13:04 | |
| *** lkarm has joined #senlin | 13:08 | |
| *** yanyan has joined #senlin | 13:13 | |
| *** Qiming has quit IRC | 13:13 | |
| *** Qiming has joined #senlin | 13:16 | |
| *** yanyanhu has quit IRC | 13:16 | |
| *** Qiming has quit IRC | 13:17 | |
| *** Qiming has joined #senlin | 13:18 | |
| *** Liuqing_ has joined #senlin | 13:31 | |
| *** Liuqing has quit IRC | 13:32 | |
| *** yanyan is now known as yanyanhu | 13:43 | |
| Qiming | #endmeeting | 14:00 |
| *** Qiming has quit IRC | 14:01 | |
| *** Qiming has joined #senlin | 14:02 | |
| *** Qiming has quit IRC | 14:02 | |
| *** Qiming has joined #senlin | 14:03 | |
| *** elynn has quit IRC | 14:04 | |
| *** yanyanhu has quit IRC | 14:06 | |
| openstackgerrit | lvdongbing proposed openstack/senlin: Update node status when fail to create nova instance https://review.openstack.org/238753 | 14:12 |
| openstackgerrit | Merged openstack/senlin: Enable update of network properties for nova server profile https://review.openstack.org/241561 | 14:31 |
| *** jdandrea has joined #senlin | 14:38 | |
| openstackgerrit | Merged openstack/senlin-dashboard: Add event-list api https://review.openstack.org/249077 | 14:41 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Remove 'trigger' from api layer https://review.openstack.org/249261 | 15:00 |
| openstackgerrit | lvdongbing proposed openstack/senlin: Update node status when fail to create nova instance https://review.openstack.org/238753 | 15:05 |
| Qiming | lvdongbing, still online? | 15:07 |
| *** lvdongbing has quit IRC | 15:07 | |
| *** Qiming has quit IRC | 15:24 | |
| *** pratikmallya has joined #senlin | 15:27 | |
| *** Liuqing_ has quit IRC | 15:54 | |
| *** pratikmallya has quit IRC | 15:54 | |
| *** pratikmallya has joined #senlin | 15:59 | |
| *** pratikma_ has joined #senlin | 16:03 | |
| *** pratikmallya has quit IRC | 16:05 | |
| *** zhenguo has quit IRC | 16:29 | |
| *** rebase has joined #senlin | 16:30 | |
| *** rebase has quit IRC | 16:34 | |
| *** pratikmallya has joined #senlin | 17:54 | |
| *** pratikmallya has quit IRC | 17:54 | |
| *** pratikmallya has joined #senlin | 17:54 | |
| *** pratikma_ has quit IRC | 17:56 | |
| *** pratikmallya has quit IRC | 18:04 | |
| *** lkarm has quit IRC | 18:47 | |
| *** pratikmallya has joined #senlin | 18:50 | |
| *** pratikmallya has quit IRC | 18:50 | |
| *** pratikmallya has joined #senlin | 18:51 | |
| *** pratikmallya has quit IRC | 19:05 | |
| *** pratikmallya has joined #senlin | 19:05 | |
| *** lkarm has joined #senlin | 19:17 | |
| *** lkarm has quit IRC | 19:57 | |
| *** pratikmallya has quit IRC | 21:40 | |
| *** pratikmallya has joined #senlin | 21:46 | |
| *** pratikmallya has quit IRC | 22:14 | |
| *** pratikmallya has joined #senlin | 22:46 | |
| *** Qiming has joined #senlin | 23:22 | |
| *** lkarm has joined #senlin | 23:51 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!