mike-tutkowski commented on issue #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command URL: https://github.com/apache/cloudstack/pull/2402#issuecomment-357557367 @rhtyd @rafaelweingartner @DaanHoogland I don't think it's really a blocker. Perhaps we can get it into the first maintenance release for 4.11. It causes two problems: 1) For non-managed storage: If you create a volume snapshot from, say, storage pool 1 and later delete storage pool 1, then you can no longer use the volume snapshot (because we can't find an applicable host for the copy operation with this new logic). 2) For managed storage: The SAN-assisted volume snapshot use case fails. This only applies to XenServer and only when locationtype=secondary is passed into createSnapshot (that parameter is only used for managed storage and primary is the default value for that parameter). As far as I know, only one company currently uses this API this way. This company also is accustomed to patching CloudStack and using those modified versions in production, so I don't think we need to hold this particular release up because they can use the 4.11 release with this PR in their production environment (even before it "officially" comes out in a maintenance release).
---------------------------------------------------------------- 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
