lets try with service principal

chris-kruining · Nov 18, 2024 · 80eec44 · 80eec44
1 parent 27281af
commit 80eec44
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/.github/workflows/app.yml b/.github/workflows/app.yml
@@ -55,10 +55,18 @@ jobs:
           docker build . --file Dockerfile --tag ${{ secrets.ACR_LOGIN_SERVER }}/$IMAGE_NAME:${{needs.versionize.outputs.semver}}
           docker build . --file Dockerfile --tag ${{ secrets.ACR_LOGIN_SERVER }}/$IMAGE_NAME:latest
 
-      - name: Login to ACR
-        uses: azure/docker-login@v2
+      - name: Az CLI login
+        id: login
+        uses: azure/login@v2
         with:
-          login-server: ${{ secrets.ACR_LOGIN_SERVER }}
+          client-id: ${{ secrets.CALQUE_PRD_CLIENT_ID }}
+          tenant-id: ${{ secrets.CALQUE_PRD_TENANT_ID }}
+          subscription-id: ${{ secrets.CALQUE_PRD_SUBSCRIPTION_ID }}
+
+      # - name: Login to ACR
+      #   uses: azure/docker-login@v2
+      #   with:
+      #     login-server: ${{ secrets.ACR_LOGIN_SERVER }}
 
       - name: Push container images
         working-directory: src