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

refactor: Allow option to specify virtual network gateway ip configuration names #921

Merged
merged 5 commits into from
Jan 7, 2025

Conversation

oZakari
Copy link
Contributor

@oZakari oZakari commented Dec 6, 2024

Overview/Summary

This pull request focuses on enhancing the configuration of virtual network gateways by adding new parameters for IP configuration names. It also includes updates to parameter files to reflect these changes.

Enhancements to IP Configuration:

  • Added ipConfigurationName and ipConfigurationActiveActiveName parameters to multiple configuration objects in hubNetworking-multiRegion.bicep and hubNetworking.bicep. [1] [2] [3] [4] [5] [6]

Resource Configuration Updates:

  • Updated resGateway and resGatewaySecondaryLocation resources to use the new IP configuration name parameters in hubNetworking-multiRegion.bicep and hubNetworking.bicep. [1] [2] [3] [4] [5] [6]

Parameter File Updates:

  • Modified various parameter files to include the new IP configuration parameters:
    • hubNetworking.parameters.all.json [1] [2]
    • hubNetworking.parameters.az.all.json [1] [2]
    • hubNetworking.parameters.az.multiRegion.all.json [1] [2] [3]
    • hubNetworking.parameters.min.json [1] [2]
    • mc-hubNetworking.parameters.all.json [1] [2]
    • mc-hubNetworking.parameters.min.json [1] [2]

Related Issues/Work Items

Closes #920
Related to #863

Breaking Changes

  1. Revert default value for the virtual network gateway IP configuration name back to vnetGatewayConfig hubNetworking.bicep. However, there is now a parameter to set this value to avoid redeployment of the resource.

Testing Evidence

Replace this with any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).

As part of this Pull Request I have

oZakari and others added 2 commits December 4, 2024 17:25
…d module (#903)

* Fix private dns zone list in policy assignment

* Remove unused dns zone ids and update param names

* Remove unused dns zone ids and update param names

* Update generated docs

* Add param to accelerator

* Generate Parameter Markdowns [oZakari/1618d0f4]

* Fix value for param

* Add prefix to dependabot title

* Update test values

* Updated param table

* Generate Parameter Markdowns [oZakari/640a1dbc]

* Update logic

* Update spacing

* Add params to parameter file

* Generate Parameter Markdowns [oZakari/0a582834]

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
@oZakari oZakari temporarily deployed to BicepUpdateDocumentation December 6, 2024 22:40 — with GitHub Actions Inactive
@oZakari oZakari closed this Dec 7, 2024
@oZakari oZakari reopened this Dec 7, 2024
@oZakari oZakari had a problem deploying to BicepUpdateDocumentation December 7, 2024 13:28 — with GitHub Actions Failure
@oZakari
Copy link
Contributor Author

oZakari commented Dec 7, 2024

/azp run validateazcloud

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@oZakari oZakari force-pushed the main branch 4 times, most recently from d1792a5 to a97b298 Compare December 7, 2024 15:17
@oZakari oZakari had a problem deploying to BicepUpdateDocumentation December 7, 2024 15:21 — with GitHub Actions Failure
@oZakari oZakari requested a review from sebassem December 18, 2024 03:07
@oZakari oZakari requested a deployment to BicepUpdateDocumentation December 18, 2024 03:09 — with GitHub Actions Waiting
@oZakari oZakari temporarily deployed to BicepUpdateDocumentation December 18, 2024 03:11 — with GitHub Actions Inactive
@oZakari oZakari requested a deployment to BicepUpdateDocumentation December 18, 2024 03:15 — with GitHub Actions Waiting
@oZakari oZakari requested a deployment to BicepUpdateDocumentation December 18, 2024 03:15 — with GitHub Actions Waiting
@oZakari oZakari added Area: Networking 🌐 Issues / PR's related to Networking auto-merge ✔️ Automatically merges labels Dec 18, 2024
@oZakari
Copy link
Contributor Author

oZakari commented Dec 18, 2024

/azp run validateazcloud

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@oZakari oZakari merged commit 04b3a3a into main Jan 7, 2025
17 of 19 checks passed
@oZakari oZakari deleted the fix-bugs branch January 7, 2025 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Networking 🌐 Issues / PR's related to Networking auto-merge ✔️ Automatically merges
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Placeholder value in alzDefaultPolicyAssignments.parameters.min.json has been overwritten by a real guid.
1 participant