[OSSTEST PATCH] cri-args-hostlists: Do not include transcript in reports

Ian Jackson posted 1 patch 23 weeks ago
Failed in applying to current master (apply log)
cri-args-hostlists | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)

[OSSTEST PATCH] cri-args-hostlists: Do not include transcript in reports

Posted by Ian Jackson 23 weeks ago
These transcripts are huge and not useful.

Still include them if sg-secute-flight failed.

Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
---
 cri-args-hostlists | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/cri-args-hostlists b/cri-args-hostlists
index 3578fe1c..28d576db 100644
--- a/cri-args-hostlists
+++ b/cri-args-hostlists
@@ -86,8 +86,11 @@ execute_flight () {
         ./sg-execute-flight $1 $2 >tmp/$1.transcript 2>&1
 	local rc=$?
 	set -e
-        cat tmp/$1.transcript
-	test $rc = 0
+        if [ "$rc" != 0 ]; then
+		cat tmp/$1.transcript
+		echo "rc=$rc"
+		exit 1
+	fi
 }
 
 start_email () {
-- 
2.20.1