* [OSSTEST PATCH] Executive HTML output: Use #888888 (grey) for queued jobs
@ 2015-09-30 15:05 Ian Jackson
2015-09-30 15:11 ` Ian Jackson
0 siblings, 1 reply; 3+ messages in thread
From: Ian Jackson @ 2015-09-30 15:05 UTC (permalink / raw)
To: xen-devel; +Cc: Ian Jackson, Ian Campbell
Either the flight hasn't started yet, or the job is blocked waiting
for other jobs to finish. In any case their state is not very
interesting.
Most usefully this change visually distinguishes, in the plan summary,
jobs which are waiting for prior jobs to finish, from ones which have
entered the planning queue.
Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
Osstest/Executive.pm | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/Osstest/Executive.pm b/Osstest/Executive.pm
index aeb8c25..d127bbb 100644
--- a/Osstest/Executive.pm
+++ b/Osstest/Executive.pm
@@ -221,6 +221,7 @@ our $red= '#ff8888';
our $yellow= '#ffff00';
our $purple= '#ff00ff';
our $blue= '#0000ff';
+our $grey= '#888888';
sub report_run_getinfo ($) {
# $f is a joined flight/job row, must contain at least
@@ -272,8 +273,10 @@ END
};
} elsif ($status eq 'blocked') {
return $single->("blocked", $purple),
+ } elsif ($status eq '(queued)') {
+ return $single->("($status)", $grey),
} else {
- return $single->("($f->{status})", $yellow);
+ return $single->("($status)", $yellow);
}
}
--
1.7.10.4
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [OSSTEST PATCH] Executive HTML output: Use #888888 (grey) for queued jobs
2015-09-30 15:05 [OSSTEST PATCH] Executive HTML output: Use #888888 (grey) for queued jobs Ian Jackson
@ 2015-09-30 15:11 ` Ian Jackson
2015-09-30 15:28 ` Ian Campbell
0 siblings, 1 reply; 3+ messages in thread
From: Ian Jackson @ 2015-09-30 15:11 UTC (permalink / raw)
To: xen-devel, Ian Campbell
Ian Jackson writes ("[OSSTEST PATCH] Executive HTML output: Use #888888 (grey) for queued jobs"):
> Either the flight hasn't started yet, or the job is blocked waiting
> for other jobs to finish. In any case their state is not very
> interesting.
>
> Most usefully this change visually distinguishes, in the plan summary,
> jobs which are waiting for prior jobs to finish, from ones which have
> entered the planning queue.
I didn't test this. Ideally it would be tested by applying to the
thing which is currently generating /~ianc/summary.html on the
production VM.
> + } elsif ($status eq '(queued)') {
> + return $single->("($status)", $grey),
The '(queued)' is wrong.
Ian.
>From f0d8434ae0414b97898e67be5681c6d7e5318e14 Mon Sep 17 00:00:00 2001
From: Ian Jackson <ian.jackson@eu.citrix.com>
Date: Wed, 30 Sep 2015 16:02:49 +0100
Subject: [OSSTEST PATCH] Executive HTML output: Use #888888 (grey) for queued
jobs
Either the flight hasn't started yet, or the job is blocked waiting
for other jobs to finish. In any case their state is not very
interesting.
Most usefully this change visually distinguishes, in the plan summary,
jobs which are waiting for prior jobs to finish, from ones which have
entered the planning queue.
Also replace a $f->{status} with $status, which is less confusing.
Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
v2: Get test right.
Update commit message.
---
Osstest/Executive.pm | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/Osstest/Executive.pm b/Osstest/Executive.pm
index aeb8c25..4cc34c7 100644
--- a/Osstest/Executive.pm
+++ b/Osstest/Executive.pm
@@ -221,6 +221,7 @@ our $red= '#ff8888';
our $yellow= '#ffff00';
our $purple= '#ff00ff';
our $blue= '#0000ff';
+our $grey= '#888888';
sub report_run_getinfo ($) {
# $f is a joined flight/job row, must contain at least
@@ -272,8 +273,10 @@ END
};
} elsif ($status eq 'blocked') {
return $single->("blocked", $purple),
+ } elsif ($status eq 'queued') {
+ return $single->("($status)", $grey),
} else {
- return $single->("($f->{status})", $yellow);
+ return $single->("($status)", $yellow);
}
}
--
1.7.10.4
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [OSSTEST PATCH] Executive HTML output: Use #888888 (grey) for queued jobs
2015-09-30 15:11 ` Ian Jackson
@ 2015-09-30 15:28 ` Ian Campbell
0 siblings, 0 replies; 3+ messages in thread
From: Ian Campbell @ 2015-09-30 15:28 UTC (permalink / raw)
To: Ian Jackson, xen-devel
On Wed, 2015-09-30 at 16:11 +0100, Ian Jackson wrote:
> Ian Jackson writes ("[OSSTEST PATCH] Executive HTML output: Use #888888
> (grey) for queued jobs"):
> > Either the flight hasn't started yet, or the job is blocked waiting
> > for other jobs to finish. In any case their state is not very
> > interesting.
> >
> > Most usefully this change visually distinguishes, in the plan summary,
> > jobs which are waiting for prior jobs to finish, from ones which have
> > entered the planning queue.
>
> I didn't test this. Ideally it would be tested by applying to the
> thing which is currently generating /~ianc/summary.html on the
> production VM.
I've just thrown it in there.
> > + } elsif ($status eq '(queued)') {
> > + return $single->("($status)", $grey),
>
> The '(queued)' is wrong.
>
> Ian.
>
> From f0d8434ae0414b97898e67be5681c6d7e5318e14 Mon Sep 17 00:00:00 2001
> From: Ian Jackson <ian.jackson@eu.citrix.com>
> Date: Wed, 30 Sep 2015 16:02:49 +0100
> Subject: [OSSTEST PATCH] Executive HTML output: Use #888888 (grey) for
> queued
> jobs
>
> Either the flight hasn't started yet, or the job is blocked waiting
> for other jobs to finish. In any case their state is not very
> interesting.
>
> Most usefully this change visually distinguishes, in the plan summary,
> jobs which are waiting for prior jobs to finish, from ones which have
> entered the planning queue.
>
> Also replace a $f->{status} with $status, which is less confusing.
>
> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
> ---
> v2: Get test right.
> Update commit message.
> ---
> Osstest/Executive.pm | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/Osstest/Executive.pm b/Osstest/Executive.pm
> index aeb8c25..4cc34c7 100644
> --- a/Osstest/Executive.pm
> +++ b/Osstest/Executive.pm
> @@ -221,6 +221,7 @@ our $red= '#ff8888';
> our $yellow= '#ffff00';
> our $purple= '#ff00ff';
> our $blue= '#0000ff';
> +our $grey= '#888888';
>
> sub report_run_getinfo ($) {
> # $f is a joined flight/job row, must contain at least
> @@ -272,8 +273,10 @@ END
> };
> } elsif ($status eq 'blocked') {
> return $single->("blocked", $purple),
> + } elsif ($status eq 'queued') {
> + return $single->("($status)", $grey),
> } else {
> - return $single->("($f->{status})", $yellow);
> + return $single->("($status)", $yellow);
> }
> }
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-09-30 15:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-30 15:05 [OSSTEST PATCH] Executive HTML output: Use #888888 (grey) for queued jobs Ian Jackson
2015-09-30 15:11 ` Ian Jackson
2015-09-30 15:28 ` Ian Campbell
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).