From 4063b20af4fec2440a6e3c6554fc8183a0c2c558 Mon Sep 17 00:00:00 2001
From: jinyuanliu <jayonlau@gmail.com>
Date: Sun, 17 Oct 2021 01:51:47 -0400
Subject: [PATCH] Fix helm test method about cinder

This patch makes the helm test command more standard by calling script
of common.

Change-Id: Icb4bdb5c63b1630cb63d3e43a1e5595af0f4e6bf
---
 tools/deployment/multinode/110-cinder.sh | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/tools/deployment/multinode/110-cinder.sh b/tools/deployment/multinode/110-cinder.sh
index 55f3af0ad4..d5e74c60f1 100755
--- a/tools/deployment/multinode/110-cinder.sh
+++ b/tools/deployment/multinode/110-cinder.sh
@@ -15,6 +15,7 @@ set -xe
 
 #NOTE: Get the over-rides to use
 : ${OSH_EXTRA_HELM_ARGS_CINDER:="$(./tools/deployment/common/get-values-overrides.sh cinder)"}
+: ${RUN_HELM_TESTS:="yes"}
 
 #NOTE: Lint and package chart
 make cinder
@@ -79,6 +80,7 @@ sleep 30 #NOTE(portdirect): Wait for ingress controller to update rules and rest
 openstack volume type list
 openstack volume type list --default
 
-# Delete the test pod if it still exists
-kubectl delete pods -l application=cinder,release_group=cinder,component=test --namespace=openstack --ignore-not-found
-helm test cinder --timeout 900
+# Run helm tests
+if [ "x${RUN_HELM_TESTS}" != "xno" ]; then
+    ./tools/deployment/common/run-helm-tests.sh cinder
+fi