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 :| -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list