Skip to content
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 VDC Group support for NSX-T Edge Gateways #440

Merged
merged 8 commits into from
Mar 21, 2022

Conversation

Didainius
Copy link
Collaborator

@Didainius Didainius commented Mar 1, 2022

This PR adds support for NSX-T Edge Gateway in VDC Groups.

  • NSX-T Edge Gateway now supports VDC Groups by switching from OrgVdc to OwnerRef field.
    Additional methods NsxtEdgeGateway.MoveToVdc(), Org.GetNsxtEdgeGatewayByNameAndOwnerId(),
    VdcGroup.GetNsxtEdgeGatewayByName(), VdcGroup.GetAllNsxtEdgeGateways(), org.GetVdcGroupById
  • Additional helper functions OwnerIsVdcGroup(), OwnerIsVdc(), VdcGroup.GetCapabilities(),
    VdcGroup.IsNsxt()

Additionally

  • test Test_NsxtEdgeVdcGroup which aims to test all the new added things.
  • test function test_CreateVdcGroup is moved to separate file govcd/vdc_group_common_test.go so that it can be shared on multiple build tags and reused and create

Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
@Didainius Didainius marked this pull request as ready for review March 2, 2022 07:40
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
@Didainius Didainius changed the title Add support for NSX-T Edge Gateways Add VDC Group support for NSX-T Edge Gateways Mar 7, 2022
@Didainius Didainius requested review from mikeletux and adambarreiro and removed request for dataclouder and vbauzys March 9, 2022 07:08
@Didainius Didainius requested a review from vbauzys March 15, 2022 06:27
Copy link
Contributor

@vbauzys vbauzys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small changes needed, overall looks good.

.github/CODEOWNERS Outdated Show resolved Hide resolved
govcd/nsxt_edgegateway.go Show resolved Hide resolved
govcd/nsxt_edgegateway.go Show resolved Hide resolved
govcd/nsxt_edgegateway.go Outdated Show resolved Hide resolved
govcd/vdc_group.go Outdated Show resolved Hide resolved
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Copy link
Contributor

@vbauzys vbauzys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small things to fix and should be fine

.changes/v2.15.0/440-improvements.md Outdated Show resolved Hide resolved
govcd/nsxt_edgegateway_test.go Show resolved Hide resolved
govcd/vdc_group_common_test.go Outdated Show resolved Hide resolved
govcd/vdc_group_common_test.go Show resolved Hide resolved
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Copy link
Collaborator

@adambarreiro adambarreiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Contributor

@mikeletux mikeletux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Approved!

@Didainius Didainius merged commit 3cf7a8f into vmware:main Mar 21, 2022
@Didainius Didainius deleted the vdc-grp-pr branch March 21, 2022 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants