Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update docs for DefaultAzureCredential regarding AZURE_CLIENT_ID / managedIdentityClientId relationship #16402

Closed
christothes opened this issue Jan 28, 2021 · 2 comments · Fixed by #16584

Comments

@christothes
Copy link
Member

Add documentation to DefaultAzureCredential indicating that setting the AZURE_CLIENT_ID environment variable is equivalent to setting the managedIdentityClientId.

@chlowell
Copy link
Member

Closing this because it's already documented on the class (feel free to reopen if you meant to have it documented somewhere else):

:keyword str managed_identity_client_id: The client ID of a user-assigned managed identity. Defaults to the value
of the environment variable AZURE_CLIENT_ID, if any. If not specified, a system-assigned identity will be used.

@christothes
Copy link
Member Author

christothes commented Jan 29, 2021

The other languages have a section about Managed Identities here that explains how to configure a custom user created managed identity client ID. Python doesn't have this (not sure if that is intentional), but it would go there if so.

The reason for the issue is that .NET had a customer question about this behavior which is not documented.

@christothes christothes reopened this Jan 29, 2021
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-python that referenced this issue Nov 26, 2021
Recovery services backup october release (Azure#16865)

* RecoveryServicesBackup: Added ValidateOperationAsync APIs to new API version 2021-10-01 (Azure#16291)

* Added base for new API version 2021-10-01 from previous 2021-08-01

* Updates Readme

* Updates API version in new specs and examples

* Updated CRR related API in readme

* Added PreCheckAsync new APIs

* Removed incorrect long-running-operation tag

* Renamed API as per PR review

* RecoveryServicesBackup: Minor update to new ValidateOperation async API (Azure#16402)

* Added long running operation tag to async API

* Updated examples with renamed tracking API

* Recovery services backup october release (Azure#16303)

* Adds base for updating Microsoft.RecoveryServices from version stable/2021-08-01 to 2021-10-01

* Update Swagger spec and examples

* Update readme

* Resolve Avocado and PrettierCheck issues

* Resolve Model Validation issues

* Adding package for active and passive stamp (Azure#16695)

* Updating readme to split active and passive stamp

* Update

* Updating readme.go

* Fixing model validation errors

* Fixing model validation

* Adding enums for protectionIntent

* Adding identity based restore details

Co-authored-by: abhishek-goyal1 <[email protected]>
Co-authored-by: Abhinit Kumar <[email protected]>
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-python that referenced this issue Nov 26, 2021
Recovery services backup october release (Azure#16865)

* RecoveryServicesBackup: Added ValidateOperationAsync APIs to new API version 2021-10-01 (Azure#16291)

* Added base for new API version 2021-10-01 from previous 2021-08-01

* Updates Readme

* Updates API version in new specs and examples

* Updated CRR related API in readme

* Added PreCheckAsync new APIs

* Removed incorrect long-running-operation tag

* Renamed API as per PR review

* RecoveryServicesBackup: Minor update to new ValidateOperation async API (Azure#16402)

* Added long running operation tag to async API

* Updated examples with renamed tracking API

* Recovery services backup october release (Azure#16303)

* Adds base for updating Microsoft.RecoveryServices from version stable/2021-08-01 to 2021-10-01

* Update Swagger spec and examples

* Update readme

* Resolve Avocado and PrettierCheck issues

* Resolve Model Validation issues

* Adding package for active and passive stamp (Azure#16695)

* Updating readme to split active and passive stamp

* Update

* Updating readme.go

* Fixing model validation errors

* Fixing model validation

* Adding enums for protectionIntent

* Adding identity based restore details

Co-authored-by: abhishek-goyal1 <[email protected]>
Co-authored-by: Abhinit Kumar <[email protected]>
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants