-
Notifications
You must be signed in to change notification settings - Fork 986
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
chore: Drop ec2:DescribeAvailabilityZones
permissions
#6932
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/karpenter snapshot
✅ Deploy Preview for karpenter-docs-prod canceled.
|
Snapshot successfully published to
|
f82852e
to
642019f
Compare
Pull Request Test Coverage Report for Build 10716911962Details
💛 - Coveralls |
642019f
to
07fafdc
Compare
07fafdc
to
f04db7d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/karpenter snapshot
Snapshot successfully published to
|
LGTM 🚀 |
Fixes #N/A
Description
ec2:DescribeAvailabilityZones
permission has not been needed for a little bit since we dropped this call from our GetInstanceTypes() call path. We can remove this READ permission from our policyHow was this change tested?
Does this change impact docs?
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.