Monday, 2016-03-07

*** luobin has quit IRC01:12
*** luobin has joined #openstack-smaug01:52
*** luobin has quit IRC02:05
*** wanghao_ has joined #openstack-smaug02:33
*** x00350071 is now known as xiangxinyong03:34
openstackgerritchenying proposed openstack/smaug: The RESTAPI of resource protectables  https://review.openstack.org/28526403:48
*** yuvalbrik has joined #openstack-smaug06:50
*** yuval has quit IRC06:54
*** gampel has joined #openstack-smaug06:55
*** yuvalbrik is now known as yuval07:04
gampelzengyingzhe: hi07:13
zengyingzhehi, gampel, do you think we'd better change a name for protectable plugin method fetch_child_resources?07:14
zengyingzheI got many feedbacks saying that it's a little confusing.07:14
gampelwhat is the proposed  new name ?07:15
zengyingzheWait a min. Let me think about it.07:17
zengyingzheYou got any suggestion?07:17
gampelLets think about it, i agree that it is a bit confusing07:18
gampeldid you see my comment about the directory structure07:18
*** luobin has joined #openstack-smaug07:18
zengyingzheYes. I'm fixing it.07:18
gampelwe need all the team to be coordinated on what ever stricture we decide07:19
zengyingzhesure07:19
zengyingzheAnd another thing.07:21
gampelwhat about the protactabole t pagination params07:21
gampelsorry yes07:21
zengyingzheLet's talk about the paging stuff later.07:22
openstackgerritsmile-luobin proposed openstack/smaug: Implement swift bank plugin  https://review.openstack.org/27992907:22
gampelok07:22
zengyingzheI see you suggest that check if the parent_resource is really the parent type07:23
zengyingzhebut in the implementation of protectableregistry07:23
zengyingzhethe fetch_dependent_resources method already confirms that the parent_resource is a parent type of given plugin.07:24
*** rong__ has joined #openstack-smaug07:25
zengyingzheis that necessary to check it again?07:25
gampelOk i missed that let me check ? can you point me to the patch07:26
openstackgerritsmile-luobin proposed openstack/smaug: Implement swift bank plugin  https://review.openstack.org/27992907:26
zengyingzhehttps://review.openstack.org/#/c/285611/207:26
rong__eran, i am rong hu. honestly, I can not understand what is your suggestion,can you give more ditail?07:27
zengyingzheIt'll make the plugin more complex. for instance, volume plugin, if i want to check if parent_resource is a server indeed, i must construct a server client.07:28
gampeli mean to the rotectableregistry patch07:28
zengyingzhehttps://review.openstack.org/#/c/281783/8/smaug/services/protection/protectable_registry.py07:28
gampelok07:29
gampel rong__: what are we talking about07:29
rong__gampel:we need to support the optional name in cider restore  The name of the volume to which you want to restore a backup. firstly, I do not understand which should I do in this commit.07:31
rong__gampel: "this is an optional in cinder so we need to support by default  auto generate value  Missing default val". secondly, this sentence I also can not understand what should I do? it is means I should get the default vulue from cinder?07:33
gampelrong__:  I mean that if you look in cider backup API the vol uuid is optional07:45
gampelit means that the backup driver will auto generate uuid for the volume07:45
gampelwe need to add the the schema  a default value for each parameter that is not mandatory  does this make sense07:47
rong__yes, use the uuid1, is it?07:47
gampelI mean this parameter in my view should have a default value of auto generated uuid07:48
rong__probably, i agree with you07:49
gampelfor the name cinder backup support optional name, i think that we should support it our cinder schema07:49
gampelthe schema  and the parameters should reflect the driver in this case cinder max options07:50
gampel rong__:  do you agree ?07:51
rong__intuitively, can I say that we should increase parmeters as more as we can do in the cinder options?07:52
gampelgreat lets think together  how to reflect it in the schema in the most intuitive and user friendly way07:54
*** c00281451_ is now known as chenzeng07:54
gampeli have to go to a meeting , is is clear now my comments07:55
rong__Ok, let me think about it.07:56
openstackgerritwangliuan proposed openstack/smaug: Implement Provider and ProviderRegistry  https://review.openstack.org/28161807:59
openstackgerritsmile-luobin proposed openstack/smaug: Implement swift bank plugin  https://review.openstack.org/27992907:59
*** luobin has quit IRC08:04
*** luobin has joined #openstack-smaug08:05
*** rong__ has quit IRC08:20
openstackgerritxiangxinyong proposed openstack/smaug-dashboard: Initial Cookiecutter for smaug-dashboard  https://review.openstack.org/28645609:08
openstackgerritxiangxinyong proposed openstack/smaug-dashboard: Initial Cookiecutter for smaug-dashboard  https://review.openstack.org/28645609:21
openstackgerritYingzhe Zeng proposed openstack/smaug: Implement cinder protectable plugin  https://review.openstack.org/28561109:30
openstackgerritYingzhe Zeng proposed openstack/smaug: Implement nova protectable plugin  https://review.openstack.org/28654209:50
*** luobin has quit IRC10:07
*** yuval has quit IRC11:16
*** wanghao_ has quit IRC11:37
*** yuval has joined #openstack-smaug12:59
*** yuvalbrik has joined #openstack-smaug13:30
*** gampel has quit IRC13:31
*** gampel has joined #openstack-smaug13:32
*** yuval has quit IRC13:34
*** yuvalbrik has quit IRC14:06
*** yuvalbrik has joined #openstack-smaug14:06
*** zhonghua-lee has quit IRC14:21
*** zhonghua-lee has joined #openstack-smaug14:22
gampel saggi: start with this one https://review.openstack.org/#/c/284282/715:12
openstackgerritMerged openstack/smaug: The RESTAPI of resource Restore  https://review.openstack.org/28428216:18
*** gampel has quit IRC16:19
*** chenying_ has joined #openstack-smaug16:56
*** chenying has quit IRC16:59
openstackgerritEran Gampel proposed openstack/smaug: The RESTAPI of resource protectables  https://review.openstack.org/28526418:11
openstackgerritMerged openstack/smaug: The RESTAPI of resource protectables  https://review.openstack.org/28526418:22
openstackgerritMerged openstack/python-smaugclient: Initial Cookiecutter for python-smaugclient  https://review.openstack.org/28635619:51
*** yuvalbrik has quit IRC19:58
*** yuvalbrik has joined #openstack-smaug20:03
openstackgerritEran Gampel proposed openstack/smaug: implement time trigger  https://review.openstack.org/27119620:08
openstackgerritEran Gampel proposed openstack/smaug: The RESTAPI of resource providers  https://review.openstack.org/28565120:17
openstackgerritMerged openstack/smaug: The RESTAPI of resource providers  https://review.openstack.org/28565120:54
*** yuvalbr has joined #openstack-smaug20:58
*** yuvalbrik has quit IRC21:01

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