-
Notifications
You must be signed in to change notification settings - Fork 1.2k
kvm: support qemu-system-x86>=5.2 #5755
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
kvm: support qemu-system-x86>=5.2 #5755
Conversation
@blueorangutan package |
@weizhouapache a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
Packaging result: ✔️ el7 ✔️ el8 ✔️ debian ✔️ suse15. SL-JID 1859 |
@blueorangutan test |
@weizhouapache a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
Trillian test result (tid-2641)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, how does this relate to the qemu-kvm-ev package? can/should we include that as well as a acceptable dependency?
@DaanHoogland I have not tested Ubuntu 21.04 (which has qemu-system-x86 but does not have qemu-kvm) . Without this PR, cloudstack-agent cannot be installed. |
ok, so the qemu-kvm-ev discussion only applies to redhat varieties? tnx |
@DaanHoogland exactly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - we'll need to maintain the arm build to have qemu-system-arm
then.
LGTM, checked packages.ubuntu.com - this will ensure qemu-system/x86 is installed on ubuntu 21.04 and later. |
Description
This PR fixes #5350
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?