On Wed, Dec 14, 2022 at 09:56:28AM -0800, Andrea Bolognani wrote:
> On Tue, Dec 13, 2022 at 05:30:59PM -0700, Jim Fehlig wrote:
> > @@ -450,7 +446,6 @@ Requires: module-init-tools
> > Requires: iproute
> > # for /sbin/tc
> > Requires: iproute-tc
> > -Requires: polkit >= 0.112
> > %if %{with_dmidecode}
> > # For virConnectGetSysinfo
> > Requires: dmidecode
> > @@ -464,6 +459,21 @@ Requires: numad
> > Requires: dbus
> > # For uid creation during pre
> > Requires(pre): shadow-utils
>
> Have you verified that all of these runtime dependencies are really
> only needed when running the monolithic daemon?
>
> I would expect that, for example, the iproute dependency would have
> to be moved to the libvirt-daemon-driver-network package and so on.
>
> Some of them might even need to be duplicated to ensure that a pure
> split daemon scenario works correctly.
And I think the %pre snippet that handles creation of the libvirt
group needs to be moved to the common package as well, to allow
members of the group to perform privileged operations in a pure split
daemon deployment.
--
Andrea Bolognani / Red Hat / Virtualization
