Skip to content

feat(eks-v2): graduate to stable 🚀#36950

Merged
mergify[bot] merged 36 commits intomainfrom
leonmk/eksv2
Feb 20, 2026
Merged

feat(eks-v2): graduate to stable 🚀#36950
mergify[bot] merged 36 commits intomainfrom
leonmk/eksv2

Conversation

@leonmk-aws
Copy link
Contributor

@leonmk-aws leonmk-aws commented Feb 11, 2026

Description of changes

We are excited to graduate the @aws-cdk/aws-eks-v2-alpha module to STABLE.
It now lives on as aws-cdk-lib/aws-eks-v2.

Deprecated properties removed:

  • The deprecated instanceType property has been removed, use instanceTypes instead

Checklist


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@github-actions github-actions bot added the p2 label Feb 11, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team February 11, 2026 13:21
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Feb 11, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 11, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results936 ran936 passed
TestResult
No test annotations available

@github-actions
Copy link
Contributor

github-actions bot commented Feb 11, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ☑️SkippedFailed ❌️
Security Guardian Results with resolved templates936 ran935 passed1 failed
TestResult
Security Guardian Results with resolved templates
packages/@aws-cdk-testing/framework-integ/test/aws-eks-v2/test/integ.eks-cluster-imported.js.snapshot/aws-cdk-eks-import-cluster-test.template.json
iam-role-root-principal-needs-conditions.guard❌ failure

@leonmk-aws leonmk-aws added the pr/needs-integration-tests-deployment Requires the PR to deploy the integration test snapshots. label Feb 11, 2026
@aws-cdk-automation aws-cdk-automation added the pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes. label Feb 11, 2026
@kumsmrit kumsmrit added the pr/request-cli-integ-tests Request CLI integ tests to be run. You will need to review the code and approve the deployment. label Feb 17, 2026
@kumsmrit kumsmrit had a problem deploying to deployment-integ-test February 17, 2026 15:17 — with GitHub Actions Failure
@leonmk-aws leonmk-aws had a problem deploying to deployment-integ-test February 18, 2026 17:01 — with GitHub Actions Failure
@aemada-aws aemada-aws self-assigned this Feb 19, 2026
Comment on lines +247 to +252
/**
* @deprecated use `addToPrincipalPolicy()`
*/
public addToPolicy(statement: PolicyStatement): boolean {
return this.addToPrincipalPolicy(statement).statementAdded;
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we remove this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we can remove it because this class implements IPrincipal and addToPolicy is required:

addToPolicy(statement: PolicyStatement): boolean;

@kumsmrit kumsmrit had a problem deploying to deployment-integ-test February 19, 2026 23:33 — with GitHub Actions Failure
@aws-cdk-automation aws-cdk-automation removed the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Feb 20, 2026
@aws-cdk-automation
Copy link
Collaborator

➡️ PR build request submitted to test-main-pipeline ⬅️

A maintainer must now check the pipeline and add the pr-linter/cli-integ-tested label once the pipeline succeeds.

@kumsmrit kumsmrit removed pr/request-cli-integ-tests Request CLI integ tests to be run. You will need to review the code and approve the deployment. pr/needs-integration-tests-deployment Requires the PR to deploy the integration test snapshots. labels Feb 20, 2026
@mergify
Copy link
Contributor

mergify bot commented Feb 20, 2026

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit a7de51c into main Feb 20, 2026
39 of 43 checks passed
@mergify mergify bot deleted the leonmk/eksv2 branch February 20, 2026 10:00
@mergify
Copy link
Contributor

mergify bot commented Feb 20, 2026

Merge Queue Status

Rule: default-squash


  • Entered queue2026-02-20 10:00 UTC
  • Checks passed · in-place
  • Merged2026-02-20 10:00 UTC · at d448fe83169178a45ccbace0a72cb6dce4654841

This pull request spent 6 seconds in the queue, with no time running CI.

Required conditions to merge

@github-actions
Copy link
Contributor

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

contribution/core This is a PR that came from AWS. p2 pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes. pr-linter/cli-integ-tested Assert that any CLI changes have been integ tested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants