From eb80519a95ecf6a243c3fdfdb7a631f39cc075d8 Mon Sep 17 00:00:00 2001 From: Yuan Chen Date: Thu, 30 Apr 2026 15:56:40 -0700 Subject: [PATCH] fix(recipes): correct nvsentinel registry default to OCI source MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The `nvsentinel` registry entry declared: defaultRepository: https://helm.ngc.nvidia.com/nvidia defaultChart: nvidia/nvsentinel But the chart isn't published to the HTTPS NGC index — only to the OCI registry at `oci://ghcr.io/nvidia/nvsentinel`. The defaults are silently ignored today: every nvsentinel-using overlay sets its own `source: oci://ghcr.io/nvidia` + chart `nvsentinel`, so the broken HTTPS default never resolves. But anyone relying on the registry defaults (e.g. via `aicr bundle` without explicit overlay overrides on this entry) would hit the dead path. Update the defaults to match what every overlay already uses: defaultRepository: oci://ghcr.io/nvidia defaultChart: nvsentinel Same shape as the kai-scheduler entry post-#720 (OCI registry path in `defaultRepository`, bare chart name in `defaultChart`). Verified locally: $ helm pull oci://ghcr.io/nvidia/nvsentinel --version v1.3.0 Pulled. $ aicr bundle -r recipe.yaml -o /tmp/bundle ... generates upstream.env with CHART='oci://ghcr.io/nvidia/nvsentinel' REPO='' VERSION='v1.3.0' Note: other NGC HTTPS entries in the registry (gpu-operator, network-operator, nodewright-operator, nvidia-dra-driver-gpu) are unchanged — those charts are genuinely served by the HTTPS NGC index. nvsentinel is special because it ships only via OCI. Refs: #698 (Phase 1 follow-up #2) --- recipes/registry.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/recipes/registry.yaml b/recipes/registry.yaml index af170a7b1..aa959b44c 100644 --- a/recipes/registry.yaml +++ b/recipes/registry.yaml @@ -231,8 +231,8 @@ components: healthCheck: assertFile: checks/nvsentinel/health-check.yaml helm: - defaultRepository: https://helm.ngc.nvidia.com/nvidia - defaultChart: nvidia/nvsentinel + defaultRepository: oci://ghcr.io/nvidia + defaultChart: nvsentinel defaultVersion: v1.3.0 defaultNamespace: nvsentinel nodeScheduling: