David Shrewsbury
David Shrewsbury
Have to revamp this a bit. The config objects come into play for `transmit` and `process` phases with this change, but did not originally. This could cause data transformations that...
~~I think this needs modified to _at _least__ make sure that the `BaseConfig` object initialization does not do any active changes (making directories and the like) that will not be...
Looks like this was resolved. Closing.
@AlanCoding Since this touches `awx_display.py`, will let you or someone from your team check it out before approving.
We really need to add a test for a full string input of the galaxy value to make certain that we don't break that. There is a good example in...
Thanks for your PR. On changes like this, we have to ask someone from the Controller/AWX team to test the change to validate that it won't adversely affect that system...
Random out loud thought... might be worth considering removing `env` at [this point in the code](https://github.com/ansible/ansible-runner/blob/f95972b530045a0bf2701240ea4e5bcc03eced50/src/ansible_runner/runner.py#L106) so that suppressing will work even outside of the streaming protocol. Also, unclear to...
Thanks for the PR. It's a little problematic (through no fault of your own) in that it suffers from a problem we've been trying to address within runner when introducing...