Skip to content

[release-1.19] Update systemd unit conditions to execute nvidia-smi and re-add restart logic#1826

Merged
cdesiniotis merged 3 commits into
release-1.19from
backport-1825-to-release-1.19
May 14, 2026
Merged

[release-1.19] Update systemd unit conditions to execute nvidia-smi and re-add restart logic#1826
cdesiniotis merged 3 commits into
release-1.19from
backport-1825-to-release-1.19

Conversation

@github-actions
Copy link
Copy Markdown

🤖 Automated backport of #1825 to release-1.19

✅ Cherry-pick completed successfully with no conflicts.

Original PR: #1825
Original Author: @cdesiniotis

Cherry-picked commits (2):

  • 66b50d2 Update systemd unit conditions to execute nvidia-smi
  • d5177c7 Reintroduce restart logic to nvidia-cdi-refresh.service

This backport was automatically created by the backport bot.

This ensures that the nvidia-cdi-refresh.service does not start until
after the driver is loaded and available. The added benefit of running
nvidia-smi is that it will load the module (if not already loaded) and
create the device nodes (if not already created).

Signed-off-by: Christopher Desiniotis <cdesiniotis@nvidia.com>
(cherry picked from commit 66b50d2)
The restart logic was removed when a dependency on the multi-user-target was added in
5fe6b42.
However, this change had unforeseen consequences and was later reverted in
5eee5ce.
When reverting this change, the restart logic was not re-added to the service.

Signed-off-by: Christopher Desiniotis <cdesiniotis@nvidia.com>
(cherry picked from commit d5177c7)
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 14, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Signed-off-by: Christopher Desiniotis <cdesiniotis@nvidia.com>
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 25890642706

Coverage remained the same at 43.341%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 14866
Covered Lines: 6443
Line Coverage: 43.34%
Coverage Strength: 0.48 hits per line

💛 - Coveralls

@cdesiniotis
Copy link
Copy Markdown
Contributor

/ok to test df93c2e

@cdesiniotis cdesiniotis merged commit b3d3e75 into release-1.19 May 14, 2026
10 of 11 checks passed
@cdesiniotis cdesiniotis deleted the backport-1825-to-release-1.19 branch May 14, 2026 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants