On Thu, Dec 04, 2008 at 05:44:30PM +0100, Guido Günther wrote: > On Thu, Dec 04, 2008 at 01:26:40PM +0000, Daniel P. Berrange wrote: > > On Thu, Dec 04, 2008 at 02:11:07PM +0100, Guido G?nther wrote: > > > Hi, > > > attached patch is from Debian Bug #507553: > > > > > > The logfile is opened with O_APPEND (to make logrotate work with > > > copytruncate) and without O_TRUNC (to not lose log information e.g. with > > > stop and start of a VM). > > > > > > Using collectd or munin to monitor VMs creates quiet a bit of qemu > > > monitor output so the ability to use logrotate makes a lot of sense to > > > me. > > > > And is there a corresponding logrotate config file from Debian we > > can use ? If we're going to let log files append, then we need to > > ship a logrotate config by default too. > I've changed the patch to only use O_APPEND as root so we don't fill up > the user's log when running as qemu:///session > > > ACK if we add a logrotate config too > New version attached. O.k. to apply? Looks fine to me, we will need to update the spec file but that can be done later, Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@xxxxxxxxxxxx | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/ -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list