harikrishna-patnala commented on code in PR #367:
URL: 
https://github.com/apache/cloudstack-documentation/pull/367#discussion_r1477697422


##########
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:
   @shwstppr may be we can continue merging this and handle that steps in a 
different PR. @GutoVeronezi can you please raise a separate PR for that.



-- 
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

Reply via email to