Skip to content

Add mnnvl-group annotation validation and extend webhook to PodClique…#535

Merged
shmuel-runai merged 3 commits into
ai-dynamo:mainfrom
shmuel-runai:grove-417/main-group-annotation
Apr 16, 2026
Merged

Add mnnvl-group annotation validation and extend webhook to PodClique…#535
shmuel-runai merged 3 commits into
ai-dynamo:mainfrom
shmuel-runai:grove-417/main-group-annotation

Conversation

@shmuel-runai

Copy link
Copy Markdown
Contributor

…TemplateSpec level

/kind feature

What this PR does / why we need it:

Introduce grove.io/mnnvl-group annotation support with DNS-1123 label validation, conflict detection (auto-mnnvl: disabled + mnnvl-group), and immutability enforcement.
Extend MNNVL webhook validation from PCS metadata to the full spec hierarchy (PCS → template → clique).

Which issue(s) this PR fixes:

Refs #417

NONE

@copy-pr-bot

copy-pr-bot Bot commented Apr 15, 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 force-pushed the grove-417/main-group-annotation branch from fcb46b3 to 1046c11 Compare April 16, 2026 08:15
Comment thread operator/internal/mnnvl/webhook.go Outdated
Comment thread operator/internal/mnnvl/webhook.go Outdated
Comment thread operator/internal/mnnvl/webhook.go
…TemplateSpec level

Introduce grove.io/mnnvl-group annotation support with DNS-1123 label
validation, conflict detection (auto-mnnvl: disabled + mnnvl-group),
and immutability enforcement. Extend MNNVL webhook validation from
PCS metadata to the full spec hierarchy (PCS → template → clique).
@shmuel-runai shmuel-runai force-pushed the grove-417/main-group-annotation branch from 1046c11 to 8a6083a Compare April 16, 2026 12:40
@shmuel-runai shmuel-runai merged commit 8cdd90b into ai-dynamo:main Apr 16, 2026
26 of 27 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