Matt Martz
Matt Martz
Based on that output, you have `connection: local` or `ansible_connection: local` configured somewhere. It's properly limiting to the correct host, but ansible has been told to use the `local` connection...
It's really hard to say without an actual reproducer than we can run, or look at in it's completeness.
The above change has been merged to ansible/ansible. Please ensure that this isn't causing any duplicate events in runner.
This is most certainly a runner bug. The problem is that the `DOCUMENTATION` for `awx_display` only extends from `default_callback` and the fact that there is dynamic class building means that...
fwiw, this was how I tested with just ansible core: ```shell export ANSIBLE_CALLBACKS_ENABLED=tree,awx_display export ANSIBLE_STDOUT_CALLBACK=awx_display export ORIGINAL_STDOUT_CALLBACK=tree export AWX_ISOLATED_DATA_DIR=$PWD/cache mkdir -p $AWX_ISOLATED_DATA_DIR RUNNER=$PWD/ansible-runner/ansible_runner if [ -e "${RUNNER}/callbacks" ]; then export...
It looks like the JSON encoder class is missing support for `Mapping` which [core has](https://github.com/ansible/ansible/blob/a12e0a0e874c6c0d18a1a2d83dcb106d207136af/lib/ansible/module_utils/common/json.py#L46): https://github.com/ansible/ansible-runner/blob/d467662f585f5bb6ed2c5e8d57f130772760031a/ansible_runner/display_callback/callback/awx_display.py#L76 I didn't specifically test this playbook, but I'd guess it has something to do...
> Well, we can't import from ansible in that context Yep, I wasn't recommending that. Just that you could import `collections.abc.Mapping` and mirror the logic that core has, by casting...
If the plans are to extend the ignores in the `collection_template`, switching to using `manifest` over `build_ignore` would be an improvement, even without listing any explicit directives, as the defaults...
The Core team has discussed this, and have decided that ansible-core will _not_ be adopting this proposal. We _may_ adopt the change to rename the license files to their SPDX...
A few of the items I remember from our discussion... 1. In the end we just agreed that adding `SPDX-License-Identifier` didn't offer a benefit for us 1. We don't realistically...