Skip to content

Commit

Permalink
Merge pull request openshift#3492 from ggiguash/fix_retries
Browse files Browse the repository at this point in the history
USHIFT-3136: Fix remove_vm function to keep info files when not fully deleted
  • Loading branch information
openshift-merge-bot[bot] committed Jun 17, 2024
2 parents 6d6945d + 3349eeb commit 9fe470a
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions test/bin/scenario.sh
Original file line number Diff line number Diff line change
Expand Up @@ -647,14 +647,14 @@ remove_vm() {
# Remove the pool directory
# ShellCheck: Using "${var:?}" to ensure this never expands to '/*'
rm -rf "${VM_DISK_BASEDIR:?}/${vm_pool_name}"

# Remove the info file so something processing the VMs does not
# assume the file exists. This is most useful in a local setting.
rm -rf "${SCENARIO_INFO_DIR}/${SCENARIO}/vms/${vmname}"
else
# Remove VM disk files
rm -f "${VM_DISK_BASEDIR}/${vm_pool_name}/*"
fi

# Remove the info file so something processing the VMs does not
# assume the file exists. This is most useful in a local setting.
rm -rf "${SCENARIO_INFO_DIR}/${SCENARIO}/vms/${vmname}"
}

# Configure the firewall in the VM based on the instructions in the documentation.
Expand Down

0 comments on commit 9fe470a

Please sign in to comment.