| *** yinwei_computer has quit IRC | 00:54 | |
| *** yinwei_computer has joined #openstack-smaug | 00:55 | |
| yinwei_computer | hi, guys | 00:58 |
|---|---|---|
| yinwei_computer | we have irc meeting today? | 00:59 |
| yinwei_computer | odd week utc 9:00 am? | 00:59 |
| yinwei_computer | 4:00 pm china time? | 01:00 |
| *** zhurong has joined #openstack-smaug | 01:03 | |
| *** zhangshuai has joined #openstack-smaug | 01:10 | |
| xiangxinyong | it seems the patch has not been merged. | 01:10 |
| xiangxinyong | https://review.openstack.org/#/c/322713/ | 01:10 |
| xiangxinyong | This is even week. and the utc time is 15:00, so it is 23:00 | 01:12 |
| yinwei_computer | today is odd week, right? | 01:28 |
| yinwei_computer | xiangxingyong: today is odd week | 01:28 |
| yinwei_computer | xiangxinyong: but I can understand today there's no meeting in 4:00 pm china time since the patch hasn't merged yet | 01:28 |
| xiangxinyong | yinwei_computer: but how to calculate the 4:00 pm? | 01:37 |
| xiangxinyong | the time 15:00 UTC and 09:00 UTC | 01:38 |
| *** pzchen has joined #openstack-smaug | 01:40 | |
| yinwei_computer | 9:00 utc am is 4:00pm beijing time | 01:44 |
| *** pzchen has quit IRC | 01:59 | |
| *** smile-luobin has joined #openstack-smaug | 02:05 | |
| openstackgerrit | smile-luobin proposed openstack/smaug: Fix bugs for the heat_client create https://review.openstack.org/321983 | 03:13 |
| *** zhangshuai has quit IRC | 03:21 | |
| *** zhangshuai has joined #openstack-smaug | 03:21 | |
| openstackgerrit | zhangshuai proposed openstack/smaug: Fix fullstack tests of the resource plan and provider https://review.openstack.org/322725 | 03:50 |
| *** zhangshuai_ has joined #openstack-smaug | 03:59 | |
| *** zhangshuai has quit IRC | 04:02 | |
| *** zhangshuai_ has quit IRC | 04:04 | |
| *** zhangshuai_ has joined #openstack-smaug | 04:05 | |
| *** zhangshuai_ has quit IRC | 04:26 | |
| *** zhangshuai_ has joined #openstack-smaug | 04:26 | |
| *** zhangshuai_ has quit IRC | 04:34 | |
| *** zhangshuai_ has joined #openstack-smaug | 04:34 | |
| xiangxinyong | yinwei_computer: OK | 06:17 |
| openstackgerrit | zhangshuai proposed openstack/smaug: Fix fullstack tests of the resource plan and provider https://review.openstack.org/322725 | 06:30 |
| *** zhangshuai_ has quit IRC | 06:34 | |
| *** zhangshuai_ has joined #openstack-smaug | 06:34 | |
| *** zhonghua-lee has joined #openstack-smaug | 06:35 | |
| *** yuval has joined #openstack-smaug | 06:36 | |
| *** saggi has joined #openstack-smaug | 07:09 | |
| openstackgerrit | zhangshuai proposed openstack/smaug: Fix fullstack tests of the resource plan and provider https://review.openstack.org/322725 | 07:15 |
| chenying | hi https://review.openstack.org/323458 plz help reviewing this patch. The provider fullstack tests failed because of this bug. | 07:17 |
| chenying | Hi saggi | 07:18 |
| saggi | chenying: hi | 07:25 |
| chenying | How are you? | 07:26 |
| openstackgerrit | zhangshuai proposed openstack/smaug: Fix fullstack tests of the resource plan and provider https://review.openstack.org/322725 | 07:28 |
| *** gampel has joined #openstack-smaug | 07:35 | |
| openstackgerrit | smile-luobin proposed openstack/smaug: Add "sort_dir" parameter for list_objects in BankPlugin https://review.openstack.org/324175 | 07:42 |
| saggi | chenying: Good, how are you? | 07:43 |
| xiangxinyong | saggi; hello | 07:55 |
| chenying | fine. saggi, Could you see whether these patches can be merged? | 07:56 |
| saggi | xiangxinyong: hi | 07:56 |
| chenying | https://review.openstack.org/#/c/325910/ https://review.openstack.org/#/c/321983/ https://review.openstack.org/#/c/322414/ | 07:57 |
| chenying | https://review.openstack.org/#/c/323458/ | 07:58 |
| *** saggi has quit IRC | 07:59 | |
| *** saggi has joined #openstack-smaug | 08:00 | |
| xiangxinyong | saggi: i want to push the youtube video into the mailing list. | 08:00 |
| xiangxinyong | saggi: you introduce smaug in the video. | 08:01 |
| saggi | What do you mean push? | 08:01 |
| xiangxinyong | saggi: i want to push this link to mailing list | 08:02 |
| saggi | Again? | 08:02 |
| saggi | What mailing list? | 08:02 |
| xiangxinyong | saggi: https://www.youtube.com/watch?v=tVYuW_YMB8&feature=youtu.be | 08:03 |
| xiangxinyong | openstack-dev | 08:03 |
| xiangxinyong | saggi: did you send this link into openstack-dev mailing list before? | 08:04 |
| saggi | yes | 08:04 |
| saggi | After the summit | 08:04 |
| xiangxinyong | OK | 08:04 |
| xiangxinyong | sorry I miss your email | 08:04 |
| chenying | Hi yuval saggi The patches above have get some review comments. Hope that you could review these patches. | 08:05 |
| chenying | https://review.openstack.org/#/c/325910/ | 08:06 |
| chenying | https://review.openstack.org/#/c/321983/ | 08:06 |
| chenying | https://review.openstack.org/#/c/322414/ | 08:06 |
| chenying | https://review.openstack.org/#/c/323458 | 08:06 |
| openstackgerrit | Merged openstack/smaug: Expect exceptions in Protection Service https://review.openstack.org/325910 | 08:16 |
| openstackgerrit | Merged openstack/smaug: Fix bugs for the heat_client create https://review.openstack.org/321983 | 08:17 |
| yuval | saggi: chenying: this also https://review.openstack.org/#/c/304212/ | 08:24 |
| openstackgerrit | chenying proposed openstack/smaug: Fix list protectable API error https://review.openstack.org/321329 | 08:36 |
| chenying | Hi yuval saggi This patch have been rebased. I think it is ready for being merged. | 08:38 |
| chenying | https://review.openstack.org/321329 | 08:38 |
| *** yinwei_computer has quit IRC | 08:39 | |
| chenying | Hi yuval about your comment in this patch. https://review.openstack.org/#/c/323161/ | 08:40 |
| yuval | chenying: yes | 08:40 |
| yuval | > | 08:40 |
| yuval | ? | 08:40 |
| chenying | wait for the looping call: | 08:40 |
| chenying | sync_status.wait() I am not sure that it is a must here. | 08:40 |
| openstackgerrit | zhangshuai proposed openstack/smaug: Fix fullstack tests of the resource plan and provider https://review.openstack.org/322725 | 08:41 |
| yuval | otherwise the task will be completed immediately | 08:42 |
| yuval | same goes for protect and restore | 08:42 |
| yuval | zhangshuai_: hey | 08:44 |
| yuval | zhangshuai_: are you here? | 08:44 |
| zhangshuai_ | yes | 08:44 |
| zhangshuai_ | i am here | 08:45 |
| *** pzchen has joined #openstack-smaug | 08:45 | |
| yuval | how are you? :) | 08:46 |
| *** pzchen has left #openstack-smaug | 08:46 | |
| chenying | yuval If you have time, could you help review zhuangshuai's fullstack test. | 08:47 |
| zhangshuai_ | Yes,fine,thank you. | 08:47 |
| yuval | zhangshuai_: could you open a bug about polling in the cleanup ( https://review.openstack.org/#/c/322725/10..11/smaug/tests/fullstack/smaug_base.py ) | 08:47 |
| yuval | chenying: yep, I'm on it | 08:47 |
| yuval | chenying: zhangshuai_: there are also cheyning's comments in: https://review.openstack.org/#/c/323166/ | 08:48 |
| zengchen | ping saggi | 08:51 |
| zengchen | About the patch to apply smaug as a big-tent project, I have 3 comments. | 08:51 |
| zengchen | 1. The 'Backup and Restore' should be changed to 'Protect and Restore'. | 08:51 |
| zengchen | Because if using the 'Backup', it means we can only protect data by backup, | 08:51 |
| zengchen | but actually we can adopt another way, like replication, snapshot etc. | 08:51 |
| zengchen | 2. At present, we use 'project' instead of 'tenant', so consider to use 'project' instead, such as 'tenant-level' etc. | 08:51 |
| zengchen | 3. simple syntax error about 'All smaug code is being code reviewed in OpenStack CI' at 'Open Development' section, | 08:51 |
| zengchen | which should be changed to 'All smaug codes are being reviewed in OpenStack CI'. | 08:51 |
| *** chenpengzi has joined #openstack-smaug | 08:51 | |
| zhangshuai_ | i'll fix this patch after | 08:52 |
| zhangshuai_ | (yuval) https://review.openstack.org/#/c/323166/ | 08:52 |
| chenying | The test could be run in jenkins fullstack job after being reviewed and merged. | 08:53 |
| openstackgerrit | Merged openstack/smaug: The versioned object and db operation of operation_log https://review.openstack.org/304212 | 09:04 |
| yuval | ping zengchen | 09:14 |
| zengchen | yuval:hi | 09:14 |
| yuval | zengchen: hey :) | 09:14 |
| yuval | zengchen: seems like https://review.openstack.org/#/c/323648/ is in merge conflict | 09:14 |
| zengchen | yuval:how are you. | 09:15 |
| yuval | zengchen: very well, you? | 09:15 |
| zengchen | yuval:good, ok, I will update it .thank you. | 09:15 |
| openstackgerrit | zengchen proposed openstack/smaug: Add list function to scheduled operation log https://review.openstack.org/323648 | 09:28 |
| zhangshuai_ | chenying:is checkpoint show and checkpoint delete ok? | 09:28 |
| chenying | checkpoint delete flow need one patch: Implement cinder volume backup deletion | 09:38 |
| *** zhangshuai_ has quit IRC | 09:39 | |
| chenying | checkpoint show need wanglian's one patch. | 09:39 |
| *** zhangshuai_ has joined #openstack-smaug | 09:39 | |
| chenying | hi yuval I do a test about sync_status.wait() if the task can't be completed immediately, the rpcclient of api service will failed because of time out. | 09:49 |
| chenying | : MessagingTimeout: Timed out waiting for a reply to message | 09:49 |
| chenying | You know the sync_status may take several minutes if we have some backups which need be deleted. | 09:49 |
| chenying | So I think the update of status should run in background. the task don't need wait for it. | 09:52 |
| chenying | yuval What's your opinion about it? | 09:53 |
| chenying | There are only two volume backups in my test. | 09:56 |
| *** chenpengzi has quit IRC | 10:04 | |
| *** zhangshuai_ has quit IRC | 10:09 | |
| *** zhangshuai_ has joined #openstack-smaug | 10:10 | |
| *** zhurong has quit IRC | 10:10 | |
| *** zhangshuai_ has quit IRC | 10:11 | |
| *** zhangshuai_ has joined #openstack-smaug | 10:12 | |
| *** zhangshuai_ has quit IRC | 10:17 | |
| *** zhangshuai_ has joined #openstack-smaug | 10:17 | |
| *** zhangshuai_ has quit IRC | 10:38 | |
| *** zhangshuai has joined #openstack-smaug | 10:39 | |
| yuval | chenying: did you try the .wait() in what flow? Restore/backup/delete? | 11:03 |
| openstackgerrit | WeAreFormalGroup proposed openstack/smaug: Add "sort_dir" parameter for list_objects in BankPlugin https://review.openstack.org/324175 | 11:25 |
| openstackgerrit | WeAreFormalGroup proposed openstack/smaug: Implement list checkpoints https://review.openstack.org/310710 | 11:32 |
| *** zengchen has quit IRC | 12:21 | |
| *** zhangshuai has quit IRC | 12:49 | |
| *** zhurong has joined #openstack-smaug | 13:07 | |
| *** gampel has quit IRC | 14:07 | |
| openstackgerrit | Yuval Brik proposed openstack/smaug: Fix stevedore of Protectable Registry https://review.openstack.org/312951 | 14:16 |
| *** yuval has quit IRC | 14:37 | |
| *** gsagie has joined #openstack-smaug | 14:46 | |
| *** gsagie has left #openstack-smaug | 14:46 | |
| *** yuval has joined #openstack-smaug | 14:53 | |
| *** gampel has joined #openstack-smaug | 14:56 | |
| gampel | it is in #openstack-meeting | 15:00 |
| gampel | i see another meeting going around there | 15:01 |
| *** zhurong has quit IRC | 15:03 | |
| yuval | there is no meeting in #openstack-meeting-alt | 15:03 |
| yuval | disregard, there is | 15:03 |
| gampel | meeting-3 is free | 15:04 |
| yuval | #meeting-3 ? | 15:05 |
| yuval | saggi: ? | 15:05 |
| gampel | I mean #openstack-meeting-3 | 15:05 |
| saggi | come to #openstack-meeting-3 | 15:08 |
| yuval | there | 15:08 |
| *** yuval_ has joined #openstack-smaug | 15:29 | |
| *** yuval has quit IRC | 15:29 | |
| *** yuval_ is now known as yuval | 15:29 | |
| *** gampel has left #openstack-smaug | 15:50 | |
| *** saggi has quit IRC | 16:08 | |
| *** yuval has quit IRC | 16:09 | |
| *** saggi has joined #openstack-smaug | 16:33 | |
| *** saggi has quit IRC | 16:49 | |
| *** saggi has joined #openstack-smaug | 20:29 | |
| openstackgerrit | Merged openstack/smaug: Fix wrong definition of schema method https://review.openstack.org/323458 | 20:43 |
| openstackgerrit | Merged openstack/smaug: Fix fullstack tests of the resource plan and provider https://review.openstack.org/322725 | 20:43 |
| *** saggi has quit IRC | 21:00 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!