JoaoJandre commented on PR #8911: URL: https://github.com/apache/cloudstack/pull/8911#issuecomment-2225422982
> > Hey @mlsorensen, @weizhouapache, @rohityadavcloud, @borisstoyanov, @DaanHoogland > > I've pushed the new change, now we keep the VM as linked clone if we are going from/to NFS/SMP/Local storage. We only consolidate the VM if the template does not exist anymore (i.e. force removed). > > thanks @JoaoJandre for the update I have not reviewed and tested the changes yet. the logic looks gooood to me. can you please update the title of this PR ? @JoaoJandre > > > I've tested creating a VM and migrating between NFS/SMP/Local storage, all migrations were successful, the template was copied to the storage and the VM was not corrupted. > > I've also tested creating a VM, force deleting its template and then migrating it, the migration consolidated the volume, and the VM was not corrupted. @weizhouapache I've updated the title and description, thanks for the reminder -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
