Skip to content

Conversation

@rahulait
Copy link
Contributor

@rahulait rahulait commented Jan 23, 2026

Description

Checklist

  • No secrets, sensitive information, or unrelated changes
  • Lint checks passing (make lint)
  • Generated assets in-sync (make validate-generated-assets)
  • Go mod artifacts in-sync (make validate-modules)
  • Test cases are added for new code paths

Testing

Manually deployed a cluster with gpu-operator image from this PR.

Verified that all pods are able to come up fine and additional drivers are also validated correctly.

time="2026-01-23T20:08:10Z" level=info msg="version: v25.10.1, commit: a7709fe6"
time="2026-01-23T20:08:10Z" level=info msg="Attempting to validate a pre-installed driver on the host"
time="2026-01-23T20:08:10Z" level=info msg="No pre-installed driver detected on the host: no 'nvidia-smi' file present on the host: lstat /host/usr/bin/nvidia-smi: no such file or directory"
time="2026-01-23T20:08:10Z" level=info msg="Validating containerized driver installation"
time="2026-01-23T20:08:10Z" level=info msg="Driver is not pre-installed on the host and is managed by GPU Operator. Checking driver container status."
...
time="2026-01-23T20:09:35Z" level=info msg="Attempting to validate a driver container installation"
...
running command sh with args [-c lsmod | grep -E '^gdrdrv\s']
time="2026-01-23T20:09:35Z" level=info msg="Validating additional driver component: gdrcopy"
...

Signed-off-by: Rahul Sharma <rahulsharm@nvidia.com>
@copy-pr-bot
Copy link

copy-pr-bot bot commented Jan 23, 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.

@rahulait
Copy link
Contributor Author

/ok to test a7709fe

@cdesiniotis cdesiniotis enabled auto-merge January 23, 2026 20:42
@cdesiniotis cdesiniotis merged commit b9a19ec into NVIDIA:main Jan 23, 2026
15 of 16 checks passed
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.

3 participants