Skip to content

Commit 305811b

Browse files
authored
Merge pull request #189 from DrDroidLab/feature/new_github_action_workflow
Simplified the version update to use a single find command
2 parents 1c097e8 + 19419ac commit 305811b

File tree

1 file changed

+10
-18
lines changed

1 file changed

+10
-18
lines changed

.github/workflows/release.yaml

Lines changed: 10 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -86,35 +86,27 @@ jobs:
8686
- name: Update Helm chart versions
8787
run: |
8888
VERSION="${{ steps.version.outputs.version }}"
89+
CHART_PATH="${{ env.HELM_CHART_PATH }}"
8990
90-
# Update subchart versions
91-
sed -i "s/^version:.*/version: ${VERSION}/" ${{ env.HELM_CHART_PATH }}/charts/redis/Chart.yaml
92-
sed -i "s/^version:.*/version: ${VERSION}/" ${{ env.HELM_CHART_PATH }}/charts/celery_beat/Chart.yaml
93-
sed -i "s/^version:.*/version: ${VERSION}/" ${{ env.HELM_CHART_PATH }}/charts/celery_worker/Chart.yaml
91+
# Update all version fields in all Chart.yaml files to the release version
92+
# This handles both chart versions and dependency versions
93+
find ${CHART_PATH} -name "Chart.yaml" -exec sed -i "s/\(version:\s*\)[0-9][0-9.]*/\1${VERSION}/g" {} \;
9494
95-
# Update redis dependency version in celery-beat and celery-worker
96-
sed -i "s/version: 0\..*/version: ${VERSION}/" ${{ env.HELM_CHART_PATH }}/charts/celery_beat/Chart.yaml
97-
sed -i "s/version: 0\..*/version: ${VERSION}/" ${{ env.HELM_CHART_PATH }}/charts/celery_worker/Chart.yaml
98-
99-
# Update parent Chart.yaml version and appVersion
100-
sed -i "s/^version:.*/version: ${VERSION}/" ${{ env.HELM_CHART_PATH }}/Chart.yaml
101-
sed -i "s/^appVersion:.*/appVersion: \"${VERSION}\"/" ${{ env.HELM_CHART_PATH }}/Chart.yaml
102-
103-
# Update dependency versions in parent Chart.yaml
104-
sed -i "/dependencies:/,$ s/version:.*/version: ${VERSION}/" ${{ env.HELM_CHART_PATH }}/Chart.yaml
95+
# Update appVersion in parent Chart.yaml
96+
sed -i "s/^appVersion:.*/appVersion: \"${VERSION}\"/" ${CHART_PATH}/Chart.yaml
10597
10698
# Display updated Chart files
10799
echo "=== Parent Chart.yaml ==="
108-
cat ${{ env.HELM_CHART_PATH }}/Chart.yaml
100+
cat ${CHART_PATH}/Chart.yaml
109101
echo ""
110102
echo "=== redis/Chart.yaml ==="
111-
cat ${{ env.HELM_CHART_PATH }}/charts/redis/Chart.yaml
103+
cat ${CHART_PATH}/charts/redis/Chart.yaml
112104
echo ""
113105
echo "=== celery_beat/Chart.yaml ==="
114-
cat ${{ env.HELM_CHART_PATH }}/charts/celery_beat/Chart.yaml
106+
cat ${CHART_PATH}/charts/celery_beat/Chart.yaml
115107
echo ""
116108
echo "=== celery_worker/Chart.yaml ==="
117-
cat ${{ env.HELM_CHART_PATH }}/charts/celery_worker/Chart.yaml
109+
cat ${CHART_PATH}/charts/celery_worker/Chart.yaml
118110
119111
- name: Package Helm chart
120112
run: |

0 commit comments

Comments
 (0)