[Xen-devel] [OSSTEST PATCH 4/5] ms-flights-summary: Treat preparing jobs outside the plan as `queued'

[Ian Jackson <ian.jackson@xxxxxxxxxxxxx>]

These jobs are still to run but we have no time estimate for them.

So do not add them to UnqueuedJobs; they then contribute to making the
`End of phase/flight' indication *not* bold when there are such jobs.

And bolden their figure in the table.

Signed-off-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
---
 ms-flights-summary | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/ms-flights-summary b/ms-flights-summary
index 9a814ea..e4ce03b 100755
--- a/ms-flights-summary
+++ b/ms-flights-summary
@@ -132,7 +132,9 @@ END
     }
 
     $f->{NrJobs} = keys %{$f->{Jobs}};
-    $f->{UnqueuedJobs} = $f->{NrJobs} - ($f->{Stats}{queued}//0);
+    $f->{UnqueuedJobs} = $f->{NrJobs}
+        - ($f->{Stats}{queued}//0)
+        - ($f->{Stats}{preparing}//0);
 }
 
 sub sort_stats($) {
@@ -407,7 +409,8 @@ foreach my $f (sort keys %flights) {
     print "  <td align=right><strong>$fi->{NrJobs}</strong></td>\n";
     foreach (@summarycounts) {
        my $s = ($fi->{Stats}{$_} || '');
-       $s = "<strong>$s</strong>" if $s && $_ eq 'queued';
+       $s = "<strong>$s</strong>"
+            if $s && ($_ eq 'queued' || $_ eq 'preparing');
        print "  <td align=right>$s</td>";
     }
     my $expectedend = fmttime($fi->{ExpectedEnd});
-- 
2.1.4


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel