* [PATCH] OSSTest: some more info when capturing logs
@ 2015-06-24 16:11 Dario Faggioli
2015-06-25 8:48 ` Ian Campbell
0 siblings, 1 reply; 4+ messages in thread
From: Dario Faggioli @ 2015-06-24 16:11 UTC (permalink / raw)
To: xen-devel; +Cc: Ian Jackson, Ian Campbell
[-- Attachment #1.1: Type: text/plain, Size: 987 bytes --]
more specifically:
* '-n' switch to xl info, to show the topology
of the host,
* 'xl vcpu-list', to show what the various vcpus
are up to.
Signed-off-by: Dario Faggioli <dario.faggioli@citrix.com>
---
ts-logs-capture | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/ts-logs-capture b/ts-logs-capture
index 2078459..2498416 100755
--- a/ts-logs-capture
+++ b/ts-logs-capture
@@ -153,9 +153,10 @@ sub fetch_logs_host_guests () {
foreach my $cmd
(
- 'xl info',
+ 'xl info -n',
'cat /proc/cpuinfo',
'xl list',
+ 'xl vcpu-list',
'xm list',
'xm list --long',
'xenstore-ls -fp',
--
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 181 bytes --]
[-- Attachment #2: Type: text/plain, Size: 126 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] OSSTest: some more info when capturing logs
2015-06-24 16:11 [PATCH] OSSTest: some more info when capturing logs Dario Faggioli
@ 2015-06-25 8:48 ` Ian Campbell
2015-06-25 9:11 ` Dario Faggioli
0 siblings, 1 reply; 4+ messages in thread
From: Ian Campbell @ 2015-06-25 8:48 UTC (permalink / raw)
To: Dario Faggioli; +Cc: Ian Jackson, xen-devel
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"?
IOW could we just collect xl info -n and be done with it?
> * 'xl vcpu-list', to show what the various vcpus
> are up to.
>
> Signed-off-by: Dario Faggioli <dario.faggioli@citrix.com>
> ---
> ts-logs-capture | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/ts-logs-capture b/ts-logs-capture
> index 2078459..2498416 100755
> --- a/ts-logs-capture
> +++ b/ts-logs-capture
> @@ -153,9 +153,10 @@ sub fetch_logs_host_guests () {
>
> foreach my $cmd
> (
> - 'xl info',
> + 'xl info -n',
> 'cat /proc/cpuinfo',
> 'xl list',
> + 'xl vcpu-list',
> 'xm list',
> 'xm list --long',
> 'xenstore-ls -fp',
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] OSSTest: some more info when capturing logs
2015-06-25 8:48 ` Ian Campbell
@ 2015-06-25 9:11 ` Dario Faggioli
2015-06-25 9:27 ` Ian Campbell
0 siblings, 1 reply; 4+ messages in thread
From: Dario Faggioli @ 2015-06-25 9:11 UTC (permalink / raw)
To: Ian Campbell; +Cc: Ian Jackson, xen-devel
[-- Attachment #1.1: Type: text/plain, Size: 1054 bytes --]
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?
Dario
--
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 181 bytes --]
[-- Attachment #2: Type: text/plain, Size: 126 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] OSSTest: some more info when capturing logs
2015-06-25 9:11 ` Dario Faggioli
@ 2015-06-25 9:27 ` Ian Campbell
0 siblings, 0 replies; 4+ messages in thread
From: Ian Campbell @ 2015-06-25 9:27 UTC (permalink / raw)
To: Dario Faggioli; +Cc: Ian Jackson, xen-devel
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>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-06-25 9:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-06-24 16:11 [PATCH] OSSTest: some more info when capturing logs Dario Faggioli
2015-06-25 8:48 ` Ian Campbell
2015-06-25 9:11 ` Dario Faggioli
2015-06-25 9:27 ` Ian Campbell
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.