Skip to content

Commit f1e8ea5

Browse files
Shivs11claude
andauthored
Add helm dependency build step to release workflow (#279)
## Summary - Add `helm dependency build` before `helm package` in the release workflow's helm job - This fixes the controller Helm chart push which has been failing 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 — `release.yml` was missed ## Context The v1.5.0, v1.5.1, and v1.5.2 releases all failed to push the controller Helm chart to OCI with: ``` Error: found in Chart.yaml, but missing in charts/ directory: cert-manager ``` 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent f8fc3c2 commit f1e8ea5

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

.github/workflows/release.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -139,6 +139,9 @@ jobs:
139139
git commit -m "Bump chart version to $NEW_VERSION [skip ci]"
140140
git push
141141
142+
- name: Build chart dependencies
143+
run: helm repo add jetstack https://charts.jetstack.io && helm dependency build ./helm/temporal-worker-controller
144+
142145
- name: Package and Push Helm charts
143146
run: |
144147
# Use version from previous step

0 commit comments

Comments
 (0)