🤖 feat: use Terraform profile-based AWS auth with refresh#28
Conversation
|
@codex review Please review this follow-up change for AWS credential auto-refresh behavior in Terraform and README guidance. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a8eeaaed73
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
@codex review Addressed AZ selection hardening: availability zones are now filtered to standard |
|
@codex review Addressed both review items:
|
|
Codex Review: Didn't find any major issues. Chef's kiss. ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
Summary
Harden Terraform AWS authentication and AZ selection for EKS: keep default credential-chain compatibility, support refreshable profile-based auth when desired, and avoid selecting unsupported Local/Wavelength zones for cluster subnets.
Background
Two reliability issues were addressed:
profile = terraformbroke environments that depend on the default AWS credential chain (CI, role-based runtime creds, etc.).state=availablezones can pick Local/Wavelength zones in opted-in accounts, which EKS does not support for control plane subnets.Implementation
terraform/variables.tf:aws_profileis now optional (default = null) so the default AWS SDK credential chain remains intactterraform/versions.tf:profile = var.aws_profileterraform/vpc.tf:data "aws_availability_zones" "available"now filterszone-type = availability-zoneterraform/README.md:credential_processplan,apply,destroy) or be set once viaTF_VAR_aws_profileValidation
terraform -chdir=terraform fmt -recursive -checkterraform -chdir=terraform init -backend=false -input=falseterraform -chdir=terraform validateRisks
Low risk. Scope is limited to Terraform config and docs. Backward compatibility improves for non-profile environments, while profile-based flows remain supported and documented.
Generated with
mux• Model:openai:gpt-5.3-codex• Thinking:xhigh• Cost:$0.53