Skip to content

fix dra-driver version in alm-example - #2960

Merged
rahulait merged 1 commit into
NVIDIA:mainfrom
rahulait:fix-dra-driver-in-alm-example
Sep 23, 2026
Merged

rahulait merged 1 commit into
NVIDIA:mainfrom
rahulait:fix-dra-driver-in-alm-example

Conversation

@rahulait

@rahulait rahulait commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Description

The GPUCluster alm-example in the OLM bundle referenced an outdated DRA driver version. This PR removes the explicit image reference so the operator uses the default DRA driver image configured through DRA_DRIVER_IMAGE.

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

Signed-off-by: Rahul Sharma <rahulsharm@nvidia.com>
@rahulait
rahulait requested a review from a team as a code owner September 23, 2026 19:33
@coderabbitai

coderabbitai Bot commented Sep 23, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: NVIDIA/gpu-operator/.coderabbit.yaml

Review profile: QUIET

Plan: Enterprise

Run ID: bd8573cb-a421-42c3-89ae-1b4694bb8059

📥 Commits

Reviewing files that changed from the base of the PR and between 42052c4 and a3aa5d6.

📒 Files selected for processing (3)
  • .github/scripts/update-csv-images.py
  • bundle/manifests/gpu-operator-certified.clusterserviceversion.yaml
  • config/samples/nvidia_v1alpha1_gpucluster.yaml
💤 Files with no reviewable changes (1)
  • bundle/manifests/gpu-operator-certified.clusterserviceversion.yaml

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.


📝 Walkthrough

Walkthrough

The GPUCluster sample now uses the nvcr.io/nvidia repository and version v0.5.0 for its DRA driver image. The CSV GPUCluster example removes the DRA driver image settings and retains computeDomains. The CSV update script no longer updates DRA driver fields for GPUCluster examples. It continues to update NVIDIADriver examples.

Merge Risk: ⚪ Minimal · up to a3aa5

The packaged example uses the configured DRA driver image when its fields are omitted. No actionable merge risk is evident.


Comment @coderabbitai help to get the list of available commands.

@rahulait

Copy link
Copy Markdown
Contributor Author

/cherry-pick release-26.7

@rahulait
rahulait enabled auto-merge September 23, 2026 20:30
@rahulait
rahulait merged commit 60526e3 into NVIDIA:main Sep 23, 2026
21 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

🤖 Backport PR created for release-26.7: #2961 ✅

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