-
Notifications
You must be signed in to change notification settings - Fork 339
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
vmware_guest_snapshot - remove datacenter and folder parameter #1982
base: main
Are you sure you want to change the base?
Conversation
Build failed. ✔️ ansible-tox-linters SUCCESS in 9m 39s |
Build failed. ✔️ ansible-tox-linters SUCCESS in 9m 51s |
The datacenter and folder parameters are referred to by the get_vm method in the PyVmomi class, so please do not remove them. |
@sky-joker @Nina2244 I though you've left this collection. Great to see you again! I didn't find the time yet to have a closer look at this PR, but FYI I'm preparing a new major release so we can do some breaking changes if you think they're needed or make the collection more healthy. edit: Last day to release a new major version and get it into the Ansible Community Package 11 would be 2024-11-04, but I would like to do the next major release sooner. But we do have the opportunity to implement breaking changes if we think they make sense ATM 😉 |
SUMMARY
Remove datacenter and folder parameter because its not used anywhere in the code.
ISSUE TYPE
COMPONENT NAME
vmware_guest_snapshot.py
ADDITIONAL INFORMATION