From: Ian Jackson <ian.jackson@eu.citrix.com>
Date: Thu, 20 Apr 2017 13:04:36 +0000 (+0000)
Subject: sg-report-flight: Do not report many non-regressions as allowable
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=ccc34a7d17ec7759ab416120f8acf47fff2be600;p=people%2Fiwj%2Fosstest.git

sg-report-flight: Do not report many non-regressions as allowable

In bd648aea2ebe
 sg-report-flight: report allowable regressions separately in summary
whether the regression was allowable was put in $allow, but this
was erroneously not used.

In 4a210cda9cc8
 sg-report-flight: fix allowable failure logic not to reuse $allow for two purposes (!)
that use of the variable, which was never used, was removed.

However, what ought to happen, is that $failv->{Allow} would be set to
1 if the entry matched an allow pattern, and only if it was a blocker.
That makes the logic in "report ... separately" correct.

Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
---

diff --git a/sg-report-flight b/sg-report-flight
index 5b8f67a3..5e362b91 100755
--- a/sg-report-flight
+++ b/sg-report-flight
@@ -888,6 +888,7 @@ END
             print DEBUG " allow $item?";
             foreach my $allowpat (@allows) {
                 next unless $item =~ m/$allowpat/;
+                $failv->{Allow}= 1;
                 $blocker='';
                 print DEBUG " allowed";
                 last;
@@ -895,7 +896,6 @@ END
             print DEBUG "\n";
         }
 	$failv->{Blocker}= $blocker;
-        $failv->{Allow}= 1;
 	$failv->{SummaryCore}= $rtups[0][2];
 	$failv->{SummaryRefFlight}= $rtups[0][3];
 	$failv->{Summary}= $failv->{SummaryCore};