-
Notifications
You must be signed in to change notification settings - Fork 329
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_guest_network: Fix to port group changes to work properly #401
vmware_guest_network: Fix to port group changes to work properly #401
Conversation
LGTM |
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
@mariolenz Do you think we can go ahead and merge this? Thanks. |
I successfully ran some test playbooks in our environment and the code changes make sense to me. So I'd say: Yes, go ahead. |
@mariolenz Thanks for the heads-up and feedback. |
Thank you @mariolenz @Akasurde ! |
SUMMARY
Fix to port group changes to work properly.
The vmware_guest_network module has been an error that occurs or can't change the port group when changing the port group.
The cause of the error is each object type didn't compare well, wronged object structure.
This PR is to fix that bug.
And NSX-T port group supported.
fixes: #88 #204 #339 #352
ISSUE TYPE
COMPONENT NAME
plugins/modules/vmware_guest_network.py
ADDITIONAL INFORMATION
tested on vCenter/ESXi 7.0/6.7 and NSX-T 3.0