-
Notifications
You must be signed in to change notification settings - Fork 1.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
Prevent multiple ScaledObjects managing one HPA #6198
Conversation
3f2401c
to
04dc8c5
Compare
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.
LGTM!
Just a note about the changelog
/run-e2e internal |
112e1f5
to
2d26918
Compare
Sorry, I missed a line of code when commit, which caused the e2e test in I've rebased main, to solve the other two failed. |
Signed-off-by: wangrushen <[email protected]>
Signed-off-by: wangrushen <[email protected]>
2d26918
to
de0f603
Compare
/run-e2e internal Update: You can check the progress here |
/run-e2e internal |
PTAL @zroubalik @wozniakjan |
Signed-off-by: wangrushen <[email protected]>
/run-e2e internals |
* Prevent multiple ScaledObjects managing one HPA Signed-off-by: wangrushen <[email protected]> * fix: e2e test and mv changelog to improvements Signed-off-by: wangrushen <[email protected]> * check nil for spec.advanced.horizontalPodAutoscalerConfig Signed-off-by: wangrushen <[email protected]> --------- Signed-off-by: wangrushen <[email protected]>
* Prevent multiple ScaledObjects managing one HPA Signed-off-by: wangrushen <[email protected]> * fix: e2e test and mv changelog to improvements Signed-off-by: wangrushen <[email protected]> * check nil for spec.advanced.horizontalPodAutoscalerConfig Signed-off-by: wangrushen <[email protected]> --------- Signed-off-by: wangrushen <[email protected]> Signed-off-by: Chirag Bhatia <[email protected]>
Provide a description of what has been changed
Prevent multiple ScaledObjects managing one HPA
Checklist
Fixes #6130
Relates to #