Skip to content

Commit 385d799

Browse files
committed
Uses build-and-push outputs for container deployment
Ensures consistent environment references by retrieving registry data from prior steps Prevents deployment if critical environment variables are unset Facilitates more reliable container publication and deployment
1 parent dcdfe02 commit 385d799

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

.github/workflows/docker-deploy.yml

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@ jobs:
99
runs-on: ubuntu-latest
1010
env:
1111
REGISTRY: tpsappscriptingacr.azurecr.io
12+
outputs:
13+
registry: ${{ steps.registry.outputs.registry }}
1214
steps:
1315
- uses: actions/checkout@v4
1416

@@ -48,8 +50,12 @@ jobs:
4850
docker push ${{ steps.registry.outputs.registry }}/app-scripting-editor-api:latest
4951
5052
deploy-to-appservice:
51-
if: ${{ secrets.EDITOR_APP_NAME != '' && secrets.RESOURCE_GROUP != '' && secrets.EDITOR_API_APP_NAME != '' }}
5253
needs: build-and-push
54+
env:
55+
EDITOR_APP_NAME: ${{ secrets.EDITOR_APP_NAME }}
56+
RESOURCE_GROUP: ${{ secrets.RESOURCE_GROUP }}
57+
EDITOR_API_APP_NAME: ${{ secrets.EDITOR_API_APP_NAME }}
58+
if: ${{ env.EDITOR_APP_NAME != '' && env.RESOURCE_GROUP != '' && env.EDITOR_API_APP_NAME != '' }}
5359
runs-on: ubuntu-latest
5460
steps:
5561
- name: Azure Login
@@ -59,8 +65,8 @@ jobs:
5965

6066
- name: Deploy editor to App Service (container)
6167
run: |
62-
az webapp config container set --name ${{ secrets.EDITOR_APP_NAME }} --resource-group ${{ secrets.RESOURCE_GROUP }} --docker-custom-image-name ${{ steps.registry.outputs.registry }}/app-scripting-editor:latest
68+
az webapp config container set --name ${{ env.EDITOR_APP_NAME }} --resource-group ${{ env.RESOURCE_GROUP }} --docker-custom-image-name ${{ needs.build-and-push.outputs.registry }}/app-scripting-editor:latest
6369
6470
- name: Deploy editor-api to App Service (container)
6571
run: |
66-
az webapp config container set --name ${{ secrets.EDITOR_API_APP_NAME }} --resource-group ${{ secrets.RESOURCE_GROUP }} --docker-custom-image-name ${{ steps.registry.outputs.registry }}/app-scripting-editor-api:latest
72+
az webapp config container set --name ${{ env.EDITOR_API_APP_NAME }} --resource-group ${{ env.RESOURCE_GROUP }} --docker-custom-image-name ${{ needs.build-and-push.outputs.registry }}/app-scripting-editor-api:latest

0 commit comments

Comments
 (0)