cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <>
Subject [GitHub] mike-tutkowski commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025.
Date Thu, 01 Jan 1970 00:00:00 GMT
mike-tutkowski commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025.

 File path: engine/schema/src/com/cloud/host/dao/
 @@ -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:
   On XenServer, making use of a volume snapshot that resides on primary storage needs to
leverage a XenServer host that supports UUID resigning. UUID resigning is available as a patch
for XenServer (@syed created it in 2016). When the CSMS connects with a XenServer host, it
checks to see if this patch is installed. The results of the check are stored in cloud.host_details
(the supportsResign property). If any host in the cluster supports UUID resigning, then the
"convenience" property, also called supportsResign, in cloud.cluster_details is set to true.
That being the case, we need to find one of these hosts in the zone when leveraging a managed
snapshot on XenServer.

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:

With regards,
Apache Git Services

View raw message