[Xen-devel] [OSSTEST PATCH 01/26] Executive: Do update flights_harness_touched for play flights

[Ian Jackson <ian.jackson@xxxxxxxxxxxxx>]

Have \bplay\b simply bypass the blessing check, but not the harness
revision update which follows.

Signed-off-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
---
 Osstest/JobDB/Executive.pm | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/Osstest/JobDB/Executive.pm b/Osstest/JobDB/Executive.pm
index c0af21c..51c1ebb 100644
--- a/Osstest/JobDB/Executive.pm
+++ b/Osstest/JobDB/Executive.pm
@@ -108,9 +108,11 @@ END
 
     die "modifying flight $fl but flight not found\n"
         unless defined $bless;
-    return if $bless =~ m/\bplay\b/;
-    die "modifying flight $fl blessing $bless expected @$flok\n"
-        unless grep { $_ eq $bless } @$flok;
+
+    unless ($bless =~ m/\bplay\b/) {
+       die "modifying flight $fl blessing $bless expected @$flok\n"
+           unless grep { $_ eq $bless } @$flok;
+    }
 
     my $rev = get_harness_rev();
 
-- 
2.1.4


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