-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
feat: Add support for Karpenter v1 controller IAM role permissions #3126
feat: Add support for Karpenter v1 controller IAM role permissions #3126
Conversation
I will need to check if this is backwards compatible with Karpenter versions < 1.0 before we can proceed |
This reverts commit f0e5c79.
any plans when this would be merged and available for hee? |
I quickly tried this PR with Karpenter 0.37.0, and I found an issue with launching new nodes.
Likely due to the additions of |
## [20.24.0](v20.23.0...v20.24.0) (2024-08-19) ### Features * Add support for Karpenter v1 controller IAM role permissions ([#3126](#3126)) ([e317651](e317651))
This PR is included in version 20.24.0 🎉 |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
This PR updates the IAM role for the Karpenter Controller to reflect all the changes made for Karpenter v1.0.0
Motivation and Context
Without this, karpenter fails to start nodes when upgraded to v1.0.0. Solves errors such as;
They removed the
sqs:GetQueueAttributes
in this changeThis change shows all the changes in the controller IAM policy (taken from website/content/en/docs/getting-started/getting-started-with-karpenter/cloudformation.yaml)
Breaking Changes
How Has This Been Tested?
Tested in my lab setup, changes applied and nodes now spin up.
examples/*
to demonstrate and validate my change(s)examples/*
projectspre-commit run -a
on my pull request