Skip to content

Conversation

@MikeWedderburn-Clarke
Copy link
Contributor

Clarified conditions for enabling routing intent in Azure Firewall and added details about User Defined Routes in Spokes. Many customers have interpreted the original statement to say that they cannot have UDRs in Spokes.

Clarified conditions for enabling routing intent in Azure Firewall and added details about User Defined Routes in Spokes.
Many customers have interpreted the original statement to say that they cannot have UDRs in Spokes.
@prmerger-automator
Copy link
Contributor

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

@prmerger-automator
Copy link
Contributor

@MikeWedderburn-Clarke : 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 ad64484:

✅ Validation status: passed

File Status Preview URL Details
articles/virtual-wan/how-to-routing-policies.md ✅Succeeded

For more details, please refer to the build report.

@ttorble ttorble requested a review from Copilot December 15, 2025 16:20
@ttorble
Copy link
Contributor

ttorble commented Dec 15, 2025

@wtnlee

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 tracking label for the PR review team label Dec 15, 2025
@wtnlee
Copy link
Contributor

wtnlee commented Dec 15, 2025

@ttorble and @MikeWedderburn-Clarke

Please make the following changes to the PR:

However, you can associate User Defined Routes (UDR) to spoke Virtual Networks connected to the Virtual WAN hub. To ensure the spoke Virtual Network learns routes from Virtual WAN, ensure "Propagate gateway routes' is set to "Yes" for the user-defined route table. For more information, see troubleshooting data path.

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Clarified conditions for enabling routing intent in Azure Firewall and added details about User Defined Routes in Spokes. Many customers have interpreted the original statement to say that they cannot have UDRs in Spokes. Included recommended changes from @wtnlee
@MikeWedderburn-Clarke
Copy link
Contributor Author

Requested changes have been made.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 024a085:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/virtual-wan/how-to-routing-policies.md ⚠️Warning Details

articles/virtual-wan/how-to-routing-policies.md

  • Line 170, Column 279: [Warning: file-not-found - See documentation] Invalid file link: 'virtual-networks-udr-overview.md'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 0177a9d:

✅ Validation status: passed

File Status Preview URL Details
articles/virtual-wan/how-to-routing-policies.md ✅Succeeded

For more details, please refer to the build report.

@MikeWedderburn-Clarke
Copy link
Contributor Author

Tagging @aparnabhat-msftgh who helped with troubleshooting the original customer issue and also with writing the proposed updates. Thanks Aparna!

@v-dirichards
Copy link
Contributor

@wtnlee Could you review this proposed update to your article and enter #sign-off in a comment if it's ready to merge?

Thanks!

@aparnabhatms
Copy link

Tagging @aparnabhat-msftgh who helped with troubleshooting the original customer issue and also with writing the proposed updates. Thanks Aparna!

thanks @MikeWedderburn-Clarke. :)

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.

5 participants