*** openstack has joined #openstack-smaug | 00:03 | |
*** zhurong has quit IRC | 00:03 | |
*** openstack has joined #openstack-smaug | 00:54 | |
*** zhurong has joined #openstack-smaug | 01:04 | |
*** zhangshuai has joined #openstack-smaug | 01:06 | |
*** yinweiishere has joined #openstack-smaug | 01:10 | |
yinweiishere | is chenying there? | 01:10 |
---|---|---|
yinweiishere | ping chenying | 01:11 |
*** zhurong has quit IRC | 01:35 | |
*** zhurong has joined #openstack-smaug | 01:36 | |
chenying | Hi | 01:36 |
*** chenhuayi has joined #openstack-smaug | 01:37 | |
*** openstackgerrit has quit IRC | 01:49 | |
*** openstackgerrit has joined #openstack-smaug | 01:57 | |
*** pzchen has joined #openstack-smaug | 02:07 | |
yinwei_computer | chenying | 02:36 |
*** openstack has joined #openstack-smaug | 02:38 | |
*** zhonghua-lee has quit IRC | 02:40 | |
*** zhangshuai_ has joined #openstack-smaug | 02:42 | |
*** zhangshuai has quit IRC | 02:42 | |
*** zhonghua-lee has joined #openstack-smaug | 02:43 | |
openstackgerrit | Merged openstack/smaug: Enable heat in the local conf https://review.openstack.org/320835 | 02:44 |
chenying | yinwei_computer: Hi | 02:52 |
yinwei_computer | as we used to meet an issue: resources being protected in plan get deleted anonymously, which will cause protection failure | 02:56 |
yinwei_computer | I mentioned this issue in cinder replication bp review. One of cinder core suggests we propose another bp to discuss this issue and proposed changes. | 02:57 |
openstackgerrit | chenying proposed openstack/smaug: Fix list protectable API error https://review.openstack.org/321329 | 02:58 |
yinwei_computer | I will have wangliuan to propose a bp about open reference counting of cinder resources to other services. | 02:58 |
yinwei_computer | just to let you know, since I have reported this issue as a bug. | 02:59 |
*** zhurong has quit IRC | 02:59 | |
yinwei_computer | what do you think? | 02:59 |
yinwei_computer | chenying? | 02:59 |
*** zhurong has joined #openstack-smaug | 03:00 | |
chenying | That sounds good. open reference counting of cinder resources to other services? Can we add a check before calling "protect" rpc interface? | 03:04 |
*** mingyu has joined #openstack-smaug | 03:05 | |
*** pzchen has quit IRC | 03:06 | |
yinwei_computer | the point is the resource being protected should not be deleted | 03:08 |
yinwei_computer | it could be delete at any time point | 03:09 |
yinwei_computer | during protection | 03:09 |
yinwei_computer | passively checking won't work | 03:09 |
openstackgerrit | smile-luobin proposed openstack/smaug: Add serialize and deserialize functions for resource_graph https://review.openstack.org/320804 | 03:09 |
yinwei_computer | the reasonable procedure should be: first stop protection; then edit plan and remove the resource from the plan; and delete resource at last | 03:11 |
yinwei_computer | same order as delete an in-use volume | 03:11 |
*** mingyu has quit IRC | 03:14 | |
*** pzchen has joined #openstack-smaug | 03:19 | |
chenying | You think this procedure is done in smaug service internally. A user or tenant don't need know the protected resource witch have been deleted. Is that right? | 03:19 |
yinwei_computer | Not sure if I got your point. | 03:21 |
openstackgerrit | chenying proposed openstack/smaug: Fix create checkpoint API errors in protection service https://review.openstack.org/320754 | 03:21 |
*** chenying_ has joined #openstack-smaug | 03:21 | |
yinwei_computer | basically, I think resources in cinder may get referenced by other top layer services, like smaug, or nova | 03:22 |
yinwei_computer | here comes the safe deletion issue: resource being referenced shouldn't be deleted directly by cinder | 03:22 |
yinwei_computer | cinder should keep a reference counting | 03:22 |
yinwei_computer | the problem is how to notify cinder that your volume is referenced by others. we need an API here. | 03:23 |
yinwei_computer | Nova is a special case, which directly changes cinder volume status to in-use to solve this safe deletion issue. | 03:24 |
yinwei_computer | but that's not a generic solution. | 03:24 |
chenying_ | Can we push a bp to cinder to mark this volume being protected. cann't be delted. or must add force parameter and can be delted. | 03:24 |
*** chenying has quit IRC | 03:24 | |
yinwei_computer | it could be | 03:24 |
yinwei_computer | either an additional status field or a reference counting, they are actually the same in functionality. | 03:25 |
chenying_ | IMO, We can look other protject how to solve this problom. like freeze or dragon. | 03:26 |
yinwei_computer | what do you mean by dragon? | 03:26 |
yinwei_computer | flow dragon | 03:26 |
yinwei_computer | yes, definitely | 03:26 |
chenying_ | yes flow dragon | 03:26 |
yinwei_computer | we could check existed ways and propose our bp | 03:26 |
yinwei_computer | it could be a generic API: reference_resource/dereference_resource, or it could be a specific requirement: adding a field for smaug protection | 03:27 |
yinwei_computer | not sure if they're the same case | 03:28 |
*** zhangshuai_ has quit IRC | 03:59 | |
*** pzchen has quit IRC | 04:00 | |
*** smcginnis has quit IRC | 04:22 | |
*** smcginnis has joined #openstack-smaug | 04:22 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/smaug: Updated from global requirements https://review.openstack.org/319891 | 04:42 |
*** zhangshuai has joined #openstack-smaug | 04:44 | |
*** mingyu has joined #openstack-smaug | 06:07 | |
*** chenying_ has quit IRC | 06:19 | |
*** yuval has joined #openstack-smaug | 06:22 | |
*** zhangshuai has quit IRC | 06:29 | |
*** zhangshuai has joined #openstack-smaug | 06:30 | |
*** saggi has joined #openstack-smaug | 06:36 | |
*** chenying has joined #openstack-smaug | 06:39 | |
*** mingyu has quit IRC | 06:44 | |
*** saggi has quit IRC | 06:48 | |
chenying | ping gampel | 06:51 |
*** openstack has joined #openstack-smaug | 06:54 | |
yuval | chenying: hey, posted 2 questions on https://review.openstack.org/320754 | 07:02 |
chenying | yuval: thanks I will address these comments. | 07:03 |
yuval | chenying: regarding the fullstack being 'NOT_REGISTERED', I'll look into it. Meanwhile you can try asking in #openstack-dev | 07:04 |
chenying | OK Thanks. | 07:05 |
yuval | chenying: can you please review https://review.openstack.org/310385 ? | 07:15 |
chenying | I will review it later. | 07:17 |
*** mingyu has joined #openstack-smaug | 07:23 | |
*** mingyu has quit IRC | 07:23 | |
*** mingyu has joined #openstack-smaug | 07:23 | |
yuval | chenying: and this: https://review.openstack.org/#/c/317437/ | 07:39 |
chenying | ok | 07:43 |
*** wangfeng has joined #openstack-smaug | 07:54 | |
*** zhurong_ has joined #openstack-smaug | 07:57 | |
*** zhurong has quit IRC | 08:01 | |
*** zhangshuai has quit IRC | 08:05 | |
*** yuval_b has joined #openstack-smaug | 08:16 | |
openstackgerrit | Yuval Brik proposed openstack/smaug: Fix stevedore of Protectable Registry https://review.openstack.org/312951 | 09:26 |
*** mingyu has quit IRC | 09:31 | |
*** mingyu has joined #openstack-smaug | 09:31 | |
*** zhangshuai has joined #openstack-smaug | 09:33 | |
*** zhangshuai_ has joined #openstack-smaug | 09:53 | |
*** zhangshuai has quit IRC | 09:57 | |
*** zhangshuai_ has quit IRC | 10:06 | |
*** zhurong_ has quit IRC | 10:08 | |
openstackgerrit | wangliuan proposed openstack/smaug: Implement Glance restoration https://review.openstack.org/321454 | 10:13 |
*** zhangshuai_ has joined #openstack-smaug | 10:27 | |
*** zhangshuai_ has quit IRC | 10:32 | |
*** zhangshuai_ has joined #openstack-smaug | 10:32 | |
openstackgerrit | Eran Gampel proposed openstack/smaug: [API] Add name property to Resource model https://review.openstack.org/301778 | 10:39 |
*** chenhuayi has quit IRC | 11:15 | |
openstackgerrit | Merged openstack/smaug: [API] Add name property to Resource model https://review.openstack.org/301778 | 11:31 |
yuval | ping chenying | 11:36 |
chenying | Hi | 11:36 |
yuval | chenying: wanted to ask what is missing for the full tests, so I can start testing the new design | 11:41 |
openstackgerrit | chenying proposed openstack/smaug: Fix create checkpoint API errors in protection service https://review.openstack.org/320754 | 11:42 |
chenying | https://review.openstack.org/320754 If this patch is merged, volume protect can work. volume protect full test will be submitted today or tomorrow. | 11:45 |
yuval | chenying: that's it? :) | 11:46 |
chenying | volume restore need a patch to fix bug. My environment block the verification. I think it will be submitted tomorrow. and with volume restore full test. | 11:48 |
openstackgerrit | Yuval Brik proposed openstack/smaug: Implement the Resource Flow https://review.openstack.org/310386 | 11:50 |
chenying | and volume delete need another patch. I think the base full test have been done. It also can be submmited tomorrow. And then I think you can continue your new design and run fullstack tests in you local environment. | 11:53 |
chenying | Hi Eran I think this patch can be merged. https://review.openstack.org/#/c/320732/ | 11:57 |
gampel | I will look now thx | 11:58 |
*** mingyu has quit IRC | 12:03 | |
openstackgerrit | Yuval Brik proposed openstack/smaug: Implement the Resource Flow https://review.openstack.org/310386 | 12:03 |
openstackgerrit | Merged openstack/smaug: Fix bug in unpack_graph https://review.openstack.org/320732 | 12:04 |
chenying | re-installing a full devstack environment always need a day or two in china because of poor network. It waste a lot of time. :( | 12:06 |
gampel | sorry maybe we need to create cache system in chengdu | 12:08 |
*** chenying has quit IRC | 12:14 | |
*** chenying has joined #openstack-smaug | 12:15 | |
chenying | volume-restore works now in my environment. only need a small fix. | 12:43 |
*** mingyu has joined #openstack-smaug | 12:48 | |
*** zhangshuai_ has quit IRC | 12:49 | |
*** zhangshuai_ has joined #openstack-smaug | 12:49 | |
*** zhangshuai_ has quit IRC | 12:50 | |
*** zhurong has joined #openstack-smaug | 12:51 | |
*** zhangshuai_ has joined #openstack-smaug | 12:52 | |
*** zhangshuai_ has quit IRC | 12:52 | |
*** zhangshuai_ has joined #openstack-smaug | 12:53 | |
*** zhangshuai_ has joined #openstack-smaug | 12:53 | |
*** zhangshuai_ has quit IRC | 12:58 | |
yuval | chenying: great! :) | 12:58 |
yuval | what's missing? | 12:58 |
*** mingyu has quit IRC | 13:10 | |
*** mingyu has joined #openstack-smaug | 13:10 | |
*** zhurong has quit IRC | 14:20 | |
*** zhurong has joined #openstack-smaug | 14:21 | |
*** zhurong has quit IRC | 14:34 | |
*** yuval has quit IRC | 15:01 | |
*** wangfeng has quit IRC | 15:45 | |
*** mingyu has quit IRC | 17:00 | |
*** mingyu has joined #openstack-smaug | 18:01 | |
*** mingyu has quit IRC | 18:06 | |
*** zhurong has joined #openstack-smaug | 23:41 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!