Skip to content

feat: Update AWS LB Controller IAM policy and add AGA policy#497

Merged
allamand merged 2 commits intoaws-ia:mainfrom
neilkuan:feat/update-alb-controller-iam-policy
Mar 9, 2026
Merged

feat: Update AWS LB Controller IAM policy and add AGA policy#497
allamand merged 2 commits intoaws-ia:mainfrom
neilkuan:feat/update-alb-controller-iam-policy

Conversation

@neilkuan
Copy link
Contributor

@neilkuan neilkuan commented Mar 9, 2026

Summary

Test plan

  • Verify terraform validate passes
  • Verify terraform plan with enable_aws_load_balancer_controller = true includes ec2:DescribeRouteTables
  • Verify terraform plan with enable_aws_load_balancer_controller_aga = true creates the AGA policy and attachment
  • Verify AGA policy is not created when enable_aws_load_balancer_controller_aga = false (default)

Add missing ec2:DescribeRouteTables permission to the main IAM policy
and add an independent Global Accelerator (AGA) IAM policy that can be
optionally enabled via enable_aws_load_balancer_controller_aga variable.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@neilkuan neilkuan requested a review from a team as a code owner March 9, 2026 11:55
Auto-generated by terraform-docs pre-commit hook.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@neilkuan
Copy link
Contributor Author

neilkuan commented Mar 9, 2026

@bryantbiggs
Please take look thank.

@bryantbiggs
Copy link
Contributor

I'm no longer with AWS so I don't have access to this project - cc @allamand

@neilkuan
Copy link
Contributor Author

neilkuan commented Mar 9, 2026

I'm no longer with AWS so I don't have access to this project - cc @allamand

I am sorry. I didn't know that, but thank you reply this. Thank you

@allamand allamand merged commit d88c48f into aws-ia:main Mar 9, 2026
1 check failed
@allamand
Copy link
Contributor

allamand commented Mar 9, 2026

What is your current usage of this repo ? We don't recommend keeping it usage and recommend to move for a gitops approach instead using tools like Argocd/kro/ack

@neilkuan neilkuan deleted the feat/update-alb-controller-iam-policy branch March 10, 2026 01:30
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.

3 participants