mike-tutkowski commented on a change in pull request #2315: A comprehensive 
solution for #CLOUDSTACK-9025.
URL: https://github.com/apache/cloudstack/pull/2315#discussion_r151234607
 
 

 ##########
 File path: engine/schema/src/com/cloud/host/dao/HostDao.java
 ##########
 @@ -103,4 +95,13 @@
     List<HostVO> listByType(Type type);
 
     HostVO findByIp(String ip);
+
+    /**
+     * This method will look for a host that is connected to the storage pool 
where the volume of the Snapshot is stored.
+     * <ul>
+     * <li>If the storage pool found for the volume of the snapshotId has more 
than a host, we will choose one randomly;
+     * <li>If no host is found, we throw a runtime exception
+     * </ul>
+     */
+    HostVO findHostToOperateOnSnapshot(long snapshotId);
 
 Review comment:
   One workaround I already have in place, though, is the following (currently 
only supported on XenServer): You can pass in a parameter when taking the 
volume snapshot that tells us to put the volume snapshot on secondary storage 
instead of keeping it on primary storage. If, however, you have already created 
the snapshot on primary storage, then you'd need to create a volume from it and 
then snapshot that volume to secondary storage.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to