Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[eks/alb-controller] Update ALB controller IAM policy #821

Merged
merged 4 commits into from Aug 19, 2023
Merged

[eks/alb-controller] Update ALB controller IAM policy #821

merged 4 commits into from Aug 19, 2023

Conversation

Nuru
Copy link
Sponsor Contributor

@Nuru Nuru commented Aug 17, 2023

what

  • [eks/alb-controller] Update ALB controller IAM policy

why

  • Previous policy had error preventing the creation of the ELB service-linked role

@Nuru Nuru added the bugfix Change that restores intended behavior label Aug 17, 2023
@Nuru Nuru requested review from a team as code owners August 17, 2023 05:00
milldr
milldr previously approved these changes Aug 17, 2023
View reviewed changes
@Nuru Nuru enabled auto-merge (squash) August 17, 2023 23:25
@Nuru Nuru requested a review from milldr August 18, 2023 01:04
@Nuru Nuru disabled auto-merge August 18, 2023 07:49
@Nuru Nuru merged commit 9da133f into main Aug 19, 2023
4 checks passed
@Nuru Nuru deleted the alb-controller-iam branch August 19, 2023 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aknysh aknysh aknysh approved these changes

@milldr milldr Awaiting requested review from milldr

Assignees
No one assigned
Labels
bugfix Change that restores intended behavior
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Nuru @aknysh @milldr