Adding logic to add availability zones in parameters files for the networking platform resources depending on the selected region

[sebassem]

Pull Request

Issue

Issue Azure/Azure-Landing-Zones#2524 and Azure/Azure-Landing-Zones#2530

Description

Adding a new private function to determine the availability zones for the selected regions and edit the parameters files of the relevant networking modules to include them by default.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the projects associated license.

[oZakari]

Hey @sebassem, this works perfect from my initial testing! What are your thoughts on potentially adding a unit test for this function? I can't see us modifying this function too much long term, but just curious if you have any thoughts.

[sebassem]

Hey @sebassem, this works perfect from my initial testing! What are your thoughts on potentially adding a unit test for this function? I can't see us modifying this function long term, but just curious if you have any thoughts.

sure thing , I can give it a try and see if its easy to implement.

[sebassem]

@oZakari I added a unit test for the function, please let me know if its ok

[jaredfholgate]

Hi @sebassem. I just saw this pull request is in progress. I have been working on a major refactor and have most likely broken your work, but given it looks like this is Bicep specific, you should be able to update the code to work with the new code base. Let me know if you need help though and apologies for pipping you to the post with my changes.

[sebassem]

Hi @sebassem. I just saw this pull request is in progress. I have been working on a major refactor and have most likely broken your work, but given it looks like this is Bicep specific, you should be able to update the code to work with the new code base. Let me know if you need help though and apologies for pipping you to the post with my changes.

No worries @jaredfholgate I just created a new branch of the new changes and submitted a PR