Skip to content
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 (backport to 1.29) #373

Merged

Conversation

coord-e
Copy link
Member

@coord-e coord-e commented Oct 30, 2024

This PR backports changes from #371 and #372 into the 1.29 release branch.

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 is a cherry-picked commit with changes in #372
@coord-e coord-e force-pushed the coord-e/backport-karpenter-v1-controller-policy-1-29 branch from a3daf64 to c171d0b Compare October 30, 2024 06:50
@coord-e coord-e marked this pull request as ready for review October 30, 2024 11:02
@coord-e coord-e requested a review from a team as a code owner October 30, 2024 11:02
@coord-e coord-e requested review from errm, a team and mozamimy October 30, 2024 11:02
@nekketsuuu
Copy link

nekketsuuu commented Nov 1, 2024

Changes for test/cluster_test.go might be forgotten to be cherry-picked.

@coord-e
Copy link
Member Author

coord-e commented Nov 6, 2024

release-1-29 branch runs tests with Karpenter 0.37.0, which does not support v1 resources, so I didn't include that change in this PR. https://github.com/cookpad/terraform-aws-eks/blob/release-1-29/test/cluster_test.go#L94

@coord-e coord-e merged commit 03820cf into release-1-29 Nov 6, 2024
3 checks passed
@coord-e coord-e deleted the coord-e/backport-karpenter-v1-controller-policy-1-29 branch November 6, 2024 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants