Philippe Pepiot

Hi @ecks , I'll be happy to review a PR from you, thanks! I think the `+` character can be used but I'd rather use it to separate the "whole"...

Hi thanks for opening the issue, I was planning to open it :) First I think maybe we could always run shell with "-l", if there's no drawback with this....

Hi, thanks for contributing ! I'm not sure all arguments of subprocess are useful here, but `cwd` definitively is. But the way you've implemented it only works with "local" backend....

I think this is a good idea

Hi, dynamically starting/stopping services during tests could require to wait/retry for instance to wait up to 10 second the service is running: ```python host.run("sudo systemctl start zookeeper") zookeeper = host.service("zookeeper")...

Hi @aheumaier I don't have code fully in mind and known nothing about windows & powershell. But my advice would be either to * add a `shell` or `terminal` parameter...

Totally agree with @ssbarnea . First step is indeed to make some kind of windows CI working (cf https://github.com/pytest-dev/pytest-testinfra/issues/429). Then If you plan to work on this, don't assume I...

This is a private key used testinfra own test suite. We might want to generate a key on the fly here.

If you don't use ssh-agent, password will be promted for each command testinfra need to run over ssh (for instance using `host.package()` require some commands to be run to detect...

Hi, thanks for reporting this! I'd expect testinfra to use the settings that are defined in the inventory file / ansible.cfg. Looks like a bug, under what version of ansible...