Azure / azure-cli

Azure Command-Line Interface
MIT License
3.93k stars 2.9k forks source link

Customer feedback | az containerapp secret | Clarify the expected values for 'identity' #29238

Open dbradish-microsoft opened 1 month ago

dbradish-microsoft commented 1 month ago

Describe the bug

This GitHub issue is being moved from MicrosoftDocs/azure-docs-cli #4408 as it is given on autogenerated source code and needs attention from the az containerapp engineering team.


Type of issue Missing information

Feedback For az containerapp secret set, need to clarify that when using a system-assigned managed identity, the value of 'identity' is system, as indicated in this doc ("for system assigned identity, use system instead of the resource ID").

Could we update the CLI doc to reflect this? E.g:: Change from the following...: A list of secret(s) for the container app. Space-separated values in 'key=value' or 'key=keyvaultref:keyvaulturl,identityref:identity' format (where 'key' cannot be longer than 20 characters).

...to the following: A list of secret(s) for the container app. Space-separated values in 'key=value' or 'key=keyvaultref:keyvaulturl,identityref:identity' format (where 'key' cannot be longer than 20 characters, and where 'identity' is either system or the resource id of the user-assigned managed identity).

Page URL https://learn.microsoft.com/en-us/cli/azure/containerapp/secret?view=azure-cli-latest#az-containerapp-secret-set

Content source URL https://github.com/MicrosoftDocs/azure-docs-cli/blob/main/latest/docs-ref-autogen/containerapp/secret.yml


Related command

az containerapp secret

Errors

n/a

Issue script & Debug output

n/a

Expected behavior

n/a

Environment Summary

n/a

Additional context

n/a

yonzhan commented 1 month ago

Thank you for opening this issue, we will look into it.

microsoft-github-policy-service[bot] commented 1 month ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @calvinsID.