Skip to content

Commit 454f2a3

Browse files
Fix App Configuration ID retrieval in deployment workflow and enable admin user for container registry
1 parent 54c8e50 commit 454f2a3

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

.github/workflows/deploy-aca-package.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ jobs:
127127
- name: Assign AcrPull to Container App
128128
run: |
129129
CONTAINER_APP_ID=$(az containerapp show --name ${{ env.container-name }} --resource-group ${{ env.resource-group-name }} --query "identity.principalId" -o tsv)
130-
APPCONFIG_ID=$(az acr show --name ${{ env.app-config-name }} --resource-group ${{ env.resource-group-name }} --query "id" -o tsv)
130+
APPCONFIG_ID=$(az appconfig show --name ${{ env.app-config-name }} --resource-group ${{ env.resource-group-name }} --query "id" -o tsv)
131131
az role assignment create \
132132
--assignee $CONTAINER_APP_ID \
133133
--role "App Configuration Data Reader" \

iac/container-registry.bicep

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ resource acrResource 'Microsoft.ContainerRegistry/registries@2021-06-01-preview'
1919
name: acrSku
2020
}
2121
properties: {
22-
adminUserEnabled: false
22+
adminUserEnabled: true
2323
}
2424
}
2525

0 commit comments

Comments
 (0)