-
Notifications
You must be signed in to change notification settings - Fork 67
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
fix: GPU bootstrap, refresh driver versions and list of supported GPU VM SKUs #587
Conversation
Pull Request Test Coverage Report for Build 12092148753Details
💛 - Coveralls |
GPUNeedsFabricManager and related changes can be split into a separate PR since karpenter doesn't have MIG support today right? https://learn.microsoft.com/en-us/azure/aks/gpu-multi-instance?tabs=azure-cli |
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, lets wait on removing NC1 etc. or do it in a separate PR.
Yeah seems weird to add this support now. |
Fixes #579, #517
Description
How was this change tested?
make presubmit
Does this change impact docs?
Release Note