Update patch set 18

Patch Set 18:

(1 comment)

Still leave some questions here due to the time zone diff.

Patch-set: 18
This commit is contained in:
Gerrit User 19346 2015-12-30 03:08:52 +00:00 committed by Gerrit Code Review
parent 147856eb44
commit 4d1bfd2397

View File

@ -111,6 +111,24 @@
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "fa69d971_85b0318d",
"filename": "doc/source/api/bank.md",
"patchSetId": 18
},
"lineNbr": 51,
"author": {
"id": 19346
},
"writtenOn": "2015-12-30T03:08:52Z",
"side": 1,
"message": "Due to the time shift, I will leave my questions here:\n1. so the dependency here in protection_definition doesn\u0027t mean the attachment info (metadata of vm). Could you pls. take an example for the dependency field of the protection_definition?\n2. we do have requirement that the IP won\u0027t change after restoration in hybrid cloud. So that apps won\u0027t be aware of the restoration.\n3. so it\u0027s the responsibility of vm protection plugin to store the metadata instead of bank? any available vm plugin offers this capability? AFAK,openstack suggests backup vm and metadata(db) separately.",
"parentUuid": "1a7b0d38_2f5681ce",
"revId": "d192b3e1a8b323e22f379c90c7f6052747d2c273",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "1a7b0d38_185bd688",