[libvirt PATCH 2/8] Revert "remote: move timeout arg into sysconf file"

[Daniel P. Berrangé]

On Wed, Apr 01, 2020 at 08:53:39PM +0200, Andrea Bolognani wrote:
> There is nothing really systemd-specific about passing extra
> arguments to daemons so it's reasonable, although not currently the
> case, that startup scripts written for other init systems might want
> to source these sysconf files; for those init systems, which likely
> do not support socket activation, making the daemon quit after a
> timeout has expired is probably not a good idea.
> 
> More generally, the sysconf files should not reflect the default
> behavior, but only contain overrides explicitly put in place by the
> admin; now that we have a mechanism to disable timeouts regardless
> of the default set in the service file, that argument for having the
> default timeout in the sysconf file is moot as well.

The effect on this though is that --timeout arg now has to be
specified twice so we'll get a running process of

   "libvirtd --timeout 120 --timeout 0"

which I find quite unappealing, so I'm not really in favour of
this revert, especially as we don't actually use the sysconf
files from other init systems


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 :|