[Libguestfs] [PATCH Fedora libguestfs] Don't depend on libvirt-daemon-kvm monolith.

[Daniel P. Berrangé]

On Fri, Jan 10, 2020 at 02:26:35PM +0000, Richard W.M. Jones wrote:
> On Fri, Jan 10, 2020 at 02:15:10PM +0000, Daniel P. Berrangé wrote:
> > Do you use the libvirt "secret" APIs at all (disk encryption, network
> > disk auth passwords) ? If so you will need "libvirt-daemon-driver-secret"
> > too. How about any other libvirt sub-driver APIs ?  Networking ? Host
> > dev, etc  ?
> 
> The full list of APIs we use is attached, assuming I got my regexp
> correct.
> 
> > > +Recommends:    libvirt-daemon-driver-storage-disk
> > > +Recommends:    libvirt-daemon-driver-storage-gluster
> > > +Recommends:    libvirt-daemon-driver-storage-iscsi
> > > +Recommends:    libvirt-daemon-driver-storage-logical
> > > +Recommends:    libvirt-daemon-driver-storage-rbd
> > > +Recommends:    libvirt-daemon-driver-storage-scsi
> 
> In his response Pino questioned if we need these at all.  I may not
> understand exactly what they do, but we *do* use libvirt XML
> containing disk elements to refer to disks, as well as elements
> referring to gluster, iscsi, ceph, etc.

These are specific impls of the libvirt storage pool APIs.


If you're only usng domain <disk>s which directly reference
gluster/iscsi/etc, you don't need the storage pool APIs.

If you're using domain <disk>s which use type=volume,
then you'll need the storage pool APIs & any impls
that are relevant


> virStoragePoolFree (
> virStoragePoolGetInfo (
> virStoragePoolLookupByName (
> virStorageVolFree (
> virStorageVolGetInfo (
> virStorageVolGetPath (
> virStorageVolLookupByName (

So something in libguestfs is using storage pools, which would
mean you want  libvirt-daemon-driver-storage-core, and one or
more of the pool impls that you use.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|