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

feat(aws-eks): add support for EKS version 1.30 #30443

Closed
2 tasks
cristianmagana opened this issue Jun 3, 2024 · 3 comments · Fixed by #30454
Closed
2 tasks

feat(aws-eks): add support for EKS version 1.30 #30443

cristianmagana opened this issue Jun 3, 2024 · 3 comments · Fixed by #30454
Assignees
Labels
@aws-cdk/aws-eks Related to Amazon Elastic Kubernetes Service effort/medium Medium work item – several days of effort feature-request A feature should be added or improved. p2

Comments

@cristianmagana
Copy link

Describe the feature

Amazon EKS announced support for Kubernetes version 1.30

https://aws.amazon.com/about-aws/whats-new/2024/05/amazon-eks-distro-kubernetes-version-1-30/

CDK support for aws_eks.KubernetesVersion would be needed to mark this feature as complete

Kubectl lambda layer is waiting approval to support v1.30

Use Case

EKS Blueprints deployed from cdk need to be manually upgraded to v1.30

Proposed Solution

No response

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

CDK version used

2.144.0

Environment details (OS name and version, etc.)

EKS

@cristianmagana cristianmagana added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Jun 3, 2024
@github-actions github-actions bot added the @aws-cdk/aws-eks Related to Amazon Elastic Kubernetes Service label Jun 3, 2024
@pahud
Copy link
Contributor

pahud commented Jun 4, 2024

depends on cdklabs/awscdk-asset-kubectl#838

@pahud pahud added p2 effort/medium Medium work item – several days of effort and removed needs-triage This issue or PR still needs to be triaged. labels Jun 4, 2024
@pahud pahud self-assigned this Jun 4, 2024
@mergify mergify bot closed this as completed in #30454 Jun 6, 2024
Copy link

github-actions bot commented Jun 6, 2024

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

mergify bot pushed a commit that referenced this issue Jun 6, 2024
### Issue # (if applicable)

- Similar to #29040, this PR adds 1.30 support.
- Update all existing integ tests for `V1_29` and make sure they deploy with `V1_30`.

DependOn:
- [x] cdklabs/awscdk-asset-kubectl#838
- [x] cdklabs/awscdk-asset-kubectl#860

Closes #30443

### Reason for this change



### Description of changes



### Description of how you validated changes

```ts
export class EksClusterLatestVersion extends Stack {
  constructor(scope: Construct, id: string, props: StackProps) {
    super(scope, id, props);

    const vpc = new ec2.Vpc(this, 'Vpc', { natGateways: 1 });
    const mastersRole = new iam.Role(this, 'Role', {
      assumedBy: new iam.AccountRootPrincipal(),
    });

    // create eks.Cluster of the latest k8s version
    new eks.Cluster(this, 'Cluster', {
      vpc,
      mastersRole,
      version: eks.KubernetesVersion.V1_30,
      kubectlLayer: new KubectlV30Layer(this, 'KubectlLayer'),
      defaultCapacity: 1,
    });
  }
}
```

$ kubectl get no
NAME                           STATUS   ROLES    AGE     VERSION
ip-172-31-3-166.ec2.internal   Ready    <none>   5m38s   v1.30.0-eks-036c24b



### Checklist
- [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Leo10Gama pushed a commit to Leo10Gama/aws-cdk that referenced this issue Jun 11, 2024
### Issue # (if applicable)

- Similar to aws#29040, this PR adds 1.30 support.
- Update all existing integ tests for `V1_29` and make sure they deploy with `V1_30`.

DependOn:
- [x] cdklabs/awscdk-asset-kubectl#838
- [x] cdklabs/awscdk-asset-kubectl#860

Closes aws#30443

### Reason for this change



### Description of changes



### Description of how you validated changes

```ts
export class EksClusterLatestVersion extends Stack {
  constructor(scope: Construct, id: string, props: StackProps) {
    super(scope, id, props);

    const vpc = new ec2.Vpc(this, 'Vpc', { natGateways: 1 });
    const mastersRole = new iam.Role(this, 'Role', {
      assumedBy: new iam.AccountRootPrincipal(),
    });

    // create eks.Cluster of the latest k8s version
    new eks.Cluster(this, 'Cluster', {
      vpc,
      mastersRole,
      version: eks.KubernetesVersion.V1_30,
      kubectlLayer: new KubectlV30Layer(this, 'KubectlLayer'),
      defaultCapacity: 1,
    });
  }
}
```

$ kubectl get no
NAME                           STATUS   ROLES    AGE     VERSION
ip-172-31-3-166.ec2.internal   Ready    <none>   5m38s   v1.30.0-eks-036c24b



### Checklist
- [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
mazyu36 pushed a commit to mazyu36/aws-cdk that referenced this issue Jun 22, 2024
### Issue # (if applicable)

- Similar to aws#29040, this PR adds 1.30 support.
- Update all existing integ tests for `V1_29` and make sure they deploy with `V1_30`.

DependOn:
- [x] cdklabs/awscdk-asset-kubectl#838
- [x] cdklabs/awscdk-asset-kubectl#860

Closes aws#30443

### Reason for this change



### Description of changes



### Description of how you validated changes

```ts
export class EksClusterLatestVersion extends Stack {
  constructor(scope: Construct, id: string, props: StackProps) {
    super(scope, id, props);

    const vpc = new ec2.Vpc(this, 'Vpc', { natGateways: 1 });
    const mastersRole = new iam.Role(this, 'Role', {
      assumedBy: new iam.AccountRootPrincipal(),
    });

    // create eks.Cluster of the latest k8s version
    new eks.Cluster(this, 'Cluster', {
      vpc,
      mastersRole,
      version: eks.KubernetesVersion.V1_30,
      kubectlLayer: new KubectlV30Layer(this, 'KubectlLayer'),
      defaultCapacity: 1,
    });
  }
}
```

$ kubectl get no
NAME                           STATUS   ROLES    AGE     VERSION
ip-172-31-3-166.ec2.internal   Ready    <none>   5m38s   v1.30.0-eks-036c24b



### Checklist
- [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
@aws-cdk-automation
Copy link
Collaborator

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.

@aws aws locked as resolved and limited conversation to collaborators Jul 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
@aws-cdk/aws-eks Related to Amazon Elastic Kubernetes Service effort/medium Medium work item – several days of effort feature-request A feature should be added or improved. p2
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants