[Libvir] [PATCH] output virsh log to file

[Nobuhiro Itou]

Hi, Daniel

Thank you for your comment.

>   My experience with systematic logging at command line level is that
> you end up polluting your file system with (hidden) log files, and while it
> may be fine for a developper it's not something you want to inflict to
> your customers. Also the fixed name for the log file makes it easilly
> useless, a random run in the same directory will just wipe out the
> data you tried to collect in a separate process. 
>   I think having permanent systematic logging to a fixed file is not
> proper, I would rather not use that.
>   But since the patch is relatively simple based on existing virsh logging
> code, I think this could go as a command line option for virsh, for example
>    --log filename
> where the detailed logs can then be saved if needed when a problem occurs.
> I think this would avoid the main drawbacks of your proposed patch.

I agree about a command line option.
So, I remaked the patch which --log option is added for virsh.
How about this one?

Signed-off-by: Nobuhiro Itou <fj0873gn at aa.jp.fujitsu.com>


Thanks,
Nobuhiro Itou.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: virsh_log.patch
Type: application/octet-stream
Size: 12819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20070601/6a73b11f/attachment-0001.obj>