| *** pratikmallya has quit IRC | 00:03 | |
| *** xuhaiwei has joined #senlin | 01:18 | |
| *** Qiming has joined #senlin | 01:20 | |
| *** Yanyanhu has joined #senlin | 01:41 | |
| openstackgerrit | Merged openstack/senlin: Optimize Zone Distribution Get Logic for Placement https://review.openstack.org/265086 | 01:49 |
|---|---|---|
| *** Qiming_ has joined #senlin | 02:01 | |
| *** Qiming has quit IRC | 02:04 | |
| openstackgerrit | Merged openstack/senlin: Remove 'priority' and 'level' from API layer https://review.openstack.org/266805 | 02:07 |
| *** gongysh has quit IRC | 02:12 | |
| *** gongysh has joined #senlin | 02:12 | |
| *** elynn has joined #senlin | 02:35 | |
| elynn | morning | 02:36 |
| lixinhui | morning, elynn | 02:44 |
| elynn | morning lixinhui :) | 02:44 |
| openstackgerrit | Merged openstack/senlin: use oslo.utils.reflection extract the class name https://review.openstack.org/266807 | 03:02 |
| *** yuanying_ has quit IRC | 03:34 | |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Fix action sort https://review.openstack.org/266778 | 03:44 |
| *** yuanying has joined #senlin | 03:46 | |
| *** Qiming_ has quit IRC | 03:54 | |
| *** yuanying_ has joined #senlin | 03:56 | |
| *** yuanying has quit IRC | 03:59 | |
| *** yuanying_ has quit IRC | 04:00 | |
| *** yuanying has joined #senlin | 04:06 | |
| *** yuanying has quit IRC | 04:07 | |
| *** yuanying_ has joined #senlin | 04:07 | |
| *** Qiming_ has joined #senlin | 04:11 | |
| openstackgerrit | Merged openstack/senlin: Trival: Remove unused logging import https://review.openstack.org/266863 | 04:37 |
| openstackgerrit | ting wang proposed openstack/senlin: Remove H402 rule of HACKING check https://review.openstack.org/267298 | 04:40 |
| openstackgerrit | ting wang proposed openstack/senlin: ADD E402 rule of HACKING check https://review.openstack.org/267298 | 04:46 |
| openstackgerrit | ting wang proposed openstack/senlin: ADD H402 rule of HACKING check and fix flake8 https://review.openstack.org/267298 | 05:13 |
| openstackgerrit | ting wang proposed openstack/senlin: ADD H402 rule of HACKING check and fix flake8 https://review.openstack.org/267298 | 05:23 |
| openstackgerrit | Merged openstack/senlin: Remove 'level' and 'cooldown' from policies https://review.openstack.org/266879 | 05:44 |
| openstackgerrit | Merged openstack/senlin: Remove old properties from cluster-policy-binding (API) https://review.openstack.org/266892 | 05:44 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add metadata updating support for os.nova.server profile https://review.openstack.org/262658 | 05:46 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Remove old properties of c-p binding (engine) https://review.openstack.org/266931 | 05:50 |
| openstackgerrit | Cindia-blue proposed openstack/senlin: Add Description about Recover Function in Profile https://review.openstack.org/264727 | 05:55 |
| elynn | Hi do you have time to review https://review.openstack.org/#/c/266778/ ? | 06:13 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add metadata update support for os.nova.server profile https://review.openstack.org/262658 | 06:13 |
| openstackgerrit | Merged openstack/senlin: Add Description about Recover Function in Profile https://review.openstack.org/264727 | 06:14 |
| xuhaiwei | Engine lock problem happened again when having a test | 06:28 |
| xuhaiwei | currently we are modifying a lot for api and db, not sure why this happened | 06:29 |
| xuhaiwei | I am testing cluster-scale-out by using placement policy | 06:29 |
| Qiming_ | elynn, either you or I have to rebase I guess | 06:30 |
| elynn | lock breaker are not enable for now. | 06:30 |
| elynn | Qiming_, I can do a rebase. | 06:30 |
| Qiming_ | xuhaiwei, at this stage, the whole service is not suitable for a testing | 06:30 |
| Qiming_ | you may be well aware that we are in the middle of a huge change | 06:31 |
| openstackgerrit | Merged openstack/senlin: Remove old properties of c-p binding (engine) https://review.openstack.org/266931 | 06:31 |
| xuhaiwei | yes, I will test it later again | 06:31 |
| elynn | Qiming_, Oh... both of our patch are merging, let's see what will happend. | 06:31 |
| Qiming_ | for example, http://git.openstack.org/cgit/openstack/senlin/tree/senlin/engine/actions/base.py#n383 | 06:31 |
| Qiming_ | the line elynn is modifying | 06:32 |
| Qiming_ | and the for loop on line 390 | 06:32 |
| Qiming_ | this is all broken | 06:32 |
| Qiming_ | after we are removing 'priority', 'level' and 'cooldown' | 06:32 |
| Qiming_ | we don't know which policy will be checked first, which will be checked next | 06:32 |
| Qiming_ | after all these disruptive changes are in, we will need Yanyan's patch : https://review.openstack.org/#/c/264670/ | 06:34 |
| openstackgerrit | Merged openstack/senlin: Fix action sort https://review.openstack.org/266778 | 06:34 |
| Qiming_ | so ... it doesn't make any sense to do a placement policy test | 06:34 |
| Qiming_ | at least at this moment | 06:34 |
| Qiming_ | we have higer priority things to do | 06:34 |
| xuhaiwei | yes, so lets make all of these done first | 06:35 |
| Qiming_ | elynn, both patches merged, conflict solved by git automatically | 06:36 |
| Qiming_ | \o/ | 06:36 |
| elynn | that's good. | 06:36 |
| xuhaiwei | don't understand well why gerrit has a conflict lists at the patch page | 06:37 |
| Qiming_ | it is confusing | 06:37 |
| Qiming_ | it is actually listing some patch dependencies, but it is calling them as 'conflicts' | 06:38 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: [WIP]Rework policy check for built-in policy types https://review.openstack.org/264670 | 06:45 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Rework policy check for built-in policy types https://review.openstack.org/264670 | 06:51 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Fix action columns in db migration scripts https://review.openstack.org/267343 | 07:06 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Revise policy functional test https://review.openstack.org/267352 | 07:21 |
| openstackgerrit | Merged openstack/senlin: Fix action columns in db migration scripts https://review.openstack.org/267343 | 07:32 |
| openstackgerrit | Merged openstack/senlin: Revise policy functional test https://review.openstack.org/267352 | 07:34 |
| openstackgerrit | Merged openstack/senlin: Add metadata update support for os.nova.server profile https://review.openstack.org/262658 | 07:36 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Further cleanse useless properties of cp/policies https://review.openstack.org/267363 | 07:39 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Remove level, priority and cooldown kwargs from cluster methods https://review.openstack.org/267364 | 07:42 |
| Yanyanhu | lost all my draft comments after refreshing the webpage of patch review... sigh | 07:46 |
| Yanyanhu | need to rewrite them all | 07:47 |
| xuhaiwei | xixi | 07:47 |
| Yanyanhu | didn't know what happened... | 07:48 |
| elynn | bad gerrit | 07:48 |
| Yanyanhu | yea | 07:48 |
| xuhaiwei | it should be there | 07:48 |
| Qiming_ | it is still there | 07:48 |
| xuhaiwei | you saved it , right? | 07:48 |
| Qiming_ | you will need to switch to the proper patchset, but clicking the upper right corner | 07:48 |
| Qiming_ | switch to the patchset | 07:49 |
| xuhaiwei | I think so | 07:49 |
| Yanyanhu | ah, I see | 07:49 |
| Yanyanhu | I first replied qiming's comments | 07:50 |
| Yanyanhu | than haiwei's | 07:50 |
| Yanyanhu | but those reply comments in draft status will not be shown in the same reply post | 07:50 |
| Yanyanhu | sigh, that is because you guys comments are not in the same patchset | 07:51 |
| xuhaiwei | of course | 07:51 |
| Yanyanhu | my fault. I shouldn't blame gerrit for this :) | 07:52 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Add 'service' table in db https://review.openstack.org/267369 | 07:52 |
| xuhaiwei | I often make the same mistake | 07:52 |
| Yanyanhu | :) | 08:00 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Make senlin-manage a console_script entry point https://review.openstack.org/267374 | 08:09 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Make senlin-api a console script entry point https://review.openstack.org/267375 | 08:09 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Make senlin-engine a console script entry point https://review.openstack.org/267376 | 08:09 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Revise user docs for policy and bindings changes https://review.openstack.org/267378 | 08:13 |
| openstackgerrit | Merged openstack/senlin: Replace deprecated library function os.popen() with subprocess https://review.openstack.org/266849 | 08:14 |
| Yanyanhu | hi, Qiming_, around? | 08:39 |
| Qiming_ | yes sir | 08:39 |
| Yanyanhu | do you think it's necessary to add 'policy_type' attribute to cluster_policy db model? | 08:39 |
| Yanyanhu | for filtering | 08:39 |
| Qiming_ | wait | 08:40 |
| Yanyanhu | ok | 08:40 |
| Qiming_ | we already have a policy in it | 08:42 |
| Qiming_ | as a backref | 08:42 |
| Qiming_ | if needed, I'd suggest we add a db api for this purpose instead of adding a redundant column into the 'cluster_policy' table | 08:43 |
| Qiming_ | I understand the requirement to do a policy type based filtering though | 08:43 |
| Qiming_ | that can be done via a DB API | 08:44 |
| Yanyanhu | Qiming_, understand, I think this is better to add 'policy_type' property | 08:48 |
| Yanyanhu | thanks | 08:48 |
| Yanyanhu | s/better to/better than | 08:48 |
| openstackgerrit | Cindia-blue proposed openstack/senlin: Add Recover into Node Actions and Node Model https://review.openstack.org/264568 | 08:55 |
| openstackgerrit | Cindia-blue proposed openstack/senlin: Add Recover as a Cluster Action https://review.openstack.org/267419 | 09:07 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Support to filter cluster_policy with policy_type https://review.openstack.org/267422 | 09:15 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Further cleanse developer doc https://review.openstack.org/267424 | 09:16 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Add db api for service table https://review.openstack.org/267425 | 09:18 |
| Yanyanhu | hi, elynn, which patch is the last one in this dependency chain? https://review.openstack.org/#/c/267374/ | 09:21 |
| Yanyanhu | so patch for senlin-manage depends on the one for senlin-api? | 09:21 |
| Yanyanhu | and the one for senlin-api relies on the one for senlin-engine? | 09:22 |
| elynn | senlin-engine depends on senlin-api | 09:22 |
| Yanyanhu | oh | 09:22 |
| elynn | senlin-api depends on senlin-manage | 09:22 |
| Yanyanhu | I see | 09:22 |
| elynn | from bottom to top | 09:22 |
| Yanyanhu | got it | 09:22 |
| openstackgerrit | Merged openstack/senlin: Further cleanse useless properties of cp/policies https://review.openstack.org/267363 | 09:23 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Revert "Fix action columns in db migration scripts" https://review.openstack.org/267430 | 09:35 |
| Qiming_ | hi, we were getting some complaints a few weeks before from some early users | 09:35 |
| Qiming_ | we are not gonna change the DB schema freely | 09:35 |
| Qiming_ | we have to start managing DB schema in a stricter way | 09:36 |
| elynn | OK, let's follow the guided way to migrate our db schema. | 09:36 |
| Qiming_ | right | 09:37 |
| Qiming_ | chuck sent me an email complaining that after reinstall senlin service and do db migration, we were deleting all database entries | 09:37 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Remove 'level', 'priority' and 'cooldown' from DB https://review.openstack.org/267432 | 09:38 |
| Qiming_ | that is another problem | 09:38 |
| Qiming_ | xuhaiwei, let's keep those columns in database | 09:38 |
| xuhaiwei | why | 09:38 |
| elynn | I think it's better to add another script to remove deleted columns. | 09:38 |
| Qiming_ | elynn, we can improve the script to do a clean install or an upgrade | 09:39 |
| Qiming_ | xuhaiwei, okay, you are only removing them from the db models, that is fine | 09:40 |
| elynn | ok, let me have a try. | 09:40 |
| Qiming_ | we are not so sure whether things will work properly after this huge change | 09:40 |
| Qiming_ | xuhaiwei, so I'd suggest we keep those columns in database until we are pretty sure | 09:41 |
| xuhaiwei | I am afraid it will harm something if we dont clean them all | 09:42 |
| Yanyanhu | I agree with keeping these columes since some users have started using senlin now | 09:42 |
| Yanyanhu | otherwise, they have to rebuild their db... | 09:43 |
| Yanyanhu | or we add a DB version migration? | 09:44 |
| Qiming_ | xuhaiwei, we are not using them | 09:44 |
| Qiming_ | we are not sure whether 'cooldown' should be just an in-memory status or a db column, until we complete the scaling policy revision | 09:45 |
| openstackgerrit | Merged openstack/senlin: Revert "Fix action columns in db migration scripts" https://review.openstack.org/267430 | 09:49 |
| Qiming_ | elynn, reverted the previous change to action table | 09:49 |
| xuhaiwei | Qiming_, in fact it really harm things, if these elements remain in DB, when attaching a policy to a cluster, it will fail | 09:49 |
| openstackgerrit | Merged openstack/senlin: Remove 'level', 'priority' and 'cooldown' from DB https://review.openstack.org/267432 | 09:49 |
| xuhaiwei | I am afraid we need to remove them all not only models | 09:50 |
| elynn | Qiming_, I saw it. I will work on another patch to improve scripts. | 09:50 |
| Qiming_ | we will need another patch for removing useless columns, including profile.permission | 09:50 |
| Qiming_ | xuhaiwei, tell me a case why and where things fail | 09:50 |
| xuhaiwei | just made a test: create a policy and a cluster, then attach the policy to the cluster | 09:51 |
| Qiming_ | how it fails? | 09:51 |
| xuhaiwei | wait a minite | 09:52 |
| Qiming_ | do you have this patch? | 09:52 |
| Qiming_ | https://review.openstack.org/266931 | 09:52 |
| Qiming_ | are you helping rework the client side to remove those 'level', 'cooldown', 'priority' properties? | 09:52 |
| Yanyanhu | hi, xuhaiwei, I guess that error is resolved by this patch https://review.openstack.org/267363 | 09:53 |
| xuhaiwei | I have the latest code I think | 09:53 |
| Yanyanhu | I also met it when doing the functional test, but I think it's gone after applying this patch | 09:53 |
| xuhaiwei | I am not starting client fix for this | 09:54 |
| Yanyanhu | error happened during cluster policy attaching | 09:54 |
| Qiming_ | we are not supposed to do functional tests | 09:54 |
| Qiming_ | we are crazy for fixing a lot of things | 09:54 |
| Yanyanhu | Qiming_, I'm trying to test my change on lb_policy | 09:54 |
| Qiming_ | anyone is working on sdk side fixes besides complaining? | 09:54 |
| Yanyanhu | so I run lb_policy functional test separately | 09:54 |
| Qiming_ | Yanyanhu, that may mean something is still not cleansed | 09:55 |
| xuhaiwei | :q | 09:55 |
| Yanyanhu | Qiming_, yes | 09:55 |
| Yanyanhu | but I think that error is gone after 267363 was merged | 09:55 |
| xuhaiwei | that is a quit command | 09:55 |
| Yanyanhu | I also proposed a change for this issue and then abandoned it since it duplicated with 267363 :) | 09:56 |
| Qiming_ | please take a look at the todos | 09:56 |
| Qiming_ | remove level/cooldown from consts | 09:56 |
| Qiming_ | remove level/cooldown from api doc | 09:56 |
| Qiming_ | remove level/cooldown from sdk | 09:56 |
| Qiming_ | remove level/cooldown from client | 09:56 |
| Qiming_ | add rpc test for cluster_policy_attach/detach/ | 09:56 |
| Qiming_ | remove cluster_policy_enable/disable sdk call | 09:56 |
| Qiming_ | remove node_join/leave api | 09:56 |
| Qiming_ | remove node_join/leave from rpc | 09:56 |
| Qiming_ | remove node_join/leave from node_actions | 09:56 |
| Qiming_ | remove node_join/leave from doc | 09:56 |
| Qiming_ | remove node_join/leave from api doc | 09:56 |
| Qiming_ | remove node_join/leave from SDK | 09:56 |
| Qiming_ | remove node_join/leave from client | 09:56 |
| Qiming_ | until all parts of code get cleansed | 09:56 |
| Qiming_ | I'm not interested in any functional test results | 09:56 |
| Yanyanhu | ok, I see | 09:56 |
| Yanyanhu | so will keep on fixing builtin policy issue after the code is cleansed | 09:58 |
| xuhaiwei | :q | 09:58 |
| xuhaiwei | sorry again | 09:58 |
| Qiming_ | never mind | 09:59 |
| Qiming_ | just want to make sure I understand if something is really broken | 09:59 |
| xuhaiwei | I will show you soon | 10:00 |
| Qiming_ | xuhaiwei, have you merged this? https://review.openstack.org/267363 | 10:00 |
| xuhaiwei | http://paste.openstack.org/show/483858/ | 10:01 |
| xuhaiwei | yes, 267363 is also merged | 10:02 |
| Qiming_ | that one is an error from api layer | 10:02 |
| Qiming_ | it hasn't reached the RPC layer yet | 10:02 |
| xuhaiwei | it is not because those elements are still in DB? | 10:03 |
| elynn | Is that because we are not remove them from openstacksdk? | 10:03 |
| Qiming_ | the client and the sdk side are not yet changed, so they are sending in a priority value which is not acceptable any more at api | 10:03 |
| Qiming_ | xuhaiwei, no, the request was blocked by the api layer | 10:04 |
| Yanyanhu | I didn't find error when directly invoking API interface of cluster_policy_attach | 10:04 |
| Qiming_ | I'm planning to do that | 10:04 |
| Qiming_ | but I don't want to propose too many trivial patches to SDK | 10:04 |
| Qiming_ | I was trying to propose a change to SDK after node-join and node-leave is removed | 10:05 |
| Qiming_ | and permission is removed ('hidden') from apis | 10:05 |
| Qiming_ | then with a single patch, we can get the SDK side work right | 10:05 |
| xuhaiwei | ok, I will fix client | 10:05 |
| Qiming_ | then finally we will fix the client | 10:05 |
| xuhaiwei | you fix client? | 10:05 |
| Qiming_ | anyone can do that | 10:06 |
| *** elynn has quit IRC | 10:06 | |
| Qiming_ | there are other things missing if you are looking for something to do, :) | 10:06 |
| Yanyanhu | actually I tried to fix client about an hour ago, but I guessed you guys was also doing that, so I stopped working on it :) | 10:07 |
| Qiming_ | rpc client module, we don't have test cases for cluster_policy_attach/detach | 10:07 |
| Qiming_ | okay, that is a happy pain | 10:07 |
| Qiming_ | we don't have unit tests for the receivers releated calls in senlin/engine/service | 10:08 |
| Qiming_ | Regarding the error you pasted, xuhaiwei, I don't like this error report in its current format | 10:09 |
| Qiming_ | I mean the http://paste.openstack.org/show/483858/ | 10:09 |
| Qiming_ | maybe we can return a more useful information to client instead of a stack trace | 10:09 |
| xuhaiwei | that error report is from API side | 10:10 |
| Qiming_ | so you will see it from client side? | 10:10 |
| xuhaiwei | no | 10:11 |
| Qiming_ | okay | 10:12 |
| Qiming_ | there are some cases we are showing call stack dumps at client side | 10:12 |
| Qiming_ | it is crazy ... | 10:12 |
| Qiming_ | another change proposal, guys | 10:12 |
| xuhaiwei | The server could not comply with the request since it is either malformed or otherwise incorrect. (HTTP 400) (Request- ID: req-49cb9e47-969d-4b0f-8b3e-f5c724ae5cd9) | 10:12 |
| Qiming_ | since we don't have a systematic way for access control | 10:12 |
| xuhaiwei | in client side, I only got this | 10:13 |
| Qiming_ | xuhaiwei, yes, that is not very helpful | 10:13 |
| xuhaiwei | yes | 10:13 |
| Qiming_ | I'm not sure how we can improve that | 10:13 |
| xuhaiwei | I am not sure either | 10:13 |
| Qiming_ | it may imply that for all 400 errors, a client will see this "uniform"/"official" error message | 10:14 |
| xuhaiwei | this error comes from keystoneauth1 | 10:14 |
| Qiming_ | which is, sadly, useless | 10:14 |
| xuhaiwei | every time when we got exceptions from keystoneauth1, it is in bad format | 10:14 |
| Qiming_ | if we parse the exception carefully, the useful message should be inside the exception | 10:14 |
| xuhaiwei | I am doubting keystoneauth1 is making things bad | 10:15 |
| Qiming_ | maybe, maybe not | 10:15 |
| Qiming_ | having a single source of exception, on the other hand, does help us manage exceptions | 10:16 |
| xuhaiwei | keystoneauth1.exceptions.http.BadRequest: The server could not comply with the request since it is either malformed or otherwise incorrect. (HTTP 400) (Request-ID: req-0b2abacb-7617-4f4e-a302-0723aa2c1d6b) | 10:16 |
| Qiming_ | the only problem as I see it is about extracting meaningful message and display them to users | 10:16 |
| xuhaiwei | this is the message when I use '-d' to show the debug details | 10:16 |
| Qiming_ | ... | 10:17 |
| Qiming_ | each HTTPError class has a consructor: | 10:17 |
| Qiming_ | 65 class HttpError(base.ClientException): | 10:17 |
| Qiming_ | 66 """The base exception class for all HTTP exceptions.""" | 10:17 |
| Qiming_ | 67 http_status = 0 | 10:17 |
| Qiming_ | 68 message = "HTTP Error" | 10:17 |
| Qiming_ | 69 | 10:17 |
| Qiming_ | 70 def __init__(self, message=None, details=None, | 10:17 |
| Qiming_ | 71 response=None, request_id=None, | 10:17 |
| Qiming_ | 72 url=None, method=None, http_status=None): | 10:17 |
| Qiming_ | 73 self.http_status = http_status or self.http_status | 10:17 |
| Qiming_ | 74 self.message = message or self.message | 10:17 |
| Qiming_ | 75 self.details = details | 10:17 |
| Qiming_ | 76 self.request_id = request_id | 10:17 |
| Qiming_ | 77 self.response = response | 10:17 |
| Qiming_ | 78 self.url = url | 10:17 |
| Qiming_ | 79 self.method = method | 10:17 |
| Qiming_ | 80 formatted_string = "%s (HTTP %s)" % (self.message, self.http_status) | 10:17 |
| Qiming_ | 81 if request_id: | 10:18 |
| Qiming_ | 82 formatted_string += " (Request-ID: %s)" % request_id | 10:18 |
| Qiming_ | 83 super(HttpError, self).__init__(formatted_string) | 10:18 |
| Qiming_ | can we examine the details field? | 10:18 |
| xuhaiwei | I read this source before | 10:18 |
| xuhaiwei | it's not good I think, I also made a patch to fix it, but they want me to add a test case | 10:18 |
| xuhaiwei | honstly, some exceptions have not good format, so it's difficult to parse it | 10:20 |
| Qiming_ | anyone want to help remove 'permission' from profiles? | 10:29 |
| Qiming_ | it is a similar problem like the properties of a policy | 10:30 |
| xuhaiwei | I can do it, but must do it tomorrow | 10:31 |
| Qiming_ | alright | 10:31 |
| Qiming_ | it will involve a lot of changes | 10:31 |
| Qiming_ | engine, rpc, api, doc | 10:31 |
| Qiming_ | I'll start killing node-join and node-leave | 10:32 |
| xuhaiwei | After I fixed SDK, senlin client, and DB, cluster-policy-attach still failed | 10:32 |
| Qiming_ | hopefully, by tomorrow, we are well situated to propose a single patch to sdk | 10:32 |
| Qiming_ | leaving, very tired a day | 10:34 |
| Qiming_ | bye guys | 10:34 |
| xuhaiwei | bye | 10:34 |
| Yanyanhu | see u tomorrow | 10:34 |
| Yanyanhu | hi, xuhaiwei, you mean use the latest code with fix in senlinclient and sdk, cluster-policy-attach still failed? | 10:35 |
| xuhaiwei | sorry Yanyanhu | 10:37 |
| xuhaiwei | I missed one thing in client | 10:37 |
| xuhaiwei | damn | 10:37 |
| openstackgerrit | Merged openstack/senlin: add debug testenv in tox https://review.openstack.org/266854 | 10:37 |
| *** Qiming_ has quit IRC | 10:39 | |
| Yanyanhu | also prepare to leave, see U guys tomorrow | 10:39 |
| xuhaiwei | see u | 10:39 |
| openstackgerrit | xu-haiwei proposed openstack/python-senlinclient: Remove 'level', 'cooldown' and 'priority' properties in client https://review.openstack.org/267458 | 10:45 |
| *** Yanyanhu has quit IRC | 10:51 | |
| *** dixiaoli has joined #senlin | 11:06 | |
| *** Qiming_ has joined #senlin | 11:30 | |
| *** Qiming_ has quit IRC | 11:30 | |
| *** Qiming_ has joined #senlin | 11:30 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Remove node actions from api layer https://review.openstack.org/267489 | 11:49 |
| *** dixiaoli has quit IRC | 12:01 | |
| *** Qiming_ has quit IRC | 12:14 | |
| *** Qiming_ has joined #senlin | 12:14 | |
| openstackgerrit | Merged openstack/senlin: Revise user docs for policy and bindings changes https://review.openstack.org/267378 | 12:19 |
| openstackgerrit | Merged openstack/senlin: Further cleanse developer doc https://review.openstack.org/267424 | 12:19 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Remove node actions from api layer https://review.openstack.org/267489 | 12:47 |
| openstackgerrit | Merged openstack/senlin: Remove node actions from api layer https://review.openstack.org/267489 | 13:14 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Remove node actions from engine service https://review.openstack.org/267546 | 13:15 |
| openstackgerrit | Merged openstack/python-senlinclient: Revise client to support new sorting option https://review.openstack.org/265497 | 13:26 |
| openstackgerrit | Merged openstack/senlin: Remove node actions from engine service https://review.openstack.org/267546 | 13:52 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Revise docs about node-join and node-leave https://review.openstack.org/267580 | 13:56 |
| openstackgerrit | Merged openstack/senlin: Fix sample placement policies https://review.openstack.org/264773 | 14:00 |
| *** dixiaoli has joined #senlin | 14:12 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Remove 'permission' from profiles https://review.openstack.org/267589 | 14:13 |
| openstackgerrit | Merged openstack/senlin: Revise docs about node-join and node-leave https://review.openstack.org/267580 | 15:20 |
| openstackgerrit | Merged openstack/senlin: Remove 'permission' from profiles https://review.openstack.org/267589 | 15:20 |
| *** pratikmallya has joined #senlin | 15:22 | |
| *** pratikma_ has joined #senlin | 15:31 | |
| *** pratikmallya has quit IRC | 15:34 | |
| *** Qiming_ has quit IRC | 15:38 | |
| *** dixiaoli has quit IRC | 16:26 | |
| *** dixiaoli has joined #senlin | 16:26 | |
| *** dixiaoli has quit IRC | 17:02 | |
| *** pratikma_ has quit IRC | 19:39 | |
| *** pratikmallya has joined #senlin | 19:46 | |
| *** pratikmallya has quit IRC | 20:05 | |
| *** pratikmallya has joined #senlin | 21:06 | |
| *** pratikmallya has quit IRC | 21:07 | |
| *** pratikmallya has joined #senlin | 21:08 | |
| *** pratikma_ has joined #senlin | 21:09 | |
| *** pratikmallya has quit IRC | 21:13 | |
| *** pratikma_ has quit IRC | 21:37 | |
| *** pratikmallya has joined #senlin | 22:17 | |
| *** xuhaiwei has quit IRC | 22:19 | |
| *** pratikmallya has quit IRC | 23:25 | |
| *** Qiming has joined #senlin | 23:46 | |
| *** Qiming has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!