Skip to content

Azure Docs: Azure Private DNS Zone#586

Merged
remotesynth merged 2 commits intomainfrom
docs/azure-private-dns-zone
May 8, 2026
Merged

Azure Docs: Azure Private DNS Zone#586
remotesynth merged 2 commits intomainfrom
docs/azure-private-dns-zone

Conversation

@paolosalvatori
Copy link
Copy Markdown
Contributor

This pull request adds an article on Azure Private DNS Zone to the Azure emulator documentation.

Fixes DOC-182

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 22, 2026

Deploying localstack-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2eb87b9
Status: ✅  Deploy successful!
Preview URL: https://7154339a.localstack-docs.pages.dev
Branch Preview URL: https://docs-azure-private-dns-zone.localstack-docs.pages.dev

View logs

1. **Copy/paste error**: The text about creating a zone “in the **second** resource group … cleanup” did not match the guide (only `rg-dns-demo`). It is replaced with accurate copy that points to the **recommended Blob private zone name** and defers linking to the next section.
2. **Deletion order vs Azure**: Deleting only the VNet link and then the zone, while the private endpoint still had a DNS zone group pointing at that zone, does not match how dependencies work in Azure (you normally remove the endpoint’s **DNS zone group** first, then remove links, then delete the zone). The page now includes `az network private-endpoint dns-zone-group delete` and explains this in one sentence with links to Microsoft’s zone delete behavior.
3. **Samples link**: The Cosmos sample URL without `tree/main/` was not a standard GitHub browse URL; it now matches the repo layout that exists under `main`.
4. **Features wording**: The bullet titled **Auto-registration** suggested behavior that the **Limitations** section correctly denies. It is reworded so it’s clear you only persist **`registrationEnabled`**, not VM auto-registration.
Copilot AI review requested due to automatic review settings May 8, 2026 19:43
Copy link
Copy Markdown

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 was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@remotesynth
Copy link
Copy Markdown
Collaborator

Addressed some accuracy issues:

  1. Copy/paste error: The text about creating a zone “in the second resource group … cleanup” did not match the guide (only rg-dns-demo). It is replaced with accurate copy that points to the recommended Blob private zone name and defers linking to the next section.
  2. Deletion order vs Azure: Deleting only the VNet link and then the zone, while the private endpoint still had a DNS zone group pointing at that zone, does not match how dependencies work in Azure (you normally remove the endpoint’s DNS zone group first, then remove links, then delete the zone). The page now includes az network private-endpoint dns-zone-group delete and explains this in one sentence with links to Microsoft’s zone delete behavior.
  3. Samples link: The Cosmos sample URL without tree/main/ was not a standard GitHub browse URL; it now matches the repo layout that exists under main.
  4. Features wording: The bullet titled Auto-registration suggested behavior that the Limitations section correctly denies. It is reworded so it’s clear you only persist registrationEnabled, not VM auto-registration.

@remotesynth remotesynth merged commit 0cd4461 into main May 8, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants