David Shrewsbury

Results 88 comments of David Shrewsbury

Without _some_ information on how to reproduce this, this would be very difficult to track down. Is there a particular playbook or role you are running that might help us?

Could you please elaborate? Are you asking for the code to NOT set this? Is the ability to use `.retry` files something you are wanting to do with `ansible-runner`?

> @sivel Thanks for providing detailed background! I have no objection to the policy, but I want to know details a bit. > > If we strictly follow pep508, even...

Removed association with issue #364. This PR would pass through unrecognized (and potentially invalid) requirements.

What is this "consolidating" with? I'm not sure I see the improvement here. If the only reason is to guarantee we always use the same version (and I'm not convinced...

> First, requirements.txt is for runtime requirements, not test ones. pre-commit takes care of version locking itself and can automatically make pull requests to update the linters, see how it...

You are missing required arguments. The `--container-image` option alone is not enough to cause runner to use containers for process isolation. See: https://ansible.readthedocs.io/projects/runner/en/stable/standalone/#running-with-process-isolation Or, if you want to set these...

I have no idea why this option is used, and there is no context within the code that I can see. I'm assuming this has something to do with an...

I looked at the change that introduced this `--ipc` option. I cannot, for the life of me, see any reason that change needed `--ipc=host`. All the data that change needs...

So it turns out, on a system with SELinux enabled, mounting `SSH_AUTH_SOCK` in a rootless container, and then doing something like `ssh-add -l`, will generate an audit event, causing the...