Update private-link-and-dns-integration-at-scale.md
Before selecting the "Create pull request" button:
-
Enter a meaningful title above^, using a prefix if necessary and keywords "New" or "Update" indicating the nature of changes.
-
Describe the summary, scope, and intent of this PR:
Using Azure firewall as a DNS proxy should be mentioned or recommended in some way in the ready step for CAF. -
Insert links(s) to any related work item(s) or supporting detail:
AFTER YOUR PR HAS BEEN CREATED, expand this section for tips and additional instructions.
These are common guidelines for contributions across the repos managed by the Cloud Architecture Content Team (CACT). Some repositories may have additional specific requirements that are not listed here.
Guidance for all contributors
| Topic | Guidance |
|---|---|
| Draft PR | If your PR will be a work-in-progress for more than a day or two, select the Convert to draft link in the upper right of the page (under Reviewers) to change it to a draft. For future reference, you can also do this using the Create pull request button drop-down during PR creation. |
| ms.date metadata |
|
| Placement and linking | If you're creating a new article or articles, include updates to the related TOC.yml file to propose where the article(s) should be placed. Also consider other places within the document set where it would be beneficial to cross-reference and link to your new article(s). |
| PR build | After you open your PR, and for each successive commit that you push to your branch, the publishing platform will run validation on the files in your pull request. A summary of the build results for each file will be inserted inline into your pull request, which includes any build suggestions/warnings/errors. PRs cannot be merged until all build errors and most warnings are resolved. |
| Publishing | Following a successful merge, most repos publish to the live site at least once per (business) day, usually around 10am Pacific. |
| Additional resources |
Additional guidance for private repos and internal contributors
| Topic | Guidance |
|---|---|
| PR size | If your PR is more than ~5 lines of changes, or you'd like for the changes to go through editorial or larger review, open a contribution request at https://aka.ms/Contribution and include a link to the PR in response #8. Once it's processed, you'll be notified of the next steps. |
| PR title prefix | Select the Edit button to the right of the PR title if you need to revise it. The following prefixes are reserved for specific contribution types:
|
| PR preview | Following successful build of your PR, publishable files will also include Preview URL links to staged previews of your new/updated articles. Be sure to review these for verification of your intended contributions, or to send to other internal contributors for review. |
| PR sign-off (public repo) | If an article you own is updated in a public repo PR, you are responsible for sign-off. You will be automatically notified via email. The PR will not be merged until you've had a chance to review and sign-off. |
| PR sign-off (private repo) | After you've completed your proposed changes, addressed build warnings, and completed all review work, you can begin the sign-off process for review and merge:
|
| Additional resources |
|
@hlokensgard : Thanks for your contribution! The author(s) have been notified to review your proposed change.
Learn Build status updates of commit 1260e29:
:warning: Validation status: warnings
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/ready/azure-best-practices/private-link-and-dns-integration-at-scale.md | :warning:Warning | Details |
docs/ready/azure-best-practices/private-link-and-dns-integration-at-scale.md
- Line 24, Column 109: [Warning: hard-coded-locale - See documentation]
Link 'https://learn.microsoft.com/en-us/samples/azure/azure-quickstart-templates/azure-firewall-dns-proxy/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites. - Line 24, Column 109: [Suggestion: docs-link-absolute - See documentation]
Absolute link 'https://learn.microsoft.com/en-us/samples/azure/azure-quickstart-templates/azure-firewall-dns-proxy/' will be broken in isolated environments. Replace with a relative link.
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.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
@JefferyMitchell
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team
#label:"pending-content-team/business-approval"
Learn Build status updates of commit a83ecf8:
:white_check_mark: Validation status: passed
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/ready/azure-best-practices/private-link-and-dns-integration-at-scale.md | :white_check_mark:Succeeded |
For more details, please refer to the build report.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
@MicrosoftDocs/patterns-and-practices-team-pr-reviewers - Please review the proposed changes. If you want to move forward with the content, add a sign-off.
#label:"aq-pr-triaged" #label:"pending-content-team/business-approval" Cc: @MicrosoftDocs/public-repo-pr-review-team
Learn Build status updates of commit e2f9b34:
:white_check_mark: Validation status: passed
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/ready/azure-best-practices/private-link-and-dns-integration-at-scale.md | :white_check_mark:Succeeded |
For more details, please refer to the build report.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
PnP #sign-off
#remove-label:"pnp-review-in-progress" #remove-label:"do-not-merge" #label:"ready-to-merge"
Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @JefferyMitchell