On a Wednesday in 2021, Daniel P. Berrangé wrote:
>This is in prep for switching to use the modular daemons by default. In
>testing that setup I hit a couple of minor bugs.
>
>Daniel P. Berrangé (10):
> remote: use absolute path to check for daemons
> remote: stop checking for errors from socket allocations
> remote: move open flags enum to sockets header
> remote: consistently use flags for passing ro/user/autostart props
> remote: change socket helper to return full daemon path
> remote: don't populate daemon path if autostart is not required
> rpc: remove "spawnDaemon" parameter
> remote: move proxy/mode defaults after URI parsing
> remote: fix regression connecting to remote session daemon
> remote: use virtproxyd if no URI is given
>
> src/admin/admin_remote.c | 2 +-
> src/locking/lock_driver_lockd.c | 1 -
> src/logging/log_manager.c | 1 -
> src/lxc/lxc_monitor.c | 2 +-
> src/qemu/qemu_migration.c | 3 +-
> src/remote/remote_daemon_dispatch.c | 4 +-
> src/remote/remote_driver.c | 135 ++++++++++++----------------
> src/remote/remote_sockets.c | 95 +++++++++++++-------
> src/remote/remote_sockets.h | 15 ++--
> src/remote/remote_ssh_helper.c | 16 ++--
> src/rpc/virnetclient.c | 5 +-
> src/rpc/virnetclient.h | 3 +-
> src/rpc/virnetsocket.c | 26 ++----
> src/rpc/virnetsocket.h | 3 +-
> tests/virnetsockettest.c | 4 +-
> 15 files changed, 157 insertions(+), 158 deletions(-)
>
With the unused variable fixed:
Reviewed-by: Ján Tomko <jtomko@redhat.com>
Jano