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

Feature/cni custom network #777

Closed
wants to merge 3 commits into from

Conversation

qtruong77
Copy link

@qtruong77 qtruong77 commented Jul 15, 2022

What does this PR do?

Enable provision EKS cluster with secondary CIDR with CNI custom networking.

Motivation

This feature will be a high demand where we don't want to consume primary CIDR range for EKS environment with enterprise environment

More

  • Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
  • Yes, I have added a new example under examples to support my PR
  • Yes, I have created another PR for add-ons under add-ons repo (if applicable)
  • Yes, I have updated the docs for this feature
  • Yes, I ran pre-commit run -a with this PR

Note: Not all the PRs required examples and docs except a new pattern or add-on added.

For Moderators

  • E2E Test successfully complete before merge?

Additional Notes

An initial PR to have the team review on the changes.
The implementation is based on this tutorial:

  • Create an optional step to configure custom network right after the control plan is up with auth configmap (this step is enable by provide input pod_subnet_ids
  • Use null_resource to update the daemonset environment variable and apply the manifest due to limitation of the kubernetes_daemonset and kubernetes_manifest resource of kubernetes provider
  • Update the max_pod value for worker node

@askulkarni2
Copy link
Contributor

This was done in #778

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants