-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Support GPU in OpenStack #13330
Support GPU in OpenStack #13330
Conversation
da428f9
to
5b79d40
Compare
@olemarkus as you did that AWS support, could you check this? With this PR I can get GPUs visible in OpenStack and everything looks good. |
/test pull-kops-e2e-kubernetes-aws |
2 similar comments
/test pull-kops-e2e-kubernetes-aws |
/test pull-kops-e2e-kubernetes-aws |
/hold |
1 similar comment
/hold |
4bdc6e5
to
ae139b8
Compare
0c9d7cd
to
c01e5a6
Compare
/hold cancel |
/test pull-kops-e2e-kubernetes-aws /test pull-kops-verify-golangci-lint |
9e96167
to
ab86510
Compare
9252953
to
28caf02
Compare
upup/pkg/fi/cloudup/apply_cluster.go
Outdated
} | ||
|
||
if config.NvidiaGPU != nil && config.NvidiaGPU.DriverPackage == "" { | ||
config.NvidiaGPU.DriverPackage = "nvidia-headless-460-server" |
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.
we should perhaps store this const to same common place. This is now defined in two files as hardcoded, which would be the correct place to store this information? there are wellknownports etc should we add something new?
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.
It could certainly be moved to a const. Perhaps in the API package.
Here we are doing the merging in two locations as well. Didn't it work to move this to a function similar to how we merge workpool?
/test pull-kops-e2e-kubernetes-aws |
ping @olemarkus |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: olemarkus The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
The Kubernetes project has merge-blocking tests that are currently too flaky to consistently pass. This bot retests PRs for certain kubernetes repos according to the following rules:
You can:
/retest |
1 similar comment
The Kubernetes project has merge-blocking tests that are currently too flaky to consistently pass. This bot retests PRs for certain kubernetes repos according to the following rules:
You can:
/retest |
fixes #12750