[libvirt] [PATCH] Do not keep empty log files for deleted domains

Peter Krempa pkrempa at redhat.com
Wed Jul 3 06:55:15 UTC 2019


On Wed, Jul 03, 2019 at 08:32:21 +0200, Jan Zerebecki wrote:
> On 03/07/2019 08.03, Peter Krempa wrote:
> > I'm not sure that this is the right thing to do. virtlogd has some
> > internal log rotation mechanisms
> 
> logrotate is already in use here and this patch doesn't change what is
> rotated nor how often it is rotated.
> 
> > and also the SIGUSR1 action is reserved
> > for re-exec updates and not for dealing with modified files
> 
> The mechanism (re-exec) and implementation details for both uses do not
> conflict.

The problem is that the logrotate configs predate use of virtlogd and I
don't think they were adapted to be used together. In fact if I remember
correctly the idea of virtlogd was to avoid using logrotate altogether.

Additionally virtlogd is currently used only with the qemu VM log files.
Everything else logs itself, so your changes to libvirtd/lxc and others
seem to be unnecessary.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20190703/9df237b6/attachment-0001.sig>


More information about the libvir-list mailing list