diff options
| author |   Markos Chandras <mchandras@suse.de> | 2017-01-24 10:37:16 +0000 |
|---|---|---|
| committer | Markos Chandras <mchandras@suse.de> | 2017-01-24 10:39:45 +0000 |
| commit | 586a5cf54400c439f29df9e213a5ababa5d112c3 (patch) | |
| tree | f52e5f15324b3e957116f7903563268adbad7961 /jjb/infra | |
| parent | 387c212e28b6050a74affbb787c9525e6337ce08 (diff) | |
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>
Diffstat (limited to 'jjb/infra')
| -rwxr-xr-x | jjb/infra/bifrost-verify.sh | 2 |
1 files changed, 1 insertions, 1 deletions
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 |