Skip to content

Fix failing windows modules with community.vmware.vmware_tools module on ansible-core 2.21.1#2554

Open
chschenk wants to merge 2 commits into
ansible-collections:mainfrom
chschenk:unquote-pwsh
Open

Fix failing windows modules with community.vmware.vmware_tools module on ansible-core 2.21.1#2554
chschenk wants to merge 2 commits into
ansible-collections:mainfrom
chschenk:unquote-pwsh

Conversation

@chschenk

Copy link
Copy Markdown
Contributor
SUMMARY

Currently the community.vmware.vmware_tools connection plugin does not work on windows targets with ansible-core 2.21.1. The underlying issues are the same as described here: ansible-collections/community.general#12161

  1. The stdout and stderr return values of the exec_command function should be of type bytes and not str.
  2. The values in -EncodedCommand and -EncodedArguments must not be quoted otherwise powershell refuses to parse these parameters.
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

community.vmware.vmware_tools

@sonarqubecloud

Copy link
Copy Markdown

@centosinfra-prod-github-app

Copy link
Copy Markdown

@mariolenz

mariolenz commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Sorry, I thought I've already answered. Either I've started and didn't sent the comment, or I did and something went wrong.

  1. I don't use this functionality in our environment, so I don't know how to test this. And we still don't have a solution to run integration tests (again) for this collection. So I guess I just have to trust you that this really fixes things.
  2. Please add a changelog fragment.

Maybe something like changelogs/fragments/2554-fix-vmware_tools.yml:

bugfixes:
  - vmware_tools - Fix an issue with ansible-core 2.21 (https://github.com/ansible-collections/community.vmware/pull/2554).

You might have to create the changelogs/fragments directory first, though.

Out of curiosity, do you run into the same issue with ansible-core 2.21.0? I would be really surprised if the module works there and just started to fail with 2.21.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants