| *** jruano has quit IRC | 00:10 | |
| *** lkarm has joined #senlin | 00:48 | |
| *** lkarm has quit IRC | 00:52 | |
| *** zhenguo has joined #senlin | 00:57 | |
| *** LiuWei has joined #senlin | 01:01 | |
| *** LiuWei has quit IRC | 01:01 | |
| *** Qiming has joined #senlin | 01:03 | |
| *** lawrance_ has joined #senlin | 01:36 | |
| *** mathspanda has joined #senlin | 01:52 | |
| openstackgerrit | xu-haiwei proposed stackforge/senlin: Handle exception in neutron_v2 module https://review.openstack.org/215051 | 01:54 |
|---|---|---|
| openstackgerrit | Merged stackforge/senlin-dashboard: Updated from global requirements https://review.openstack.org/217966 | 01:58 |
| openstackgerrit | xu-haiwei proposed stackforge/senlin: Handle SDK exception of sdk module https://review.openstack.org/218047 | 02:02 |
| *** elynn has joined #senlin | 02:05 | |
| *** Yanyanhu has joined #senlin | 02:06 | |
| *** lawranc__ has joined #senlin | 02:10 | |
| *** Yanyanhu has quit IRC | 02:11 | |
| *** lawrance_ has quit IRC | 02:13 | |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Add network support for os.nova.server profile https://review.openstack.org/217546 | 02:24 |
| *** Yanyanhu has joined #senlin | 02:25 | |
| Qiming | zhenguo, I was trying the dashboard | 02:37 |
| Qiming | I can see the clusters panel listed once I logged in | 02:37 |
| Qiming | but when I click "clusters", I am getting "connection refused" message | 02:38 |
| *** wizardYVE has joined #senlin | 02:47 | |
| *** lkarm has joined #senlin | 02:48 | |
| Qiming | Yanyanhu, you patch 217546 | 02:49 |
| Yanyanhu | yes? | 02:49 |
| xuhaiwei_ | hi guys, when I tried to create a node, I got this error, is this related to delete the context in drivers layer?? http://paste.openstack.org/show/430388/ | 02:49 |
| Qiming | Don't understand why we are emphasizing this "this property is only available for node creation." | 02:49 |
| Qiming | xuhaiwei_, could be | 02:50 |
| Qiming | please LOG.error() self.conn_params | 02:50 |
| Yanyanhu | Qiming, since user shouldn't specify the fixed-ip or port in profile spec when they want to use this profile to create cluster | 02:50 |
| Yanyanhu | hi, xuhaiwei_, can you also paste your operations to produce this error | 02:51 |
| xuhaiwei_ | create a node | 02:51 |
| Yanyanhu | just create a nova using nova server profile? | 02:51 |
| xuhaiwei_ | by senlin node-create, yanyanhu | 02:51 |
| xuhaiwei_ | yes | 02:51 |
| Qiming | it could be cause by param username or user_name | 02:51 |
| Yanyanhu | hmm, just made serveral tests for other patches this morning | 02:51 |
| Yanyanhu | didn't find this error | 02:52 |
| Qiming | Yanyanhu, got you points | 02:52 |
| Yanyanhu | :) | 02:52 |
| *** lkarm has quit IRC | 02:52 | |
| Yanyanhu | hi, xuhaiwei_ , can you try to rebase the master and try again? | 02:52 |
| Qiming | the message is confusing though | 02:53 |
| xuhaiwei_ | ok | 02:53 |
| xuhaiwei_ | maybe my code is old | 02:53 |
| Yanyanhu | Qiming, umm, yes, not very clear | 02:53 |
| Yanyanhu | ok | 02:53 |
| Qiming | it reads to me like: you can only use these properties when creating a node, not updating a node | 02:54 |
| Qiming | Better we remove that confusing "help" for now | 02:54 |
| Qiming | users would know what they are doing | 02:54 |
| Yanyanhu | ok | 02:54 |
| Yanyanhu | oh we give some hints in example profile spec? | 02:55 |
| Qiming | there are other cases ... for example the server name | 02:55 |
| Yanyanhu | ah, yes | 02:55 |
| Yanyanhu | the same problem | 02:55 |
| Qiming | users will know what they are doing | 02:55 |
| Yanyanhu | ok, will remove them. I believe user should know this | 02:55 |
| Qiming | or we can revisit this when we have a better alternative | 02:56 |
| Yanyanhu | ok | 02:56 |
| zhenguo | Qiming: you are using keystone v3, right? | 02:56 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Add network support for os.nova.server profile https://review.openstack.org/217546 | 02:57 |
| Qiming | zhenguo, yes | 02:58 |
| zhenguo | Qiming: oh sorry, I didn't test v3, it's workd fine with v2 | 02:59 |
| zhenguo | Qiming: as we discussed yesterday, I guess v3 may need to pass project_domain_name or id | 02:59 |
| zhenguo | Qiming: seems project may not unique in multi domains | 03:00 |
| Qiming | from the code, I saw you were passing project_id | 03:00 |
| Qiming | that should be okay | 03:00 |
| Qiming | if you are passing project_name, you will need project_domain_name | 03:00 |
| zhenguo | yeah, but you got connection refused... | 03:00 |
| Qiming | right | 03:00 |
| Qiming | anyway, will continue look into this | 03:00 |
| zhenguo | will check if it's because lost project_domain_id | 03:01 |
| xuhaiwei_ | got the reason | 03:03 |
| xuhaiwei_ | Yanyanhu, I think it's a bug, https://review.openstack.org/#/c/216529/3/senlin/profiles/base.py line204 should be 'user_name' | 03:03 |
| Yanyanhu | hi, xuhaiwei_ , username is the correct key I think | 03:05 |
| Yanyanhu | for sdk | 03:05 |
| Yanyanhu | hmm, let me try again in my env | 03:05 |
| xuhaiwei_ | I mean in this self.context.get('username') | 03:06 |
| xuhaiwei_ | should be username=self.context.get('user_name') | 03:06 |
| xuhaiwei_ | yes, sdk use username | 03:07 |
| Yanyanhu | umm, I remeber this part was also changed | 03:07 |
| Yanyanhu | let me have a check | 03:07 |
| Yanyanhu | did you provide customized context in profile spec? | 03:08 |
| Yanyanhu | e.g. nova server spec | 03:08 |
| xuhaiwei_ | no | 03:08 |
| Yanyanhu | http://git.openstack.org/cgit/stackforge/senlin/tree/senlin/drivers/openstack/keystone_v3.py#n161 | 03:09 |
| Yanyanhu | http://git.openstack.org/cgit/stackforge/senlin/tree/senlin/common/context.py#n122 | 03:09 |
| Yanyanhu | hmm, if no customized context is provided, service_credential/context will be used to initialize profile context | 03:10 |
| Yanyanhu | and the key name should be 'username' now | 03:10 |
| Yanyanhu | although in senlin RequestContext, the keyname is still 'user_name' | 03:11 |
| Qiming | Yanyanhu, https://review.openstack.org/#/c/216529/3/senlin/profiles/base.py | 03:12 |
| Qiming | line 204 after change | 03:12 |
| Qiming | getting "username" from context will fail | 03:12 |
| Yanyanhu | umm, why? | 03:14 |
| Qiming | what is the self.context? | 03:14 |
| Yanyanhu | if no customized context is provided, self.context is context.get_service_context | 03:14 |
| Yanyanhu | otherwise, it will be updated using context field defined in spec | 03:15 |
| Qiming | I see | 03:15 |
| Yanyanhu | hmm, is that possible that haiwei need to recreate profiles? | 03:16 |
| Qiming | yep | 03:17 |
| xuhaiwei_ | so the RequestContext is useless now? | 03:17 |
| xuhaiwei_ | I will recreate it later | 03:17 |
| Yanyanhu | no, xuhaiwei_ , it is still useful in upper layers like engine service, action | 03:18 |
| Yanyanhu | just we removed it from driver layer since it is not needed when senlin talks with other services through sdk | 03:19 |
| Yanyanhu | it just makes thing confusing | 03:19 |
| xuhaiwei_ | thanks Yanyanhu | 03:20 |
| Yanyanhu | so we decided to eliminated it :) | 03:20 |
| Yanyanhu | no problem | 03:20 |
| xuhaiwei_ | after recreating profile, it goes well now | 03:20 |
| xuhaiwei_ | go for lunch now | 03:21 |
| Yanyanhu | ttyl | 03:21 |
| zhenguo | Qiming: around? | 03:38 |
| Qiming | yes | 03:38 |
| Qiming | in call | 03:39 |
| zhenguo | I just test v3, but can't reproduce your error | 03:39 |
| zhenguo | use default domain | 03:40 |
| zhenguo | ok, it's lunch time, ttyl | 03:41 |
| *** lawranc__ has quit IRC | 03:43 | |
| *** wizardYVE has quit IRC | 03:53 | |
| Qiming | okay | 03:59 |
| *** wizardYVE has joined #senlin | 04:17 | |
| *** lkarm has joined #senlin | 05:04 | |
| xuhaiwei_ | Qiming, it seems what we did in this patch is totally wrong https://review.openstack.org/#/c/216139/4 | 05:07 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: Revise response of action_get api interface https://review.openstack.org/218093 | 05:07 |
| xuhaiwei_ | firstly, the original exception like ResourceNotFound returns physical_id back, now we can't not get the physical | 05:09 |
| xuhaiwei_ | physical_id | 05:09 |
| *** lkarm has quit IRC | 05:09 | |
| Qiming | I don't think it is totoally wrong, just something need to improve | 05:10 |
| xuhaiwei_ | Secondly, because we delete .ResourceNotFound and also the error message like 'flavor not found', the error message from SDK is not sufficient, it only returns 'Client Error' | 05:10 |
| openstackgerrit | Yanyan Hu proposed stackforge/python-senlinclient: Add resource_key for Action resource https://review.openstack.org/218095 | 05:10 |
| Qiming | agree we need to do something at the profile layer, not delaying everything to the node layer | 05:11 |
| openstackgerrit | Merged stackforge/senlin: Handle SDK exception of sdk module https://review.openstack.org/218047 | 05:12 |
| xuhaiwei_ | yes | 05:12 |
| Qiming | okay, then we need to revert the profile layer exception handling | 05:13 |
| xuhaiwei_ | and there is another problem, Yanyanhu, can you create a node successfully? | 05:13 |
| xuhaiwei_ | though I provided the right image, the driver still complains 'image not found' | 05:14 |
| Yanyanhu | hi, xuhaiwei_, I think so | 05:16 |
| Yanyanhu | you still have problem creating node? | 05:17 |
| xuhaiwei_ | yes, are you using keystone v3? | 05:17 |
| Yanyanhu | yes | 05:18 |
| Yanyanhu | what error you met | 05:18 |
| xuhaiwei_ | image can't be found | 05:19 |
| xuhaiwei_ | nova driver complains | 05:19 |
| xuhaiwei_ | I am using keystonev2 | 05:19 |
| Yanyanhu | hmm, you can check the detail error info from engine log since currently, the image not found exception will be given for all kinds of error happened in image_get progress | 05:21 |
| Yanyanhu | did this error happen when you used keystone_v3 driver? | 05:22 |
| xuhaiwei_ | yes, I tried just now | 05:22 |
| xuhaiwei_ | it also happened | 05:22 |
| Qiming | cirros 3.2 3.4 ? | 05:23 |
| xuhaiwei_ | no, I created it by myself | 05:23 |
| Yanyanhu | ok, what is the error in engine log? | 05:24 |
| xuhaiwei_ | just a moment | 05:25 |
| Yanyanhu | ok | 05:25 |
| openstackgerrit | xu-haiwei proposed stackforge/senlin: Handle exception in neutron_v2 module https://review.openstack.org/215051 | 05:25 |
| openstackgerrit | Merged stackforge/python-senlinclient: Add resource_key for Action resource https://review.openstack.org/218095 | 05:26 |
| xuhaiwei_ | http://paste.openstack.org/show/430488/ | 05:27 |
| xuhaiwei_ | another problem is caused by this patch https://review.openstack.org/#/c/214448/7/senlin/profiles/os/nova/server.py | 05:28 |
| openstackgerrit | Merged stackforge/senlin: Revise response of action_get api interface https://review.openstack.org/218093 | 05:29 |
| xuhaiwei_ | the try: except should not contain the server_create method, in fact, don't need the try | 05:29 |
| Yanyanhu | hi, xuhaiwei_ , seems error happened during nova server create progress | 05:33 |
| xuhaiwei_ | yes | 05:34 |
| Yanyanhu | about the try except, it is for catching the exception happened during nova server creation and prevent it going to node_action layer directly | 05:34 |
| xuhaiwei_ | why the driver can't find the image | 05:34 |
| Yanyanhu | the node_action will check the return value from profile layer to decide whether the physical resource behind the node is created successfully | 05:34 |
| Yanyanhu | hmm, I guess the image is there | 05:35 |
| xuhaiwei_ | because it is in different project from the image creater | 05:35 |
| xuhaiwei_ | ? | 05:35 |
| Yanyanhu | oh | 05:35 |
| Yanyanhu | that's possible | 05:35 |
| xuhaiwei_ | so the project_domain_name affacts? | 05:36 |
| Yanyanhu | no, just the project takes effect I think | 05:36 |
| Yanyanhu | one user can't see images created by other users in different project | 05:37 |
| Yanyanhu | I guess so | 05:37 |
| xuhaiwei_ | why project is different | 05:37 |
| Yanyanhu | you just said it :) | 05:38 |
| Yanyanhu | 'because it is in different project from the image creater' | 05:38 |
| xuhaiwei_ | so in the context we passed to nova driver, the project_id is wrong? | 05:39 |
| Yanyanhu | hmm, that shouldn't happen since we didn't specify the project id when create connection using sdk | 05:40 |
| Yanyanhu | we just used user name/password and trusts | 05:40 |
| xuhaiwei_ | I have to leave for a meeting ourside the company, maybe investigate this problem at weekends | 05:41 |
| Yanyanhu | sure | 05:41 |
| xuhaiwei_ | thanks Yanyanhu | 05:41 |
| Yanyanhu | any issue, just leave msg | 05:41 |
| Yanyanhu | np :) | 05:41 |
| Qiming | zhenguo, still not able to start dashboard | 06:01 |
| Qiming | ref: http://paste.openstack.org/show/430514/ | 06:01 |
| *** mathspanda has quit IRC | 06:01 | |
| zhenguo | Qiming: ok, seems your env is out of date. | 06:02 |
| zhenguo | Qiming: are you following the readme of senlin-dashboard to run horizon? | 06:03 |
| Qiming | yes | 06:04 |
| zhenguo | Qiming: I'm in a meeting now, will be back soon | 06:13 |
| Qiming | ok | 06:13 |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Change default names for actions https://review.openstack.org/218119 | 06:58 |
| *** wizardYVE has quit IRC | 07:01 | |
| *** mathspanda has joined #senlin | 07:02 | |
| *** mathspanda has quit IRC | 07:11 | |
| *** lawrance_ has joined #senlin | 07:12 | |
| *** lkarm has joined #senlin | 07:20 | |
| *** lkarm has quit IRC | 07:24 | |
| *** mathspanda has joined #senlin | 07:24 | |
| *** mathspanda has quit IRC | 07:30 | |
| *** mathspanda has joined #senlin | 07:48 | |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Change default names for actions https://review.openstack.org/218119 | 07:53 |
| zhenguo | Qiming: there? | 08:04 |
| openstackgerrit | Merged stackforge/senlin: Handle exception in neutron_v2 module https://review.openstack.org/215051 | 08:05 |
| openstackgerrit | Merged stackforge/senlin: Add network support for os.nova.server profile https://review.openstack.org/217546 | 08:05 |
| Qiming | yes, zhenguo | 08:05 |
| zhenguo | Qiming: you can't even login the dashboard as the pasted log | 08:06 |
| Qiming | yes, when I logged in, the browser jumps directly to the error page | 08:06 |
| zhenguo | seems is not related the senlin plugin | 08:07 |
| Qiming | looks like caused by the virtualenv | 08:07 |
| zhenguo | yeas | 08:07 |
| Qiming | there are some variables referenced cannot be found | 08:07 |
| Qiming | I'm thinking if I can copy _50_senlin.py directly to horizon install | 08:08 |
| Qiming | and try it there | 08:08 |
| zhenguo | your horizon is worked now? | 08:08 |
| zhenguo | not the new cloned horizon | 08:08 |
| Qiming | it used to work, where I can see the 'clusters' panel in the UI | 08:09 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: add functional test for cluster scale in/out https://review.openstack.org/218142 | 08:09 |
| Qiming | when I click it, I saw some errors | 08:09 |
| Qiming | then I tried cloning the latest horizon | 08:10 |
| Qiming | seems everything is broken now | 08:10 |
| zhenguo | ok, you can pasted the logs on current horizon | 08:10 |
| Qiming | trying | 08:10 |
| zhenguo | but in the latest horizon, you have run python tools/install_venv.py successfuly? | 08:11 |
| Qiming | yes | 08:12 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: add functional test for cluster scale in/out https://review.openstack.org/218142 | 08:12 |
| Qiming | no luck | 08:12 |
| zhenguo | and runserver with ./run_tests.sh --runserver 0.0.0.0:8080 | 08:12 |
| Qiming | still having this problem: http://paste.openstack.org/show/430637/ | 08:15 |
| * zhenguo checking | 08:16 | |
| zhenguo | Qiming: that's strange. | 08:16 |
| Qiming | even if I remove _50_senlin.py from horizon, I still cannot use horizon now | 08:18 |
| zhenguo | Qiming: I guess you env has some issues now | 08:19 |
| Qiming | right | 08:19 |
| Qiming | maybe browser? | 08:19 |
| zhenguo | Qiming: can you reclone horizon and create venv following the instructions of senlin-dashboard. | 08:19 |
| Qiming | okay | 08:19 |
| Qiming | I'll kill the current horizon instance first? | 08:20 |
| zhenguo | you can use python runserver | 08:20 |
| zhenguo | no need to kill apache server | 08:20 |
| Qiming | okay | 08:21 |
| *** elynn has quit IRC | 08:22 | |
| Qiming | deleting horizon and recloning | 08:22 |
| zhenguo | Qiming: you can copy and paste the command in senlin-dashboard/readme | 08:23 |
| Qiming | okay | 08:24 |
| openstackgerrit | Yanyan Hu proposed stackforge/senlin: add functional test for cluster scale in/out https://review.openstack.org/218142 | 08:24 |
| zhenguo | Qiming: good luck, wait for your feedback, | 08:25 |
| Qiming | cannot login now | 08:34 |
| Qiming | zhenguo, here is what I'm getting | 08:36 |
| Qiming | I'm stuck at the login screen | 08:36 |
| Qiming | with this from command line: http://paste.openstack.org/show/430653/ | 08:36 |
| zhenguo | stuck? still can't login? | 08:38 |
| Qiming | ('Connection aborted.', error(111, 'Connection refused')) | 08:38 |
| Qiming | different now | 08:39 |
| Qiming | wait, maybe its my fault | 08:41 |
| Qiming | \o/\o/ | 08:42 |
| Qiming | finally, I'm seeing the results!!! | 08:43 |
| Qiming | two clusters showed! | 08:43 |
| zhenguo | Qiming: lol | 08:44 |
| Qiming | beautiful, the most beautiful interface I ever saw on horizon!!! | 08:44 |
| Qiming | guys, this is beautiful: http://snag.gy/kx8ix.jpg | 08:47 |
| mathspanda | yes, i like it.:) | 08:49 |
| Yanyanhu | nice :) | 08:51 |
| Yanyanhu | will make a try | 08:51 |
| *** huangtianhua has joined #senlin | 08:55 | |
| Qiming | 热烈欢迎黄师父！ | 08:56 |
| Yanyanhu | ;) | 08:56 |
| zhenguo | huangtianhua: you dedicate to senlin as well? | 08:56 |
| huangtianhua | 我来看看大家 | 08:57 |
| huangtianhua | ：） | 08:57 |
| Qiming | 嗯，大家站好点儿 | 08:57 |
| zhenguo | 哈哈 | 08:57 |
| huangtianhua | 呵呵，同志们辛苦了 | 08:58 |
| Qiming | 为人民服务！ | 08:58 |
| Yanyanhu | 老大摆桌酒吧，hoho | 08:58 |
| zhenguo | 黄core。。。 | 08:58 |
| huangtianhua | 牛core | 08:58 |
| huangtianhua | 。。。。。。 | 08:58 |
| Yanyanhu | :p | 08:58 |
| huangtianhua | 你们都忙啥呢 | 09:01 |
| Qiming | 快乐地修bug | 09:01 |
| Yanyanhu | 有时候也制造bug... | 09:02 |
| Qiming | 然后把这里的东西留给黄师父：http://git.openstack.org/cgit/stackforge/senlin/tree/FEATURES.rst | 09:02 |
| *** mathspanda has quit IRC | 09:05 | |
| zhenguo | senlin 什么时候可以挪到 openstack下面 | 09:06 |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Avoid passing rich object when reschedule https://review.openstack.org/218160 | 09:06 |
| Qiming | 马上吧 | 09:06 |
| Qiming | 人家都催了 | 09:06 |
| zhenguo | 哈哈 赶紧把 | 09:06 |
| Qiming | searching the thread on this | 09:07 |
| Yanyanhu | https://wiki.openstack.org/wiki/Stackforge_Namespace_Retirement | 09:07 |
| Yanyanhu | 赶紧签名 | 09:08 |
| Qiming | 好的 | 09:08 |
| Yanyanhu | 竟然看到了cloud-init。。。这是啥。。。 | 09:08 |
| Qiming | don't know | 09:08 |
| Yanyanhu | 是咱们平时用的那个cloud-init么。。。 | 09:08 |
| Qiming | :) | 09:08 |
| Qiming | refresh that page | 09:09 |
| Yanyanhu | 我顶！ | 09:10 |
| Qiming | see this: http://lists.openstack.org/pipermail/openstack-dev/2015-August/073071.html | 09:10 |
| zhenguo | Qiming: don't forget senlin-dashboard :P | 09:10 |
| Qiming | zhenguo, sure! | 09:10 |
| Qiming | see it yourself | 09:11 |
| Qiming | :) | 09:11 |
| zhenguo | heh | 09:11 |
| Qiming | After that, no further action is required -- the Infrastructure team | 09:11 |
| Qiming | will handle the system configuration changes needed to effect the move ... | 09:11 |
| Qiming | SO NICE! | 09:11 |
| Yanyanhu | I'm curious how will they handle fuel ;) | 09:12 |
| Qiming | yup, still a chaos there | 09:12 |
| zhenguo | a question, moving out of stackforge means that we become an offical project? | 09:12 |
| Yanyanhu | yes | 09:12 |
| zhenguo | graduated? | 09:13 |
| Qiming | zhenguo, no | 09:13 |
| Qiming | we have to get 'senlin' listed here: http://git.openstack.org/cgit/openstack/governance/tree/reference/projects.yaml | 09:14 |
| zhenguo | Qiming: so http://stackalytics.com/ will update to differ these projects from openstack offical ones | 09:15 |
| Qiming | not sure about that | 09:15 |
| Qiming | the big tent idea is to give each project an equal opportunity to grow | 09:16 |
| Qiming | but ... yes, there are company-wise assessment to your contributions | 09:16 |
| zhenguo | yes | 09:16 |
| Qiming | those are about the mindset change needed by those management people | 09:17 |
| zhenguo | especially in huawei :P | 09:17 |
| Qiming | :P | 09:18 |
| Qiming | in IBM too | 09:18 |
| zhenguo | so I have to dedicated to many projects: horizon, ironic, mistral-dashboard ... | 09:19 |
| Qiming | people come and go, companies appear and disappear | 09:20 |
| Qiming | but, long live your code! | 09:20 |
| zhenguo | truth | 09:20 |
| Qiming | what else do we code farmers want? :) | 09:21 |
| zhenguo | no, you are PTL, lol | 09:21 |
| Qiming | ... | 09:22 |
| Qiming | everyone can be | 09:22 |
| Qiming | if he/she wants to make this thing useful, make it better | 09:23 |
| zhenguo | so you mainly focus on senlin now, how to keep heat core member | 09:23 |
| Qiming | I'm not a qualified Heat core now | 09:23 |
| zhenguo | I'm not a qualified Horizon core as well ... | 09:24 |
| Qiming | I am getting less and less interested in those meaningless changes | 09:24 |
| Yanyanhu | making openstack better, this is our goal, no matter which project you are contributing to I think :) | 09:25 |
| Qiming | I used to believe convergence is good, nice to have | 09:25 |
| Qiming | now I'm doubting whether it is the right thing to do | 09:25 |
| Yanyanhu | really hope it can keep growing up and finally become something like 'Linux' | 09:25 |
| Qiming | agreed | 09:26 |
| zhenguo | agreed | 09:26 |
| Qiming | OpenStack may keep you fed, but you company may switch to something else soon, ;) | 09:26 |
| Yanyanhu | yes, that's true | 09:26 |
| zhenguo | but Linux is one project, openstack grows too big to control. | 09:26 |
| Qiming | by the way, I'm very loyal to my company | 09:27 |
| Qiming | that's very true | 09:27 |
| Yanyanhu | haha | 09:27 |
| Yanyanhu | we both are ;) | 09:27 |
| Yanyanhu | yes, openstack is too big now | 09:27 |
| zhenguo | haha | 09:27 |
| Qiming | Yanyanhu, saw your functional tests of scaling | 09:41 |
| Yanyanhu | yes | 09:41 |
| Qiming | I'm wondering if we should separate those into a dedicated module | 09:41 |
| Qiming | one module for cluster basics: create/update/delete/list/show | 09:42 |
| Yanyanhu | hmm, make sense | 09:42 |
| Qiming | one module for cluster membership management: add/delete nodes | 09:42 |
| Qiming | one module for cluster-policy associations: attach/detach/update | 09:42 |
| Qiming | one module dedicated to cluster-resize or cluster-scaling | 09:43 |
| Yanyanhu | ok, so there will be a cluster directory there which is used to maintain these modules | 09:43 |
| Yanyanhu | like tests/functional/cluster/ | 09:44 |
| Qiming | it is okay to spread these modules just under tests/functional at the moment | 09:44 |
| Yanyanhu | ok | 09:44 |
| Qiming | if will be very inconvenient to look into different subdirs for files | 09:44 |
| Qiming | if we find there are too many modules under functional, say over 40 | 09:45 |
| Qiming | it may be the time to consider a reorg of the structure | 09:45 |
| Yanyanhu | got it | 09:46 |
| Qiming | just name the modules as test_cluster_crud test_cluster_membership ... would make it not so difficult to find things you want | 09:46 |
| Yanyanhu | ok | 09:47 |
| Yanyanhu | BTW, I found the functional test failed even for those merged patches for the same reason. I guess there could be some merged change which broke the test. Or some changes happened in devstack gate side | 09:47 |
| Qiming | looking | 09:48 |
| Qiming | seems the exception decorator is failing? invoke_with_catch() takes at least 1 argument (0 given) | 09:50 |
| Yanyanhu | yes | 09:50 |
| Yanyanhu | but not sure why it happened | 09:50 |
| Yanyanhu | didn't find recently changes on it | 09:50 |
| Yanyanhu | ah, I see | 09:55 |
| Qiming | get_user_id was made a class method with translate_exception decorator? | 09:55 |
| Yanyanhu | yes | 09:55 |
| Yanyanhu | but this is ok I think | 09:55 |
| Yanyanhu | I guess the problem is cause by using this decorate on create_connection and authenticate in sdk driver module | 09:56 |
| Yanyanhu | these two methods are not classmethod or instancemethod of a class | 09:56 |
| Yanyanhu | thus the first parameter 'driver' of invoke_with_catch is not given I guess | 09:57 |
| Qiming | yep | 09:58 |
| Qiming | need to revert that change | 09:58 |
| Yanyanhu | em, or give a revision | 09:59 |
| Yanyanhu | I guess maybe we can manually trigger a functional test before each time we merge a patch if it is related to some important workflow | 10:00 |
| Yanyanhu | then we can easily locate the error | 10:00 |
| Qiming | yep | 10:00 |
| openstackgerrit | Merged stackforge/senlin: Fix errors in sample trigger spec and db model https://review.openstack.org/217526 | 10:05 |
| openstackgerrit | Merged stackforge/senlin: Enable triggers for stevedore https://review.openstack.org/217550 | 10:06 |
| *** lawrance_ has quit IRC | 10:06 | |
| openstackgerrit | Merged stackforge/senlin: Fix ceilometer alarm time constraints https://review.openstack.org/217552 | 10:06 |
| *** lawrance_ has joined #senlin | 10:07 | |
| openstackgerrit | Merged stackforge/senlin: Fix policy json file for project checking https://review.openstack.org/217553 | 10:07 |
| openstackgerrit | Merged stackforge/senlin: Forbid deletion of actions in use https://review.openstack.org/217720 | 10:08 |
| openstackgerrit | Merged stackforge/senlin: Change default names for actions https://review.openstack.org/218119 | 10:08 |
| Yanyanhu | will leave, guys, have a good weekend | 10:10 |
| Qiming | you too | 10:10 |
| *** Yanyanhu has quit IRC | 10:14 | |
| *** Qiming has quit IRC | 10:25 | |
| *** zhenguo has quit IRC | 10:40 | |
| *** lawrance_ has quit IRC | 11:07 | |
| *** huangtianhua has quit IRC | 11:35 | |
| *** Qiming has joined #senlin | 11:37 | |
| *** lawrance_ has joined #senlin | 11:55 | |
| *** lawrance_ has quit IRC | 12:19 | |
| *** lkarm has joined #senlin | 12:28 | |
| *** Qiming has quit IRC | 13:08 | |
| *** lkarm has quit IRC | 13:42 | |
| *** lkarm has joined #senlin | 13:42 | |
| *** lkarm has quit IRC | 13:47 | |
| *** lkarm has joined #senlin | 13:59 | |
| openstackgerrit | Qiming Teng proposed stackforge/senlin: Make triggers project-safe https://review.openstack.org/218282 | 14:12 |
| *** Qiming has joined #senlin | 14:15 | |
| *** zhenguo has joined #senlin | 14:22 | |
| openstackgerrit | Merged stackforge/senlin-dashboard: Add clusters table https://review.openstack.org/217793 | 14:41 |
| *** jruano has joined #senlin | 14:43 | |
| openstackgerrit | Zhenguo Niu proposed stackforge/senlin: Improve devstack plugin https://review.openstack.org/218338 | 15:10 |
| *** zhenguo has quit IRC | 15:14 | |
| *** Qiming has quit IRC | 15:25 | |
| *** lkarm has quit IRC | 15:37 | |
| *** lkarm has joined #senlin | 15:37 | |
| *** jruano has quit IRC | 16:07 | |
| *** jruano has joined #senlin | 16:57 | |
| *** lkarm has quit IRC | 17:16 | |
| *** lkarm has joined #senlin | 17:17 | |
| *** lkarm has quit IRC | 17:21 | |
| *** lkarm has joined #senlin | 17:26 | |
| *** lkarm has quit IRC | 17:46 | |
| *** lkarm has joined #senlin | 17:47 | |
| *** lkarm has quit IRC | 17:52 | |
| *** lkarm has joined #senlin | 18:27 | |
| *** lkarm_ has joined #senlin | 18:28 | |
| *** lkarm has quit IRC | 18:31 | |
| *** jruano has quit IRC | 18:48 | |
| *** jruano has joined #senlin | 19:20 | |
| *** lkarm has joined #senlin | 20:21 | |
| *** lkarm_ has quit IRC | 20:24 | |
| *** lkarm_ has joined #senlin | 20:49 | |
| *** lkarm has quit IRC | 20:52 | |
| *** lkarm_ has quit IRC | 21:23 | |
| *** jruano has quit IRC | 21:44 | |
| *** jruano has joined #senlin | 22:16 | |
| openstackgerrit | OpenStack Proposal Bot proposed stackforge/senlin: Updated from global requirements https://review.openstack.org/218524 | 23:49 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
