Re: [libvirt] [PATCH] Do not keep empty log files for deleted domains
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.
Attachments:
- signature.asc (application/pgp-signature — 833 bytes)