Skip to content

Decouple OperatorMetrics object from ClusterPolicyController object#2584

Merged
cdesiniotis merged 1 commit into
NVIDIA:mainfrom
cdesiniotis:decouple-operator-metrics-from-clusterpolicy
Jun 26, 2026
Merged

Decouple OperatorMetrics object from ClusterPolicyController object#2584
cdesiniotis merged 1 commit into
NVIDIA:mainfrom
cdesiniotis:decouple-operator-metrics-from-clusterpolicy

Conversation

@cdesiniotis

Copy link
Copy Markdown
Contributor

With this commit, any controller in the GPU Operator that needs to set operator metrics can do so without relying on state from the ClusterPolicy controller object. This will be especially relevant when we integrate the NVIDIA DRA Driver for GPUs since we will be introducing a new CRD that serves as an alternative to the ClusterPolicy CRD.

Code changes in this PR were drafted with the assistance of Claude Code.

With this commit, any controller in the GPU Operator that needs to
set operator metrics can do so without relying on state from
the ClusterPolicy controller object. This will be especially
relevant when we integrate the NVIDIA DRA Driver for GPUs since
we will be introducing a new CRD that serves as an alternative to
the ClusterPolicy CRD.

Signed-off-by: Christopher Desiniotis <cdesiniotis@nvidia.com>
@cdesiniotis cdesiniotis self-assigned this Jun 26, 2026

@karthikvetrivel karthikvetrivel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@rahulait rahulait left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cdesiniotis cdesiniotis enabled auto-merge June 26, 2026 18:35
@cdesiniotis cdesiniotis merged commit 197a7dd into NVIDIA:main Jun 26, 2026
36 of 37 checks passed
@cdesiniotis cdesiniotis added this to the v26.7 milestone Jun 26, 2026
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.

5 participants