Skip to content

Commit

Permalink
Merge pull request #93 from leanix/feature/CID-2411/fix-release-docke…
Browse files Browse the repository at this point in the history
…r-action

Use GitHub action secrets instead of env variables
  • Loading branch information
mohamedlajmileanix authored Mar 7, 2024
2 parents 650596c + f3ff6a6 commit b3de4ab
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/release-docker-image.yml
Original file line number Diff line number Diff line change
Expand Up @@ -83,15 +83,15 @@ jobs:
uses: azure/docker-login@v1
if: (steps.tag-action.outputs.tag != '')
with:
login-server: ${{ env.ACR_PUBLIC_LOGIN }}
username: ${{ env.ACR_PUBLIC_USERNAME }}
password: ${{ env.ACR_PUBLIC_PASSWORD }}
login-server: ${{ secrets.ACR_PUBLIC_LOGIN }}
username: ${{ secrets.ACR_PUBLIC_USERNAME }}
password: ${{ secrets.ACR_PUBLIC_PASSWORD }}

- name: Setup ssh
uses: webfactory/ssh-agent@v0.4.1
if: (steps.tag-action.outputs.tag != '')
with:
ssh-private-key: ${{ env.CI_GITHUB_SSH_PRIVATE_KEY }}
ssh-private-key: ${{ secrets.CI_GITHUB_SSH_PRIVATE_KEY }}
ssh-auth-sock: ${{ env.SSH_AUTH_SOCK }}

- name: Build and push image to Azure public registry
Expand All @@ -100,6 +100,6 @@ jobs:
with:
context: ./
file: Dockerfile
tags: ${{ env.ACR_PUBLIC_LOGIN }}/vsm-github-broker:${{ steps.tag-action.outputs.tag }}, ${{ env.ACR_PUBLIC_LOGIN }}/vsm-github-broker:latest
tags: ${{ secrets.ACR_PUBLIC_LOGIN }}/vsm-github-broker:${{ steps.tag-action.outputs.tag }}, ${{ secrets.ACR_PUBLIC_LOGIN }}/vsm-github-broker:latest
push: true
ssh: default=${{ env.SSH_AUTH_SOCK }}
ssh: default=${{ env.SSH_AUTH_SOCK }}

0 comments on commit b3de4ab

Please sign in to comment.