-
Notifications
You must be signed in to change notification settings - Fork 112
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
Add NSX-T VDC Group support for vcd_nsxt_security_group #814
Conversation
…oups Signed-off-by: Vaidotas Bauzys <vbauzys@vmware.com>
Signed-off-by: Vaidotas Bauzys <vbauzys@vmware.com>
Signed-off-by: Vaidotas Bauzys <vbauzys@vmware.com>
Signed-off-by: Vaidotas Bauzys <vbauzys@vmware.com>
Signed-off-by: Vaidotas Bauzys <vbauzys@vmware.com>
Signed-off-by: Vaidotas Bauzys <vbauzys@vmware.com>
Signed-off-by: Vaidotas Bauzys <vbauzys@vmware.com>
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.
Just submitting a few cosmetic things to fix as I am going deeper look into it.
Signed-off-by: Vaidotas Bauzys <vbauzys@vmware.com>
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!
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.
Just some small things.
Tested and works great! Thanks for implementing 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.
Approved! Thanks!
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!
Signed-off-by: Vaidotas Bauzys <vbauzys@vmware.com>
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. I did a lot of manual testing including migration from old configuration, tossing parent edge gateway in and out of VDC groups and creating 50+ security groups while moving edge gateway in and out. It all looks to be good.🤞🏻
Signed-off-by: Vaidotas Bauzys <vbauzys@vmware.com>
Dep: vmware/go-vcloud-director#456
This PR adds VDC Group support for:
vdc
field and instead inherits parent VDC / VDC Group from NSX-T Edge Gateway)There is a major difference in how these resources support it. Security group follows the location of parent Edge Gateway ID - move together to VDC Group and back.
Note. The goal of this PR is to preserve backward compatibility and not break anything for existing users.
vdc
fields are deprecated in favor ofedge_gateway_id
.Additionally:
Ran tests on 10.3.1 and 10.2.2 and upgrade tests