Azure / azure-cli

Azure Command-Line Interface

MIT License

3.99k stars 2.97k forks source link

az acr import docs - clarify that a name or resource ID must be specified to --registry for AAD auth #15089

Open ohadschn opened 4 years ago

ohadschn commented 4 years ago

This will not use AAD auth and will fail even if I'm logged (az login) in and have owner access to both ACRs, because AAD auth will not be used for the source ACR (it will only respect user/password): az acr import --name target --source source.azurecr.io/foo:bar

However this will succeed: az acr import --name target --registry /subscriptions/XXX/resourceGroups/YYY/providers/Microsoft.ContainerRegistry/registries/source --source foo:bar IMHO That is not trivial to understand from the docs.

Document Details

⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c608162e-cf40-9d53-da4a-51f51a0311ea
Version Independent ID: eccbb543-be68-7de2-d41d-abee55c81452
Content: az acr
Content Source: src/azure-cli/azure/cli/command_modules/acr/_help.py
Service: container-registry
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

ghost commented 4 years ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @toddysm, @yugangw-MSFT.

yonzhan commented 4 years ago

acr

yugangw-msft commented 4 years ago

This is by-design though ux is not clear. Doc should be updated at minimum

rayoef commented 1 year ago

reassign:tejawsikolli-web