Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update wellKnownPolicies.loadBalancerController to Latest Version #8286

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

LEEDOWON96
Copy link

Description

This PR updates the IAM policy for wellKnownPolicies.loadBalancerController to align with the latest recommended policy for the AWS Load Balancer Controller.

https://github.com/kubernetes-sigs/aws-load-balancer-controller/blob/main/docs/install/iam_policy.json

The main changes include adding new permissions such as ec2:GetSecurityGroupsForVpc, elasticloadbalancing:DescribeTrustStores, elasticloadbalancing:DescribeCapacityReservation, and elasticloadbalancing:ModifyCapacityReservation. All existing permissions have been retained.

Checklist

  • Added tests that cover your change (if possible)
  • Added/modified documentation as required (such as the README.md, or the userdocs directory)
  • Manually tested
  • Made sure the title of the PR is a good description that can go into the release notes
  • (Core team) Added labels for change area (e.g. area/nodegroup) and kind (e.g. kind/improvement)

BONUS POINTS checklist: complete for good vibes and maybe prizes?! 🤯

  • Backfilled missing tests for code in same general area 🎉
  • Refactored something and made the world a better place 🌟

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello LEEDOWON96 👋 Thank you for opening a Pull Request in eksctl project. The team will review the Pull Request and aim to respond within 1-10 business days. Meanwhile, please read about the Contribution and Code of Conduct guidelines here. You can find out more information about eksctl on our website

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant