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

packages: add NVIDIA k8s device plugin #1916

Merged
merged 1 commit into from
Jan 19, 2022

Conversation

arnaldo2792
Copy link
Contributor

Issue number:
N / A

Description of changes:

packages: add NVIDIA k8s device plugin

The NVIDIA k8s device plugin exposes to the kubelet the NVIDIA GPUs in
the host. By providing the k8s device plugin in the base image, the
startup time for a GPU workload decreases, since the kubelet doesn't
have to fetch yet another system container.

Testing done:
I launched a nodegroup without the NVIDIA k8s device plugin daemonset:

eksctl create nodegroup --install-nvidia-plugin=false --config-file=../k8s-1.21.yamlh

I was able to launch a pod in the new host and run nvidia-smi within the host

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

@arnaldo2792
Copy link
Contributor Author

Forced push adds releases-url

The NVIDIA k8s device plugin exposes to the kubelet the NVIDIA GPUs in
the host. By providing the k8s device plugin in the base image, the
startup time for a GPU workload decreases, since the kubelet doesn't
have to fetch yet another system container.

Signed-off-by: Arnaldo Garcia Rincon <[email protected]>
@arnaldo2792
Copy link
Contributor Author

Forced pushes address:

  • Typo in systemd unit
  • Summary description in k8s-device-plugin spec
  • Changed compilation commands in spec file to be more like other go packages

@arnaldo2792 arnaldo2792 merged commit 3b6e8e9 into bottlerocket-os:develop Jan 19, 2022
@arnaldo2792 arnaldo2792 deleted the k8s-plugin branch January 26, 2022 21:37
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.

3 participants