GutoVeronezi commented on pull request #5297: URL: https://github.com/apache/cloudstack/pull/5297#issuecomment-993625709
@DaanHoogland as I explained in https://github.com/apache/cloudstack/pull/5297#issuecomment-940007442, the errors are related to the CentOS's QEMU binary: `Operation not supported: live disk snapshot not supported with this QEMU binary` RedHat removed/blocked a lot of features in CentOS, such as VM migrations, memory hotplug, and so on. Therefore, default CentOS's QEMU binary does not support them. The tests are not passing due to QEMU binary in CentOS 7 used by the CI. If the QEMU binary in CentOS 7 used by the CI were upgraded to the alternative (`**qemu-kvm-ev**`, which supports more features than the default one) or if it used another distro, like Ubuntu, the tests would pass. Therefore, the problem is not directly linked with the patch itself. Besides that, in my point of view, such limitations in the default QEMU binary (in CentOS) make it unfeasible to build a cloud with CentOS and the default QEMU binary. Probably, most people using KVM with CentOS are using the `ev` binaries (I might be wrong though, however, that seems to be the case when looking at the users' list). Therefore, I do not see why we would impose such limitation in the ACS feature set, to support an O.S. that is in its final years, and we have a limited supported set of features with it. Also, we just need to guide users to upgrade to **qemu-kvm-ev**, like we already did many other times in the mailing list (e.g. https://lists.apache.org/thread/vg9qz7w52yso5b5xcp4rhtmrodhvdmpf). -- 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]
