-
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
Delete hpa first before deleting the resources it controls in test-cmd.sh #22466
Conversation
Labelling this PR as size/XS |
LGTM, Thanks! |
@k8s-oncall: I'm going to manually merge this once tests pass in order to fix test flakes. |
GCE e2e build/test passed for commit 0c4e13c. |
Should the reaper delete the hpa too? |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e build/test passed for commit 0c4e13c. |
Automatic merge from submit-queue |
Auto commit by PR queue bot
@Kargakis The reaper for which resource? |
If we set an hpa for a rc or a deployment (or any resource that supports autoscaling and is reapable for that matter) then the respective reaper would somehow detect that hpa and delete it. Makes sense? |
This PR is already included in the release-1.2 branch and does not require cherrypick-ing |
Fixes #21716
cc @kubernetes/kubectl @kubernetes/autoscaling