-
Notifications
You must be signed in to change notification settings - Fork 7
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
Fix advanced settings #5
Fix advanced settings #5
Conversation
customvalues --> advanced_settings
Updated for `advanced_settings`
Thanks very much for your contribution. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did some research:
advanced_settings
was added to vmware_guest in the 1.8.0 release of the community.vmware collection.
This release is part of the Ansible community package 3.2.0, which is based on ansible-core 2.10.7 (see build data).
Because this change is a breaking one, min_ansible_version
will have to be changed to 2.10.7 here.
Please also change the requirements section to
"The minimum community.vmware collection version is 1.8.0, which is part of the Ansible community package 3.2.0."
Updated documentation to be vSphere 7.0 Updated documentation to reflect new minimum requirements Updated galaxy metadata for new minimum Ansible version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please review and let me know if there are any changes you'd like to see!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect, thank you very much for the contribution.
@dotalchemy released v6.0.0 |
Fixed a thing introduced by a change made to the 'community.vmware' collection where
customsettings
was replaced withadvanced_settings
in a recent release of the collection.This caused the role to fail with
Unable to find custom value key disk.EnableUUID
See Github Issue #179
See vmware_guest_module
Tested on whatever version of Ansible is in the very latest AWX Operator build, but there shouldn't be a version dependency change as it's just a string edit in the yaml to match what
community.vmware
is expecting.