Re: [PATCH] output virsh log to file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi, Daniel

Thank you for committing virsh log patch.

>   Way simpler, that works, I just fixed a couple of things:
>     - removed defines for buffer size which were not needed anymore

Oops, sorry. I had forgotten to erase it.

>     - if no log file name was given do not try to open it

Okey. I agree.

> 
> I commited this in CVS but I'm afraid there is something missing, look at the
> following:
> 
> paphio:~/libvirt/src -> ./virsh --log logfile list
> libvir: error : no support for hypervisor (null)
> virsh: error: failed to connect to the hypervisor
> paphio:~/libvirt/src -> cat logfile
> [2007.06.06 14:26:30 virsh] ERROR failed to connect to the hypervisor
> paphio:~/libvirt/src -> 
> 
> the machine is not running Xen so the error is normal, the trouble is that
> the logfile only contains the virsh error not the libvirt one (which is the
> one the most useful), this need debugging I guess.

Yes. I also had recognized this problem.
I think so the libvirt error is the most useful, too.
I try to think about measures.

> Another question is that the log file is opened in append mode, while I guess
> this kind of option usually rewrite the file from scratch, but this is an 
> open question, I can see how both behaviour could be useful or annoying 
> depending on the circumstances.

Hum, I understood that you said.
But, I think that virsh must not overwrite the logfile without 
user's permission, because virsh doesn't know what the file user 
specified contains.
If it is append mode, the old contents of a file do not disappear.
User can delete it later if needless, and can keep it if need.


Thanks,
Nobuhiro Itou.


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]