Skip to content

feat(iam): manage AWS Load Balancer Controller policy inline#188

Merged
ciiiii merged 1 commit intomasterfrom
max/manage-aws-lb-policy
Apr 30, 2026
Merged

feat(iam): manage AWS Load Balancer Controller policy inline#188
ciiiii merged 1 commit intomasterfrom
max/manage-aws-lb-policy

Conversation

@maxsxu
Copy link
Copy Markdown
Member

@maxsxu maxsxu commented Apr 30, 2026

Motivation

Currently, the AWS Load Balancer Controller IAM role is created, but the policy must be provided via an override or managed externally. This change manages the default AWS Load Balancer Controller policy inline within the IAM module, ensuring that the controller has the necessary permissions by default when the role is created.

Modifications

  • Added aws_iam_role_policy.aws_load_balancer_controller in modules/iam/aws_load_balancer_controller.tf to define the controller policy inline.
  • Updated load_balancer_policy_arn_override variable description in modules/iam/variables.tf to clarify that it's now an optional additional policy.
  • Updated modules/iam/README.md to reflect the changes.

Verifying this change

  • Make sure that the change passes the CI checks.

This change is already covered by existing tests, such as terraform plan in consuming modules.

Documentation

Check the box below.

Need to update docs?

  • doc-required

  • no-need-doc

    Managed inline in Terraform module README.

@maxsxu maxsxu requested a review from a team as a code owner April 30, 2026 14:27
@github-actions github-actions Bot added the no-need-doc This pr does not need any document label Apr 30, 2026
@ciiiii ciiiii merged commit 682e686 into master Apr 30, 2026
8 checks passed
@ciiiii ciiiii deleted the max/manage-aws-lb-policy branch April 30, 2026 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-need-doc This pr does not need any document

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants