-
Notifications
You must be signed in to change notification settings - Fork 38.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Aggressively delete a cluster template if it exists. #9702
Conversation
return | ||
echo "Instance template ${1} already exists; deleting." >&2 | ||
if ! gcloud compute instance-template delete "$1" --project "${PROJECT}" &>/dev/null; then | ||
echo -e "${color_yellow}Failed to delete existing instance template" >&2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should end with ${color_norm}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
LGTM w/ nits |
Comments addressed, ptal. |
I'm going to merge this, as this problem is currently breaking a number of e2e runs. |
Aggressively delete a cluster template if it exists.
Sorry for not getting back, but was truly LGTM. I'll add the label for posterity :-) |
GCE e2e build/test failed for commit e920e456f7aadd30755330dec32152f6f3c4ebf8. |
GCE e2e build/test passed for commit a44f36b295d947c1bab7389802fc0b74effff221. |
GCE e2e build/test failed for commit d91fe7c2cf5e9b8d6d6c6d7232d2e50cdb76e809. |
GCE e2e build/test failed for commit dcb09e7. |
Root cause #9016. More fixes to come. |
@mbforbes @ixdy @quinton-hoole