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
docs: Update CloudFormation to handle v1beta1 permissions #4771
docs: Update CloudFormation to handle v1beta1 permissions #4771
Conversation
✅ Deploy Preview for karpenter-docs-prod canceled.
|
af73411
to
e3ac33b
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 |
e3ac33b
to
61c4fa5
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 |
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.
The cloudformation.yaml file looks good to me. I'll update my PR to match the new contents of that file: #4394
61c4fa5
to
e8868cb
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 |
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.
lgtm
Fixes #N/A
Description
Updates the CloudFormation template shipped with the Getting Started guide to use v1beta1 permissions. This also ensures that we are deploying the Karpenter controller IAM permissions with an additional policy that gives the controller the alpha permissions so that we can ensure that scenarios that are testing alpha and beta simultaneously will work.
How was this change tested?
make apply
with the new permissionsDoes 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.