-
Notifications
You must be signed in to change notification settings - Fork 5
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
Change EKS module to conform to EKS Blueprints direction #73
Comments
More info needed. @zack-is-cool are you working on stuff for this already? I remember you had mentioned it. Need better context on what is needed and what is already being worked. |
Currently working it in branch: https://github.com/defenseunicorns/iac/tree/calico_cni which will be a refactor from using We'll also be using calico to utilize network policies. @ntwkninja anything I'm missing here? |
@zack-is-cool would you mind pushing up a draft PR? Will this work that you are doing also cover #75? |
spot on @zack-is-cool VPC CNI with custom networking + calico CNI could not be deployed with EKS blueprints because it was missing the @RothAndrew Zack is working #73 & #75 |
Sounds good. Recommend we get the automated testing stuff in place before a big change like this gets merged. |
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/docs/v5-direction/DIRECTION_v5.md#notable-changes
The text was updated successfully, but these errors were encountered: