Re: [PATCH] qemu: logrotate: drop minsize directive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/26/2017 11:14 AM, Daniel P. Berrange wrote:
On Thu, Oct 26, 2017 at 11:13:23AM -0600, Jim Fehlig wrote:
On 10/26/2017 12:48 AM, Daniel P. Berrange wrote:
On Wed, Oct 25, 2017 at 03:30:46PM -0600, Jim Fehlig wrote:
On a cloud host it is possible to create 100's of unique instances
per day, each leaving behind a /var/log/libvirt/qemu/instance-name.log
file that is < 100k. With the current 'minsize 100k' directive, these
files are never rotated and hence never removed. Over months of time,
tens of thousands of these files can accumulate on the host.

Dropping 'minsize 100k' allows rotating small files, which will
increase the number of log files, but 'rotate 4' ensures they will
be removed after a month.

Signed-off-by: Jim Fehlig <jfehlig@xxxxxxxx>
---
   daemon/libvirtd.qemu.logrotate.in | 1 -
   1 file changed, 1 deletion(-)

diff --git a/daemon/libvirtd.qemu.logrotate.in b/daemon/libvirtd.qemu.logrotate.in
index 15cf019b2..cdb399ef2 100644
--- a/daemon/libvirtd.qemu.logrotate.in
+++ b/daemon/libvirtd.qemu.logrotate.in
@@ -5,5 +5,4 @@
           compress
           delaycompress
           copytruncate
-        minsize 100k
   }

Reviewed-by: Daniel P. Berrange <berrange@xxxxxxxxxx>

Only after pushing this did I remember there are logrotate files for other
hypervisor drivers: libxl, lxc, and uml. Would it be fine to push a followup
that removes minsize from those files as well?

Yep, that makes sense.

Thanks. Followup patch is now pushed.

Regards,
Jim

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux