-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
feat: Add support for AWS Gateway controller (VPC Lattice) to IRSA module #378
feat: Add support for AWS Gateway controller (VPC Lattice) to IRSA module #378
Conversation
pending name clarification |
9e18878
to
cd08216
Compare
data "aws_iam_policy_document" "api_gateway_controller" { | ||
count = var.create_role && var.attach_api_gateway_controller_policy ? 1 : 0 | ||
|
||
# https://github.com/aws/aws-application-networking-k8s/blob/main/examples/recommended-inline-policy.json |
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.
In the comment, does it make sense to point to a tag/release instead of a moving main
branch?
The rest LGTM.
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.
good point - corrected in 5ca15c6
…h EKS module output
## [5.20.0](v5.19.0...v5.20.0) (2023-05-22) ### Features * Add support for AWS Gateway controller (VPC Lattice) to IRSA module ([#378](#378)) ([fdee003](fdee003))
This PR is included in version 5.20.0 🎉 |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
cluster_autoscaler_cluster_names
andkarpenter_controller_cluster_name
which shadow their current equivalents (cluster_autoscaler_cluster_ids
andkarpenter_controller_cluster_id
). Users should now use thecluster_name
output from the EKS module (starting in v19.x) sincecluster_id
now has a different meaning on local clusters on Outposts. This change starts to correct that behavior and align the inputs/outputs to reduce confusion but without breaking compatibility. The_id
equivalents will be removed at the next breaking changeMotivation and Context
Breaking Changes
How Has This Been Tested?
examples/*
to demonstrate and validate my change(s)examples/*
projectspre-commit run -a
on my pull request