Skip to content

Set extendedResourceName on gpu.nvidia.com DeviceClass for resource.k8s.io/v1 - #2859

Open
karthikvetrivel wants to merge 1 commit into
NVIDIA:mainfrom
karthikvetrivel:kv-dra-deviceclass-extended-resource
Open

Set extendedResourceName on gpu.nvidia.com DeviceClass for resource.k8s.io/v1#2859
karthikvetrivel wants to merge 1 commit into
NVIDIA:mainfrom
karthikvetrivel:kv-dra-deviceclass-extended-resource

Conversation

@karthikvetrivel

Copy link
Copy Markdown
Member

No description provided.

@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: QUIET

Plan: Enterprise

Run ID: 4c9a4e4a-503c-4758-9e1e-cc7bdcab0f9b

📥 Commits

Reviewing files that changed from the base of the PR and between df32e7e and d890133.

📒 Files selected for processing (1)
  • manifests/state-dra-driver/0400_deviceclass-gpu.yaml

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


📝 Walkthrough

Walkthrough

The GPU device class manifest now sets extendedResourceName to nvidia.com/gpu when DeviceClassAPIVersion is resource.k8s.io/v1.

Merge Risk: ⚪ Minimal · up to d8901

GPU DeviceClasses rendered for resource.k8s.io/v1 now advertise nvidia.com/gpu as their extended resource. No current merge-blocking risk is identified.


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

…8s.io/v1

Signed-off-by: Karthik Vetrivel <kvetrivel@nvidia.com>
@karthikvetrivel
karthikvetrivel force-pushed the kv-dra-deviceclass-extended-resource branch from d890133 to 5b897f3 Compare September 4, 2026 21:00
@tariq1890

Copy link
Copy Markdown
Contributor

Thanks @karthikvetrivel, can you fill out the PR description explaining why this PR is needed?

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.

2 participants