Merge pull request #3697 from k8s-infra-cherrypick-robot/cherry-pick-3686-to-release-1.8

[release-1.8] renaming cleanup function in ci-build-azure-ccm.sh and ci-build-kubernetes.sh

Author: k8s-ci-robot

hack/verify-starlark.sh

@@ -39,7 +39,7 @@ TMP_DIR=$(mktemp -d)
 OUT="${TMP_DIR}/out.log"
 
 # cleanup on exit
-cleanup() {
+capz::verify-starlark::cleanup() {
   ret=0
   if [[ -s "${OUT}" ]]; then
     echo "Found errors:"
@@ -52,7 +52,7 @@ cleanup() {
   rm -rf "${TMP_DIR}"
   exit ${ret}
 }
-trap cleanup EXIT
+trap capz::verify-starlark::cleanup EXIT
 
 BUILDIFIER="${SCRIPT_DIR}/tools/bin/buildifier/${VERSION}/buildifier"
 

scripts/ci-build-azure-ccm.sh

@@ -90,13 +90,14 @@ can_reuse_artifacts() {
     echo "true"
 }
 
-cleanup() {
+capz::ci-build-azure-ccm::cleanup() {
+    echo "cloud-provider-azure cleanup"
     if [[ -d "${AZURE_CLOUD_PROVIDER_ROOT:-}" ]]; then
         make -C "${AZURE_CLOUD_PROVIDER_ROOT}" clean || true
     fi
 }
 
-trap cleanup EXIT
+trap capz::ci-build-azure-ccm::cleanup EXIT
 
 setup
 main

scripts/ci-build-kubernetes.sh

@@ -152,13 +152,13 @@ can_reuse_artifacts() {
     echo "true"
 }
 
-cleanup() {
+capz::ci-build-kubernetes::cleanup() {
     if [[ -d "${KUBE_ROOT:-}" ]]; then
         make -C "${KUBE_ROOT}" clean || true
     fi
 }
 
-trap cleanup EXIT
+trap capz::ci-build-kubernetes::cleanup EXIT
 
 setup
 main

scripts/ci-conformance.sh

@@ -78,11 +78,11 @@ export WINDOWS="${WINDOWS:-false}"
 # Generate SSH key.
 capz::util::generate_ssh_key
 
-cleanup() {
+capz::ci-conformance::cleanup() {
     "${REPO_ROOT}/hack/log/redact.sh" || true
 }
 
-trap cleanup EXIT
+trap capz::ci-conformance::cleanup EXIT
 
 if [[ "${WINDOWS}" == "true" ]]; then
   make test-windows-upstream

scripts/ci-e2e.sh

@@ -77,11 +77,11 @@ export CALICO_VERSION
 
 capz::util::generate_ssh_key
 
-cleanup() {
+capz::ci-e2e::cleanup() {
     "${REPO_ROOT}/hack/log/redact.sh" || true
 }
 
-trap cleanup EXIT
+trap capz::ci-e2e::cleanup EXIT
 # Image is configured as `${CONTROLLER_IMG}-${ARCH}:${TAG}` where `CONTROLLER_IMG` is defaulted to `${REGISTRY}/${IMAGE_NAME}`.
 if [[ "${BUILD_MANAGER_IMAGE}" == "false" ]]; then
   # Load an existing image, skip docker-build and docker-push.

scripts/ci-entrypoint.sh

@@ -304,7 +304,7 @@ copy_secret() {
     rm azure_json
 }
 
-on_exit() {
+capz::ci-entrypoint::on_exit() {
     if [[ -n ${KUBECONFIG:-} ]]; then
         "${KUBECTL}" get nodes -o wide || echo "Unable to get nodes"
         "${KUBECTL}" get pods -A -o wide || echo "Unable to get pods"
@@ -324,7 +324,7 @@ on_exit() {
 # setup all required variables and images
 setup
 
-trap on_exit EXIT
+trap capz::ci-entrypoint::on_exit EXIT
 export ARTIFACTS="${ARTIFACTS:-${PWD}/_artifacts}"
 
 # create cluster