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 154c012..92a5ce8 100644 (file)
--- 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