-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Azure: added auth_source to override the source of the credentials #35213
Conversation
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.
couple little things, looks great!
@@ -49,6 +55,7 @@ | |||
) | |||
|
|||
AZURE_CREDENTIAL_ENV_MAPPING = dict( | |||
auth_source='AZURE_AUTH_SOURCE', |
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.
We actually probably don't want this in the CREDENTIAL_ENV_MAPPING, since that'd be kinda chicken-or-egg... Probably want to check it explicitly if the module arg isn't set, and not support it in the credfile at all.
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.
Might also want to rename ANSIBLE_AZURE_AUTH_SOURCE
- we really should've ANSIBLE_
namespaced all those from the start. :( Just want to make sure we don't collide with something they use.
- C(auto) will follow the default precedence of module parameters -> environment variables -> default profile in credential file | ||
C(~/.azure/credentials). | ||
- C(cli) will source the credentials from the Azure CLI profile. | ||
- Can also be set vial the C(AZURE_AUTH_SOURCE) environment variable. |
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.
s/vial/via
* tweak error messages * fix auto fallback to cli * fix import issues when azure-cli not installed
rebuild_merge |
Upstream hung creating a storage account in one of the 3.6 containers; all others passed. Merging. |
…nsible#35213) * Azure: added auth_source to override the source of the credentials * removed broken cli auth and fix some review changes * missed some more cli auth stuff that isn't required * Added cli auth back in * Added docs around cli auth * fix azure CLI auth issues * tweak error messages * fix auto fallback to cli * fix import issues when azure-cli not installed
…nsible#35213) * Azure: added auth_source to override the source of the credentials * removed broken cli auth and fix some review changes * missed some more cli auth stuff that isn't required * Added cli auth back in * Added docs around cli auth * fix azure CLI auth issues * tweak error messages * fix auto fallback to cli * fix import issues when azure-cli not installed
SUMMARY
Overrides the auth source for Azure credentials.
ISSUE TYPE
COMPONENT NAME
azure_rm_common
ANSIBLE VERSION