Browse code

cleanup of extended cleanup traps

deads2k authored on 2016/01/21 03:42:20
Showing 4 changed files
... ...
@@ -19,6 +19,7 @@ for test_script in $test_scripts; do
19 19
 	echo "${STARTTIME} starting ${test_script}";
20 20
 
21 21
 	set +e
22
+	# use a subshell to prevent `exit` calls from killing this script
22 23
 	(${test_script})
23 24
 	CURR_RETURN=$?
24 25
 	set -e
... ...
@@ -23,14 +23,13 @@ export BASETMPDIR="${TMPDIR}/openshift-extended-tests/authentication"
23 23
 export EXTENDED_TEST_PATH="${OS_ROOT}/test/extended"
24 24
 export KUBE_REPO_ROOT="${OS_ROOT}/../../../k8s.io/kubernetes"
25 25
 
26
-function join { local IFS="$1"; shift; echo "$*"; }
27
-
28
-
29 26
 function cleanup()
30 27
 {
28
+	out=$?
31 29
 	docker rmi test/scratchimage
32 30
 	cleanup_openshift
33 31
 	echo "[INFO] Exiting"
32
+	return $out
34 33
 }
35 34
 
36 35
 trap "exit" INT TERM
... ...
@@ -98,7 +98,7 @@ if [[ -z ${TEST_ONLY+x} ]]; then
98 98
     out=$?
99 99
     cleanup_openshift
100 100
     echo "[INFO] Exiting"
101
-    exit $out
101
+    return $out
102 102
   }
103 103
 
104 104
   trap "exit" INT TERM
... ...
@@ -30,7 +30,7 @@ function cleanup()
30 30
 	out=$?
31 31
 	cleanup_openshift
32 32
 	echo "[INFO] Exiting"
33
-	exit $out
33
+	return $out
34 34
 }
35 35
 
36 36
 trap "exit" INT TERM