Matt Martz

Can you provide output from the `git` task causing this, with at least `-v`, or maybe `-vvv`? I cannot see why the `git` module would have returned non-utf8 data. If...

I have verified this warning, when a diff contains latin1 encoded characters. As mentioned above, we have a few potential options: 1. Don't show a diff when it includes non-utf8...

I've reclassified this as a bug, since it's not user resolvable, other than not using the diff option.

There are already 2 in flight PRs to address this, one for backports, and one for 2.18: * https://github.com/ansible/ansible/pull/83249 * https://github.com/ansible/ansible/pull/83252

I wouldn't use `jq`. Just register the output of the command, then use `from_json` filter, and jinja to access the value you need.

I think I'm more in favor of this one, since it would more closely match the behavior of sending meta results through the queue. I'm still not in love with...

We should probably switch to using `shlex.join` in `lib/ansible/plugins/shell/__init__.py::ShellBase.build_module_command`. We should stop individually quoting, and just quote everything as needed with one function. We'd need to make sure that we...

Maybe something like: ```diff diff --git a/lib/ansible/plugins/shell/__init__.py b/lib/ansible/plugins/shell/__init__.py index 5aa0a1b89e..945d8f4281 100644 --- a/lib/ansible/plugins/shell/__init__.py +++ b/lib/ansible/plugins/shell/__init__.py @@ -207,19 +207,15 @@ class ShellBase(AnsiblePlugin): return 'echo %spwd%s' % (self._SHELL_SUB_LEFT, self._SHELL_SUB_RIGHT) def build_module_command(self, env_string,...

> Is anyone aware of a workaround? Don't put your virtualenvs or python installs in a path containing spaces. From the reports above, it seems to be related to `pipx`....

Yeah, I agree that we shouldn't just replace the image. v1 and v2 schemas shouldn't be used, and should be deprecated. We need to ensure we are preventing the default...