-
Notifications
You must be signed in to change notification settings - Fork 983
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
Bottlerocket nvidia #1331
Bottlerocket nvidia #1331
Conversation
✔️ Deploy Preview for karpenter-docs-prod canceled. 🔨 Explore the source changes: df1156a 🔍 Inspect the deploy log: https://app.netlify.com/sites/karpenter-docs-prod/deploys/6206f3286d6097000732e759 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! NICE!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome work! 💯
but did not have the infrastructure in place to deploy it into a live cluster.
Let's hold on this until we can get this manual test done.
I've tested manually. Scaled up an nvidia gpu request on both amd64 and arm64 w/ Bottlerocket 🚀 |
1. Issue, if available:
If the AMI family type used is Bottlerocket and the instance type is an NVIDIA GPU instance the drivers are not available in the generic Bottlerocket AMI. The lookup for arm64 architecture was also dependent on GPU instances but they're not mutually exclusive (eg G5g).
2. Description of changes:
This adds a lookup for Bottlerocket AMIs to use the
-nvidia
variant along with arm64 AMI when needed.3. How was this change tested?
I based the code off the existing AL2 lookups and ran linting and basic tests but did not have the infrastructure in place to deploy it into a live cluster.
4. Does this change impact docs?
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.