Skip to content

Commit 6e94a97

Browse files
committed
Update chart paths
1 parent f944876 commit 6e94a97

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

.github/workflows/helm-build-push.yml

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,10 @@ on:
44
push:
55
workflow_dispatch:
66

7+
env:
8+
CHART_YAML_PATH: deployment/helm/charts/danswer/Chart.yaml
9+
VALUES_YAML_PATH: deployment/helm/charts/danswer/values.yaml
10+
711
jobs:
812
helm_chart_version_check:
913
runs-on: ubuntu-latest
@@ -20,11 +24,11 @@ jobs:
2024
# on main or a stable tag on a dev branch.
2125

2226
- name: Fail on semver pre-release chart version
23-
run: yq .version deployment/helm/Chart.yaml | grep -v '[a-zA-Z-]'
27+
run: yq .version ${{ env.CHART_YAML_PATH }} | grep -v '[a-zA-Z-]'
2428
if: ${{ github.ref_name == 'main' }}
2529

2630
- name: Fail on stable semver chart version
27-
run: yq .version deployment/helm/Chart.yaml | grep '[a-zA-Z-]'
31+
run: yq .version ${{ env.CHART_YAML_PATH }} | grep '[a-zA-Z-]'
2832
if: ${{ github.ref_name != 'main' }}
2933

3034
# To reduce resource usage images are built only on tag.
@@ -37,19 +41,19 @@ jobs:
3741
curl -H "Authorization: Bearer $(echo ${{ secrets.GITHUB_TOKEN }} | base64)"
3842
https://ghcr.io/v2/stackhpc/danswer/danswer-backend/tags/list
3943
| jq .tags
40-
| grep $( yq .appVersion deployment/helm/Chart.yaml )-$( yq .tagSuffix deployment/helm/values.yaml )
44+
| grep $( yq .appVersion ${{ env.CHART_YAML_PATH }} )-$( yq .tagSuffix ${{ env.VALUES_YAML_PATH }} )
4145
&&
4246
curl -H "Authorization: Bearer $(echo ${{ secrets.GITHUB_TOKEN }} | base64)"
4347
https://ghcr.io/v2/stackhpc/danswer/danswer-web-server/tags/list
4448
| jq .tags
45-
| grep $( yq .appVersion deployment/helm/Chart.yaml )-$( yq .tagSuffix deployment/helm/values.yaml )
49+
| grep $( yq .appVersion ${{ env.CHART_YAML_PATH }} )-$( yq .tagSuffix ${{ env.VALUES_YAML_PATH }} )
4650
4751
# Check if current chart version exists in releases already
4852
- name: Check for Helm chart version bump
4953
id: version_check
5054
run: |
5155
set -xe
52-
chart_version=$(yq .version deployment/helm/Chart.yaml)
56+
chart_version=$(yq .version ${{ env.CHART_YAML_PATH }})
5357
if [[ $(curl https://api.github.com/repos/stackhpc/danswer/releases | jq '.[].tag_name' | grep danswer-helm-$chart_version) ]]; then
5458
echo chart_version_changed=false >> $GITHUB_OUTPUT
5559
else

0 commit comments

Comments
 (0)