From 613b00cccf5d1a9c795ae8ec3b0604c052d9dbff Mon Sep 17 00:00:00 2001
From: Mark Goddard <mark@stackhpc.com>
Date: Mon, 4 Oct 2021 09:08:15 +0100
Subject: [PATCH] CI: future-proof return semantics in check-failure.sh

In case we add further check functions, use a return rather than exit.

Change-Id: Ie2a53028802ecde628bc9fe0b4e3236bdb588cf9
---
 tests/check-failure.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/check-failure.sh b/tests/check-failure.sh
index 46b5241193..dbc7bb40eb 100755
--- a/tests/check-failure.sh
+++ b/tests/check-failure.sh
@@ -30,7 +30,7 @@ check_failure() {
     # NOTE(mgoddard): monasca-thresh is a one-shot container that exits but
     # remains in place, leaving it with a status of exited. This is harmless.
     if [[ "$failed_containers" = "monasca_thresh" ]]; then
-        exit 0
+        return
     fi
 
     if [[ -n "$failed_containers" ]]; then