mike-tutkowski commented on issue #2298: CLOUDSTACK-9620: Enhancements for 
managed storage
URL: https://github.com/apache/cloudstack/pull/2298#issuecomment-356734107
 
 
   @rafaelweingartner It looks like the method that contains that code 
(createSqlFindHostConnectedToStoragePoolToExecuteCommand) is only called from 
HostDaoImpl.findHostToOperateOnSnapshotBasedOnStoragePool.
   
   In this case, we are just trying to find a host to perform a copy of the 
volume snapshot from secondary to a new template on secondary. As such, you 
shouldn't really care for managed storage if the compute cluster meets the 
restrictions of your cluster_details clause (no UUID resigning is required 
here).
   
   Even more concerning to me, though, is what happens if the storage pool 
(managed or non managed) no longer exists? For example, let's say you created a 
volume snapshot from a volume in storage pool 1 and later you removed storage 
pool 1. You then go to create a template from this volume snapshot, but you 
can't find a connected host because storage pool 1 no longer exists (it 
shouldn't have any connected hosts listed in cloud.storage_pool_host_ref).

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