| *** chenhuayi has joined #openstack-smaug | 00:35 | |
| *** zengyingzhe__ has quit IRC | 01:01 | |
| *** zhongjun_ has joined #openstack-smaug | 01:08 | |
| *** zhonghua-lee has quit IRC | 01:09 | |
| *** zhonghua-lee has joined #openstack-smaug | 01:10 | |
| *** zhangshuai has joined #openstack-smaug | 01:16 | |
| openstackgerrit | smile-luobin proposed openstack/smaug: Fix bug in unpack_graph https://review.openstack.org/320732 | 01:33 |
|---|---|---|
| *** pzchen has joined #openstack-smaug | 02:36 | |
| openstackgerrit | chenying proposed openstack/smaug: Fix create checkpoint API errors in protect service https://review.openstack.org/320754 | 03:11 |
| chenying | This patch fix the errors when I test 'protect' action with | 03:12 |
| chenying | volume protection plugins. | 03:12 |
| xiangxinyong | important patch | 03:30 |
| yinwei_computer | Let's get it reviewed | 03:51 |
| yinwei_computer | BTW, could everyone get some time to check nova protection plugin patches? | 03:52 |
| yinwei_computer | https://review.openstack.org/#/c/319158/1 | 03:52 |
| yinwei_computer | https://review.openstack.org/#/c/295618/ | 03:52 |
| *** zhangshuai_ has joined #openstack-smaug | 03:56 | |
| *** pzchen has quit IRC | 03:57 | |
| *** zhangshuai has quit IRC | 04:00 | |
| *** zhangshuai_ has quit IRC | 04:02 | |
| *** zhangshuai_ has joined #openstack-smaug | 04:02 | |
| *** zhangshuai_ has quit IRC | 04:03 | |
| *** zhangshuai_ has joined #openstack-smaug | 04:04 | |
| *** zhangshuai_ has quit IRC | 04:10 | |
| *** zhangshuai_ has joined #openstack-smaug | 04:11 | |
| *** zhangshuai_ has quit IRC | 04:37 | |
| *** zhangshuai_ has joined #openstack-smaug | 04:46 | |
| *** zhangshuai_ has quit IRC | 05:01 | |
| *** zhongjun_ has quit IRC | 05:28 | |
| *** zhongjun_ has joined #openstack-smaug | 05:31 | |
| *** zhurong has joined #openstack-smaug | 06:10 | |
| xiangxinyong | welcome zhurong | 06:11 |
| xiangxinyong | He is a expert from 99cloud. | 06:12 |
| chenzeng | welcome zhurong | 06:14 |
| chenying | welcome. | 06:23 |
| xiangxinyong | zhurong: welcome to give feedback about smaug. Looking forward to you to join smaug | 06:24 |
| *** yuval has joined #openstack-smaug | 06:27 | |
| zhurong | I just a dajiangyou(pingying) :) Thanks xiangxinyong chenzeng chenying | 06:27 |
| chenzeng | zhurong:no, we hope for your good proactice in smaug. | 06:29 |
| chenying | ping yuval | 06:31 |
| openstackgerrit | chenying proposed openstack/smaug: Fix create checkpoint API errors in protect service https://review.openstack.org/320754 | 06:31 |
| yinwei_computer | welcome, zhurong! | 06:32 |
| openstackgerrit | chenying proposed openstack/smaug: Fix create checkpoint API errors in protection service https://review.openstack.org/320754 | 06:35 |
| yuval | ping chenying | 06:40 |
| yuval | welcome zhurong! | 06:40 |
| chenying | Hi yuval https://review.openstack.org/320754 This patch fix the errors testing 'protect' action with | 06:41 |
| chenying | volume protection plugins. | 06:41 |
| chenying | in smaug master branch. | 06:41 |
| yuval | I'm on it | 06:41 |
| chenying | These are another two patches . One is fixing the bank lease errors, luobing works on it. Another one is processing plugins parameters, wangliuan will work on it. | 06:45 |
| yuval | links? | 06:45 |
| chenying | They have not submitted them yes. I will ask them finish them ASAP. | 06:45 |
| chenying | So my patch did not fix these two problems. | 06:47 |
| yuval | ok | 06:47 |
| chenying | I will submit a patch to fix 'resotre' and 'delete' action with volume plugins later. Then, I think some fullstack teses could be submiited. | 06:49 |
| openstackgerrit | smile-luobin proposed openstack/smaug: Add serialize and deserialize function for packed_resource_graph https://review.openstack.org/320804 | 06:59 |
| openstackgerrit | smile-luobin proposed openstack/smaug: Add serialize and deserialize function for packed_resource_graph https://review.openstack.org/320804 | 07:01 |
| yinwei_computer | ping saggi | 07:02 |
| yuval | ping chenying | 07:02 |
| chenying | Hi | 07:02 |
| yuval | yinwei_computer: saggi will not be here today | 07:03 |
| yuval | chenying: what is the 'bank_lease error' you speak about in https://review.openstack.org/#/c/320754/3/smaug/services/protection/checkpoint.py ? | 07:03 |
| chenying | wait a minute | 07:04 |
| openstackgerrit | smile-luobin proposed openstack/smaug: Fix bug in unpack_graph https://review.openstack.org/320732 | 07:06 |
| *** zhangshuai_ has joined #openstack-smaug | 07:08 | |
| openstackgerrit | smile-luobin proposed openstack/smaug: Fix bug in unpack_graph https://review.openstack.org/320732 | 07:08 |
| chenying | Swift bank plugin have this function renew_lease. This method will be called periodically to renew lease form swift. Now this function have bug. It doesn't work. | 07:08 |
| yuval | what do you mean "now"? it did work before | 07:09 |
| yinwei_computer | yuval, saggi gave a critical comment on list checkpoints, I need confirm with him about the requirment | 07:10 |
| yinwei_computer | he told that we won't support combined filters | 07:10 |
| yinwei_computer | if this is the case, the implementation would be much simpler | 07:11 |
| yinwei_computer | but UI and API may need get changed | 07:11 |
| yuval | I suggest you mail him your question | 07:11 |
| yinwei_computer | yuval: will saggi come to office this week? | 07:11 |
| yuval | yinwei_computer: no, afaik | 07:12 |
| yinwei_computer | he checks mail everyday? | 07:12 |
| chenying | yuval: it did work before? I don't think so. I think it never work in smaug master branch. luobin is working on it. | 07:12 |
| *** zhangshuai_ has quit IRC | 07:13 | |
| yuval | chenying: I don't get it. What do you mean by "not working"? Unit tests pass. Where have you found it doesn't work? | 07:13 |
| yuval | yinwei_computer: I think so | 07:13 |
| yuval | yinwei_computer: I'll message him to check, after you'll send your mail | 07:13 |
| yuval | yinwei_computer: (please CC me and Eran as well) | 07:13 |
| yinwei_computer | ok | 07:16 |
| yinwei_computer | thanks, yuval! | 07:16 |
| chenying | yuval: If you start protection service, you will found the POST 'renew lease form swift' print error messages periodically. | 07:16 |
| chenying | messages like this: | 07:16 |
| chenying | 2016-05-25 09:34:42.498 INFO swiftclient [-] REQ: curl -i http://10.229.50.225:8080/v1/AUTH_0fdaee000ffe41adaed13a5d82898361/leases/bf0172a6-d6cd-4a35-b91b-2f9b0129d361 -X POST -H "X-Delete-After: 100" -H "X-Auth-Token: e844c7512cbc4640..." | 07:16 |
| chenying | 2016-05-25 09:34:42.499 INFO swiftclient [-] RESP STATUS: 404 Not Found | 07:16 |
| chenying | 2016-05-25 09:34:42.499 INFO swiftclient [-] RESP HEADERS: {u'Date': u'Wed, 25 May 2016 01:34:42 GMT', u'Content-Length': u'70', u'Content-Type': u'text/html; charset=UTF-8', u'X-Trans-Id': u'txf2223e02ff58485e8ad51-0057450132'} | 07:16 |
| chenying | 2016-05-25 09:34:42.499 INFO swiftclient [-] RESP BODY: <html><h1>Not Found</h1><p>The resource could not be found.</p></html> | 07:16 |
| chenying | 2016-05-25 09:34:42.500 ERROR swiftclient [-] Object POST failed: http://10.229.50.225:8080/v1/AUTH_0fdaee000ffe41adaed13a5d82898361/leases/bf0172a6-d6cd-4a35-b91b-2f9b0129d361 404 Not Found [first 60 chars of response] <html><h1>Not Found</h1><p>The resource could not be found.< | 07:16 |
| chenying | 2016-05-25 09:34:42.500 TRACE swiftclient Traceback (most recent call last): | 07:17 |
| chenying | 2016-05-25 09:34:42.500 TRACE swiftclient File "/usr/local/lib/python2.7/dist-packages/swiftclient/client.py", line 1565, in _retry | 07:17 |
| chenying | 2016-05-25 09:34:42.500 TRACE swiftclient service_token=self.service_token, **kwargs) | 07:17 |
| chenying | 2016-05-25 09:34:42.500 TRACE swiftclient File "/usr/local/lib/python2.7/dist-packages/swiftclient/client.py", line 1324, in post_object | 07:17 |
| chenying | 2016-05-25 09:34:42.500 TRACE swiftclient http_response_content=body) | 07:17 |
| chenying | 2016-05-25 09:34:42.500 TRACE swiftclient ClientException: Object POST failed: http://10.229.50.225:8080/v1/AUTH_0fdaee000ffe41adaed13a5d82898361/leases/bf0172a6-d6cd-4a35-b91b-2f9b0129d361 404 Not Found [first 60 chars of response] <html><h1>Not Found</h1><p>The resource could not be found.< | 07:17 |
| chenying | 2016-05-25 09:34:42.500 TRACE swiftclient | 07:17 |
| chenying | 2016-05-25 09:34:42.501 ERROR smaug.services.protection.bank_plugins.swift_bank_plugin [-] acquire lease failed, err:Connection to swift failed: Object POST failed: http://10.229.50.225:8080/v1/AUTH_0fdaee000ffe41adaed13a5d82898361/leases/bf0172a6-d6cd-4a35-b91b-2f9b0129d361 404 Not Found [first 60 chars of response] <html><h1>Not Found</h1><p>The resource could not be found.<. | 07:17 |
| chenying | Oh sorry so many logs. | 07:17 |
| yinwei_computer | :) flood my screen | 07:18 |
| openstackgerrit | smile-luobin proposed openstack/smaug: Add serialize and deserialize function for packed_resource_graph https://review.openstack.org/320804 | 07:18 |
| yuval | yinwei_computer: chenying: please review https://review.openstack.org/310385 | 07:22 |
| yinwei_computer | sure | 07:23 |
| yuval | chenying: is it possible that there are some parts in "Fix create checkpoint API errors in protection service" patch that are not related to the API errors? | 07:23 |
| yinwei_computer | yuval, forgot to tell, your enhancement work is terrific! | 07:23 |
| yuval | chenying: if so, lets separate them into different patches and get them in quickly | 07:23 |
| yuval | yinwei_computer: thanks :) | 07:23 |
| yinwei_computer | I just gave several trivial comments. For the merge workflow, I think we could hold the merge until CI ready for project stability. what do you think? | 07:25 |
| yuval | yinwei_computer: yes, we should wait until the CI is ready, I agree | 07:28 |
| chenying | "Fix create checkpoint API errors in protection service' this patch is not related to the bank_lease API errors. But I comments some code about bank_lease check. These comments should be canceled when luobin submit the lease fix patch. | 07:28 |
| yinwei_computer | chenying, has smile_luobin got your comment? | 07:31 |
| yinwei_computer | or you could invite him to review your patch | 07:31 |
| chenying | I have invited him to review this patch. | 07:32 |
| yinwei_computer | ok, got it. Thanks! | 07:33 |
| openstackgerrit | zhangshuai proposed openstack/smaug-dashboard: [plan]implement plan detail https://review.openstack.org/318428 | 07:37 |
| openstackgerrit | zhangshuai proposed openstack/smaug-dashboard: [plan]implement plan detail https://review.openstack.org/318428 | 07:52 |
| openstackgerrit | xiangxinyong proposed openstack/smaug: Enable heat in the local conf https://review.openstack.org/320835 | 08:33 |
| openstackgerrit | zhangshuai proposed openstack/smaug: fix requirements.txt for smaug https://review.openstack.org/320853 | 09:28 |
| openstackgerrit | zhangshuai proposed openstack/smaug: fix requirements.txt for smaug https://review.openstack.org/320853 | 09:37 |
| *** zhangshuai_ has joined #openstack-smaug | 09:39 | |
| *** zhurong has quit IRC | 09:41 | |
| *** zhurong has joined #openstack-smaug | 09:42 | |
| *** zhangshuai_ has quit IRC | 09:52 | |
| *** zhangshuai_ has joined #openstack-smaug | 09:56 | |
| openstackgerrit | chenpengzi proposed openstack/smaug-dashboard: fix requirements.txt for smaug-dashboard https://review.openstack.org/320868 | 09:58 |
| openstackgerrit | chenpengzi proposed openstack/smaug-dashboard: fix requirements.txt for smaug-dashboard https://review.openstack.org/320868 | 10:03 |
| *** zhurong has quit IRC | 10:05 | |
| *** zhangshuai_ has quit IRC | 10:13 | |
| *** zhangshuai_ has joined #openstack-smaug | 10:13 | |
| *** chenhuayi has quit IRC | 10:20 | |
| *** zhangshuai_ has quit IRC | 10:30 | |
| *** zhangshuai_ has joined #openstack-smaug | 10:30 | |
| *** zhangshuai_ has quit IRC | 10:36 | |
| *** zhangshuai_ has joined #openstack-smaug | 10:37 | |
| *** openstackgerrit has quit IRC | 11:18 | |
| *** openstackgerrit has joined #openstack-smaug | 11:18 | |
| *** zhangshuai_ has quit IRC | 12:14 | |
| *** zhurong has joined #openstack-smaug | 12:41 | |
| *** wangfeng has quit IRC | 13:12 | |
| *** zhurong has quit IRC | 14:40 | |
| *** yuval has quit IRC | 15:03 | |
| *** chenying has quit IRC | 19:39 | |
| *** chenying has joined #openstack-smaug | 19:40 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!