Skip to content
This repository was archived by the owner on Apr 23, 2026. It is now read-only.

Document removing any CUDA packages before Upgrade#3242

Merged
mergify[bot] merged 1 commit intoinstructlab:mainfrom
derekhiggins:rm-cuda
Mar 27, 2025
Merged

Document removing any CUDA packages before Upgrade#3242
mergify[bot] merged 1 commit intoinstructlab:mainfrom
derekhiggins:rm-cuda

Conversation

@derekhiggins
Copy link
Copy Markdown
Contributor

@derekhiggins derekhiggins commented Mar 20, 2025 •
edited
Loading

Old cuda package have been seen to cause updates to fail due to dependency errors

Closes: #3241

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the
    conventional commits.
  • Changelog updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Functional tests have been added, if necessary.
  • E2E Workflow tests have been added, if necessary.

@derekhiggins derekhiggins mentioned this pull request Mar 20, 2025
Merged
derekhiggins
derekhiggins commented Mar 20, 2025
View reviewed changes
Comment thread scripts/infra/cloud-instance.sh Outdated
@derekhiggins derekhiggins changed the title Remove any CUDA packages before installing drivers Document removing any CUDA packages before Upgrade Mar 27, 2025
@mergify mergify Bot added documentation Improvements or additions to documentation ci-failure PR has at least one CI failure labels Mar 27, 2025
@mergify mergify Bot removed the ci-failure PR has at least one CI failure label Mar 27, 2025
danmcp
danmcp reviewed Mar 27, 2025
View reviewed changes
Comment thread docs/accelerators/gpu-acceleration.md Outdated
Comment thread docs/accelerators/gpu-acceleration.md Outdated
@derekhiggins
Document removing any CUDA packages before Upgrade
e087cff 
Old cuda package have been seen to cause updates
to fail due to dependency errors

Closes: instructlab#3241
Signed-off-by: Derek Higgins <[email protected]>
@mergify mergify Bot added the one-approval PR has one approval from a maintainer label Mar 27, 2025
@nathan-weinberg nathan-weinberg removed the request for review from booxter March 27, 2025 14:23
@mergify mergify Bot merged commit 5d51b64 into instructlab:main Mar 27, 2025
14 checks passed
@mergify mergify Bot removed the one-approval PR has one approval from a maintainer label Mar 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@danmcp danmcp danmcp approved these changes

@nathan-weinberg nathan-weinberg nathan-weinberg approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Rerunning install_rh_nvidia_drivers on host fails with package dependency errors

3 participants

@derekhiggins @danmcp @nathan-weinberg