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

Remove CPU limits #93

Open
asimpleidea opened this issue Feb 10, 2023 · 0 comments
Open

Remove CPU limits #93

asimpleidea opened this issue Feb 10, 2023 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@asimpleidea
Copy link
Member

asimpleidea commented Feb 10, 2023

Look at this tweet by Tom Hockins, one of the original maintainers of Kubernetes: https://twitter.com/thockin/status/1134193838841401345

When you have CPU requests, you are guaranteed to have those requests and the app works fine, but if there are no more resources than the app simply won't have them. If there is, then the app is going to use them for when it needs it, and it will perform better for that period and will return to its requested -- and, again, work fine -- in case someone else requests the available resources.

With CPU limits, the app won't take the additional resources when it needs them even though there are resources available, and that is a waste.

In short, for most of the time, the app will use its requested CPU and will just burst for a short period when it needs it, there is no point in preventing it from using it if is free and no one else has requested it.

@asimpleidea asimpleidea added the enhancement New feature or request label Feb 10, 2023
@asimpleidea asimpleidea self-assigned this Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant