Portal Network Validation Improvement

[marvinbuss]

Overview/Summary

Replace this with a brief description of what this Pull Request fixes, changes, etc.

This PR fixes/adds/changes/removes

1. This PR improves the CIDR range validation in the portal experience.

Breaking Changes

1. No Breaking changes, just improvements of the portal validation.

Testing Evidence

Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).
Here are a few screenshots:

As part of this Pull Request I have

• Checked for duplicate Pull Requests
• Associated it with relevant issues, for tracking and closure. (Azure firewall Subnet out of scope from Adress Space #731)
• Ensured my code/branch is up-to-date with the latest changes in the main branch
• Performed testing and provided evidence.
• Updated relevant and associated documentation.
• Updated the "What's New?" wiki page (located: /docs/wiki/whats-new.md)

[marvinbuss]

@jtracey93 & @krnese Let me know whether you want me to document the changes anywhere.

[krnese]

@daltondhcp - can you validate pls, for hub and spoke as well as vwan?

[jtracey93]

@jtracey93 Jack Tracey FTE & @krnese Kristian Nese FTE Let me know whether you want me to document the changes anywhere.

@marvinbuss If you could just add a note to the Wiki "What's New?" page that would be great (/docs/wiki/whats-new.md) 👍

[marvinbuss]

@jtracey93 Jack Tracey FTE & @krnese Kristian Nese FTE Let me know whether you want me to document the changes anywhere.

@marvinbuss If you could just add a note to the Wiki "What's New?" page that would be great (/docs/wiki/whats-new.md) 👍

DONE!

[daltondhcp]

@daltondhcp - can you validate pls, for hub and spoke as well as vwan?

LGTM - Tested and validated in both scenarios and works exactly as intended.

[marvinbuss]

@daltondhcp & @krnese Are you ok with the Vnet and subnet range sizes?

[daltondhcp]

@marvinbuss, noticed one more thing, would it be possible to add the same validation to the corp landing zone part as well?

[daltondhcp]

@daltondhcp & @krnese Are you ok with the Vnet and subnet range sizes?

Yes it is all good!

[marvinbuss]

@marvinbuss, noticed one more thing, would it be possible to add the same validation to the corp landing zone part as well?

@daltondhcp Please check my last commit. Checking real overlap is quite difficult at this point in time. Please see my analysis here: Azure/data-management-zone#148 (comment)

[daltondhcp]

@marvinbuss, noticed one more thing, would it be possible to add the same validation to the corp landing zone part as well?

@daltondhcp Please check my last commit.

Awesome, looks good and works well!

[krnese]

Excellent work!

[tulpy]

This is great thanks guys!