-
Notifications
You must be signed in to change notification settings - Fork 40.1k
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
Horizontal Pod Autoscaler is deleted along with namespace #13786
Conversation
GCE e2e build/test passed for commit 9d60963a6bbd8fcf459289f4ddacb3ba7448457a. |
Assigned @mwielgus who is a big fan a this attitude. |
@@ -103,6 +103,7 @@ func (rc *ResourceConsumer) sendConsumeCPUrequests(requests, millicores, duratio | |||
|
|||
// sendOneConsumeCPUrequest sends POST request for cpu consumption | |||
func (rc *ResourceConsumer) sendOneConsumeCPUrequest(millicores int, durationSec int) { | |||
defer GinkgoRecover() |
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.
why do you need to call it here?
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.
when err (from sending post request) is not nil, then we need defer GinkgoRecover().
sendOneConsumeCPUrequest is always in a new gorutine.
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.
Ok. Right
GCE e2e build/test passed for commit d5c5dc6e51e525601ef537f306ba07f7e0edc116. |
Labelling this PR as size/XS |
@socaa could you please rebase this PR? |
@piosz done |
LGTM |
GCE e2e build/test failed for commit d6d199d. |
@k8s-bot test this please |
GCE e2e build/test passed for commit d6d199d. |
Horizontal Pod Autoscaler is deleted along with namespace
Part of #11570
@piosz
@mwielgus