Re: [libvirt PATCH 0/4] remote: switch to modular daemons by default

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

 



On 6/10/21 3:43 PM, Daniel P. Berrangé wrote:
> This series first improves driver probing when using modular daemons.
> 
> Currently when URI is NULL, we connect to virtproxyd and it looks
> at which UNIX sockets exist and what binaries exist, to decide which
> modular hypervisor daemon to connect to.
> 
> This means the common case results in all traffic going via virtproxyd.
> Moving the logic out of virtproxyd into the remote client means we can
> avoid using virtproxyd by default.
> 
> With this, we can now switch to the modular daemons by default. The
> latter change primarily impacts how autostart works
> 
> When running as root we simply connect to whatever UNIX socket exists
> and rely on systemd to autostart if needed. Whether the UNIX sockets
> are for the modular daemon or libvirt doesn't matter - we'll look for
> both. Defaults are dependent on the distros' systemd presets. I intend
> to get Fedora / RHEL-9 presets changed to use the modular daemons.
> 
> When running as non-root, we again try to connect to whatever UNIX
> socket exists, whethe a modular daemon or libvirtd. When no socket
> exists though, we need to pick a daemon to start and that's where
> the meson default setting toggle comes into effect, so we prefer
> spawning modular daemons now and don't spawn libvirtd.
> 
> Daniel P. Berrangé (4):
>   remote: extract logic for probing for modular daemons
>   remote: add support for probing drivers with modular daemons
>   remote: remove probing logic from virtproxyd dispatcher
>   remote: switch to auto-spawn modular daemons by default
> 
>  meson.build                         |   6 +-
>  meson_options.txt                   |   2 +-
>  src/libvirt_remote.syms             |   5 +
>  src/remote/remote_daemon_dispatch.c | 130 +------------------
>  src/remote/remote_sockets.c         | 192 ++++++++++++++++++++++++++--
>  src/remote/remote_sockets.h         |   7 +
>  6 files changed, 202 insertions(+), 140 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@xxxxxxxxxx>

Michal




[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