From: Ian Jackson <ian.jack...@eu.citrix.com>
* When we are doing archaeology, searching for flight(s) which ran a
particular testid, ignore all flights where the testid was skipped.
* In a flight we are examining for failures we need to justify, do not
regard `skip' as a failure which requires investigation. We
thusg treat `skip' in such a flight very like `pass'.
* Assign a colour (dark grey, almost like the background) and display
priority (very low) to `skip', so that they turn up nicely in the
HTML grids.
Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com>
---
sg-report-flight | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/sg-report-flight b/sg-report-flight
index ca9ba18..dbb17be 100755
--- a/sg-report-flight
+++ b/sg-report-flight
@@ -162,6 +162,7 @@ sub findaflight ($$$$$) {
my $checkq= db_prepare(<<END);
SELECT status FROM steps WHERE flight=? AND job=? AND testid=?
+ AND status!='skip'
END
my $thatflight= $specver{that}{flight};
@@ -460,7 +461,7 @@ END
my $fi= { Job => $j, Step => $s };
- if ($s->{status} ne 'pass') {
+ if ($s->{status} ne 'pass' && $s->{status} ne 'skip') {
$fi= $storefail->($s);
$foundfail= 1;
}
@@ -1083,7 +1084,8 @@ END
sub html_status2_colour_priority ($) {
my ($st) = @_;
- return ($st eq 'pass' ? ('#008800',100) :
+ return ($st eq 'skip' ? ('#555555', 50) :
+ $st eq 'pass' ? ('#008800',100) :
$st eq 'fail' ? ('#ff8888',200) :
$st eq 'running' ? ('#666666',250) :
('#ffff00',300));
--
2.1.4
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
