From 6958ae26390fe088340d500b12ab0dc24faca2dc Mon Sep 17 00:00:00 2001 From: Shivam Saraf Date: Wed, 15 Apr 2026 10:00:48 -0400 Subject: [PATCH] Add helm dependency build step to release workflow The release workflow's helm job was missing `helm dependency build` before `helm package`, causing the controller chart push to fail with "found in Chart.yaml, but missing in charts/ directory: cert-manager". This has been broken since cert-manager was added as an optional subchart dependency. The other helm workflows (helm.yml, helm-validate.yml, helm-image-check.yml) already have this step. Co-Authored-By: Claude Opus 4.6 (1M context) --- .github/workflows/release.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index c84b1b4e..98286243 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -139,6 +139,9 @@ jobs: git commit -m "Bump chart version to $NEW_VERSION [skip ci]" git push + - name: Build chart dependencies + run: helm repo add jetstack https://charts.jetstack.io && helm dependency build ./helm/temporal-worker-controller + - name: Package and Push Helm charts run: | # Use version from previous step