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: Describe cloudformation.yaml for docs #4394
docs: Describe cloudformation.yaml for docs #4394
Conversation
✅ Deploy Preview for karpenter-docs-prod ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
I've added some basic feedback.
@chrisnegus if you'd like a more detailed review, get in touch (via the Kubernetes Slack, or by email)
a6393ff
to
4261c87
Compare
This PR has been inactive for 14 days. StaleBot will close this stale PR after 14 more days of inactivity. |
4261c87
to
95221eb
Compare
This PR has been inactive for 14 days. StaleBot will close this stale PR after 14 more days of inactivity. |
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.
My thoughts
ccec27b
to
f8e90ad
Compare
f8e90ad
to
0f1b8c8
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.
LGTM 🚀
Description: This PR adds new documentation describing how CloudFormation is used to setup permissions needed by Karpenter to manage cloud and EKS resources. This is part of a larger effort to describe Karpenter requirements for those migrating existing clusters to use Karpenter.
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.