Skip to content

Add Annotations field to PodCliqueScalingGroupConfig with MNNVL propa…#541

Merged
shmuel-runai merged 1 commit into
ai-dynamo:mainfrom
shmuel-runai:grove-417/main-group-annotation
Apr 20, 2026
Merged

Add Annotations field to PodCliqueScalingGroupConfig with MNNVL propa…#541
shmuel-runai merged 1 commit into
ai-dynamo:mainfrom
shmuel-runai:grove-417/main-group-annotation

Conversation

@shmuel-runai

Copy link
Copy Markdown
Contributor

…gation

  • Extend the PCS API with an Annotations field on PodCliqueScalingGroupConfig, allowing PCSG-level.
  • MNNVL annotations (auto-mnnvl,mnnvl-group) are propagated from PCS to PCSG when not explicitly set in the config.
  • Webhook validation enforces annotation rules and immutability on PCSG configs at create and update time.

/kind feature

What this PR does / why we need it:

Add Annotations field to PodCliqueScalingGroupConfig with MNNVL propagation

  • Extend the PCS API with an Annotations field on PodCliqueScalingGroupConfig, allowing PCSG-level.
  • MNNVL annotations (auto-mnnvl,mnnvl-group) are propagated from PCS to PCSG when not explicitly set in the config.
  • Webhook validation enforces annotation rules and immutability on PCSG configs at create and update time.

Which issue(s) this PR fixes:

Fixes #417

…gation

* Extend the PCS API with an Annotations field on PodCliqueScalingGroupConfig,
allowing PCSG-level.
* MNNVL annotations (auto-mnnvl,mnnvl-group) are propagated from PCS to PCSG when not explicitly set in
the config.
* Webhook validation enforces annotation rules and immutability on PCSG configs at create and update time.
@copy-pr-bot

copy-pr-bot Bot commented Apr 19, 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.

@shmuel-runai shmuel-runai self-assigned this Apr 19, 2026
@shmuel-runai shmuel-runai merged commit 42d9548 into ai-dynamo:main Apr 20, 2026
15 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.

Enhancement: MNNVL V2, ComputeDomain Injection Via Annotation

3 participants