From: Markos Chandras <mchandras@suse.de>
Date: Tue, 24 Jan 2017 10:37:16 +0000 (+0000)
Subject: jjb: infra: bifrost-verify.sh: Replace 'exit' with 'return'
X-Git-Tag: danube.1.0~427^2
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F43%2F27443%2F1;p=releng.git

jjb: infra: bifrost-verify.sh: Replace 'exit' with 'return'

When there are no bifrost logs (possibly because the build broke
far too early) we shouldn't exit at all. This is a valid scenario so
we should return back to the caller and exit using the bifrost
'exit code' instead.

Change-Id: I55a847dea7a6f4209b0e4fb71fad6bb00f6df08b
Signed-off-by: Markos Chandras <mchandras@suse.de>
---

diff --git a/jjb/infra/bifrost-verify.sh b/jjb/infra/bifrost-verify.sh
index dbe39762c..48f916e72 100755
--- a/jjb/infra/bifrost-verify.sh
+++ b/jjb/infra/bifrost-verify.sh
@@ -24,7 +24,7 @@ function upload_logs() {
     gsutil -q cp -Z ${WORKSPACE}/build_log.txt ${BIFROST_GS_URL}/build_log.txt
     rm ${WORKSPACE}/build_log.txt
 
-    [[ ! -d ${WORKSPACE}/logs ]] && exit 0
+    [[ ! -d ${WORKSPACE}/logs ]] && return 0
 
     pushd ${WORKSPACE}/logs/ &> /dev/null
     for x in *.log; do