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

Allow disabling the default CNI #906

Open
jaxxstorm opened this issue Aug 24, 2023 · 4 comments
Open

Allow disabling the default CNI #906

jaxxstorm opened this issue Aug 24, 2023 · 4 comments
Labels
kind/enhancement Improvements or new features

Comments

@jaxxstorm
Copy link
Contributor

It is currently not possible to disable the default CNI in the cluster.

I had expected that removing the CNI addon would be enough, but the CNI daemonset is still created.

We should have a top level parameter/option that disables the default CNI being installed and removes all default configuration for it

@jaxxstorm jaxxstorm added needs-triage Needs attention from the triage team kind/enhancement Improvements or new features labels Aug 24, 2023
@rquitales
Copy link
Member

It looks like the CNI manifests for the daemonset etc is always applied:

const yamlPath = path.join(__dirname, "../../cni/aws-k8s-cni.yaml");

We should improve this logic to only apply it when necessary.

@rquitales rquitales removed the needs-triage Needs attention from the triage team label Aug 24, 2023
@PavelGloba
Copy link

Any news about it?

@DavidJFelix
Copy link

@PavelGloba

Any news about it?

I think this is possible using defaultAddonsToRemove.

const cluster = new eks.Cluster('example',  {
  // ...
  defaultAddonsToRemove: ['vpc-cni'],
  // ...
})

But I haven't tried this.

@PavelGloba
Copy link

@DavidJFelix

@PavelGloba

Any news about it?

I think this is possible using defaultAddonsToRemove.

const cluster = new eks.Cluster('example',  {
  // ...
  defaultAddonsToRemove: ['vpc-cni'],
  // ...
})

But I haven't tried this.

This issue was created because of the conversation in the chat, where this option didn't actually work for the cni. So I'm not sure if it was fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Improvements or new features
Projects
None yet
Development

No branches or pull requests

4 participants