-
Notifications
You must be signed in to change notification settings - Fork 76
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 Org VDC network support for NSX-T VDC Groups #443
Conversation
Signed-off-by: Dainius Serplis <[email protected]>
3695df7
to
b189add
Compare
Signed-off-by: Dainius Serplis <[email protected]>
Signed-off-by: Dainius Serplis <[email protected]>
Signed-off-by: Dainius Serplis <[email protected]>
Signed-off-by: Dainius Serplis <[email protected]>
Signed-off-by: Dainius Serplis <[email protected]>
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. Just a few small improvements needed
Signed-off-by: Dainius Serplis <[email protected]>
Signed-off-by: Dainius Serplis <[email protected]>
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 nitpicks. Very good jobs @Didainius
Signed-off-by: Dainius Serplis <[email protected]>
Signed-off-by: Dainius Serplis <[email protected]>
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.
Final ask, please update PR description with the new AnyType
prefix too.
Thank you for the refactoring!
Done |
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!!! Approved 👍
This PR works to add support for NSX-T VDC Groups in Org VDC networks (routed, isolated and imported).
It also adds a new type
AnyTypeEdgeGateway
and methodsAdminOrg.GetAnyTypeEdgeGatewayById
,Org.GetAnyTypeEdgeGatewayById
,AnyTypeEdgeGateway.IsNsxt
,AnyTypeEdgeGateway.GetNsxtEdgeGateway
.Type
AnyTypeEdgeGateway
uses the same structureOpenAPIEdgeGateway
which NSX-T Cousin type uses, butAnyTypeEdgeGateway
Get
methods will retrieve both - NSX-T and NSX-V Edge gateways. This is required as a convenience functions of Org VDC network routed network (because it can be backed by NSX-T or NSX-V).Also adds some additional convenience functions: and methods:
VdcGroup.GetNsxtImportableSwitchByName
(similar toVdc.GetNsxtImportableSwitchByName
)Org.GetOpenApiOrgVdcNetworkByNameAndOwnerId
VdcGroup.GetOpenApiOrgVdcNetworkByName
VdcGroup.GetAllOpenApiOrgVdcNetworks
VdcGroup.GetCapabilities
VdcGroup.IsNsxt