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

feat: add cpu limit flag #254

Closed
wants to merge 1 commit into from
Closed

feat: add cpu limit flag #254

wants to merge 1 commit into from

Conversation

shreddedbacon
Copy link
Member

In a similar manner to memory limits #120 , do the same for CPU

@shreddedbacon shreddedbacon changed the title chore: add cpu limit flag feat: add cpu limit flag Nov 3, 2023
@smlx
Copy link
Member

smlx commented Nov 6, 2023

Please don't do this as it doesn't work the same way as memory limits. CPU requests are enough. See https://home.robusta.dev/blog/stop-using-cpu-limits

@shreddedbacon
Copy link
Member Author

Closing this due to other work on resources being done within Lagoon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants