Re: [PATCH] OSSTest: some more info when capturing logs

[Ian Campbell <ian.campbell@citrix.com>]

On Thu, 2015-06-25 at 11:11 +0200, Dario Faggioli wrote:
> On Thu, 2015-06-25 at 09:48 +0100, Ian Campbell wrote:
> > On Wed, 2015-06-24 at 18:11 +0200, Dario Faggioli wrote:
> > > more specifically:
> > > 
> > >  * '-n' switch to xl info, to show the topology
> > >    of the host,
> > 
> > Is the information from "xl info -n" a superset of that from "xl info"?
> >
> Yes.
> 
> > IOW could we just collect xl info -n and be done with it?
> > 
> That's what the patch is doing:
> 
> > > @@ -153,9 +153,10 @@ sub fetch_logs_host_guests () {
> > >  
> > >      foreach my $cmd
> > >          (
> > > -         'xl info',
> > > +         'xl info -n',
> >
> Perhaps the changelog was not so clear... in this case, "more info"
> applies right because adding '-n' adds information, wrt not using it, or
> at least that was what I meant.
> 
> Should I clarify and resend?

No need, I should just learn to read a diff properly, sorry.

Acked-by: Ian Campbell <ian.campbell@citrix.com>