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

Missing validation on instance_type #282

Open
g-iannelli opened this issue Feb 21, 2023 · 0 comments
Open

Missing validation on instance_type #282

g-iannelli opened this issue Feb 21, 2023 · 0 comments
Labels
bug Something isn't working enhancement New feature or request furyctl-ng help wanted Extra attention is needed

Comments

@g-iannelli
Copy link
Contributor

There is no validation about the EKS node pool instance_type.

For example if you use a t3.micro instance that support max 4 pods, can't run all required daemonset pods.

The following pods seems to be installed always on each node:

Namespace Daemonset
calico-system calico-node
kube-system aws-node
kube-system aws-node-termination-handler
kube-system ebs-csi-node
kube-system kube-proxy
logging infra-fluentbit
logging systemd-common-host-tailer
monitoring kube-proxy-metrics
monitoring node-exporter

Checking the max pods described in the official eks ami repository, the following instance type should not be allowed considering the 9 infrastructure pods plus 1 for custom workloads

curl -sSL https://raw.githubusercontent.com/awslabs/amazon-eks-ami/master/files/eni-max-pods.txt | grep -v -e '^\s*#' | awk '$2 <= 10 {print "-", $1}'
- a1.medium
- c6g.medium
- c6gd.medium
- c6gn.medium
- c7g.medium
- d3.xlarge
- d3en.xlarge
- g4ad.2xlarge
- g4ad.xlarge
- is4gen.medium
- m1.small
- m6g.medium
- m6gd.medium
- m7g.medium
- r6g.medium
- r6gd.medium
- r7g.medium
- t1.micro
- t2.micro
- t2.nano
- t3.micro
- t3.nano
- t3a.micro
- t3a.nano
- t3a.small
- t4g.micro
- t4g.nano
- x2gd.medium
@g-iannelli g-iannelli added bug Something isn't working enhancement New feature or request furyctl-ng labels Feb 21, 2023
@omissis omissis added the help wanted Extra attention is needed label Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request furyctl-ng help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants