[windows] Add README instructions for Windows hosts #233
Conversation
hush-hush
left a comment
hush-hush
left a comment
There was a problem hiding this comment.
Added a few comment. WDYT ?
README.md
Outdated
| [servers] | ||
| linux1 ansible_host=127.0.0.1 | ||
| linux2 ansible_host=127.0.0.2 | ||
| windows1 ansible_host=127.0.0.3 ansible_become_method=runas ansible_become_user=System |
There was a problem hiding this comment.
ansible_become_user is optional I think. By default it will use the Winrm user, no ?
There was a problem hiding this comment.
When setting ansible_become_method to runas, you have to set the ansible_become_user. Otherwise you get this error:
fatal: [windows1]: FAILED! => {"msg": "No setting was provided for required configuration plugin_type: become plugin: runas setting: become_user "}.
But actually, there's a simpler way to make the role work, without relying on a user: by setting ansible_become=no for Windows hosts.
README.md
Outdated
| ```yml | ||
| - hosts: servers | ||
| roles: | ||
| - { role: Datadog.datadog, become: yes, become_method: runas, become_user: System } |
There was a problem hiding this comment.
Same here I think, setting only { role: Datadog.datadog} and letting ansible use the right default works. I don't think we should advice people to use System by default and let them use the default value in there version of ansible.
[windows] Add README instructions for Windows hosts
What does this PR do?
Add instructions on how to make the role work with Windows hosts (
become: yesdoes not work by itself on Windows).Motivation
Windows support.
Additional Notes
For now, I put the section next to the playbook examples. Is there a better place?