Re: [PATCH] libxl: Implement domainGetMessages API

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

 



On Tue, Dec 14, 2021 at 09:14:38 +0000, Daniel P. Berrangé wrote:
> On Tue, Dec 14, 2021 at 10:05:26AM +0100, Peter Krempa wrote:
> > On Mon, Dec 13, 2021 at 17:35:36 -0700, Jim Fehlig wrote:
> > > Since commit 46783e6307a, the 'virsh dominfo' command calls
> > > virDomainGetMessages to report any messages from the domain.
> > > Hypervisors not implementing the API now get the following
> > > log message when clients invoke 'virsh dominfo'
> > > 
> > > this function is not supported by the connection driver: virDomainGetMessages
> > > 
> > > Although libxl currently does not support any tainting or
> > > deprecation messages, provide an implementation to squelch
> > > the previously unseen error message when collecting dominfo.
> > 
> > So you are fixing a symptom, but IMO the proper fix is to just make
> > virsh ignore errors when virDomainGetMessages is not supported.
> > 
> > I have nothing against the code itself, but I wouldn't describe it as a
> > fix for virsh showing an error.
> 
> I interpreted it as meaning that message appears in logs on
> libvirtd side ?

Yeah, I got to that on a second read which I did after sending the
original reply and looking at the code. The commit message put too much
emphasis on virsh and mentions 'log' only once so that's why I missed
it.




[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]

  Powered by Linux