-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[clusteragent] Remove the CPU/memory limits by default for the init container #30266
base: main
Are you sure you want to change the base?
Conversation
…configs to set the limits
d11f0fb
to
5ea4dd9
Compare
[Fast Unit Tests Report] On pipeline 46930637 (CI Visibility). The following jobs did not run any unit tests: Jobs:
If you modified Go files and expected unit tests to run in these jobs, please double check the job logs. If you think tests should have been executed reach out to #agent-devx-help |
Test changes on VMUse this command from test-infra-definitions to manually test this PR changes on a VM: inv create-vm --pipeline-id=46930637 --os-family=ubuntu Note: This applies to commit 6fd7387 |
if err != nil { | ||
return resources, err | ||
} | ||
resources.Limits[corev1.ResourceCPU] = quantity |
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.
Is it OK to not set a limit at all (same for the memory)?
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.
good for docs
What does this PR do?
Motivation
The current limits are too low for the SSI init container.
We decided to drop them and just keep the "requests".
Also, the current CPU limit (0.05 of a CPU core) is very low. I suggest to use
0.3
.Describe how to test/QA your changes
Need help for this: I don't know much about kubernetes 🙈
Possible Drawbacks / Trade-offs
Additional Notes