shwstppr commented on code in PR #367: URL: https://github.com/apache/cloudstack-documentation/pull/367#discussion_r1427892637
########## source/adminguide/storage.rst: ########## @@ -957,6 +957,9 @@ incremental backups are supported, every N backup is a full backup. | Support incremental backup | No | Yes | No | +------------------------------+------------------+------------------+-----+ + .. note:: + Between versions 4.17.x, 4.18.0 and 4.18.1, KVM volume snapshot backups were not full snapshots and they rely on the snapshots on the primary storage. To prevent any loss of data, care must be taken during revert operation and it must be ensured that the source primary storage snapshot file is present. + Review Comment: @GutoVeronezi good idea. Though I'm not sure how to structure it. Would you mind adding a change in the PR as you must be having better understanding on the process? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
