[libvirt] Open monitor logs O_APPEND

Daniel Veillard veillard at redhat.com
Fri Dec 5 13:20:08 UTC 2008


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 at veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/




More information about the libvir-list mailing list