particle-os/debian-forge-composer
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Check for unbound variables in cleanup()

Browse source
This commit is contained in:
Alexander Todorov 2024-02-29 14:48:54 +02:00 • committed by Tomáš Hozza
parent e4b514ca3a
commit 8f28da5214
1 changed files with 10 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

14
test/cases/openshift_virtualization.sh
View file

@ -14,14 +14,20 @@ BUILD_ID="${CI_JOB_ID:-$(uuidgen)}"
TEMPDIR=$(mktemp -d)
function cleanup() {
greenprint "== Script execution stopped or finished - Cleaning up =="
# TODO: this needs to be added to cloud-cleaner as well
$OC_CLI delete vm "$VM_NAME"
$OC_CLI delete pvc "$PVC_NAME"
# since this function can be called at any time, ensure that we don't expand unbound variables
OC_CLI="${OC_CLI:-}"
VM_NAME="${VM_NAME:-}"
PVC_NAME="${PVC_NAME:-}"
[[ "$OC_CLI" && "$VM_NAME" ]] && $OC_CLI delete vm "$VM_NAME"
[[ "$OC_CLI" && "$PVC_NAME" ]] && $OC_CLI delete pvc "$PVC_NAME"
sudo rm -rf "$TEMPDIR"
# Stop watching the worker journal when exiting.
sudo pkill -P "${WORKER_JOURNAL_PID}"
WORKER_JOURNAL_PID="${WORKER_JOURNAL_PID:-}"
[ "$WORKER_JOURNAL_PID" ] && sudo pkill -P "$WORKER_JOURNAL_PID"
}
trap cleanup EXIT