-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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: new module : vmware_cluster_facts #37105
Conversation
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
9611d86
to
abc2eda
Compare
@Akasurde @bedecarroll @chrrrles @dav1x @garbled1 @jjahns @kamsz @lrivallain @nafpliot-ibm @nerzhul @pdellaert @rhoop @ritzk @stravassac @tchernomax @woshihaoren As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add |
@pdellaert @tchernomax Could you please review this ? |
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
Small comment if these methods could be useful for other modules in the future?
|
||
self.cluster_objs = [cluster_obj] | ||
|
||
def get_all_cluster_objs(self, parent): |
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.
Could potentially be moved to the module_utils? (same for lines 112 -> 122)
rebuild_merge |
@pdellaert Thanks for review. |
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
SUMMARY
Signed-off-by: Abhijeet Kasurde akasurde@redhat.com
ISSUE TYPE
COMPONENT NAME
lib/ansible/modules/cloud/vmware/vmware_cluster_facts.py
test/integration/targets/vmware_cluster_facts/aliases
test/integration/targets/vmware_cluster_facts/tasks/main.yml
ANSIBLE VERSION