Skip to content

Add VGPU documentation #44

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

Merged
merged 7 commits into from
Jun 30, 2023
Merged

Add VGPU documentation #44

merged 7 commits into from
Jun 30, 2023

Conversation

jovial
Copy link
Contributor

@jovial jovial commented Jun 23, 2023

Adds documentation for NVIDIA vGPUs.

@jovial jovial requested a review from a team as a code owner June 23, 2023 19:35
@jovial jovial requested a review from oneswig June 23, 2023 19:36

Host OS configuration is done by using roles in the `stackhpc.linux <https://github.com/stackhpc/ansible-collection-linux>`_ ansible collection.

Add the following to your ansible ``requirements.yml``:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could these changes go into s-k-c?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Potentially. I was thinking this first iteration would just contain the documentation to use the role and then we eventually land the playbook in kayobe. We'd then update these docs to use to the kayobe support. What do you reckon?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For me it depends on when we expect to get it into Kayobe. If we don't think we'll have it in time to backport to the (downstream) Antelope release, then I'd suggest we commit the custom playbook. It can always be removed in favour of the Kayobe support in future, and keeps everyone in sync.

FWIW, I didn't put the Kayobe VGPU support in the infra roadmap, thinking we'd probably end up moving on after getting to this point. That said, if you think it's a reasonably small piece of work then it might be worth doing in time to backport to antelope downstream.

jovial and others added 2 commits June 28, 2023 14:41
Co-authored-by: Mark Goddard <[email protected]>
Co-authored-by: Mark Goddard <[email protected]>

Host OS configuration is done by using roles in the `stackhpc.linux <https://github.com/stackhpc/ansible-collection-linux>`_ ansible collection.

Add the following to your ansible ``requirements.yml``:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Potentially. I was thinking this first iteration would just contain the documentation to use the role and then we eventually land the playbook in kayobe. We'd then update these docs to use to the kayobe support. What do you reckon?

@jovial jovial requested a review from markgoddard June 28, 2023 15:51
@jovial jovial requested a review from markgoddard June 29, 2023 10:00
@jovial jovial merged commit d8c8b9d into master Jun 30, 2023
@jovial jovial deleted the feaure/vgpu branch June 30, 2023 09:47
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