Alan Rominger

Results 256 comments of Alan Rominger

There are a lot of lines like this: https://github.com/ansible/ansible-runner/blob/2852b4b902121e1dce01e12848cdb2c8734d07c5/ansible_runner/config/_base.py#L107 (and `defaults.AUTO_CREATE_DIR` is a simple constant) For AWX, we do a few things that require temporary files that have to be...

Because `tempfile.mkdtemp` creates things in the machine's configured tmp dir, the cleanup command uses `tempfile.gettempdir()` as the base path for miscellaneous directories. The only point here is consistency, ansible-runner needs...

Like you say, it looks like the input to override this is `container_workdir`. This is in the `interface.py` module, but that's it, no CLI option.

Yeah, that makes sense to track it separately.

I don't think `ANSIBLE_STDOUT_CALLBACK` is right, because "tree" is not a standard out callback. It saves the data to files, it doesn't output it to the console. You can see...

I don't disagree with the patch, but I lack a genuine test case. The command: ``` ANSIBLE_CALLBACKS_ENABLED=tree ANSIBLE_STDOUT_CALLBACK=tree ansible-playbook -i localhost, ping.yml --connection=local ``` Results in _no_ output and the...

I ran that playbook, and I can't get it to give any error. Could you provide more details to reproduce the issue? With the thing that it's looping over, it's...

Well, we can't import from ansible in that context, and `Mapping` is an internal import. That is, then, conditionally imported depending on python version, and for what runner supports that...

I tried again, and still can't reproduce this. Could someone hold my hand for getting the above error? added file `test/fixtures/projects/debug/project/hostvars_as_mapping.yml` ```yaml - name: Test using hostvars as mapping connection:...

We already have an upstream issue, and here it is: https://github.com/pexpect/pexpect/issues/512 There is a proposed fix at https://github.com/pexpect/ptyprocess/pull/47, which was not even in dispute, ending at > Whichever one of...