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
rpm: Update libvirt and QEMU #6657
Conversation
Oh:
|
@rmohr I think some component that used to depend on usbredir either downgraded that dependency it to a weak one, which IIRC bazeldnf will ignore, or dropped it altogether. The fix might be as simple as explicitly including usbredir in the launcherbase image, but I need to look into the situation and figure out whether this change was intentional or something that needs to be reverted. |
@andreabolognani any chances to get this fix included :). This is needed by #6672 |
Signed-off-by: Andrea Bolognani <abologna@redhat.com>
If we ignore the libvirt-daemon-kvm package, which is a meta-package that brings in not only the QEMU driver but also a number of other things that we don't care about in the context of KubeVirt, we can simplify things a bit. In addition to that, there are a few more ignores that we can drop without affecting the results. Signed-off-by: Andrea Bolognani <abologna@redhat.com>
Specifically: libvirt 7.0.0-14 -> 7.6.0-4 QEMU 5.2.0-16 -> 6.0.0-33 Note that we have to add the qemu-kvm-hw-usbredir subpackage explicitly to the launcherbase image now, as the qemu-kvm-core subpackage no longer comes with usbredir support built-in. Signed-off-by: Andrea Bolognani <abologna@redhat.com>
Currently testimage will end up with some random version of qemu-img: probably the latest one, but possibly not the same one as launcherbase. This will not necessarily cause any problem, but it's nicer to use the same version everywhere. It also saves on download time during build and disk usage for the artifact mirror. Signed-off-by: Andrea Bolognani <abologna@redhat.com>
qemu-kvm-core no longer drags in usbredir, so ignoring it explicitly has become unnecessary. Signed-off-by: Andrea Bolognani <abologna@redhat.com>
This new version of the Go bindings was released along with libvirt 7.6.0; the version number is v1.7005.1 rather than v1.7006.0 because it introduces no API changes. Signed-off-by: Andrea Bolognani <abologna@redhat.com>
2ce9f6c
to
e7896ee
Compare
@rmohr I have confirmed that the usbredir change was intentional: @alicefr The fix you're looking for is included in /retest |
/retest |
@rmohr All tests have passed. Can you please approve the MR? Thanks! :) |
/lgtm looks awesome. 🎆 |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rmohr The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
1 similar comment
/retest |
What this PR does / why we need it:
Updates libvirt and QEMU. Specifically:
Special notes for your reviewer:
The new version of libvirt contains changes that are necessary for #6162 along with the usual bug fixes and improvements.
Release note: