-
Notifications
You must be signed in to change notification settings - Fork 81
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
Allow to pass bootstrapSelfManagedAddons
argument to cluster
#1475
Comments
Hi @ahanoff thanks for the issue. Have you tried the defaultAddonsToRemove property? https://www.pulumi.com/registry/packages/eks/api-docs/cluster/#defaultaddonstoremove_nodejs I believe if you populate that property with a list of "vpc-cni", "coredns", "kube-proxy" then they should not be auto-created anymore. |
hi @t0yv0, thanks! Yes, that's what I currently set while waiting for this issue resolution
Actually I'm confused with two things:
Anyway I'm going test your suggestion this week and get back with feedback |
Great feedback, let me double-check and see if we can clarify the docstrings. |
Hey @ahanoff, But I agree with you that having two options to achieve the same goal is not ideal! |
hi @t0yv0, @flostadler, I've tested this and would like to confirm that addons are removed and Cilium CNI installed passing |
Awesome, thanks @ahanoff. I'll keep the issue open to track this enhancement once we tackled pulumi/pulumi-aws#4755. I also updated the description with the |
Hello!
Issue details
Currently EKS cluster sets by default
bootstrapSelfManagedAddons
astrue
which can not be overridden. That setting creates 3 addons:vpc-cni
,coredns
, andkube-proxy
As a workaround users can use the
defaultAddonsToRemove
input property with["vpc-cni", "coredns", and "kube-proxy"]
to remove all addons on cluster creation.Affected area/feature
Prevents from creating clean cluster without any addon which is beneficial if you use custom CNI like Cilium
The text was updated successfully, but these errors were encountered: