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: Refactor guest inventory plugin #52642
Conversation
@s-hertel As per #50645 (comment) I am moved code in Could you please review this and let me know ? Thanks |
ab02d28
to
6b63af9
Compare
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
6b63af9
to
61f05db
Compare
30d1e38
to
e52981b
Compare
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.
Would be nice if there was a test that the cache could be used to populate inventory rather than just that the cache is being set, but that could be added later. LGTM.
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
e52981b
to
799c158
Compare
SUMMARY
Fixes: #52381
Signed-off-by: Abhijeet Kasurde akasurde@redhat.com
ISSUE TYPE
COMPONENT NAME
lib/ansible/plugins/inventory/vmware_vm_inventory.py
test/integration/targets/inventory_vmware_vm_inventory/runme.sh