fix: ensure Helm chart is published only after container images are available (#7859)
fix: consolidate Helm chart release with container image build Resolve issue #7855 by consolidating the Helm chart release workflow with the container image build workflow. This ensures perfect alignment: 1. Container images are built and pushed to GHCR 2. Images are copied from GHCR to Docker Hub 3. Helm chart is published only after step 2 completes Previously, the Helm chart was published immediately on tag push before images were available in Docker Hub, causing deployment failures. Changes: - Added helm-release job to container_release_unified.yml that depends on copy-to-dockerhub job - Removed helm_chart_release.yml workflow (consolidated into unified release) Benefits: - No race conditions between image push and chart publication - Users can deploy immediately after release - Single source of truth for release process - Clearer job dependencies and execution flow
This commit is contained in:
18
.github/workflows/container_release_unified.yml
vendored
18
.github/workflows/container_release_unified.yml
vendored
@@ -223,5 +223,23 @@ jobs:
|
|||||||
|
|
||||||
echo "✓ Successfully copied ${{ matrix.variant }} to Docker Hub"
|
echo "✓ Successfully copied ${{ matrix.variant }} to Docker Hub"
|
||||||
|
|
||||||
|
helm-release:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
needs: [copy-to-dockerhub]
|
||||||
|
permissions:
|
||||||
|
contents: write
|
||||||
|
pages: write
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v6
|
||||||
|
- name: Publish Helm charts
|
||||||
|
uses: stefanprodan/helm-gh-pages@v1.7.0
|
||||||
|
with:
|
||||||
|
token: ${{ secrets.GITHUB_TOKEN }}
|
||||||
|
charts_dir: k8s/charts
|
||||||
|
target_dir: helm
|
||||||
|
branch: gh-pages
|
||||||
|
helm_version: "3.18.4"
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
23
.github/workflows/helm_chart_release.yml
vendored
23
.github/workflows/helm_chart_release.yml
vendored
@@ -1,23 +0,0 @@
|
|||||||
name: "helm: publish charts"
|
|
||||||
on:
|
|
||||||
push:
|
|
||||||
tags:
|
|
||||||
- '*'
|
|
||||||
|
|
||||||
permissions:
|
|
||||||
contents: write
|
|
||||||
pages: write
|
|
||||||
|
|
||||||
jobs:
|
|
||||||
release:
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
steps:
|
|
||||||
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8
|
|
||||||
- name: Publish Helm charts
|
|
||||||
uses: stefanprodan/helm-gh-pages@v1.7.0
|
|
||||||
with:
|
|
||||||
token: ${{ secrets.GITHUB_TOKEN }}
|
|
||||||
charts_dir: k8s/charts
|
|
||||||
target_dir: helm
|
|
||||||
branch: gh-pages
|
|
||||||
helm_version: "3.18.4"
|
|
||||||
Reference in New Issue
Block a user