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.

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

Add ec2:DescribeAvailabilityZones to control plane template. #310

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

weseven
Copy link
Contributor

@weseven weseven commented Jul 8, 2024

Due to this upstream change in the aws-cloud-controller, when using version 1.26 or later the IAM policy for control-plane nodes needs to be extended with ec2:DescribeAvailabilityZones permission.

Load balancer creation from a loadbalancer service in k8s 1.26 (with the relative aws-cloud-controller) fails because of this, e.g.:

Warning  SyncLoadBalancerFailed  26m                 service-controller  Error syncing load balancer: failed to ensure load balancer: error get availability zone types: "error describe availability zones: \"error listing AWS availability zones: \\\"UnauthorizedOperation: You are not authorized to perform this operation. User: arn:aws:sts::242036376510:assumed-role/control-plane-dan01/i-0530c5b119dffa047 is not authorized to perform: ec2:DescribeAvailabilityZones because no identity-based policy allows the ec2:DescribeAvailabilityZones action\\\\n\\\\tstatus code: 403, request id: cfe7c34d-5631-429e-9d18-66148da7abc6\\\"\""

@weseven weseven requested a review from a team as a code owner July 8, 2024 17:10
Copy link
Contributor

@paurosello paurosello left a comment

Choose a reason for hiding this comment

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

Looks good!

@weseven weseven merged commit 15aa080 into master Jul 9, 2024
5 checks passed
@weseven weseven deleted the add-missing-policy-1.26 branch July 9, 2024 07:20
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.

4 participants