Skip to content

Auto-MNNVL: add PodSpec injection for MNNVL resourceClaims#385

Merged
shmuel-runai merged 2 commits into
ai-dynamo:mainfrom
shmuel-runai:RUN-36138/main-mnnvl-pclq-inject
Feb 2, 2026
Merged

Auto-MNNVL: add PodSpec injection for MNNVL resourceClaims#385
shmuel-runai merged 2 commits into
ai-dynamo:mainfrom
shmuel-runai:RUN-36138/main-mnnvl-pclq-inject

Conversation

@shmuel-runai

Copy link
Copy Markdown
Contributor

/kind feature

ref: GREP-270

What this PR does / why we need it:

Implement InjectAutoMNNVLIntoPodSpec to inject MNNVL resourceClaims into PodSpecs for GPU workloads. The function:

  • Adds a pod-level PodResourceClaim referencing the RCT
  • Adds claim references to each container that requests GPU
  • Is idempotent (safe to call multiple times)
  • Logs injection actions at debug level

Integration with PCS + PCSG controller:

  • Inject MNNVL claims when PCS controller creates PodClique resources
  • Propagate grove.io/auto-mnnvl annotation from PCS to PCSG
  • Inject MNNVL claims into PodCliques created via PCSG path

Refactor test organization:

  • Move helper function tests to helpers_test.go

Implement InjectAutoMNNVLIntoPodSpec to inject MNNVL resourceClaims
into PodSpecs for GPU workloads. The function:
- Adds a pod-level PodResourceClaim referencing the RCT
- Adds claim references to each container that requests GPU
- Is idempotent (safe to call multiple times)
- Logs injection actions at debug level

Integration with PCS + PCSG controller:
- Inject MNNVL claims when PCS controller creates PodClique resources
- Propagate grove.io/auto-mnnvl annotation from PCS to PCSG
- Inject MNNVL claims into PodCliques created via PCSG path

Refactor test organization:
- Move helper function tests to helpers_test.go
danbar2
danbar2 previously approved these changes Feb 2, 2026
Comment thread operator/internal/mnnvl/injection.go Outdated
@shmuel-runai shmuel-runai merged commit 4d7e621 into ai-dynamo:main Feb 2, 2026
32 of 37 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