feat: upgrade CUDA and use github hosted runner#278
Closed
tlovell-sxt wants to merge 4 commits into
Closed
Conversation
Recent requirements to use ubuntu24.04 in kubernetes deployments of blitzar dependents have made it difficult or impossible to use CUDA 12.6. Nvidia does not provide an ubuntu24.04 driver image that is compatible with the current blitzar. Using CUDA 12.8 should resolve the issue.
The current runner for the gpu jobs is self-hosted and has gone unmaintained for some time. Now, github provides some larger runners with GPUs that we can use instead for easier maintenance. This PR attempts to use them.
The generic nvidia runner does not have nix installed by default. This adds a step to install it using a cachix action (single-user).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rationale for this change
This PR is a combination of #276 and #277. The CI needs to be fixed, which is what #277 aims to do, in order to upgrade CUDA, which is what #276 aims to do. However, there are now errors in the CI about CUDA/driver version mismatch, so I'm pushing this pull request to see if both changes at once resolve the issue.
What changes are included in this PR?
Are these changes tested?
These changes do not affect existing functionality, which should be verified by existing tests.