Skip to content

Fix max_pods to 110, set kubeReserved.memory using calculation scaling with max_pods and set systemReserved.memory to 200Mi#2395

Open
wcarlsen wants to merge 1 commit into
masterfrom
remove/eks/systemreserved-kubereserved-maxpods-variables
Open

Fix max_pods to 110, set kubeReserved.memory using calculation scaling with max_pods and set systemReserved.memory to 200Mi#2395
wcarlsen wants to merge 1 commit into
masterfrom
remove/eks/systemreserved-kubereserved-maxpods-variables

Conversation

@wcarlsen

@wcarlsen wcarlsen commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Describe your changes

  • kube_reserved_cpu (kubelet reserved CPU) since, this doesn't scale with max_pods. The scaling is calculated in this fashion (6% of the first core + 1% of the second core + 0.5% of the third and fourth core + 0.25% of the remainding cores.
  • kube_memory (kubelet reserved memory) scales with Max Pods, but EKS doesn't handle overwrites to to Max Pods in terms of reserved memory. The calculation is as following 255Mi + (11Mi * Max_pods).
  • Remove systemReserved.cpu since documentation say we should only set non-compressable resources see https://kubernetes.io/docs/tasks/administer-cluster/reserve-compute-resources/#general-guidelines
  • Set systemReserved.memory to 200Mi since it is needed for the node not to go in NotReady state under stress testing. Kubernetes documentation has something on the topic https://kubernetes.io/docs/tasks/administer-cluster/reserve-compute-resources/#general-guidelines
  • Added check block to throw a warning when instance types with lower than 110 max pods
  • Fix max_pods to 110 always and remove variable

Here are the actual code setting some of these parameters:

Issue ticket number and link

Checklist before requesting a review

Is it a new release?

  • Apply a release tag release:(major|minor|patch), following semantic versioning in this guide or norelease if there is no changes to the Terraform code

…g with max_pods and set systemReserved.memory to 200Mi

Signed-off-by: Willi Carlsen <carlsenwilli@gmail.com>
@wcarlsen wcarlsen requested a review from a team as a code owner July 1, 2026 13:01
@wcarlsen wcarlsen added the release:major Triggers a major release label Jul 1, 2026
@DFDS-Snyk

DFDS-Snyk commented Jul 1, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:major Triggers a major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants