Skip to content

Commit

Permalink
Publish piepline for database migrator fails due to lack of permissio…
Browse files Browse the repository at this point in the history
…ns (#658)

* fix: add docker login

* fix: add service account for databasemigrator
  • Loading branch information
tnotheis authored May 17, 2024
1 parent 3ad797e commit c201d84
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -141,6 +141,12 @@ jobs:
uses: actions/checkout@v4
- name: Install script dependencies
run: npm install --prefix ./.ci
- name: Docker Login
uses: docker/[email protected]
with:
registry: ghcr.io
username: ${{ github.actor }}
password: ${{ secrets.GITHUB_TOKEN }}
- name: Log in to Docker Hub for accessing the cloud builder
uses: docker/login-action@v3
with:
Expand Down
3 changes: 3 additions & 0 deletions helm/templates/databasemigrator/job.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,9 @@ spec:
labels:
app: {{ .Values.databasemigrator.name }}
spec:
{{- with .Values.global.serviceAccount.name}}
serviceAccountName: {{ . }}
{{- end }}
restartPolicy: Never
{{- with .Values.databasemigrator.podSecurityContext }}
securityContext:
Expand Down

0 comments on commit c201d84

Please sign in to comment.