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 External network attachment support for vcd_nsxt_edgegateway #1111

Merged
merged 32 commits into from
Oct 4, 2023

Conversation

Didainius
Copy link
Collaborator

@Didainius Didainius commented Aug 24, 2023

This PR adds support for adding additional External Networks (only NSX-T Segment backed) to vcd_nsxt_edgegateway. This is done using a block field external_network. It is in the same resource instead of a separate one because it uses the same API endpoint as Edge Gateway creation (a bit unfortunate, but safest way).

This feature requires VCD 10.4.1+ and uses new functionality in vmware/go-vcloud-director#610

Place in UI:

image

About

IP Management for Edge Gateway external network uplinks is quite simple - it doesn't support IP Spaces and only requires IP, Prefix and allocated IP count. Primary IP can either be specified manually or assigned automatically. TypeSet has enough functionality to support it therefore only one structure is introduced external_network.

A big part of the work was making sure that this works well with existing primary network uplinks and doesn't affect each other. Quite a few tests were added with the hope of ensuring interoperability.

Additional notes:

Testing

Tags used gateway nsxt network, VCD versions:

  • 10.4.1
  • 10.5.0
  • 10.4.0
  • 10.4.2

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]>
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]>
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]>
Signed-off-by: Dainius Serplis <[email protected]>
Signed-off-by: Dainius Serplis <[email protected]>
@Didainius Didainius marked this pull request as ready for review August 31, 2023 06:28
Signed-off-by: Dainius Serplis <[email protected]>
Copy link
Collaborator

@lvirbalas lvirbalas left a comment

Choose a reason for hiding this comment

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

Thanks for updates!

Signed-off-by: Dainius Serplis <[email protected]>
.changes/v3.11.0/1092-improvements.md Outdated Show resolved Hide resolved
Signed-off-by: Dainius Serplis <[email protected]>
vcd/ipv6_test.go Outdated Show resolved Hide resolved
Signed-off-by: Dainius Serplis <[email protected]>
Signed-off-by: Dainius Serplis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants