tests: common: Ensure test_type is used as part of the cluster's name

By doing this we can make sure there won't be any clash on the cluster
name created for either the k8s or the kata-deploy tests.

Signed-off-by: Fabiano Fidêncio <fabiano.fidencio@intel.com>
This commit is contained in:
Fabiano Fidêncio 2023-08-17 10:39:15 +02:00
parent 790bd3548d
commit 285e616b5e
3 changed files with 18 additions and 9 deletions

View File

@ -25,12 +25,12 @@ function main() {
case "${action}" in case "${action}" in
install-azure-cli) install_azure_cli ;; install-azure-cli) install_azure_cli ;;
login-azure) login_azure ;; login-azure) login_azure ;;
create-cluster) create_cluster ;; create-cluster) create_cluster "kata-deploy" ;;
install-bats) install_bats ;; install-bats) install_bats ;;
install-kubectl) install_kubectl ;; install-kubectl) install_kubectl ;;
get-cluster-credentials) get_cluster_credentials ;; get-cluster-credentials) get_cluster_credentials "kata-deploy" ;;
run-tests) run_tests ;; run-tests) run_tests ;;
delete-cluster) cleanup "aks" ;; delete-cluster) cleanup "aks" "kata-deploy" ;;
*) >&2 echo "Invalid argument"; exit 2 ;; *) >&2 echo "Invalid argument"; exit 2 ;;
esac esac
} }

View File

@ -14,8 +14,10 @@ source "${tests_dir}/common.bash"
AZ_RG="${AZ_RG:-kataCI}" AZ_RG="${AZ_RG:-kataCI}"
function _print_cluster_name() { function _print_cluster_name() {
test_type="${1:-k8s}"
short_sha="$(git rev-parse --short=12 HEAD)" short_sha="$(git rev-parse --short=12 HEAD)"
echo "${GH_PR_NUMBER}-${short_sha}-${KATA_HYPERVISOR}-${KATA_HOST_OS}-amd64" echo "${test_type}-${GH_PR_NUMBER}-${short_sha}-${KATA_HYPERVISOR}-${KATA_HOST_OS}-amd64"
} }
function install_azure_cli() { function install_azure_cli() {
@ -33,12 +35,14 @@ function login_azure() {
} }
function create_cluster() { function create_cluster() {
test_type="${1:-k8s}"
# First, ensure that the cluster didn't fail to get cleaned up from a previous run. # First, ensure that the cluster didn't fail to get cleaned up from a previous run.
delete_cluster || true delete_cluster "${test_type}" || true
az aks create \ az aks create \
-g "${AZ_RG}" \ -g "${AZ_RG}" \
-n "$(_print_cluster_name)" \ -n "$(_print_cluster_name ${test_type})" \
-s "Standard_D4s_v5" \ -s "Standard_D4s_v5" \
--node-count 1 \ --node-count 1 \
--generate-ssh-keys \ --generate-ssh-keys \
@ -61,15 +65,19 @@ function install_kubectl() {
} }
function get_cluster_credentials() { function get_cluster_credentials() {
test_type="${1:-k8s}"
az aks get-credentials \ az aks get-credentials \
-g "${AZ_RG}" \ -g "${AZ_RG}" \
-n "$(_print_cluster_name)" -n "$(_print_cluster_name ${test_type})"
} }
function delete_cluster() { function delete_cluster() {
test_type="${1:-k8s}"
az aks delete \ az aks delete \
-g "${AZ_RG}" \ -g "${AZ_RG}" \
-n "$(_print_cluster_name)" \ -n "$(_print_cluster_name ${test_type})" \
--yes --yes
} }

View File

@ -84,13 +84,14 @@ function run_tests() {
function cleanup() { function cleanup() {
platform="${1}" platform="${1}"
test_type="${2:-k8s}"
ensure_yq ensure_yq
echo "Gather information about the nodes and pods before cleaning up the node" echo "Gather information about the nodes and pods before cleaning up the node"
get_nodes_and_pods_info get_nodes_and_pods_info
if [ "${platform}" = "aks" ]; then if [ "${platform}" = "aks" ]; then
delete_cluster delete_cluster ${test_type}
return return
fi fi