mirror of https://github.com/linkerd/linkerd2.git
cleanup: utilise linkerd uninstall to concisely delete all the linkerd resources involved in the test (#5522)
The linkerd uninstall command is able to remove a lot of the test resources used in CI but it ends up leaving the test namespaces though. Still, the test-cleanup script can be cleaned down to a good level by getting rid of the populate_array function. Hence, this commits adds a one-liner, alongside linkerd uninstall, to deal with the deletion of all the test namespaces and the resources instead of using the big chunk of populate_array function. Fixes #5497 Signed-off-by: Yashvardhan Kukreja <yash.kukreja.98@gmail.com>
This commit is contained in:
Yashvardhan Kukreja
GitHub
parent
40192e258a
commit
06dccac35b
|
|
@ -10,24 +10,6 @@ handle_cleanup_input "$@"
|
|||
|
||||
check_linkerd_binary
|
||||
|
||||
populate_array() {
|
||||
[ $# -ge 3 ] || {
|
||||
echo "function ${FUNCNAME[0]} expects 3 mandatory inparameters but got $#
|
||||
Usage: ${FUNCNAME[0]} TARGET_ARRAY RESOURCE_TYPE LINKERD_LABEL (FRIENDLY_NAME)" >&2
|
||||
return
|
||||
}
|
||||
|
||||
# We are using the variable i even if shellsheck does not see it
|
||||
# shellcheck disable=SC2034
|
||||
local i=0
|
||||
unset -v "$1"
|
||||
while IFS= read -r "$1[i++]"; do :; done <<< "$(kubectl --context="$k8s_context" get -A "$2" -oname -l "linkerd.io/$3")"
|
||||
# ensure that the last element isn't empty
|
||||
eval "[[ \${$1[--i]} ]]" || unset "$1[i]" # ensures last element isn't empty
|
||||
# inform user if no results were returned from command
|
||||
[ -z "$(eval "[[ \${$1[0]} ]]")" ] && echo "no ${4:-$2} found" >&2
|
||||
}
|
||||
|
||||
echo "cleaning up viz extension resources, if present [${k8s_context}]"
|
||||
"$linkerd_path" viz uninstall 2> /dev/null | kubectl --context="$k8s_context" delete -f -
|
||||
|
||||
|
|
@ -37,69 +19,11 @@ echo "cleaning up multicluster resources, if present [${k8s_context}]"
|
|||
echo "cleaning up jaeger extension resources, if present [${k8s_context}]"
|
||||
"$linkerd_path" jaeger uninstall 2> /dev/null | kubectl --context="$k8s_context" delete -f -
|
||||
|
||||
# Initialize empty arrays
|
||||
namespaces_controlplane=()
|
||||
namespaces_cni=()
|
||||
namespaces_dataplane=()
|
||||
clusterrolebindings=()
|
||||
clusterroles=()
|
||||
webhookconfigs=()
|
||||
validatingconfigs=()
|
||||
podsecuritypolicies=()
|
||||
customresourcedefinitions=()
|
||||
apiservices=()
|
||||
rolebindings=()
|
||||
echo "cleaning up linkerd resources [${k8s_context}]"
|
||||
"$linkerd_path" uninstall 2> /dev/null | kubectl --context="$k8s_context" delete -f -
|
||||
|
||||
echo "cleaning up control-plane namespaces in k8s-context [${k8s_context}]"
|
||||
|
||||
populate_array namespaces_controlplane ns is-control-plane 'control-plane namespaces'
|
||||
|
||||
echo "cleaning up CNI namespaces in k8s-context [${k8s_context}]"
|
||||
|
||||
populate_array namespaces_cni ns cni-resource 'CNI namespaces'
|
||||
|
||||
echo "cleaning up data-plane namespaces in k8s-context [${k8s_context}]"
|
||||
|
||||
populate_array namespaces_dataplane ns is-test-data-plane 'data-plane namespaces'
|
||||
populate_array clusterrolebindings clusterrolebindings control-plane-ns
|
||||
populate_array clusterroles clusterroles control-plane-ns
|
||||
populate_array webhookconfigs mutatingwebhookconfigurations control-plane-ns
|
||||
populate_array validatingconfigs validatingwebhookconfigurations control-plane-ns
|
||||
populate_array podsecuritypolicies podsecuritypolicies control-plane-ns
|
||||
populate_array customresourcedefinitions customresourcedefinitions control-plane-ns
|
||||
populate_array apiservices apiservices control-plane-ns
|
||||
|
||||
# No action if all arrays are empty
|
||||
if [[ ${namespaces_controlplane[*]} || \
|
||||
${namespaces_cni[*]} || \
|
||||
${namespaces_dataplane[*]} || \
|
||||
${clusterrolebindings[*]} || \
|
||||
${clusterroles[*]} || \
|
||||
${webhookconfigs[*]} || \
|
||||
${validatingconfigs[*]} || \
|
||||
${podsecuritypolicies[*]} || \
|
||||
${customresourcedefinitions[*]} || \
|
||||
${apiservices[*]} ]]; then
|
||||
kubectl --context="$k8s_context" delete \
|
||||
${namespaces_controlplane:+"${namespaces_controlplane[@]}"} \
|
||||
${namespaces_cni:+"${namespaces_cni[@]}"} \
|
||||
${namespaces_dataplane:+"${namespaces_dataplane[@]}"} \
|
||||
${clusterrolebindings:+"${clusterrolebindings[@]}"} \
|
||||
${clusterroles:+"${clusterroles[@]}"} \
|
||||
${webhookconfigs:+"${webhookconfigs[@]}"} \
|
||||
${validatingconfigs:+"${validatingconfigs[@]}"} \
|
||||
${podsecuritypolicies:+"${podsecuritypolicies[@]}"} \
|
||||
${customresourcedefinitions:+"${customresourcedefinitions[@]}"} \
|
||||
${apiservices:+"${apiservices[@]}"}
|
||||
fi
|
||||
|
||||
echo "cleaning up rolebindings in kube-system namespace in k8s-context [${k8s_context}]"
|
||||
populate_array rolebindings rolebindings control-plane-ns
|
||||
|
||||
# No action if array is empty
|
||||
if [[ ${rolebindings[*]} ]]; then
|
||||
kubectl --context="$k8s_context" delete "${rolebindings[@]}" -n kube-system
|
||||
fi
|
||||
echo "cleaning up the all namespaces labelled with linkerd.io/is-test-data-plane"
|
||||
kubectl --context="$k8s_context" delete ns -l linkerd.io/is-test-data-plane
|
||||
|
||||
# Helm cleanup. Just the entries in `helm ls` as the resources should have already been cleaned up by the code above.
|
||||
releases=$(helm ls -A -q)
|
||||
|
|
|
|||
Loading…
Reference in New Issue