Alex Willmer

Results 363 comments of Alex Willmer

Hunting for other cases that might need fixing - `mitogen.master.DefectivePython3xMainMethod.find()` looks okay - `mitogen.master.DefectivePython3xMainMethod.find()`looks okay - `mitogen.core` looks okay - `mitogen.compat.pkgutil` is close to retirement/removal

> I've discovered the incorrect transmission of the `remote_user` variable's value when using `delegate_to` with Mitogen 3.4. For example, let's take a simple task: > > ```yaml > - name:...

@macksyma believe your example will now work, with Mitogen 0.3.13. Sorry for the long wait.

PoC reduces localhost ping time from 0.98 seconds to 0.66 ```diff diff --git a/mitogen/master.py b/mitogen/master.py index b1e0a1de..cbd8a9f2 100644 --- a/mitogen/master.py +++ b/mitogen/master.py @@ -1048,6 +1048,7 @@ class ModuleFinder(object): if sys.modules.get(name)...

A [code search for `import __main__`](https://github.com/search?q=%2F%5E+*import+__main__%2F+language%3APython&type=code) shows it's not unique in Ansible, so it may need better handling than just the above PoC

As current caretaker of Mitogen I agree with the deprecation and accept it. I won't be advocating that the Ansible project alter this decision, and I encourage others to do...

> Do I understand correctly that this could be the end of the `ansible_mitogen` strategy plugin? It is the end for this mechanism. Mitogen will need to find other ways...

`pipelining=true` appears to be a necessary condition to replicate this. So disabling pipelining may be another workaround. At one point I thought I had bisected the commit that introduced the...