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

chore: Update GPU patterns to use new AL2023 NVIDIA AMI variant and latest EKS 1.31 #2031

Merged
merged 5 commits into from
Oct 14, 2024

Conversation

bryantbiggs
Copy link
Contributor

@bryantbiggs bryantbiggs commented Oct 11, 2024

Description

  • Update GPU patterns to use new AL2023 NVIDIA AMI variant and latest EKS 1.31
  • Update EKS module to latest minor version
  • Update NVIDIA device plugin references to use latest chart and remove affinity rule; the label used on the node groups matches the default affinity so this is no longer required
  • Update EFA device plugin to latest version
  • Add most_recent = true to VPC CNI plugin since new features are being added on the latest version and this will avoid trying to use a feature on an older version of the CNI

Motivation and Context

  • Resolves #

How was this change tested?

  • Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
  • Yes, I have updated the docs for this feature
  • Yes, I ran pre-commit run -a with this PR

Additional Notes

@bryantbiggs
Copy link
Contributor Author

pending deployment testing - will filp to ready once done

@bryantbiggs
Copy link
Contributor Author

needs terraform-aws-modules/terraform-aws-eks#3177 - used local source to these changes for testing

@bryantbiggs bryantbiggs marked this pull request as ready for review October 12, 2024 17:09
@bryantbiggs bryantbiggs requested a review from a team as a code owner October 12, 2024 17:09
@bryantbiggs bryantbiggs merged commit 9ec1d47 into main Oct 14, 2024
8 checks passed
@bryantbiggs bryantbiggs deleted the chore/update-gpu-patterns branch October 14, 2024 14:36
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.

2 participants