-
Notifications
You must be signed in to change notification settings - Fork 41
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
Enable to use Karpenter v1 controller policy #371
Conversation
2465d61
to
bcf7a28
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.
Could you check reason for test failure?
04485e2
to
00744e8
Compare
00744e8
to
c820fe1
Compare
This comment was marked as outdated.
This comment was marked as outdated.
Including both v1 and v1beta policies inline in the role seems to exceed the IAM role's policy size limit. I resolved this with managed policies, now tests are passing 7257bc8 |
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!
thank you! |
This is a squashed commit with changes in #371: Enable to detach Karpenter v1beta controller policy Make v1 and v1beta policies mutually exclusive Use Karpenter v1 resources in test Revert "Make v1 and v1beta policies mutually exclusive" This reverts commit 0db1769. Separate policy documents as managed policies
This is a squashed commit with changes in #371: Enable to detach Karpenter v1beta controller policy Make v1 and v1beta policies mutually exclusive Use Karpenter v1 resources in test Revert "Make v1 and v1beta policies mutually exclusive" This reverts commit 0db1769. Separate policy documents as managed policies
This is a squashed commit with changes in #371, along with some modifications to match the state of the 1.29 release branch: Enable to detach Karpenter v1beta controller policy Make v1 and v1beta policies mutually exclusive Use Karpenter v1 resources in test Revert "Make v1 and v1beta policies mutually exclusive" This reverts commit 0db1769. Separate policy documents as managed policies
This PR attaches a controller policy for Karpenter v1 in the karpenter module. The policy content is taken from https://github.com/aws/karpenter-provider-aws/blob/v1.0.6/website/content/en/preview/getting-started/getting-started-with-karpenter/cloudformation.yaml. I've added module variables
v1
andv1beta
to help migration as you did in #368.I'll make a PR to backport this to v0.29 and v0.30 after this is merged.