Skip to content

Commit 0a83655

Browse files
committed
Refactor chart publishing workflow to correctly identify and push changed Chart.yaml files for dist and library charts [skip ci]
1 parent 738bfa6 commit 0a83655

File tree

1 file changed

+18
-18
lines changed

1 file changed

+18
-18
lines changed

.github/workflows/push-charts.yaml

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -27,17 +27,17 @@ jobs:
2727
username: ${{ github.actor }}
2828
password: ${{ secrets.GITHUB_TOKEN }}
2929

30-
- name: Get all changed library Chart.yaml files
31-
id: changed-chart-yaml-files-library
30+
- name: Get all changed dist/chart Chart.yaml files
31+
id: changed-chart-yaml-files-dist
3232
uses: tj-actions/changed-files@v47
3333
with:
3434
files: |
35-
helm/library/**/Chart.yaml
36-
- name: Push library charts to registry
37-
if: steps.changed-chart-yaml-files-library.outputs.all_changed_files != ''
35+
dist/chart/Chart.yaml
36+
- name: Push dist charts to registry
37+
if: steps.changed-chart-yaml-files-dist.outputs.all_changed_files != ''
3838
shell: bash
3939
env:
40-
ALL_CHANGED_FILES: ${{ steps.changed-chart-yaml-files-library.outputs.all_changed_files }}
40+
ALL_CHANGED_FILES: ${{ steps.changed-chart-yaml-files-dist.outputs.all_changed_files }}
4141
run: |
4242
for CHART_FILE in ${ALL_CHANGED_FILES}; do
4343
CHART_DIR=$(dirname $CHART_FILE)
@@ -46,17 +46,17 @@ jobs:
4646
helm push $CHART_PACKAGE oci://${{ env.REGISTRY }}/${{ github.repository }}/charts/
4747
done
4848
49-
- name: Get all changed bundle Chart.yaml files
50-
id: changed-chart-yaml-files-bundle
49+
- name: Get all changed library Chart.yaml files
50+
id: changed-chart-yaml-files-library
5151
uses: tj-actions/changed-files@v47
5252
with:
5353
files: |
54-
helm/bundles/**/Chart.yaml
55-
- name: Push bundle charts to registry
56-
if: steps.changed-chart-yaml-files-bundle.outputs.all_changed_files != ''
54+
helm/library/**/Chart.yaml
55+
- name: Push library charts to registry
56+
if: steps.changed-chart-yaml-files-library.outputs.all_changed_files != ''
5757
shell: bash
5858
env:
59-
ALL_CHANGED_FILES: ${{ steps.changed-chart-yaml-files-bundle.outputs.all_changed_files }}
59+
ALL_CHANGED_FILES: ${{ steps.changed-chart-yaml-files-library.outputs.all_changed_files }}
6060
run: |
6161
for CHART_FILE in ${ALL_CHANGED_FILES}; do
6262
CHART_DIR=$(dirname $CHART_FILE)
@@ -65,17 +65,17 @@ jobs:
6565
helm push $CHART_PACKAGE oci://${{ env.REGISTRY }}/${{ github.repository }}/charts/
6666
done
6767
68-
- name: Get all changed dist/chart Chart.yaml files
69-
id: changed-chart-yaml-files-dist
68+
- name: Get all changed bundle Chart.yaml files
69+
id: changed-chart-yaml-files-bundle
7070
uses: tj-actions/changed-files@v47
7171
with:
7272
files: |
73-
dist/chart/Chart.yaml
74-
- name: Push dist charts to registry
75-
if: steps.changed-chart-yaml-files-dist.outputs.all_changed_files != ''
73+
helm/bundles/**/Chart.yaml
74+
- name: Push bundle charts to registry
75+
if: steps.changed-chart-yaml-files-bundle.outputs.all_changed_files != ''
7676
shell: bash
7777
env:
78-
ALL_CHANGED_FILES: ${{ steps.changed-chart-yaml-files-dist.outputs.all_changed_files }}
78+
ALL_CHANGED_FILES: ${{ steps.changed-chart-yaml-files-bundle.outputs.all_changed_files }}
7979
run: |
8080
for CHART_FILE in ${ALL_CHANGED_FILES}; do
8181
CHART_DIR=$(dirname $CHART_FILE)

0 commit comments

Comments
 (0)