Skip to content

Commit 295de40

Browse files
fix: remove redundant --registry flag from az acr import (#1083)
## Problem `az acr import` was being called with both `--source <registry>/<image>:<tag>` (which includes the full hostname) **and** `--registry <name>`, causing this warning on every prod deploy: ``` WARNING: The login server name of "ecsdevacr.azurecr.io" in the "--source" argument will be ignored as "--registry" already supplies the same information ``` `--registry` is only needed when `--source` omits the registry hostname. Since our `--source` already contains the full `azurecr.io` hostname, `--registry` is redundant. ## Fix Drop the `--registry` flag from the `az acr import` call in `deploy-production`. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent ffaa95d commit 295de40

1 file changed

Lines changed: 0 additions & 1 deletion

File tree

.github/workflows/Build-Test-And-Deploy.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,6 @@ jobs:
189189
az acr import \
190190
--name "${PROD_ACR%.azurecr.io}" \
191191
--source "${DEV_ACR}/essentialcsharpweb:${{ github.sha }}" \
192-
--registry "${DEV_ACR%.azurecr.io}" \
193192
--image "essentialcsharpweb:${{ github.sha }}" \
194193
--image "essentialcsharpweb:latest" \
195194
--force

0 commit comments

Comments
 (0)