Skip to content

Commit e07bbf4

Browse files
author
Scott Davidson
committed
Fixup workflows
1 parent 5c53e82 commit e07bbf4

File tree

2 files changed

+9
-8
lines changed

2 files changed

+9
-8
lines changed

.github/workflows/docker-build-push-web-container-on-tag.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,11 +41,12 @@ jobs:
4141
- name: Set up Docker Buildx
4242
uses: docker/setup-buildx-action@v3
4343

44-
- name: Login to Docker Hub
44+
- name: Login to GitHub Container Registry
4545
uses: docker/login-action@v3
4646
with:
47-
username: ${{ secrets.DOCKER_USERNAME }}
48-
password: ${{ secrets.DOCKER_TOKEN }}
47+
registry: ghcr.io
48+
username: ${{ github.actor }}
49+
password: ${{ secrets.GITHUB_TOKEN }}
4950

5051
- name: Build and push by digest
5152
id: build

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

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,11 @@ jobs:
2020
# on main or a stable tag on a dev branch.
2121

2222
- name: Fail on semver pre-release chart version
23-
run: yq .version deployment/helm/Chart.yaml | grep -v '[a-zA-Z-]'
23+
run: yq .version deployment/helm/charts/danswer/Chart.yaml | grep -v '[a-zA-Z-]'
2424
if: ${{ github.ref_name == 'main' }}
2525

2626
- name: Fail on stable semver chart version
27-
run: yq .version deployment/helm/Chart.yaml | grep '[a-zA-Z-]'
27+
run: yq .version deployment/helm/charts/danswer/Chart.yaml | grep '[a-zA-Z-]'
2828
if: ${{ github.ref_name != 'main' }}
2929

3030
# To reduce resource usage images are built only on tag.
@@ -37,19 +37,19 @@ jobs:
3737
curl -H "Authorization: Bearer $(echo ${{ secrets.GITHUB_TOKEN }} | base64)"
3838
https://ghcr.io/v2/stackhpc/danswer/danswer-backend/tags/list
3939
| jq .tags
40-
| grep $( yq .appVersion deployment/helm/Chart.yaml )-$( yq .tagSuffix deployment/helm/values.yaml )
40+
| grep $( yq .appVersion deployment/helm/charts/danswer/Chart.yaml )-$( yq .tagSuffix deployment/helm/values.yaml )
4141
&&
4242
curl -H "Authorization: Bearer $(echo ${{ secrets.GITHUB_TOKEN }} | base64)"
4343
https://ghcr.io/v2/stackhpc/danswer/danswer-web-server/tags/list
4444
| jq .tags
45-
| grep $( yq .appVersion deployment/helm/Chart.yaml )-$( yq .tagSuffix deployment/helm/values.yaml )
45+
| grep $( yq .appVersion deployment/helm/charts/danswer/Chart.yaml )-$( yq .tagSuffix deployment/helm/values.yaml )
4646
4747
# Check if current chart version exists in releases already
4848
- name: Check for Helm chart version bump
4949
id: version_check
5050
run: |
5151
set -xe
52-
chart_version=$(yq .version deployment/helm/Chart.yaml)
52+
chart_version=$(yq .version deployment/helm/charts/danswer/Chart.yaml)
5353
if [[ $(curl https://api.github.com/repos/stackhpc/danswer/releases | jq '.[].tag_name' | grep danswer-helm-$chart_version) ]]; then
5454
echo chart_version_changed=false >> $GITHUB_OUTPUT
5555
else

0 commit comments

Comments
 (0)