fix scripts for perf tests (#5276)

knative · Aug 25, 2019 · c49080b · c49080b
1 parent a0a32a7
commit c49080b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/test/performance/tools/common.sh b/test/performance/tools/common.sh
@@ -69,13 +69,13 @@ function setup_user() {
 # $1 -> cluster_name, $2 -> cluster_zone, $3 -> node_count
 function create_new_cluster() {
   # create a new cluster
-  create_cluster $1 $2 $3
+  create_cluster $1 $2 $3 || abort "Failed to create the new cluster $1"
 
   # create the secret on the new cluster
-  create_secret $1 $2
+  create_secret $1 $2 || abort "Failed to create secrets on the new cluster"
 
   # update components on the cluster, e.g. serving and istio
-  update_cluster $1 $2
+  update_cluster $1 $2 || abort "Failed to update the cluster"
 }
 
 # Update resources installed on the cluster with the up-to-date code.

diff --git a/test/performance/tools/create_cluster_benchmark.sh b/test/performance/tools/create_cluster_benchmark.sh
@@ -30,9 +30,9 @@ while [[ $# -ne 0 ]]; do
     shift
 done
 
-[[ ! -z "$CLUSTER_NAME" ]] || fail_test "Cluster name not set"
-[[ ! -z "$NUM_NODES" ]] || fail_test "Number of nodes not set"
-[[ ! -z "$CLUSTER_REGION" ]] || fail_test "Cluster region not set"
+[[ ! -z "$CLUSTER_NAME" ]] || abort "Cluster name not set"
+[[ ! -z "$NUM_NODES" ]] || abort "Number of nodes not set"
+[[ ! -z "$CLUSTER_REGION" ]] || abort "Cluster region not set"
 
 create_new_cluster $CLUSTER_NAME $CLUSTER_REGION $NUM_NODES