-
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_host_facts - related datacenter #1994
vmware_host_facts - related datacenter #1994
Conversation
Build failed. ✔️ ansible-tox-linters SUCCESS in 10m 32s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 10m 19s |
@Nina2244 Judging by the sanity tests, there are disagreements in argument_specs and documentation. Needs to be fixed |
Build succeeded. ✔️ ansible-tox-linters SUCCESS in 9m 45s |
Co-authored-by: Mario Lenz <[email protected]>
Build failed. ✔️ ansible-tox-linters SUCCESS in 9m 46s |
recheck |
@ihumster The code LGTM now. What do you think, should we merge if the CI succeeds? |
Build succeeded. ✔️ ansible-tox-linters SUCCESS in 9m 39s |
@mariolenz Yes, I think we can merge it. |
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.
LGTM
Build succeeded (gate pipeline). ✔️ ansible-tox-linters SUCCESS in 10m 17s |
4ce1264
into
ansible-collections:main
SUMMARY
Add the possibility to get the related datacenter.
ISSUE TYPE
COMPONENT NAME
vmware_host_facts.py
ADDITIONAL INFORMATION