Re: [libvirt PATCH] spec: Depend on qemu-kvm-block-driver-curl in RHEL-9

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Oct 27, 2021 at 15:38:22 +0100, Daniel P. Berrangé wrote:
> On Wed, Oct 27, 2021 at 03:58:29PM +0200, Jiri Denemark wrote:
> > The curl block driver is no longer a requirement for qemu-kvm package in
> > RHEL-9, which breaks support for https network disks. Let's depend on it
> > explicitly (in libvirt-daemon-kvm meta package) until we possibly
> > reimplement the https disk support using nbdkit (tracked in bug 2016527).
> > 
> > https://bugzilla.redhat.com/show_bug.cgi?id=2014229
> > 
> > Signed-off-by: Jiri Denemark <jdenemar@xxxxxxxxxx>
> > ---
> >  libvirt.spec.in | 4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > diff --git a/libvirt.spec.in b/libvirt.spec.in
> > index 4ecb28114c..b71888653d 100644
> > --- a/libvirt.spec.in
> > +++ b/libvirt.spec.in
> > @@ -791,6 +791,10 @@ Requires: libvirt-daemon-driver-nwfilter = %{version}-%{release}
> >  Requires: libvirt-daemon-driver-secret = %{version}-%{release}
> >  Requires: libvirt-daemon-driver-storage = %{version}-%{release}
> >  Requires: qemu-kvm
> > +# qemu-kvm no longer depends on curl block driver in RHEL-9
> > +%if 0%{?rhel} > 8
> > +Requires: qemu-kvm-block-driver-curl
> > +%endif
> 
> it feels pretty dubious to me todo this in libvirt.
> 
> If curl driver is still required by valid use cases in RHEL-9, then
> 'qemu-kvm' should still depend on it, until we have a replacement
> ready.

I would agree with you if this was either permanent or the replacement
was to be implemented in QEMU. However, this is a temporary dependency
and once we have replacement ready, we can just drop this without having
to synchronize with other components and without worrying about possible
issues when QEMU and libvirt are not upgraded in a single step.

Jirka




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux