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

Option to set cpu-cfs-quota in Kubelet Argument to avoid CPU throttling #2877

Closed
sanjeevpandey19 opened this issue Mar 10, 2023 · 5 comments · Fixed by #2930
Closed

Option to set cpu-cfs-quota in Kubelet Argument to avoid CPU throttling #2877

sanjeevpandey19 opened this issue Mar 10, 2023 · 5 comments · Fixed by #2930
Assignees
Labels
area/kubernetes K8s including EKS, EKS-A, and including VMW area/settings Issues related to our settings handling status/in-progress This issue is currently being worked on type/enhancement New feature or request

Comments

@sanjeevpandey19
Copy link

What I'd like:
I would like to get the option to setup cpuCFSQuota in kubelet argument, as this is not supported as of now

Why:
We want to make this option false to avoid unnecessary CPU Throttling in in EKS when used with bottlerocket. For reference #1026 this works with aws amazon linux

Any alternatives you've considered:
We have considered removing CPU Limit from the pod constraint but that's only the workaround and we want to go back to CPU Limit in all containers.

I walked around the documentation and didn't find any flag for cfs-quota but Let me know in case there is already an option that i might have missed Or any other alternatives which I can use to achieve this

@sanjeevpandey19 sanjeevpandey19 added status/needs-triage Pending triage or re-evaluation type/enhancement New feature or request labels Mar 10, 2023
@stmcginnis
Copy link
Contributor

Thanks for the details @sanjeevpandey19! There are a few issues for different kubelet configuration options now. This looks like a good one to include if more knobs will be exposed in Bottlerocket. Thanks!

@stmcginnis stmcginnis added area/kubernetes K8s including EKS, EKS-A, and including VMW status/research This issue is being researched area/settings Issues related to our settings handling and removed status/needs-triage Pending triage or re-evaluation labels Mar 10, 2023
@sanjeevpandey19
Copy link
Author

Thanks for the details @sanjeevpandey19! There are a few issues for different kubelet configuration options now. This looks like a good one to include if more knobs will be exposed in Bottlerocket. Thanks!

Thank @stmcginnis for replying back, really appreciate this. I am hoping to get this option from bottlerocket team soon as this is critical and could be a breaking deal for us as we are doing POC and have to migrate a couple of clusters from Amazon Linux to Bottlerocket to get better security surface at OS level

@b44rawat
Copy link

+1

@vikas-gautam
Copy link

Thanks for the details @sanjeevpandey19! There are a few issues for different kubelet configuration options now. This looks like a good one to include if more knobs will be exposed in Bottlerocket. Thanks!

Thank @stmcginnis for replying back, really appreciate this. I am hoping to get this option from bottlerocket team soon as this is critical and could be a breaking deal for us as we are doing POC and have to migrate a couple of clusters from Amazon Linux to Bottlerocket to get better security surface at OS level

+1

@vikas-bisht16
Copy link

+1

@stmcginnis stmcginnis self-assigned this Mar 21, 2023
@stmcginnis stmcginnis added status/in-progress This issue is currently being worked on and removed status/research This issue is being researched labels Mar 22, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in Bottlerocket Engineering Roadmap May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/kubernetes K8s including EKS, EKS-A, and including VMW area/settings Issues related to our settings handling status/in-progress This issue is currently being worked on type/enhancement New feature or request
Projects
Development

Successfully merging a pull request may close this issue.

5 participants