Philippe Pepiot

Hi, thanks for this. IIUC limits are per process and this module is exposing "default" limits applied to the current user. I wonder if this could be integrated in the...

Hi, thanks for reporting the issue! In the API, attributes of CommandResult ``stdout`` and ``stderr`` should be string and ``stdout_bytes`` and ``stderr_bytes`` should be bytes (in general case, backends store...

Maybe we should drop the `-z` here ? What version of centos is impacted by this bug ? 6 or 7 or both ? Maybe we can detect CentOS version...

Hi, looks like a bug, thanks for reporting it! Can you paste your nginx config (especially "server" bloc with "listen" statement) ? In the meantime we have https://github.com/philpep/testinfra/issues/234 , did...

This looks related to https://github.com/philpep/testinfra/issues/345 where I suggested to add a `playbook` argument to `ansible.get_variables()`. Not sure how it will be complex to handle lookup plugins here too. Unfortunately I...

Hi, this looks interesting, thanks for this. However I think i'd be preferable to pass the playbook file in `ansible.get_variables(playbook="/path/to/playbook.yml")` (from the ansible module). Rather than from the command line....

Hi, thansk for contributing. I dont get the point of having a "salt runner" backend since it only works in local, so you can just bypass testinfra and import and...

Hi, Instead of ``` @pytest.fixture() def X509Certificate(): x = _X509Certificate return x ``` Try ``` X509Certificate = _X509Certificate.as_fixture() ``` And instead of ```c = Command(...)```: ```c = self.run(...)``` Agree that...

Hello, Indeed, for now the support on windows is limited to running commands with `run()`, `check_output()`. I think `file` module could work if someone write a windows compatible module.

Hi, I think such feature is not so hard to have (just chaining testinfra "backends") and looks very useful (ssh+docker, ssh+ssh, ssh+kubernetes, ssh+salt etc). I wonder how to describe this...