GutoVeronezi commented on pull request #4562: URL: https://github.com/apache/cloudstack/pull/4562#issuecomment-756313460
@RodrigoDLopez it's a good way to test it. Before the commit, what you described in step 5 is what was happening. To test it... 1. I created two primary storages. 2. I created one instance (using KVM as hypervisor) in one primary storage. 3. I updated hypervisor_capabilities to support storage motion of a KVM. 4. Via GUI, I tried to migrate the volume to the other primary storage. 5. Before the modifications, it would claim that the volume was migrated, but actually it would create an inconsistency in the database and in the storage pool. Now, it's throwing an exception explaining why it is not possible and how to get around the situation. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
