Skip to content

chore(eks): add AL2023_ARM_64_NVIDIA AMI type for managed node groups#35578

Merged
mergify[bot] merged 4 commits intoaws:mainfrom
amandladev:feature/eks-al2023-arm64-nvidia-support
Sep 27, 2025
Merged

chore(eks): add AL2023_ARM_64_NVIDIA AMI type for managed node groups#35578
mergify[bot] merged 4 commits intoaws:mainfrom
amandladev:feature/eks-al2023-arm64-nvidia-support

Conversation

@amandladev
Copy link
Copy Markdown
Contributor

Issue: #35571

Reason for this change:

AWS EKS now supports Amazon Linux 2023 with NVIDIA drivers for ARM64 architecture (AL2023_ARM_64_NVIDIA) for managed node groups, but this AMI type is not available in the CDK EKS constructs. Users who want to leverage ARM64-based GPU instances (like Graviton-based GPU instances) with the latest Amazon Linux 2023 optimized for NVIDIA workloads cannot specify this AMI type through CDK, forcing them to use CloudFormation directly or older AMI types.

Description of changes:

Add support for Amazon Linux 2023 with NVIDIA drivers on ARM64 architecture for EKS managed node groups:

  • Add AL2023_ARM_64_NVIDIA to NodegroupAmiType enum: New AMI type following the established naming convention
  • Include in gpuAmiTypes array: Ensures proper validation logic for GPU instance types
  • Add comprehensive JSDoc documentation: Describes the AMI as "Amazon Linux 2023 with NVIDIA drivers (ARM-64)"
  • Maintain backward compatibility: No breaking changes to existing functionality
  • Follow established patterns: Consistent with existing AMI type implementations

The implementation enables users to specify ARM64 GPU instances with the latest Amazon Linux 2023 optimized for NVIDIA workloads:

new eks.Nodegroup(this, 'GpuNodeGroup', {
  cluster: cluster,
  amiType: eks.NodegroupAmiType.AL2023_ARM_64_NVIDIA,
  instanceTypes: [new ec2.InstanceType('g5g.xlarge')], // ARM64 GPU instance
});

Description of how you validated changes:

  • Unit Tests: All existing EKS tests pass without modifications, confirming no regressions

Checklist:

@aws-cdk-automation aws-cdk-automation requested a review from a team September 25, 2025 20:18
@github-actions github-actions bot added p2 beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK labels Sep 25, 2025
Copy link
Copy Markdown
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

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

(This review is outdated)

@ozelalisen ozelalisen changed the title feat(eks): add AL2023_ARM_64_NVIDIA AMI type for managed node groups chore(eks): add AL2023_ARM_64_NVIDIA AMI type for managed node groups Sep 26, 2025
ozelalisen
ozelalisen previously approved these changes Sep 26, 2025
Copy link
Copy Markdown
Member

@ozelalisen ozelalisen left a comment

Choose a reason for hiding this comment

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

Changed to chore, as this adding new node group version which is more suitable to chore

@ozelalisen ozelalisen self-assigned this Sep 26, 2025
@aws-cdk-automation aws-cdk-automation dismissed their stale review September 26, 2025 09:37

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

@mergify mergify bot added the queued label Sep 26, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

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).

1 similar comment
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

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 bot added a commit that referenced this pull request Sep 26, 2025
@mergify mergify bot removed the queued label Sep 26, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

This pull request has been removed from the queue for the following reason: pull request dequeued.

Pull request #35578 has been dequeued. Mergify failed to merge the pull request. GitHub can't merge the pull request after 15 retries.
Waiting for the branch protection required status checks to be validated.

You can check the last failing draft PR here: #35589.

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.
If you do update this pull request, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

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).

@ozelalisen
Copy link
Copy Markdown
Member

@Mergifyio update

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

update

❌ Mergify doesn't have permission to update

Details

For security reasons, Mergify can't update this pull request. Try updating locally.
GitHub response: refusing to allow a GitHub App to create or update workflow .github/workflows/github-merit-badger.yml without workflows permission

@ozelalisen
Copy link
Copy Markdown
Member

@Mergifyio requeue

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

requeue

✅ The queue state of this pull request has been cleaned. It can be re-embarked automatically

@mergify mergify bot added the queued label Sep 26, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

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 bot added a commit that referenced this pull request Sep 26, 2025
@mergify mergify bot removed the queued label Sep 26, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

This pull request has been removed from the queue for the following reason: pull request dequeued.

Pull request #35578 has been dequeued. Mergify failed to merge the pull request. GitHub can't merge the pull request after 15 retries.
Waiting for the branch protection required status checks to be validated.

You can check the last failing draft PR here: #35595.

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.
If you do update this pull request, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

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).

@ozelalisen
Copy link
Copy Markdown
Member

@Mergifyio requeue

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

requeue

✅ The queue state of this pull request has been cleaned. It can be re-embarked automatically

@mergify mergify bot added the queued label Sep 26, 2025
mergify bot added a commit that referenced this pull request Sep 26, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

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 removed the queued label Sep 26, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

This pull request has been removed from the queue for the following reason: pull request manually updated.

The pull request #35578 has been manually updated.

You can check the last failing draft PR here: #35598.

If you want to requeue this pull request, you can post a @mergifyio requeue comment.

@mergify mergify bot dismissed ozelalisen’s stale review September 26, 2025 15:04

Pull request has been modified.

@ozelalisen
Copy link
Copy Markdown
Member

@Mergifyio dequeue

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

dequeue

☑️ The pull request is not queued

@mergify mergify bot added the queued label Sep 26, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

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).

1 similar comment
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

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 bot added a commit that referenced this pull request Sep 26, 2025
@mergify mergify bot removed the queued label Sep 26, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

This pull request has been removed from the queue for the following reason: pull request dequeued.

Pull request #35578 has been dequeued. Mergify failed to merge the pull request. GitHub can't merge the pull request after 15 retries.
Waiting for the branch protection required status checks to be validated.

You can check the last failing draft PR here: #35601.

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.
If you do update this pull request, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 26, 2025

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).

@ozelalisen
Copy link
Copy Markdown
Member

@Mergifyio refresh

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 27, 2025

refresh

✅ Pull request refreshed

@mergify mergify bot added the queued label Sep 27, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 27, 2025

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 7379a4b into aws:main Sep 27, 2025
19 checks passed
@mergify mergify bot removed the queued label Sep 27, 2025
@github-actions
Copy link
Copy Markdown
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 Sep 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK p2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants