You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
In nuke.sh, there should be a prompt(like the one in uninstall.sh) added to give users a chance to confirm before proceeding to clean up...esp nuke is a destructive script like uninstall.sh
To Reproduce
Steps to reproduce the behavior:
Run ./nuke.sh
No prompt and just proceed
Expected behavior
Added a prompt (from uninstall.sh)
echo "***"
printf "\n"
echo "This script will destroy Open Cluster Management from the current OpenShift target cluster:"
printf "\n"
oc cluster-info | head -n 1 | awk '{print $NF}'
printf "\n"
echo "If you would like to proceed with cleanup, type: DESTROY"
read -r DESTROY_YES
if [ "${DESTROY_YES}" != "DESTROY" ]; then
echo "You must type DESTROY to clean up the Hive deployed clusters"
exit 1
fi
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
OS: all applicable
Browser all applicable
Snapshot all applicable
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered:
drkho
added
bug
Something isn't working
deploy
specific to this repository... does not imply product specific issues
labels
May 20, 2021
drkho
changed the title
[BUG][DEPLOY] Should prompt users in nuke.sh for confirmation before the script proceeds
[FEATURE] Should prompt users in nuke.sh for confirmation before the script proceeds
May 20, 2021
Describe the bug
In nuke.sh, there should be a prompt(like the one in uninstall.sh) added to give users a chance to confirm before proceeding to clean up...esp nuke is a destructive script like uninstall.sh
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Added a prompt (from uninstall.sh)
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: