On Thu, Apr 16, 2009 at 03:38:40PM +0200, Guido G?nther wrote:
On Thu, Apr 16, 2009 at 01:29:08PM +0100, Daniel P. Berrange wrote:
> On Thu, Apr 16, 2009 at 02:17:29PM +0200, Guido G?nther wrote:
> > Hi,
> > currently every monitor command is dumped into the domain logfiles, this
> > can lead to huge logs[1]. Since we're not going through the normal
> > logging facilites to get per domain logfiles we should at least resepct
> > the default priority.
>
> IMHO this should just be removed. We already have ability to log all
> monitor commands via the normal logging mechanism, so we no longer have a
> need to write them out to the domain logs. We should leave the domain logs
> just for QEMU's stdout/err as we originally had them.
I'm all for it since this makes reparsing the log output safer too.
Attached patch remove the monitor output from the domain log file. To
compensate it prefixes the debug replies with the vm name so it's easier
to find out what's wrong on a crowded system.
Good idea.
ACK to this patch.
Daniel
--
|: Red Hat, Engineering, London -o-
http://people.redhat.com/berrange/ :|
|:
http://libvirt.org -o-
http://virt-manager.org -o-
http://ovirt.org :|
|:
http://autobuild.org -o-
http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|