Skip to content

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

juanandmsft
Copy link
Contributor

Linked guidance on how to remove the need of SNAT through disableSnatOnPL tag on the NVA.

Linked guidance on how to remove the need of SNAT through disableSnatOnPL tag on the NVA.
Copy link
Contributor

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

Copy link
Contributor

Learn Build status updates of commit 8f91091:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/private-link/private-endpoint-overview.md ⚠️Warning 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.

@@ -217,7 +217,7 @@ The following information lists the known limitations to the use of private endp

| Limitation | Description |
| --------- | --------- |
| SNAT is recommended always. | Due to the variable nature of the private endpoint data-plane, it's recommended to SNAT traffic destined to a private endpoint to ensure return traffic is honored. |
| SNAT is recommended. | Due to the variable nature of the private endpoint data-plane, it's recommended to SNAT traffic destined to a private endpoint to ensure return traffic is honored when going through an NVA. This limitation can be removed using [disableSnatOnPL tag](azure/private-link/private-link-disable-snat) in your NVA |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| SNAT is recommended. | Due to the variable nature of the private endpoint data-plane, it's recommended to SNAT traffic destined to a private endpoint to ensure return traffic is honored when going through an NVA. This limitation can be removed using [disableSnatOnPL tag](azure/private-link/private-link-disable-snat) in your NVA |
| SNAT is recommended. | Due to the variable nature of the private endpoint data-plane, it's recommended to SNAT traffic destined to a private endpoint to ensure return traffic is honored when going through an NVA. This limitation can be removed using [disableSnatOnPL tag](/azure/private-link/private-link-disable-snat) in your NVA. |

Suggestion to fix link and punctuation

@ShannonLeavitt
Copy link
Contributor

@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 AbdullahBell?

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 Jul 10, 2025
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.

3 participants