Thursday, 2016-05-26

*** openstack has joined #openstack-smaug00:03
*** zhurong has quit IRC00:03
*** openstack has joined #openstack-smaug00:54
*** zhurong has joined #openstack-smaug01:04
*** zhangshuai has joined #openstack-smaug01:06
*** yinweiishere has joined #openstack-smaug01:10
yinweiishereis chenying there?01:10
yinweiishereping chenying01:11
*** zhurong has quit IRC01:35
*** zhurong has joined #openstack-smaug01:36
chenyingHi01:36
*** chenhuayi has joined #openstack-smaug01:37
*** openstackgerrit has quit IRC01:49
*** openstackgerrit has joined #openstack-smaug01:57
*** pzchen has joined #openstack-smaug02:07
yinwei_computerchenying02:36
*** openstack has joined #openstack-smaug02:38
*** zhonghua-lee has quit IRC02:40
*** zhangshuai_ has joined #openstack-smaug02:42
*** zhangshuai has quit IRC02:42
*** zhonghua-lee has joined #openstack-smaug02:43
openstackgerritMerged openstack/smaug: Enable heat in the local conf  https://review.openstack.org/32083502:44
chenyingyinwei_computer: Hi02:52
yinwei_computeras we used to meet an issue: resources being protected in plan get deleted anonymously, which will cause protection failure02:56
yinwei_computerI 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
openstackgerritchenying proposed openstack/smaug: Fix list protectable API error  https://review.openstack.org/32132902:58
yinwei_computerI will have wangliuan to propose a bp about open reference counting of cinder resources to other services.02:58
yinwei_computerjust to let you know, since I have reported this issue as a bug.02:59
*** zhurong has quit IRC02:59
yinwei_computerwhat do you think?02:59
yinwei_computerchenying?02:59
*** zhurong has joined #openstack-smaug03:00
chenyingThat 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-smaug03:05
*** pzchen has quit IRC03:06
yinwei_computerthe point is the resource being protected should not be deleted03:08
yinwei_computerit could be delete at any time point03:09
yinwei_computerduring protection03:09
yinwei_computerpassively checking won't work03:09
openstackgerritsmile-luobin proposed openstack/smaug: Add serialize and deserialize functions for resource_graph  https://review.openstack.org/32080403:09
yinwei_computerthe reasonable procedure should be: first stop protection; then edit plan and remove the resource from the plan; and delete resource at last03:11
yinwei_computersame order as delete an in-use volume03:11
*** mingyu has quit IRC03:14
*** pzchen has joined #openstack-smaug03:19
chenyingYou 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_computerNot sure if I got your point.03:21
openstackgerritchenying proposed openstack/smaug: Fix create checkpoint API errors in protection service  https://review.openstack.org/32075403:21
*** chenying_ has joined #openstack-smaug03:21
yinwei_computerbasically, I think resources in cinder may get referenced by other top layer services, like smaug, or nova03:22
yinwei_computerhere comes the safe deletion issue: resource being referenced shouldn't be deleted directly by cinder03:22
yinwei_computercinder should keep a reference counting03:22
yinwei_computerthe problem is how to notify cinder that your volume is referenced by others.  we need an API here.03:23
yinwei_computerNova is a special case, which directly changes cinder volume status to in-use to solve this safe deletion issue.03:24
yinwei_computerbut 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 IRC03:24
yinwei_computerit could be03:24
yinwei_computereither 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_computerwhat do you mean by dragon?03:26
yinwei_computerflow dragon03:26
yinwei_computeryes, definitely03:26
chenying_yes  flow dragon03:26
yinwei_computerwe could check existed ways and propose our bp03:26
yinwei_computerit could be a generic API: reference_resource/dereference_resource, or it could be a specific requirement: adding a field for smaug protection03:27
yinwei_computernot sure if they're the same case03:28
*** zhangshuai_ has quit IRC03:59
*** pzchen has quit IRC04:00
*** smcginnis has quit IRC04:22
*** smcginnis has joined #openstack-smaug04:22
openstackgerritOpenStack Proposal Bot proposed openstack/smaug: Updated from global requirements  https://review.openstack.org/31989104:42
*** zhangshuai has joined #openstack-smaug04:44
*** mingyu has joined #openstack-smaug06:07
*** chenying_ has quit IRC06:19
*** yuval has joined #openstack-smaug06:22
*** zhangshuai has quit IRC06:29
*** zhangshuai has joined #openstack-smaug06:30
*** saggi has joined #openstack-smaug06:36
*** chenying has joined #openstack-smaug06:39
*** mingyu has quit IRC06:44
*** saggi has quit IRC06:48
chenyingping gampel06:51
*** openstack has joined #openstack-smaug06:54
yuvalchenying: hey, posted 2 questions on https://review.openstack.org/32075407:02
chenyingyuval: thanks I will address these comments.07:03
yuvalchenying: regarding the fullstack being 'NOT_REGISTERED', I'll look into it. Meanwhile you can try asking in #openstack-dev07:04
chenyingOK Thanks.07:05
yuvalchenying: can you please review https://review.openstack.org/310385 ?07:15
chenyingI will review it later.07:17
*** mingyu has joined #openstack-smaug07:23
*** mingyu has quit IRC07:23
*** mingyu has joined #openstack-smaug07:23
yuvalchenying: and this: https://review.openstack.org/#/c/317437/07:39
chenyingok07:43
*** wangfeng has joined #openstack-smaug07:54
*** zhurong_ has joined #openstack-smaug07:57
*** zhurong has quit IRC08:01
*** zhangshuai has quit IRC08:05
*** yuval_b has joined #openstack-smaug08:16
openstackgerritYuval Brik proposed openstack/smaug: Fix stevedore of Protectable Registry  https://review.openstack.org/31295109:26
*** mingyu has quit IRC09:31
*** mingyu has joined #openstack-smaug09:31
*** zhangshuai has joined #openstack-smaug09:33
*** zhangshuai_ has joined #openstack-smaug09:53
*** zhangshuai has quit IRC09:57
*** zhangshuai_ has quit IRC10:06
*** zhurong_ has quit IRC10:08
openstackgerritwangliuan proposed openstack/smaug: Implement Glance restoration  https://review.openstack.org/32145410:13
*** zhangshuai_ has joined #openstack-smaug10:27
*** zhangshuai_ has quit IRC10:32
*** zhangshuai_ has joined #openstack-smaug10:32
openstackgerritEran Gampel proposed openstack/smaug: [API] Add name property to Resource model  https://review.openstack.org/30177810:39
*** chenhuayi has quit IRC11:15
openstackgerritMerged openstack/smaug: [API] Add name property to Resource model  https://review.openstack.org/30177811:31
yuvalping chenying11:36
chenyingHi11:36
yuvalchenying: wanted to ask what is missing for the full tests, so I can start testing the new design11:41
openstackgerritchenying proposed openstack/smaug: Fix create checkpoint API errors in protection service  https://review.openstack.org/32075411:42
chenyinghttps://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
yuvalchenying: that's it? :)11:46
chenyingvolume 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
openstackgerritYuval Brik proposed openstack/smaug: Implement the Resource Flow  https://review.openstack.org/31038611:50
chenyingand 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
chenyingHi Eran  I think this patch can be merged.  https://review.openstack.org/#/c/320732/11:57
gampelI will look now thx11:58
*** mingyu has quit IRC12:03
openstackgerritYuval Brik proposed openstack/smaug: Implement the Resource Flow  https://review.openstack.org/31038612:03
openstackgerritMerged openstack/smaug: Fix bug in unpack_graph  https://review.openstack.org/32073212:04
chenyingre-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
gampelsorry maybe we need to create cache system in chengdu12:08
*** chenying has quit IRC12:14
*** chenying has joined #openstack-smaug12:15
chenyingvolume-restore works now in my environment. only need a small fix.12:43
*** mingyu has joined #openstack-smaug12:48
*** zhangshuai_ has quit IRC12:49
*** zhangshuai_ has joined #openstack-smaug12:49
*** zhangshuai_ has quit IRC12:50
*** zhurong has joined #openstack-smaug12:51
*** zhangshuai_ has joined #openstack-smaug12:52
*** zhangshuai_ has quit IRC12:52
*** zhangshuai_ has joined #openstack-smaug12:53
*** zhangshuai_ has joined #openstack-smaug12:53
*** zhangshuai_ has quit IRC12:58
yuvalchenying: great! :)12:58
yuvalwhat's missing?12:58
*** mingyu has quit IRC13:10
*** mingyu has joined #openstack-smaug13:10
*** zhurong has quit IRC14:20
*** zhurong has joined #openstack-smaug14:21
*** zhurong has quit IRC14:34
*** yuval has quit IRC15:01
*** wangfeng has quit IRC15:45
*** mingyu has quit IRC17:00
*** mingyu has joined #openstack-smaug18:01
*** mingyu has quit IRC18:06
*** zhurong has joined #openstack-smaug23:41

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!