Skip to content

Conversation

@khaliddermoumi
Copy link

Before this change, the text reads "subnets must have IP ranges under 10.0.0.0/16, 172.16.0.0/12 or 192.168.0.0/16, which are class A, B or C private address ranges reserved for private networking" Are you sure if this is correct? It smells like an error, because the reserved private address range of class A is "10.0.0.0/8". /16 is not a class A network, so there is an inconsistency anyway. Would you please check this? This is an important architecture detail.

Before this change, the text reads "subnets must have IP ranges under `10.0.0.0/16`, `172.16.0.0/12` or `192.168.0.0/16`, which are class A, B or C private address ranges reserved for private networking"
Are you sure if this is correct? It smells like an error, because the reserved private address range of class A is "10.0.0.0/8". /16 is not a class A network, so there is an inconsistency anyway. 
Would you please check this? This is an important architecture detail.
@prmerger-automator
Copy link
Contributor

@khaliddermoumi : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit fbce9d4:

✅ Validation status: passed

File Status Preview URL Details
articles/ai-foundry/agents/how-to/virtual-networks.md ✅Succeeded

For more details, please refer to the build report.

@ktoliver
Copy link
Contributor

@aahill

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged C+L Pull Request Review Team label label Nov 24, 2025
@georgeollis
Copy link

10.0.0.0/8 does work - so believe it is a typo

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR corrects a technical error in the documentation regarding private IP address ranges for virtual network subnets. The change updates the Class A private address range from the incorrect 10.0.0.0/16 to the correct 10.0.0.0/8 as defined in RFC 1918.

Key changes:

  • Corrected the Class A private IP address range CIDR notation from /16 to /8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants