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

New Resource - azurerm_route_server_bgp_connection, azurerm_route_server #16578

Merged
merged 8 commits into from
Jul 20, 2022

Conversation

xiaxyi
Copy link
Contributor

@xiaxyi xiaxyi commented Apr 28, 2022

Adding new resource Azure Route Server

acc tests:

--- PASS: TestAccRouteServer_requiresImport (1455.63s)
--- PASS: TestAccRouteServer_basic (1527.78s)
--- PASS: TestAccRouteServer_complete (1648.24s)
--- PASS: TestAccRouteServerBgpConnection_basic (1659.84s)
--- PASS: TestAccRouteServerBgpConnection_requiresImport (1730.02s)
--- PASS: TestAccRouteServer_update (1779.95s)
PASS
ok  	github.com/hashicorp/terraform-provider-azurerm/internal/services/network	1780.465s

Copy link
Contributor

@tombuildsstuff tombuildsstuff left a comment

Choose a reason for hiding this comment

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

hi @xiaxyi

The Provider already supports Route Server (as azurerm_virtual_hub, which is what this is called in the API / before the marketing rebrand). Whilst this has been rebranded in Azure it's only the marketing name and not the resource name (within the API) which has changed - so I'm not sure we should be renaming this resource?

Thanks!

@xiaxyi
Copy link
Contributor Author

xiaxyi commented Apr 28, 2022

Thanks @tombuildsstuff for the comment. It's two separate resources, they are using the same API. Virtual Hub is created when specifying the virtual_wan while route server is an independent resource.

just for your reference, PowerShell's practice:

virtual hub:https://docs.microsoft.com/en-us/powershell/module/az.network/new-azvirtualhub?view=azps-7.5.0
route server:https://docs.microsoft.com/en-us/azure/route-server/quickstart-configure-route-server-powershell

@katbyte
Copy link
Collaborator

katbyte commented Jul 18, 2022

compile failure:
image

@xiaxyi
Copy link
Contributor Author

xiaxyi commented Jul 19, 2022

@katbyte Thanks for the comment, I updated the sdk and tests are passing now.

image

Copy link
Collaborator

@katbyte katbyte left a comment

Choose a reason for hiding this comment

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

LGTM 🦕

@katbyte katbyte changed the title New Resource - Azure Route Server New Resource - azurerm_route_server_bgp_connection, azurerm_route_server Jul 20, 2022
@katbyte katbyte merged commit c79ab48 into hashicorp:main Jul 20, 2022
@github-actions github-actions bot added this to the v3.15.0 milestone Jul 20, 2022
katbyte added a commit that referenced this pull request Jul 20, 2022
@github-actions
Copy link

This functionality has been released in v3.15.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

@xiaxyi xiaxyi deleted the routeServer branch August 1, 2022 02:21
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants