From 2f6539549dbd755399fff8bbed9433cd4c11cdcc Mon Sep 17 00:00:00 2001 From: Yuan Chen Date: Wed, 29 Apr 2026 16:32:21 -0700 Subject: [PATCH] fix(examples): remove stale nvsentinel manifest reference in aks-training The example references `components/nvsentinel/manifests/allow-intra- namespace.yaml`, which does not exist in the tree, so `aicr bundle examples/recipes/aks-training.yaml` fails: failed to load manifest components/nvsentinel/manifests/ allow-intra-namespace.yaml for component nvsentinel: open ...: file does not exist Bug history: - #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 blocked intra-namespace traffic (DCGM port 5555) and API-server -> webhook traffic (cert-manager port 443). - #309 (`feat: move to latest NVSentinel`) bumped nvsentinel past v0.7.0 and removed the workaround source file. The reference in the existing examples was cleaned up at that time. - #415 (`feat: Add AKS H100 recipe overlays`) added the new AKS example by copying the template structure but kept the now- orphaned `manifestFiles:` block. The reference has been broken since #415 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. Verified locally: aicr bundle -r examples/recipes/aks-training.yaml -o /tmp/bundle ... succeeds, no missing-manifest error --- examples/recipes/aks-training.yaml | 2 -- 1 file changed, 2 deletions(-) diff --git a/examples/recipes/aks-training.yaml b/examples/recipes/aks-training.yaml index e980f6d45..3104d452d 100644 --- a/examples/recipes/aks-training.yaml +++ b/examples/recipes/aks-training.yaml @@ -124,8 +124,6 @@ componentRefs: dependencyRefs: - cert-manager - gpu-operator - manifestFiles: - - components/nvsentinel/manifests/allow-intra-namespace.yaml - name: prometheus-adapter namespace: monitoring chart: prometheus-adapter