Re: [libvirt PATCH 4/8] logging, locking: Set default timeout of 120 seconds

On Wed, Apr 01, 2020 at 08:53:41PM +0200, Andrea Bolognani wrote: > [Service] > EnvironmentFile=-@sysconfdir@/sysconfig/virtlogd > -ExecStart=@sbindir@/virtlogd $VIRTLOGD_ARGS > +ExecStart=@sbindir@/virtlogd --timeout 120 $VIRTLOGD_ARGS > ExecReload=/bin/kill -USR1 $MAINPID > # Loosing the logs is a really bad thing that will > # cause the machine to be fenced (rebooted), so make I'm fairly sure this is not safe on its own. virLogDaemonInhibitor only inhibits timer shutdown for the unprivileged daemon. This setting a timeout will cause the virtlogd to shutdown even when log files are open. I can't remember why I special cased this in the code now, but fairly sure we'll need to fix that first.

Can you test to ensure that they don't prematurely shut down when logs or locks are held.