sureshanaparti commented on code in PR #9478:
URL: https://github.com/apache/cloudstack/pull/9478#discussion_r1738290039


##########
api/src/main/java/org/apache/cloudstack/api/command/user/snapshot/CopySnapshotCmd.java:
##########
@@ -84,6 +85,15 @@ public class CopySnapshotCmd extends BaseAsyncCmd implements 
UserCmd {
                     "Do not specify destzoneid and destzoneids together, 
however one of them is required.")
     protected List<Long> destZoneIds;
 
+    @Parameter(name = ApiConstants.STORAGE_ID_LIST,
+            type=CommandType.LIST,
+            collectionType = CommandType.UUID,
+            entityType = StoragePoolResponse.class,
+            required = false,
+            description = "A comma-separated list of IDs of the storage pools 
in other zones in which the snapshot will be made available. " +

Review Comment:
   any case to copy to different pools in the same zone where snapshot doesn't 
exists?



-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to