-
Notifications
You must be signed in to change notification settings - Fork 199
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
ansible_ssh_password
support
#1114
Conversation
That was unexpected. The new tests failed across the board https://dev.azure.com/mitogen-hq/mitogen/_build/results?buildId=908, e.g.
|
This is probably because See
|
f1222c5
to
fe27434
Compare
Survey of attributes/expressions currently used ansible_mitogen.transport_config.PlayContextSpec
Notes to self
|
New plugins should define
From https://docs.ansible.com/ansible/10/dev_guide/developing_plugins.html#vars-plugins
|
4e82475
to
7b74a22
Compare
4d6c4b5
to
15a1e1f
Compare
That was a hell of a journey. Some washup items
I think
Worth it's own issue, #1135. |
15a1e1f
to
d9bda21
Compare
…h_password This switches `ansible_mitogen.transport_config.PlayContextSpec.password()` to Ansible's plugin option framework. As a result - The relatively recent `ansible_ssh_password` variable is now respected. - The SSH connection password can be templated and specified as a play variable. Task variables will probably also work, but testing was blocked by mitogen-hq#1132. There is a chance this change will cause a regression in another connection plugin (e.g. mitogen_docker), but nothing turned up in the test suite. I intend ot migrate other connection configuration to `ansible_mitogen.transport_config.PlayContextSpec._connect_option()`, the next candidate is the remote port. fixes mitogen-hq#1106
d9bda21
to
551690e
Compare
Based on original PR #1107 by @madsi1m, which I accidentally clobbered (see #1107 (comment)).
Fixes #1106.