Skip to content

Make EKS Kubernetes version configurable via module variable#39

Merged
basebandit merged 3 commits into
chore/refactor-with-enhancementsfrom
copilot/sub-pr-37-again
Feb 9, 2026
Merged

Make EKS Kubernetes version configurable via module variable#39
basebandit merged 3 commits into
chore/refactor-with-enhancementsfrom
copilot/sub-pr-37-again

Conversation

Copilot AI commented Feb 9, 2026

Copy link
Copy Markdown

The cluster_version variable in staging was defined but unused—the EKS module hardcoded kubernetes_version = "1.32".

Changes

  • EKS module: Added kubernetes_version variable (defaults to "1.32")
  • EKS module: Use var.kubernetes_version instead of hardcoded value
  • Staging config: Pass local.cluster_version to module's kubernetes_version input
# infrastructure/staging/eks.tf
module "eks" {
  source = "../modules/eks"
  
  name               = "${local.env}-${local.cluster_name}"
  environment        = local.env
  kubernetes_version = local.cluster_version  # Now configurable
  # ...
}

The Kubernetes version is now configurable per environment while maintaining the same default behavior.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: basebandit <8973567+basebandit@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on refactor with enhancements from PR #37 Make EKS Kubernetes version configurable via module variable Feb 9, 2026
Copilot AI requested a review from basebandit February 9, 2026 05:46
@basebandit basebandit marked this pull request as ready for review February 9, 2026 05:59
@basebandit basebandit merged commit 0a65bd6 into chore/refactor-with-enhancements Feb 9, 2026
@basebandit basebandit deleted the copilot/sub-pr-37-again branch February 9, 2026 05:59
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