From: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
Date: Tue, 15 Oct 2019 16:17:34 +0000 (+0200)
Subject: Always run the result parsing
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;h=c6729fc7c1f5e2c3c14c017d59a0fff641375206;p=ci-management.git

Always run the result parsing

This fixes the application builds where we have no skiplist for most.
Before this we bailed out on return 0 and did never parse the result.
Now the result parsing is it's own function and always being called.

Bug-AGL: SPEC-2882

.

Change-Id: I58c05fa48dd9121e0eeb494975513dcc88290dc5
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>
---

diff --git a/jjb/common/include-agl-lava-jobs-submit.sh b/jjb/common/include-agl-lava-jobs-submit.sh
index 154c0124..92a5ce80 100644
--- a/jjb/common/include-agl-lava-jobs-submit.sh
+++ b/jjb/common/include-agl-lava-jobs-submit.sh
@@ -102,7 +102,8 @@ handle_skiplist() {
 	done
 
 	if [ ! -e "$SKIPLIST" ];then
-		echo "DEBUG: $SKIPLIST does not exists"
+		echo "DEBUG: $SKIPLIST does not exist"
+		touch $SKIPLIST
 		return 0
 	fi
 
@@ -110,7 +111,9 @@ handle_skiplist() {
 		echo "DEBUG: $SKIPLIST is empty"
 		return 0
 	fi
+}
 
+handle_result() {
 	# example of testline: * lava.lava-test-retry [fail]
 	while read testline
 	do
@@ -137,6 +140,7 @@ grep '\[fail\]$' job-result > job-result.fail || true
 
 if [ -s job-result.fail ];then
 	handle_skiplist
+	handle_result
 fi
 
 set -x