-
Notifications
You must be signed in to change notification settings - Fork 314
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
Add support for disable_instance_discovery #1442
Conversation
Hello, Thanks for the review. It looks to me like azure-identity is already updated to 1.14.0. This was done in Commit 21f6e15 Thanks! |
@p3ck Please add a description of the parameter 'disable_instance_discovery' to the file ~/plugins/doc_fragments/ aze. py, which has the default value. Thank you! |
30efd1b
to
729b239
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small change request!
plugins/doc_fragments/azure.py
Outdated
that the configured authority host is valid and trustworthy. | ||
- Set via credential file profile or the C(AZURE_DISABLE_INSTANCE_DISCOVERY) environment variable. | ||
type: bool | ||
version_added: '2.2.1' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version-added-must-be-major-or-minor, such as '2.3.0'
Added support in ansible common to support setting disable_instance_discovery when using azure clouds and you don't have access to login.microsoftonline.com. This can be specified as a module argument, environment variable or in credential profile. To enable set to True, default if False which doesn't change the current behaviour. Updated doc fragment to explain how to use it. fix ansible-collections#1236
SUMMARY
Added support in ansible common to support setting disable_instance_discovery when using azure clouds and you don't have access to login.microsoftonline.com.
This can be specified as a module argument, environment variable or in credential profile. To enable set to True, default if False which doesn't change the current behaviour.
Fixes #1236
ISSUE TYPE
COMPONENT NAME
azure_rm_common
ADDITIONAL INFORMATION
This issue affects users who are using the Azure US Government cloud
As of today the Azure Collection fails to connect in this environment:
Network access is allowed to Azure US Government cloud. (login.microsoftonline.us)
Network access is not allowed to Azure Public cloud. (login.microsoftonline.com)
Trying to use the Azure collection in this environment fails with attempts to connect to login.microsoftonline.com.