*** luobin has quit IRC | 01:12 | |
*** luobin has joined #openstack-smaug | 01:52 | |
*** luobin has quit IRC | 02:05 | |
*** wanghao_ has joined #openstack-smaug | 02:33 | |
*** x00350071 is now known as xiangxinyong | 03:34 | |
openstackgerrit | chenying proposed openstack/smaug: The RESTAPI of resource protectables https://review.openstack.org/285264 | 03:48 |
---|---|---|
*** yuvalbrik has joined #openstack-smaug | 06:50 | |
*** yuval has quit IRC | 06:54 | |
*** gampel has joined #openstack-smaug | 06:55 | |
*** yuvalbrik is now known as yuval | 07:04 | |
gampel | zengyingzhe: hi | 07:13 |
zengyingzhe | hi, gampel, do you think we'd better change a name for protectable plugin method fetch_child_resources? | 07:14 |
zengyingzhe | I got many feedbacks saying that it's a little confusing. | 07:14 |
gampel | what is the proposed new name ? | 07:15 |
zengyingzhe | Wait a min. Let me think about it. | 07:17 |
zengyingzhe | You got any suggestion? | 07:17 |
gampel | Lets think about it, i agree that it is a bit confusing | 07:18 |
gampel | did you see my comment about the directory structure | 07:18 |
*** luobin has joined #openstack-smaug | 07:18 | |
zengyingzhe | Yes. I'm fixing it. | 07:18 |
gampel | we need all the team to be coordinated on what ever stricture we decide | 07:19 |
zengyingzhe | sure | 07:19 |
zengyingzhe | And another thing. | 07:21 |
gampel | what about the protactabole t pagination params | 07:21 |
gampel | sorry yes | 07:21 |
zengyingzhe | Let's talk about the paging stuff later. | 07:22 |
openstackgerrit | smile-luobin proposed openstack/smaug: Implement swift bank plugin https://review.openstack.org/279929 | 07:22 |
gampel | ok | 07:22 |
zengyingzhe | I see you suggest that check if the parent_resource is really the parent type | 07:23 |
zengyingzhe | but in the implementation of protectableregistry | 07:23 |
zengyingzhe | the fetch_dependent_resources method already confirms that the parent_resource is a parent type of given plugin. | 07:24 |
*** rong__ has joined #openstack-smaug | 07:25 | |
zengyingzhe | is that necessary to check it again? | 07:25 |
gampel | Ok i missed that let me check ? can you point me to the patch | 07:26 |
openstackgerrit | smile-luobin proposed openstack/smaug: Implement swift bank plugin https://review.openstack.org/279929 | 07:26 |
zengyingzhe | https://review.openstack.org/#/c/285611/2 | 07:26 |
rong__ | eran, i am rong hu. honestly, I can not understand what is your suggestion,can you give more ditail? | 07:27 |
zengyingzhe | It'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 |
gampel | i mean to the rotectableregistry patch | 07:28 |
zengyingzhe | https://review.openstack.org/#/c/281783/8/smaug/services/protection/protectable_registry.py | 07:28 |
gampel | ok | 07:29 |
gampel | rong__: what are we talking about | 07: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 |
gampel | rong__: I mean that if you look in cider backup API the vol uuid is optional | 07:45 |
gampel | it means that the backup driver will auto generate uuid for the volume | 07:45 |
gampel | we need to add the the schema a default value for each parameter that is not mandatory does this make sense | 07:47 |
rong__ | yes, use the uuid1, is it? | 07:47 |
gampel | I mean this parameter in my view should have a default value of auto generated uuid | 07:48 |
rong__ | probably, i agree with you | 07:49 |
gampel | for the name cinder backup support optional name, i think that we should support it our cinder schema | 07:49 |
gampel | the schema and the parameters should reflect the driver in this case cinder max options | 07: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 |
gampel | great lets think together how to reflect it in the schema in the most intuitive and user friendly way | 07:54 |
*** c00281451_ is now known as chenzeng | 07:54 | |
gampel | i have to go to a meeting , is is clear now my comments | 07:55 |
rong__ | Ok, let me think about it. | 07:56 |
openstackgerrit | wangliuan proposed openstack/smaug: Implement Provider and ProviderRegistry https://review.openstack.org/281618 | 07:59 |
openstackgerrit | smile-luobin proposed openstack/smaug: Implement swift bank plugin https://review.openstack.org/279929 | 07:59 |
*** luobin has quit IRC | 08:04 | |
*** luobin has joined #openstack-smaug | 08:05 | |
*** rong__ has quit IRC | 08:20 | |
openstackgerrit | xiangxinyong proposed openstack/smaug-dashboard: Initial Cookiecutter for smaug-dashboard https://review.openstack.org/286456 | 09:08 |
openstackgerrit | xiangxinyong proposed openstack/smaug-dashboard: Initial Cookiecutter for smaug-dashboard https://review.openstack.org/286456 | 09:21 |
openstackgerrit | Yingzhe Zeng proposed openstack/smaug: Implement cinder protectable plugin https://review.openstack.org/285611 | 09:30 |
openstackgerrit | Yingzhe Zeng proposed openstack/smaug: Implement nova protectable plugin https://review.openstack.org/286542 | 09:50 |
*** luobin has quit IRC | 10:07 | |
*** yuval has quit IRC | 11:16 | |
*** wanghao_ has quit IRC | 11:37 | |
*** yuval has joined #openstack-smaug | 12:59 | |
*** yuvalbrik has joined #openstack-smaug | 13:30 | |
*** gampel has quit IRC | 13:31 | |
*** gampel has joined #openstack-smaug | 13:32 | |
*** yuval has quit IRC | 13:34 | |
*** yuvalbrik has quit IRC | 14:06 | |
*** yuvalbrik has joined #openstack-smaug | 14:06 | |
*** zhonghua-lee has quit IRC | 14:21 | |
*** zhonghua-lee has joined #openstack-smaug | 14:22 | |
gampel | saggi: start with this one https://review.openstack.org/#/c/284282/7 | 15:12 |
openstackgerrit | Merged openstack/smaug: The RESTAPI of resource Restore https://review.openstack.org/284282 | 16:18 |
*** gampel has quit IRC | 16:19 | |
*** chenying_ has joined #openstack-smaug | 16:56 | |
*** chenying has quit IRC | 16:59 | |
openstackgerrit | Eran Gampel proposed openstack/smaug: The RESTAPI of resource protectables https://review.openstack.org/285264 | 18:11 |
openstackgerrit | Merged openstack/smaug: The RESTAPI of resource protectables https://review.openstack.org/285264 | 18:22 |
openstackgerrit | Merged openstack/python-smaugclient: Initial Cookiecutter for python-smaugclient https://review.openstack.org/286356 | 19:51 |
*** yuvalbrik has quit IRC | 19:58 | |
*** yuvalbrik has joined #openstack-smaug | 20:03 | |
openstackgerrit | Eran Gampel proposed openstack/smaug: implement time trigger https://review.openstack.org/271196 | 20:08 |
openstackgerrit | Eran Gampel proposed openstack/smaug: The RESTAPI of resource providers https://review.openstack.org/285651 | 20:17 |
openstackgerrit | Merged openstack/smaug: The RESTAPI of resource providers https://review.openstack.org/285651 | 20:54 |
*** yuvalbr has joined #openstack-smaug | 20:58 | |
*** yuvalbrik has quit IRC | 21:01 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!