Skip to content
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

Increase limits for metrics-server. #4017

Merged

Conversation

amshuman-kr
Copy link

@amshuman-kr amshuman-kr commented May 10, 2021

How to categorize this PR?

/area control-plane auto-scaling
/kind post-mortem

What this PR does / why we need it:

Increase limits for metrics-server. This is a temporary fix until we have non-circular auto-scaling for metrics-server.

Which issue(s) this PR fixes:
Fixes #

https://github.tools.sap/kubernetes-live/issues-live/issues/453#issuecomment-213566

Special notes for your reviewer:

Follow up issue after this PR goes in -> #4018

cc @timuthy @schrodit @tedteng @dguendisch @vlerenc

Release note:

Increase limits for metrics-server. This is a temporary fix until we have non-circular auto-scaling for metrics-server.

@amshuman-kr amshuman-kr requested a review from a team as a code owner May 10, 2021 14:45
@gardener-robot gardener-robot added area/auto-scaling Auto-scaling (CA/HPA/VPA/HVPA, predominantly control plane, but also otherwise) related area/control-plane Control plane related kind/post-mortem Bug that requires deeper analysis after immediate issues were resolved (usually after downtime) needs/review size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels May 10, 2021
@gardener-robot
Copy link

@amshuman-kr You have mentioned internal references in the public. Please check.

@vpnachev
Copy link
Member

/assign

@gardener-robot
Copy link

@amshuman-kr You have mentioned internal references in the public. Please check.

@amshuman-kr
Copy link
Author

@vpnachev I have set the limits to cpu: 500m, memory: 1Gi as discussed. Can you PTAL?

@amshuman-kr
Copy link
Author

@vpnachev I have renamed the variables as you suggested. Can you PTAL?

Copy link
Member

@vpnachev vpnachev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/auto-scaling Auto-scaling (CA/HPA/VPA/HVPA, predominantly control plane, but also otherwise) related area/control-plane Control plane related kind/post-mortem Bug that requires deeper analysis after immediate issues were resolved (usually after downtime) size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants