-
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
docs(vmware_object_role_permission): Clarify vCenter compatibility #1705
docs(vmware_object_role_permission): Clarify vCenter compatibility #1705
Conversation
This update to the documentation should clarify that the module is compatible with ESXi and vCenter which previously wasn't clear by reading through the module description.
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 36s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 34s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 9m 46s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 33s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 38s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 36s |
@p-fruck I think the CI fails because of the vSphere eval license used there expired. I'll try to ping people to fix this so we can get this PR merged. |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 29s |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 44s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 39s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 9m 00s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 33s |
Still some remaining licensing issues? |
Yep, CI still is broken :-( |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 41s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 52s |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 40s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 58s |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 36s |
recheck |
1 similar comment
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 35s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 24s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 9m 22s |
Build failed. ✔️ ansible-tox-linters SUCCESS in 9m 06s |
recheck |
Build failed. ✔️ ansible-tox-linters SUCCESS in 8m 30s |
recheck |
Build succeeded. ✔️ ansible-tox-linters SUCCESS in 8m 35s |
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
Thanks @p-fruck!
Build succeeded (gate pipeline). ✔️ ansible-tox-linters SUCCESS in 8m 28s |
SUMMARY
This update to the documentation should clarify that the module is compatible with ESXi and vCenter which previously wasn't clear by reading through the module description.
ISSUE TYPE
COMPONENT NAME
vmware_object_role_permission and vmware_object_role_permission_info
ADDITIONAL INFORMATION
This PR is similar to #1678