From 0d0804d563573fc457fb8f6714f9d5aa36b7e9b3 Mon Sep 17 00:00:00 2001 From: Asad Iqbal Date: Fri, 24 Apr 2026 21:24:40 +0500 Subject: [PATCH] fix(ci): pass CI values file to release lint so --strict passes MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The release workflow's `helm lint --strict ./client` step was running without a values file. Schema enforces minLength:1 on clientId and clientPassword; the defaults in values.yaml are deliberately empty (forcing operators to supply real credentials). Result: strict lint always failed, and no release since the schema constraint was added has published — this is why 1.0.4, 1.0.5, 1.0.6, 1.0.7 never reached gh-pages. Mirror helm-ci.yaml's approach and pass `-f client/ci/eks-values.yaml` (which sets ci-test credentials) so the chart lints cleanly for packaging. Exhaustive multi-platform linting continues to run in helm-ci.yaml on every PR. --- .github/workflows/release-helm-chart.yaml | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.github/workflows/release-helm-chart.yaml b/.github/workflows/release-helm-chart.yaml index 8d8696a..5593c24 100644 --- a/.github/workflows/release-helm-chart.yaml +++ b/.github/workflows/release-helm-chart.yaml @@ -36,7 +36,13 @@ jobs: version: v3.15.4 - name: Lint chart - run: helm lint --strict ./client + # Use a CI values file so `--strict` sees non-empty clientId / + # clientPassword (the defaults in values.yaml are deliberately empty + # to force operators to supply real credentials, and the schema + # enforces minLength:1). Exhaustive multi-platform linting happens in + # helm-ci.yaml on every PR — here we just need the chart to lint + # cleanly for packaging. + run: helm lint --strict ./client -f client/ci/eks-values.yaml - name: Package tracebloc chart id: package