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 route advertisement support to vcd_network_routed_v2 #1203

Merged
merged 2 commits into from
Mar 7, 2024

Conversation

Didainius
Copy link
Collaborator

@Didainius Didainius commented Jan 30, 2024

This PR adds route advertisement support to vcd_network_routed_v2 via field route_advertisement_enabled which is requested in #1105.

Documentation is updated with details, but this feature can be enabled when IP Spaces are being used in the configuration of parent provider gateway [vcd_external_network_v2] (uses IP Space uplink).

Closes #1105

Tests passed with tags network and nsxt on 10.5.1. Also all network tests starting with TestAccVcdNetworkRoutedV2 worked on 10.4.0.

…ce and data source

Signed-off-by: Dainius Serplis <dserplis@vmware.com>
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.

Looks very clean! Thanks!

@oliverwiegers
Copy link

@Didainius I tested it. It works :)

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. I have to run the tests, then I'll approve. Thanks!

@Didainius Didainius merged commit 5a6c87d into vmware:main Mar 7, 2024
3 checks passed
@Didainius Didainius deleted the orgvdc-routed-ra branch March 7, 2024 18:50
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.

vcd_network_routed_v2 does not have an attribute for Route Advertisement
5 participants