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 vmware_vswitch #36091
Conversation
c0bf769
to
5bedb78
Compare
@pdellaert @tchernomax Could you please review this ? |
self.esxi_hostname = module.params['esxi_hostname'] | ||
esxi_hostname = module.params['esxi_hostname'] | ||
|
||
hosts = get_all_objs(self.content, [vim.HostSystem]) |
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.
Can we use the refactored logic of host/cluster here to simplify?
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.
Yes.
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
The test
|
5bedb78
to
c4e5a12
Compare
3d5e5e8
to
f34a8a1
Compare
@Akasurde could you tell when this change will get merged |
* Update documentation * Update logic * Added idempotency * Added Error handling Fixes: ansible#36030 Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
f34a8a1
to
d51a511
Compare
@tchernomax @pdellaert @umrikar Thanks for the reviews. |
* Update documentation * Update logic * Added idempotency * Added Error handling Fixes: ansible#36030 Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
* Update documentation * Update logic * Added idempotency * Added Error handling Fixes: ansible#36030 Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
* Update documentation * Update logic * Added idempotency * Added Error handling Fixes: ansible#36030 Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
* Update documentation * Update logic * Added idempotency * Added Error handling Fixes: ansible#36030 Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
* Update documentation * Update logic * Added idempotency * Added Error handling Fixes: ansible#36030 Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
* Update documentation * Update logic * Added idempotency * Added Error handling Fixes: ansible#36030 Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
SUMMARY
Fixes: #36030
Signed-off-by: Abhijeet Kasurde akasurde@redhat.com
ISSUE TYPE
COMPONENT NAME
lib/ansible/modules/cloud/vmware/vmware_vswitch.py
ANSIBLE VERSION