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


##########
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. " +
+                    "The snapshot will always be made available in the zone in 
which the volume is present.")

Review Comment:
   ```suggestion
                       "The snapshot will always be made available in the zone 
in which the volume is present. Currently supported for StorPool only")
   ```
   
   update wherever applicable if ok



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