DaanHoogland commented on code in PR #367: URL: https://github.com/apache/cloudstack-documentation/pull/367#discussion_r1426554866
########## 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: ```suggestion 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. ``` -- 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