fix(examples): remove stale nvsentinel manifest reference in aks-training

[yuanchen8911]

Summary

Remove an orphaned manifestFiles: reference in examples/recipes/aks-training.yaml that points to a non-existent file (components/nvsentinel/manifests/allow-intra-namespace.yaml). The reference has been broken since the AKS example was added.

Motivation / Context

aicr bundle examples/recipes/aks-training.yaml currently fails:

failed to load manifest components/nvsentinel/manifests/allow-intra-namespace.yaml
for component nvsentinel: open ...: file does not exist

Bug history

• fix: add NetworkPolicy workaround for nvsentinel metrics-access restriction #68 — fix: add NetworkPolicy workaround for nvsentinel metrics-access restriction — introduced the manifest as a workaround for nvsentinel < v0.7.0, which shipped a metrics-access NetworkPolicy with podSelector: {} that isolated all pods in the namespace and blocked:
  • intra-namespace pod-to-pod traffic (e.g. DCGM port 5555)
  • API-server → webhook traffic (e.g. cert-manager webhook port 443)
• feat: move to latest NVSentinel #309 — feat: move to latest NVSentinel — bumped nvsentinel past v0.7.0 (the upstream NVSentinel/PR#789 fix landed) and removed the workaround source file. References in existing examples were cleaned up at that time.
• feat: Add AKS (Azure Kubernetes Service) H100 recipe overlays #415 — feat: Add AKS (Azure Kubernetes Service) H100 recipe overlays — added the new AKS example by copying the template structure from another example, but kept the now-orphaned manifestFiles: block. The reference has been broken since this PR merged.

The nvsentinel pin in this example is v0.10.0, well past the v0.7.0 cutoff, so the workaround is irrelevant. Removing the two orphaned lines lets the example bundle cleanly.

Surfaced during phase 1 of the recipe version-bump campaign (#698 / #715), where Codex's verification flagged this as a pre-existing bundling failure on main not introduced by the bump PR. Filing as a standalone fix per scoping discussion.

Fixes: bundling failure on aicr bundle examples/recipes/aks-training.yaml
Related: #698, #715

Type of Change

• Bug fix (non-breaking change that fixes an issue)

Component(s) Affected

• Recipe engine / data (pkg/recipe) — example only

Implementation Notes

Two-line deletion. No values or version changes. Other example recipes (kind.yaml, eks-training.yaml, eks-gb200-ubuntu-training-with-validation.yaml) bundle cleanly and don't reference the missing manifest.

Testing

$ aicr bundle -r examples/recipes/aks-training.yaml -o /tmp/aicr/bundle-aks-test
... succeeds, no missing-manifest error

$ make lint    # 0 issues
$ yamllint examples/recipes/aks-training.yaml   # clean

Risk Assessment

• Low — Example-only change, two-line deletion, restores broken bundling. No production code touched.

Rollout notes: None. Example file only.

Checklist

• Tests pass locally (make test with -race)
• Linter passes (make lint)
• I did not skip/disable tests to make CI green
• I added/updated tests for new functionality (N/A — orphaned reference cleanup)
• I updated docs if user-facing behavior changed (N/A — example fix)
• Changes follow existing patterns in the codebase
• Commits are cryptographically signed (git commit -S)

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: f63c124a-eb81-46a9-9d97-a3181f3a6a9f

📥 Commits

Reviewing files that changed from the base of the PR and between 0c939ce and 2f65395.

📒 Files selected for processing (1)

• examples/recipes/aks-training.yaml

💤 Files with no reviewable changes (1)

• examples/recipes/aks-training.yaml

---

📝 Walkthrough

Walkthrough

The pull request modifies the nvsentinel component reference in the AKS training recipe by removing the allow-intra-namespace.yaml manifest from its manifestFiles list. This alters the set of manifests deployed as part of the component configuration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: removing a stale nvsentinel manifest reference from the aks-training recipe example.
Description check	✅ Passed	The description is comprehensive and directly related to the changeset, detailing the bug history, motivation, testing, and risk assessment.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Review rate limit: 9/10 reviews remaining, refill in 6 minutes.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[yuanchen8911]

Absorbing this fix into #715 — the 2-line deletion now lands as part of the phase-1 recipe bumps PR (commit 6441ab79 in #715, with Closes: #716 in the commit trailer). Same diff, same verification; just rolling it into the larger PR to avoid sequencing two PRs that both need to land. Bug history preserved in the #715 commit body.