-
Notifications
You must be signed in to change notification settings - Fork 21.9k
Linked guidance on removal of SNAT requirement through tag #127362
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
base: main
Are you sure you want to change the base?
Conversation
Linked guidance on how to remove the need of SNAT through disableSnatOnPL tag on the NVA.
|
@juanandmsft : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. @abell |
|
Learn Build status updates of commit 8f91091:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| articles/private-link/private-endpoint-overview.md | Details |
articles/private-link/private-endpoint-overview.md
- Line 220, Column 253: [Warning: file-not-found - See documentation]
Invalid file link: 'azure/private-link/private-link-disable-snat'.
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.
|
@AbdullahBell - The metadata in this article has an incorrect author value. Could you open a PR in the private repo to change the author value to IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
|
@AbdullahBell Could you review this proposed update to your article and enter Thanks! |
|
Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @abell |
|
I sent an email to the content owner today. @MicrosoftDocs/public-repo-pr-review-team |
|
Hello @juanandmsft, please correct this error on line 220:
|
|
@juanandmsft - Can you respond to the requested changes or let us know if you would like us to commit them? @AbdullahBell - When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
Updated link format for disableSnatOnPL tag in UDR section.
|
@v-dirichards, link should now be fixed. |
|
Learn Build status updates of commit 1091495: ✅ Validation status: passed
For more details, please refer to the build report. |
|
@AbdullahBell - When the changes are ready for publication, adding a |
There was a problem hiding this 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 updates documentation to provide more specific guidance on SNAT requirements for private endpoints. The change clarifies when SNAT is needed and provides a link to documentation about removing this requirement using the disableSnatOnPL tag.
- Updated SNAT limitation description to be more specific about NVA scenarios
- Added reference to disableSnatOnPL tag documentation for removing SNAT requirement
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Linked guidance on how to remove the need of SNAT through disableSnatOnPL tag on the NVA.