Skip to content

Conversation

@clstokes
Copy link
Collaborator

No description provided.

@clstokes clstokes changed the title terraform: add aws-eks terraform: add aws-eks-operator Dec 20, 2025
@clstokes clstokes marked this pull request as ready for review December 20, 2025 01:40
@clstokes
Copy link
Collaborator Author

@rajsinghtech, I'd appreciate your review when you have a chance.

Copy link

Copilot AI left a 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.

This comment was marked as resolved.

This comment was marked as resolved.

Copy link

Copilot AI left a 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.

Copy link

Copilot AI left a 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.

Copy link

Copilot AI left a 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.

@clstokes clstokes requested a review from jaxxstorm December 23, 2025 21:09
@rajsinghtech
Copy link
Contributor

rajsinghtech commented Dec 26, 2025

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 cluster.local or setup the proxygroup ingress and egress and dnsconfig so that ts.net resolves.

WIll approve for now but please review the feedback 🥇

@clstokes clstokes merged commit a95fae5 into main Jan 6, 2026
3 checks passed
@clstokes clstokes deleted the clstokes/eks-operator branch January 6, 2026 01:01
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.

4 participants