From e16ce07b8023ac538dff9683bd9fdfed01c50d34 Mon Sep 17 00:00:00 2001
From: Lukas Wuttke <lukas@tracebloc.io>
Date: Wed, 24 Jun 2026 20:17:20 +0200
Subject: [PATCH] ci(helm): drop redundant schema job; rename colliding
 Lint/Unit tests jobs
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- The `schema` job re-rendered each platform's values to /dev/null — exactly
  what `template` already does (and `template` also kubeconform-validates the
  output), so `schema` added 4 matrix jobs of zero extra coverage. Removed.
  (Not a required status check — branch protection requires only Lint + Unit
  tests.)
- helm-ci's `lint`/`unittest` jobs were named "Lint"/"Unit tests", colliding
  with the same-named required checks that standard-checks.yml emits
  unconditionally on every PR. Renamed to "Helm lint"/"Helm unit tests" so the
  required contexts come unambiguously from standard-checks.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 .github/workflows/helm-ci.yaml | 26 ++------------------------
 1 file changed, 2 insertions(+), 24 deletions(-)

diff --git a/.github/workflows/helm-ci.yaml b/.github/workflows/helm-ci.yaml
index 4dda9d4..ad46c83 100644
--- a/.github/workflows/helm-ci.yaml
+++ b/.github/workflows/helm-ci.yaml
@@ -26,7 +26,7 @@ concurrency:
 jobs:
   lint:
     timeout-minutes: 10
-    name: Lint
+    name: Helm lint
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
@@ -91,7 +91,7 @@ jobs:
 
   unittest:
     timeout-minutes: 10
-    name: Unit tests
+    name: Helm unit tests
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
@@ -107,28 +107,6 @@ jobs:
       - name: Run unit tests
         run: helm unittest ./client
 
-  schema:
-    timeout-minutes: 10
-    name: Schema validation
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        platform: [aks, bm, eks, oc]
-    steps:
-      - uses: actions/checkout@v4
-
-      - name: Set up Helm
-        uses: azure/setup-helm@v4
-        with:
-          version: v3.15.4
-
-      - name: Validate values against schema — ${{ matrix.platform }}
-        run: |
-          helm template test-${{ matrix.platform }} ./client \
-            -f client/ci/${{ matrix.platform }}-values.yaml \
-            > /dev/null
-          echo "Schema validation passed for ${{ matrix.platform }}"
-
   ingestor-multiarch:
     timeout-minutes: 10
     # Guard: the ingestor image the cluster spawns must be a multi-arch index