Skip to content

[release-1.19] fix systemd unit conditions to work on WSL2#1800

Merged
cdesiniotis merged 1 commit into
release-1.19from
backport-1782-to-release-1.19
May 7, 2026
Merged

[release-1.19] fix systemd unit conditions to work on WSL2#1800
cdesiniotis merged 1 commit into
release-1.19from
backport-1782-to-release-1.19

Conversation

@github-actions

@github-actions github-actions Bot commented May 6, 2026

Copy link
Copy Markdown

🤖 Automated backport of #1782 to release-1.19

✅ Cherry-pick completed successfully with no conflicts.

Original PR: #1782
Original Author: @elezar

Cherry-picked commits (1):

  • fe5a920 fix systemd unit conditions to work on WSL2

This backport was automatically created by the backport bot.

On WSL2, nvidia-smi lives at /usr/lib/wsl/lib/nvidia-smi rather than
/usr/bin or /usr/sbin, and the NVIDIA kernel modules (nvidia.ko /
nvidia-current.ko) are not present since GPU access goes through the
DirectX Graphics infrastructure (/dev/dxg).

Add /usr/lib/wsl/lib/nvidia-smi as a third OR ConditionPathExists and
extend ExecCondition to succeed when /dev/dxg exists so the service
runs on WSL2 without breaking the check on standard driver installs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
(cherry picked from commit fe5a920)
@copy-pr-bot

copy-pr-bot Bot commented May 6, 2026

Copy link
Copy Markdown

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.

@cdesiniotis cdesiniotis enabled auto-merge May 6, 2026 16:31
@cdesiniotis cdesiniotis disabled auto-merge May 7, 2026 19:34
@cdesiniotis cdesiniotis merged commit 997ed49 into release-1.19 May 7, 2026
1 check passed
@cdesiniotis cdesiniotis deleted the backport-1782-to-release-1.19 branch May 7, 2026 19:34
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.

2 participants