feat: update AWS provider in aws-eks module#940
feat: update AWS provider in aws-eks module#940felixglezprefapp wants to merge 10 commits intomainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the aws-eks Terraform module to require a newer AWS provider version and aligns module documentation accordingly (per issue #939).
Changes:
- Bump
hashicorp/awsrequired provider constraint from~> 5.83to~> 6.33in the module. - Refresh generated README provider/version references to reflect the new AWS provider constraint.
- Tidy the Support link formatting in module docs/footer and README.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
modules/aws-eks/versions.tf |
Updates AWS provider version constraint to ~> 6.33. |
modules/aws-eks/docs/footer.md |
Adjusts Support section link formatting. |
modules/aws-eks/README.md |
Updates documented AWS provider version and Support link formatting. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
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.
| fargate_profiles_map = { | ||
| for profile in var.fargate_profiles : | ||
| profile.name => { | ||
| name = profile.name | ||
| selectors = profile.selectors | ||
| tags = profile.tags |
| # Adaptar fargate_profiles de lista a mapa para el módulo oficial | ||
| fargate_profiles_map = { |
| { | ||
| # Si el usuario pasa pre_bootstrap_user_data, lo movemos a launch_template.user_data | ||
| launch_template = ( | ||
| contains(keys(group), "pre_bootstrap_user_data") ? merge( | ||
| lookup(group, "launch_template", {}), | ||
| { | ||
| user_data = group.pre_bootstrap_user_data | ||
| } | ||
| ) : ( | ||
| lookup(group, "launch_template", null) | ||
| ) | ||
| ) | ||
| }, |
|
|
||
| source = "terraform-aws-modules/eks/aws//modules/karpenter" | ||
| version = "20.33.1" | ||
| version = "21.0.0" |
Closes #939