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

Note that `az login` works for Ansible, too. #52835

Merged
merged 3 commits into from Mar 21, 2019

Conversation

Projects
None yet
3 participants
@mavit
Copy link
Contributor

mavit commented Feb 22, 2019

SUMMARY

Explain that one can use the az login command from the Azure CLI to authenticate Ansible to Azure.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

Azure modules.

@acozine

This comment has been minimized.

Copy link
Contributor

acozine commented Mar 11, 2019

@mavit thanks for contributing to the Ansible documentation!

Do you use an Ansible task to sign in via the Azure CLI? Are you managing your Azure authentication outside of Ansible?

@acozine acozine removed the needs_triage label Mar 11, 2019

@mavit

This comment has been minimized.

Copy link
Contributor Author

mavit commented Mar 16, 2019

For now I'm running az login before ansible-playbook. This has the advantage that I don't need to fiddle around with storing shared secrets in Ansible Vault.

@acozine

This comment has been minimized.

Copy link
Contributor

acozine commented Mar 20, 2019

Thanks for the clarification @mavit - if you'll accept my suggestion, I'll merge this PR.

Explain more clearly when to run `az login`.
Co-Authored-By: mavit <github.com@mavit.org.uk>

@ansibot ansibot removed the stale_ci label Mar 21, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

ansibot commented Mar 21, 2019

The test ansible-test sanity --test pep8 [explain] failed with 1 error:

lib/ansible/plugins/doc_fragments/azure.py:97:161: E501 line too long (175 > 160 characters)

click here for bot help

@acozine

This comment has been minimized.

Copy link
Contributor

acozine commented Mar 21, 2019

@mavit I'll fix the test failure and merge, since it was my suggestion that caused the failure in the first place

@ansibot ansibot removed the ci_verified label Mar 21, 2019

@acozine acozine merged commit d1d859f into ansible:devel Mar 21, 2019

1 check passed

Shippable Run 115254 status is SUCCESS.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.