-
Notifications
You must be signed in to change notification settings - Fork 3
terraform: add aws-eks-operator #44
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
Conversation
|
@rajsinghtech, I'd appreciate your review when you have a chance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds a new Terraform module for deploying an AWS EKS cluster with the Tailscale Kubernetes operator, enabling secure cluster access through Tailscale's mesh network.
- Introduces a complete Terraform configuration for EKS with Tailscale operator integration
- Configures provider authentication using AWS EKS token-based authentication
- Includes comprehensive documentation with deployment and verification instructions
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| terraform/aws/aws-eks-operator/versions.tf | Defines Terraform version constraints and configures AWS, Kubernetes, and Helm providers with EKS authentication |
| terraform/aws/aws-eks-operator/variables.tf | Declares Tailscale OAuth credentials as sensitive variables with validation rules |
| terraform/aws/aws-eks-operator/outputs.tf | Exports VPC ID, cluster name, namespace, and kubeconfig commands for cluster access |
| terraform/aws/aws-eks-operator/main.tf | Implements the main infrastructure including VPC, EKS cluster, and Tailscale operator Helm deployment |
| terraform/aws/aws-eks-operator/README.md | Provides comprehensive documentation for prerequisites, deployment steps, and verification procedures |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 12 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
…ples-infrastructure-as-code into clstokes/eks-operator
|
Looks pretty good, thanks @clstokes, there is much more we could expand on here, for example run a kind: connector resource and advertise the pod and service cidr's via an app connector for the cluster domain of WIll approve for now but please review the feedback 🥇 |
Co-authored-by: Copilot <[email protected]> Co-authored-by: Raj Singh <[email protected]>
Co-authored-by: Raj Singh <[email protected]>
No description provided.