-
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
mitogen_via + transport=smart throws KeyError: u'smart' #548
Comments
dw
added a commit
that referenced
this issue
Feb 19, 2019
The idea behind transport=smart is to select between paramiko and OpenSSH given the availability of connection multiplexing and/or OSX kernel bugs. We need to make no such choice.
This is now on the master branch and will make it into the next release. To be updated when a new release is made, subscribe to https://networkgenomics.com/mail/mitogen-announce/ Thanks for reporting this! |
I can confirm v2.5 -> master fixed this. |
Hi Florient! IIRC you were having troubles with mitogen_via= / connection delegation. How are things on recent releases? Did >0.2.4 solve everything? |
dw
added a commit
that referenced
this issue
Mar 6, 2019
* origin/026: docs: update Changelog for release. Bump version for release. issue #555: ansible: workaround ancient reload(sys) hack. issue #554: mitogen_action_script fix issue #554: fix Ansible 2.4 compatibility issue #554: don't rely on tmp_path autoremoval in test. issue #554: track and remove multiple make_tmp_path() calls. docs: update Changelog. docs: drastically simplify install/changelog. issue #552: include process identity in log messages. issue #550: update Changelog. issue #550: parent: add explanatory comment. issue #550: fix up TTY ioctls on WSL 2016 Anniversary Update docs: update Changelog. service: make service list optional. docs: update Changelog; closes #548. issue #548: always treat transport=smart as 'ssh' for mitogen_via=. docs: better intro paragraph. .ci: copy private key file to tempdir. os_fork: more doc tweaks os_fork: more doc tweaks os_fork: yet more doc tidyup os_fork: more doc tweaks os_fork: clean up docs .ci: import soak scripts. .ci: allow containers for different jobs to run simultaneously os_fork: python 3 fixes and tests. issue #535: activate Corker on 2.4 in master too. issue #535: update Changelog. issue #535: wire mitogen.os_fork into Broker and Pool. issue #535: parent: add create_socketpair(size=..) parameter. issue #535: introduce mitogen.os_fork module and Corker class. issue #535: docs: update Changelog issue #535: service: support Pool.defer() like Broker.defer() issue #535: core: unicode.encode() may take importer lock on 2.x issue #535: docs: fix up Select doc issue #535: docs: update Changelog. issue #535: core/select: support selecting from Latches. core: increase cookie field lengths to 64-bit; closes #545. tests: ensure serialization restrictions are in effect tests/bench: set process affinity in throughput.py. docs: update copyright year. docs: update Changelog. core: Make Latch.put(obj=) optional. docs: change 'unreleased' Changelog format and add a hint. docs: update Changelog; closes #542. issue #542: return of select poller, new selection logic issue #542: .ci: move some tests to Azure and enable Mac job. ansible: create stub __init__.py for sdist.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello,
I'm getting following error while executing Ansible with mitogen:
After some tests I found out that this error happens only when:
When I tried to set ansible_transport = ssh, ping worked.
Also, when I set ansible_ssh_common_args = ' -o ProxyJump ...' instead of mitogen_via, it also worked.
Please let me know if you need more information.
The text was updated successfully, but these errors were encountered: