| *** zhenguo has joined #senlin | 00:45 | |
| openstackgerrit | lvdongbing proposed openstack/senlin: Move release note of 'remove-trigger-support' to right location https://review.openstack.org/252121 | 00:59 |
|---|---|---|
| *** pm90_ has joined #senlin | 01:01 | |
| *** Qiming has joined #senlin | 01:02 | |
| openstackgerrit | Merged openstack/senlin: Move release note of 'remove-trigger-support' to right location https://review.openstack.org/252121 | 01:21 |
| *** pm90_ has quit IRC | 01:24 | |
| *** Yanyanhu has joined #senlin | 01:29 | |
| openstackgerrit | junxu proposed openstack/senlin: Store the actual UUID of obj_id in Senlin database for webhook create https://review.openstack.org/251187 | 01:38 |
| openstackgerrit | lvdongbing proposed openstack/senlin: Pass environment variables of proxy to tox https://review.openstack.org/252137 | 01:53 |
| *** elynn has joined #senlin | 01:54 | |
| elynn | morning | 01:55 |
| Qiming | morning | 01:57 |
| Yanyanhu | morning | 01:57 |
| openstackgerrit | Merged openstack/senlin: Pass environment variables of proxy to tox https://review.openstack.org/252137 | 02:15 |
| *** lixinhui_ has joined #senlin | 02:34 | |
| openstackgerrit | xu-haiwei proposed openstack/python-senlinclient: Fix profile update parameters passing error https://review.openstack.org/252153 | 02:40 |
| openstackgerrit | junxu proposed openstack/senlin: Store the actual UUID of obj_id in Senlin database for webhook create https://review.openstack.org/251187 | 02:51 |
| *** Qiming has quit IRC | 03:10 | |
| *** Qiming has joined #senlin | 03:10 | |
| openstackgerrit | Merged openstack/senlin: Store the actual UUID of obj_id in Senlin database for webhook create https://review.openstack.org/251187 | 03:10 |
| *** elynn has quit IRC | 03:55 | |
| *** lixinhui_ has quit IRC | 04:01 | |
| *** jruano has quit IRC | 04:22 | |
| *** yuanying has joined #senlin | 04:42 | |
| *** lixinhui has joined #senlin | 05:01 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Check new_profile before updating node https://review.openstack.org/252188 | 05:02 |
| *** elynn has joined #senlin | 05:06 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Revise action api interfaces used in functional test https://review.openstack.org/252191 | 05:08 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix status code returned from API https://review.openstack.org/251771 | 05:08 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Revise action api interfaces used in functional test https://review.openstack.org/252191 | 05:29 |
| *** elynn has quit IRC | 05:42 | |
| *** elynn has joined #senlin | 05:53 | |
| openstackgerrit | xu-haiwei proposed openstack/python-senlinclient: Fix profile update parameters passing error https://review.openstack.org/252153 | 05:59 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Check new_profile before updating node https://review.openstack.org/252188 | 06:00 |
| *** elynn_ has joined #senlin | 06:04 | |
| *** elynn has quit IRC | 06:04 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Check new_profile before updating node https://review.openstack.org/252188 | 06:10 |
| *** elynn_ has quit IRC | 06:18 | |
| *** elynn has joined #senlin | 06:19 | |
| *** pm90_ has joined #senlin | 06:20 | |
| openstackgerrit | Merged openstack/python-senlinclient: Fix help message error for node_create https://review.openstack.org/251730 | 06:24 |
| openstackgerrit | Merged openstack/senlin: Check new_profile before updating node https://review.openstack.org/252188 | 06:33 |
| *** gongysh has joined #senlin | 06:33 | |
| openstackgerrit | Merged openstack/senlin: Fix status code returned from API https://review.openstack.org/251771 | 06:34 |
| Yanyanhu | hi, Qiming, are you around? | 06:52 |
| Qiming | yes | 06:52 |
| Yanyanhu | I found after applying this patch, the resp code of some API interfaces are still old one when I did some tests | 06:53 |
| Yanyanhu | e.g. node create will return 200 rather than 202 | 06:53 |
| Qiming | yes | 06:54 |
| Yanyanhu | not sure if it's my local env's problem | 06:54 |
| Qiming | read this: https://review.openstack.org/#/c/251771/1/senlin/common/wsgi.py | 06:54 |
| Yanyanhu | oh, I see | 06:55 |
| Qiming | it will take several patches to fix all of the return code issue | 07:00 |
| Qiming | there are cases we will need to customize the header ('Location') field of the Reponse object | 07:00 |
| Yanyanhu | ok | 07:01 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Reorganize documentation https://review.openstack.org/252217 | 07:05 |
| Yanyanhu | hi, Qiming, I think maybe we can change all status code at the first step since the code about 'success' could confuse the user if the actual status_code is different | 07:06 |
| Yanyanhu | or we can leave a comment there | 07:06 |
| Qiming | what do you mean? | 07:07 |
| Qiming | users won't see the 'success' variable | 07:07 |
| *** zhenguo has quit IRC | 07:07 | |
| Qiming | actual status code can vary, it could be a 401, 404, 400 for sure | 07:08 |
| Qiming | what we are changing here is only about the status_code for success | 07:08 |
| Yanyanhu | oh, I mean the developer or some guys who want to check the code by themselves | 07:08 |
| Qiming | success=202 | 07:08 |
| Yanyanhu | yes, I mean the first time people see the code, they could feel xxx_create will return 202 | 07:08 |
| Qiming | this doesn't make sense to you? | 07:08 |
| Yanyanhu | but actual resp code is still 200 | 07:09 |
| Qiming | that is something I'm working on | 07:09 |
| Qiming | they will never see 200 for xxx_create when patches are all in | 07:09 |
| Qiming | they will only see the status_code set to the value of 'success' if the call succeeds | 07:09 |
| Qiming | the actual response code is still 200 because the whole change has not yet landed | 07:10 |
| Yanyanhu | yes, understand. Just feel maybe we should let user know the status_code adjustment is still in progress before all patches are in | 07:10 |
| Qiming | no users will know that | 07:11 |
| Qiming | only developers | 07:11 |
| Qiming | so far it is not impacting any code logic | 07:11 |
| Qiming | I was doing this very very carefully in case it will break existing code | 07:11 |
| Yanyanhu | ok | 07:11 |
| Qiming | at the moment, 'success' is there just as any garbage | 07:12 |
| Qiming | it is not used, why should any developer care? | 07:12 |
| Qiming | as developers ourselves, we know we are in the middle of some changes | 07:12 |
| Yanyanhu | ok, make sense | 07:13 |
| Qiming | I wasn't pushing in more patches in this thread (API revision) because I want to avoid dependencies | 07:16 |
| Qiming | I'm now working on publishing the docs | 07:16 |
| Qiming | when that thing gets straighten up, and this "baby step" is reviewed and merged, I will switch back to this thread | 07:17 |
| *** pm90_ has quit IRC | 07:17 | |
| Yanyanhu | no problem. publishing doc should be the one have higher priority | 07:18 |
| Qiming | I have to switch between several threads ... | 07:19 |
| Yanyanhu | yea, we really need more hands... | 07:19 |
| Qiming | still cannot find a guide how to publish docs | 07:21 |
| Qiming | sigh ... | 07:21 |
| Yanyanhu | I found only less than 10 projects have doc published here http://developer.openstack.org/api-ref-compute-v2.1.html | 07:23 |
| Yanyanhu | is there any limitation? | 07:23 |
| Qiming | no | 07:23 |
| Yanyanhu | oh, 11 ones | 07:23 |
| Yanyanhu | but I saw Manila's doc | 07:23 |
| Qiming | okay, we are talking about different things again | 07:24 |
| Yanyanhu | it's also new for openstack | 07:24 |
| Qiming | you are talking about API docs | 07:24 |
| Qiming | I'm talking about developer docs | 07:24 |
| Yanyanhu | ohh | 07:24 |
| Yanyanhu | about doc for developers from different openstack projects, didn't see anything officially before... | 07:26 |
| Qiming | http://docs.openstack.org/developer/openstack-projects.html | 07:28 |
| Yanyanhu | wow | 07:29 |
| Yanyanhu | nice | 07:29 |
| openstackgerrit | Merged openstack/python-senlinclient: Fix profile update parameters passing error https://review.openstack.org/252153 | 07:29 |
| openstackgerrit | Merged openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/251668 | 07:29 |
| Qiming | in additional to understand how to organize the docs, how to publish them | 07:31 |
| Qiming | I need to find out how to get 'Clustering' doc listed here | 07:31 |
| Yanyanhu | understand | 07:32 |
| Qiming | asking questions on openstack-api channel is a waste of time | 07:36 |
| Qiming | no one is answering questions there | 07:36 |
| Yanyanhu | ... | 07:36 |
| elynn | Find an old bug to publish docs, maybe it could help you. https://bugs.launchpad.net/python-stevedore/+bug/1374073 | 07:42 |
| openstack | Launchpad bug 1374073 in stevedore "need to publish docs to docs.openstack.org/developers" [High,Fix released] - Assigned to Doug Hellmann (doug-hellmann) | 07:42 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: [Tentative]Disable multi-engine to debug functional test https://review.openstack.org/252231 | 07:42 |
| elynn | And a new merged patch to publish docs. https://github.com/openstack-infra/project-config/commit/749e0576aa313eed11991ba7b6e89d1c11da0b59 | 07:43 |
| Qiming | not sure if this is all that is required: openstack-server-publish-jobs | 07:45 |
| Qiming | there are also openstack-publish-jobs | 07:45 |
| elynn | https://review.openstack.org/#/c/251292/ | 07:45 |
| Qiming | there are jobs about gating | 07:45 |
| Qiming | .... | 07:45 |
| elynn | Submit a patch to project-config first and I think other cores will help you review ;) | 07:47 |
| Qiming | yes, that is the last resort | 07:47 |
| Qiming | not sure if this is it: | 07:48 |
| Qiming | https://review.openstack.org/252230 | 07:48 |
| elynn | I think openstack-server-publish-jobs and python-jobs need to add to layout.yaml and openstack-publish-jobs need to add to projects.yaml | 07:48 |
| elynn | I guess... | 07:49 |
| Qiming | then that is all | 07:49 |
| Qiming | check patch #252230 | 07:49 |
| elynn | Just saw it, let's wait for jenkins... | 07:50 |
| Qiming | this | 08:18 |
| Qiming | is | 08:18 |
| Qiming | gonna | 08:18 |
| Qiming | be | 08:18 |
| Qiming | BIG | 08:18 |
| Qiming | https://review.openstack.org/252245 | 08:18 |
| Yanyanhu | @_@ | 08:19 |
| Yanyanhu | It's HUGE! | 08:19 |
| Qiming | the whole API | 08:20 |
| Qiming | there are things to be changed too | 08:20 |
| xuhaiwei | this is your excellent job this afternoon? | 08:20 |
| Qiming | not excellent at all | 08:20 |
| xuhaiwei | don't want to review it :( | 08:20 |
| elynn | Qiming: you are superman! | 08:20 |
| Qiming | things to do regarding the api doc | 08:22 |
| xuhaiwei | should we put it there? | 08:23 |
| xuhaiwei | the location | 08:23 |
| Qiming | 1) version_info response is not an object 2) response status codes are not accurate 3) some response bodies will be changed | 08:23 |
| Qiming | that is the central location for all api refs | 08:24 |
| Qiming | the result will be published here: http://developer.openstack.org/api-ref.html | 08:24 |
| xuhaiwei | when having this samples, it's easy to judge whether the api has bugs | 08:25 |
| Qiming | there is a huge effort to migrate doc composition from WADL to RST, using Swagger | 08:25 |
| Qiming | but I'm not sure when that will land | 08:25 |
| Qiming | if the api-site has started to block any new api docs, at least they will tell us where is the 'new location' | 08:26 |
| Qiming | the current documentation has some defects | 08:29 |
| Qiming | we are not clearly documenting the status_code for failures | 08:29 |
| openstackgerrit | Shu Muto proposed openstack/python-senlinclient: Remove py26 support https://review.openstack.org/252252 | 08:30 |
| xuhaiwei | in fact, I am a little afraid of that | 08:30 |
| Qiming | there is another proposal here: http://git.openstack.org/cgit/openstack/api-wg/tree/guidelines/api-docs.rst | 08:31 |
| Qiming | it is not preventing us from adding the existing api docs to api-site | 08:32 |
| Qiming | although in future, a reST/YAML style API doc is preferred | 08:33 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Webhook creation refactor https://review.openstack.org/252258 | 08:38 |
| Yanyanhu | hi, xuhaiwei, around? | 08:42 |
| Yanyanhu | I remember I said you met an error like this one when creating cluster? | 08:42 |
| Yanyanhu | cluster_create_4909a4b5[2e1e61b7-cc05-430c-9217-d8aead4b232f] CLUSTER_CREATE - FAILED: CLUSTER_CREATE [2e1e61b7] timeout | 08:42 |
| Yanyanhu | did you know how to reproduce it. | 08:43 |
| Yanyanhu | I found this error happened in gate side | 08:43 |
| openstackgerrit | Merged openstack/senlin: Revise action api interfaces used in functional test https://review.openstack.org/252191 | 08:43 |
| Yanyanhu | but just can't reproduce it locally | 08:44 |
| xuhaiwei | hi Yanyanhu | 08:44 |
| xuhaiwei | yes, that's what I met yesterday | 08:45 |
| xuhaiwei | that happened when I tried to create a cluster, but after the Error happened, the cluster created soon | 08:46 |
| Yanyanhu | so you just met this error once or it always happened | 08:46 |
| xuhaiwei | let me have a try now | 08:46 |
| Yanyanhu | ok | 08:46 |
| xuhaiwei | I created 3 clusters, neither has this error | 08:48 |
| xuhaiwei | none of them | 08:48 |
| Yanyanhu | sigh... | 08:49 |
| Yanyanhu | difficult to locate this error | 08:49 |
| Yanyanhu | will wait for the functional test result in gate side | 08:49 |
| Yanyanhu | thanks | 08:50 |
| xuhaiwei | maybe a bug, maybe not | 08:51 |
| xuhaiwei | the default timeout value is how long? | 08:55 |
| xuhaiwei | should be longer than we see? | 08:55 |
| *** lixinhui has quit IRC | 08:57 | |
| Yanyanhu | xuhaiwei, it's 3600 seconds | 08:57 |
| xuhaiwei | so timeout should not happen? | 08:57 |
| xuhaiwei | that is a bug | 08:58 |
| elynn | Maybe race condition? | 08:58 |
| Yanyanhu | so I guess could be some bugs when calculating action lapsed time | 08:58 |
| Yanyanhu | elynn, you mean? | 08:58 |
| elynn | Is that because we have multi workers and some kind of race when acquire resources? | 08:59 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Webhook creation refactor https://review.openstack.org/252258 | 09:00 |
| elynn | Then a deadlock happened? | 09:00 |
| Yanyanhu | hmm, I disabled multi-engine tentatively when executing functional test | 09:00 |
| Yanyanhu | so theoretically, that shouldn't happen... | 09:01 |
| Qiming | I'd suggestion nailing down the problem step by step | 09:01 |
| Qiming | first of all, I am seeing a lot profile still in use errors | 09:01 |
| Qiming | that means the functional test is not structured properly | 09:01 |
| Qiming | every test_xyz methods are in theory executed in parallel | 09:01 |
| Yanyanhu | Qiming, that is because profile can't be cleaned correctly since they are still being used | 09:01 |
| Qiming | why they are still in use? | 09:02 |
| Yanyanhu | since cluster created failed and cluster is in error status | 09:02 |
| Qiming | if the cluster is in error status, why attempt to delete profile? | 09:02 |
| Yanyanhu | so related profile can't be deleted as expected when a test case is done | 09:02 |
| xuhaiwei | do you ever meet this situation that every single test case is succeeded, but when test them all, some of them will fail ? | 09:03 |
| Qiming | xuhaiwei, yes | 09:03 |
| Qiming | that happened a long time ago in Heat | 09:03 |
| Yanyanhu | hmm, currently, profile creation and deletion are put into setup and teardown method of test class | 09:03 |
| xuhaiwei | that's mock's problem? | 09:03 |
| Qiming | the fix was to give each temporary stack a dedicated name | 09:03 |
| xuhaiwei | I met this problem in senlin client test | 09:03 |
| Qiming | or else some test cases will try delete stacks created by other test cases | 09:04 |
| Yanyanhu | but based on the log of devstack-gate, all functional test are run serially | 09:04 |
| Yanyanhu | no in parallel | 09:05 |
| Yanyanhu | not | 09:05 |
| Yanyanhu | hmm, I saw the log in both senlin-api and engine | 09:06 |
| Qiming | ... | 09:09 |
| Qiming | maybe the logs are concatenated? | 09:09 |
| Qiming | don't ever trust I/O results | 09:09 |
| Qiming | those things are buffered at every layers | 09:09 |
| Yanyanhu | hmm, that is possible. I think a thing need to do is given a random name for all resources created during test progress | 09:10 |
| Yanyanhu | currently, the same name are using in different test cases | 09:10 |
| Yanyanhu | the second time test passed | 09:11 |
| Yanyanhu | hi, xuhaiwei, just saw your patch about webhook rework https://review.openstack.org/#/c/252258 | 09:20 |
| xuhaiwei | yes | 09:20 |
| Yanyanhu | I think incorrect service could be returned if only service type is used to filter the list result. | 09:20 |
| Yanyanhu | since Keystone allows creating multiple services with the same service_type | 09:20 |
| Yanyanhu | they give an example about this in API doc | 09:21 |
| Yanyanhu | this place https://review.openstack.org/#/c/252258/2/senlin/drivers/openstack/keystone_v3.py, line 69 | 09:21 |
| xuhaiwei | yes? | 09:22 |
| xuhaiwei | from my test, the 'services' contains all the services | 09:22 |
| xuhaiwei | and each one has this format: attrs={u'description': u'OpenStack Telemetry Service', u'links': {u'self': u'http://192.168.11.81:5000/v3/services/6bcb49a47b7b41e581c9f3516077bc0d'}, u'type': u'metering', u'enabled': True, u'id': u'6bcb49a47b7b41e581c9f3516077bc0d', u'name': u'ceilometer' | 09:23 |
| xuhaiwei | this is for ceilometer | 09:23 |
| Yanyanhu | yes, that's because in our current env, each service has unique service_type | 09:23 |
| Yanyanhu | but we can't guarantee that I guess | 09:24 |
| Yanyanhu | you know that Monasca doesn't mark their own service type to 'metering' to avoid conflict with ceilometer | 09:24 |
| xuhaiwei | you mean the 'type' could not be unique? | 09:24 |
| Yanyanhu | yes | 09:24 |
| Yanyanhu | e.g. two different services in Openstack are all 'metering' service, they compete with each other | 09:25 |
| xuhaiwei | so we may need to test 'type' and 'name'? | 09:25 |
| Yanyanhu | yes, I guess so | 09:25 |
| xuhaiwei | ok, that makes sense | 09:25 |
| xuhaiwei | I will fix it later | 09:25 |
| Yanyanhu | thanks, other parts are ok for me | 09:26 |
| *** zhangguoqing has joined #senlin | 09:30 | |
| Qiming | Yanyanhu, I don't thing there are two metering services in OpenStack | 09:36 |
| Qiming | s/thing/think/ | 09:36 |
| Qiming | a more authorative information source would be keystoneauth | 09:37 |
| Qiming | maybe we need to check their assumptions on this, instead of being unnecessarily cautious | 09:37 |
| Yanyanhu | yes, currently only ceilomter's service type is metering, Monasca's service type is monitoring | 09:38 |
| Yanyanhu | agree that we should get the answer from the community | 09:38 |
| Qiming | in real openstack deployments, if there are more than one services bearing the same type, it would cause a lot trouble | 09:39 |
| Qiming | for example, in Heat, in Murano, ... | 09:39 |
| Yanyanhu | yes. | 09:40 |
| Qiming | another argument is that openstack sdk is talking to services using service type | 09:40 |
| Yanyanhu | Just beacuse keystone allows to do this, so I guess we need to be aware of this possible issue. | 09:41 |
| Qiming | if there are two services using the same type, it will break in a minute | 09:41 |
| xuhaiwei | currently when try to get keystone service, only 'identity' type is provided | 09:41 |
| Qiming | because it requires an admin role? | 09:42 |
| xuhaiwei | {'service_type': 'identity', 'interface': 'public', 'version': None, 'region_name': None, 'service_name': None} | 09:43 |
| xuhaiwei | only these information | 09:43 |
| Yanyanhu | so I guess we'd better get an answer from keystone or the community | 09:44 |
| Qiming | xuhaiwei, you may need an admin role to get the full list | 09:44 |
| Qiming | xuhaiwei, check this: http://git.openstack.org/cgit/openstack/keystone/tree/etc/policy.json#n19 | 09:45 |
| Qiming | although I think that is stupid | 09:45 |
| xuhaiwei | so when creating a webhook, we have a 'admin' role? | 09:46 |
| Yanyanhu | (openstack) service list | 09:46 |
| Yanyanhu | +----------------------------------+------------+----------------+ | 09:46 |
| Yanyanhu | | ID | Name | Type | | 09:46 |
| Yanyanhu | +----------------------------------+------------+----------------+ | 09:46 |
| Yanyanhu | | 11af2389070449f98f294c5cf2446a5f | service1 | compute | | 09:46 |
| Yanyanhu | | 4a203daa7df745f2bbb21bad0fe4302e | service2 | compute | | 09:46 |
| Yanyanhu | I created two fake compute services in my local env | 09:47 |
| xuhaiwei | that's true, you have two compute node, i think | 09:47 |
| Yanyanhu | just used openstackclient to do this test | 09:49 |
| xuhaiwei | anyway, I made a mistake in that patch | 09:50 |
| xuhaiwei | the exception handling logic is wrong | 09:50 |
| xuhaiwei | we should keep the original type | 09:50 |
| xuhaiwei | sorry for that | 09:50 |
| Qiming | Yanyanhu, then you try heat and see if it is possible to run a thing? | 09:50 |
| Qiming | today, most service catalog discovery is done based solely on service_type | 09:52 |
| Yanyanhu | yes, that's true | 09:53 |
| xuhaiwei | I want to go home | 09:57 |
| Yanyanhu | prepare to leave too | 09:57 |
| Yanyanhu | ttyl | 09:57 |
| Qiming | lol | 09:57 |
| xuhaiwei | so the conclution is? | 09:58 |
| Yanyanhu | maybe throw a question to mailing list? | 09:58 |
| Qiming | I'm okay with the name as an additional filter | 09:58 |
| Qiming | but I'm curious how useful that would be | 09:58 |
| Yanyanhu | or add a comment here to delete it someday? | 09:58 |
| Qiming | recall that the other day we are trying to get a 'compute' service named 'senlin' | 09:59 |
| Qiming | don't know why we ended up such a weird condition | 09:59 |
| Yanyanhu | :) | 09:59 |
| Yanyanhu | I guess the filter didn't work as expected? | 10:00 |
| Yanyanhu | since it is passed to sdk's 'services' function call | 10:00 |
| xuhaiwei | the original filter didn't work , so I fixed it | 10:00 |
| Qiming | yes, my feeling was that we are introducing some problems to fix | 10:01 |
| Qiming | but I'm fine with it if we are also solving this at the sdk side | 10:01 |
| xuhaiwei | in my opinion, the service_type should be unique in the whole openstack environment | 10:02 |
| xuhaiwei | but making a double check is also not wrong | 10:03 |
| Yanyanhu | maybe some bugs in sdk side | 10:03 |
| Yanyanhu | anyway, will make a test on services function call | 10:03 |
| Yanyanhu | using those fakes services :) | 10:03 |
| Qiming | it was really a torture --> https://review.openstack.org/#/c/252245/ | 10:06 |
| Qiming | there are so many unnecessary json format checks | 10:06 |
| Yanyanhu | ... | 10:07 |
| Qiming | I tried that, Yanyanhu | 10:07 |
| Qiming | http://paste.openstack.org/show/480597/ | 10:08 |
| Qiming | that thing worked | 10:08 |
| Qiming | so I am not strongly against adding service name check | 10:08 |
| Qiming | I was simply saying that it may be not necessary, and it seems revealing bugs in sdk | 10:09 |
| Yanyanhu | ok, actually I also tested it locally before adding that code | 10:09 |
| Yanyanhu | yes, so maybe we just add TODO for removing it in future | 10:09 |
| Qiming | \o/ | 10:10 |
| xuhaiwei | the patch is coming | 10:10 |
| Yanyanhu | I'm also curious whether TC will allow the service type conflict if a new project propose application for joining bigtent :P | 10:10 |
| Qiming | patch #252245 passed! | 10:10 |
| Yanyanhu | con! | 10:10 |
| Qiming | the first checklinks gate is not caused by this patch | 10:10 |
| Yanyanhu | that means one project are yelling to another project 'I'm your competitor' :) | 10:11 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Webhook creation refactor https://review.openstack.org/252258 | 10:14 |
| xuhaiwei | since OpenStack is a whole family, it should not allow two project has the same type | 10:14 |
| Yanyanhu | :P | 10:15 |
| Yanyanhu | I also think so | 10:15 |
| Yanyanhu | I think the follower won't choose the same service type as an existing service | 10:16 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Reorganize documentation https://review.openstack.org/252217 | 10:16 |
| xuhaiwei | ok, time to go | 10:17 |
| xuhaiwei | see you tomorrow | 10:17 |
| Qiming | me too | 10:17 |
| Qiming | leaving | 10:17 |
| Yanyanhu | will leave too | 10:18 |
| Yanyanhu | see U guys | 10:18 |
| *** Qiming has quit IRC | 10:22 | |
| *** Yanyanhu has quit IRC | 10:23 | |
| *** elynn has quit IRC | 10:24 | |
| *** yuanying has quit IRC | 10:57 | |
| *** Qiming has joined #senlin | 11:28 | |
| *** lixinhui_ has joined #senlin | 11:30 | |
| *** Qiming has quit IRC | 11:34 | |
| *** pm90_ has joined #senlin | 12:10 | |
| *** lixinhui_ has quit IRC | 12:32 | |
| *** Qiming has joined #senlin | 12:56 | |
| openstackgerrit | Merged openstack/senlin: Webhook creation refactor https://review.openstack.org/252258 | 13:08 |
| *** Qiming has quit IRC | 13:08 | |
| *** elynn has joined #senlin | 13:41 | |
| openstackgerrit | Merged openstack/python-senlinclient: Remove py26 support https://review.openstack.org/252252 | 13:44 |
| *** pm90__ has joined #senlin | 13:48 | |
| *** pm90_ has quit IRC | 13:50 | |
| *** pm90__ has quit IRC | 14:35 | |
| *** elynn has quit IRC | 14:42 | |
| openstackgerrit | Merged openstack/senlin: Add more configurable parameters to wait_for_server https://review.openstack.org/251623 | 14:53 |
| *** pm90_ has joined #senlin | 15:00 | |
| *** pm90__ has joined #senlin | 15:02 | |
| *** pm90_ has quit IRC | 15:06 | |
| *** pm90__ is now known as pratikmallya | 15:12 | |
| *** zhangguoqing has quit IRC | 15:34 | |
| *** xuhaiwei has quit IRC | 15:34 | |
| *** gongysh has quit IRC | 15:34 | |
| *** pratikmallya has quit IRC | 15:34 | |
| *** jdandrea has quit IRC | 15:35 | |
| *** zhangguoqing has joined #senlin | 15:40 | |
| *** gongysh has joined #senlin | 15:42 | |
| *** xuhaiwei has joined #senlin | 15:42 | |
| *** pratikmallya has joined #senlin | 15:59 | |
| *** pratikmallya has quit IRC | 16:09 | |
| *** pratikmallya has joined #senlin | 16:34 | |
| *** pratikmallya has quit IRC | 16:45 | |
| *** pratikmallya has joined #senlin | 16:46 | |
| *** pratikmallya has quit IRC | 17:06 | |
| *** rebase_ has joined #senlin | 18:20 | |
| *** pratikmallya has joined #senlin | 19:06 | |
| *** jdandrea has joined #senlin | 19:59 | |
| *** jdandrea has quit IRC | 21:54 | |
| *** jdandrea has joined #senlin | 22:13 | |
| *** pratikmallya has quit IRC | 22:38 | |
| *** yuanying has joined #senlin | 23:00 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Make asynchronous action return 202 https://review.openstack.org/251669 | 23:41 |
| openstackgerrit | Merged openstack/senlin: Reorganize documentation https://review.openstack.org/252217 | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!