Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make compatible with ansible 2.10.3 in windows target/adoptopenjdk-fallback combination #119

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rockdreamer
Copy link
Contributor

Updates to accomodate change in the returned api page and to better deal with the result of win_tempfile.
This makes is possible for me to use the role on macos 11, provisioning a win2019 VM, using ansible 2.10.3

@rockdreamer
Copy link
Contributor Author

Hi, any chance of getting these merged? Thanks :)

These changes were required for me to download openjdk 11 from the adoptopenjdk.
Target machine was running windows 2019. Machine where ansible is running is macos 11, ansible 2.10.3
According to recent win_tempfile documentation, its result is a dictionary, with the temporary path under the path key.
https://docs.ansible.com/ansible/latest/collections/ansible/windows/win_tempfile_module.html
Updating uses of the result to make use of the .path value
This makes is possible for me to use the role on macos 11, provisioning a win2019 VM, using ansible 2.10.3
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.

1 participant