Skip to content

Add existing private dns linking for vnet#61

Open
ChrisChapman-gh wants to merge 2 commits intoAzure:mainfrom
bjss:add-existing-private-dns-linking-for-vnet
Open

Add existing private dns linking for vnet#61
ChrisChapman-gh wants to merge 2 commits intoAzure:mainfrom
bjss:add-existing-private-dns-linking-for-vnet

Conversation

@ChrisChapman-gh
Copy link
Contributor

Description

Closes #60
Depends on #50

Adds linking between the created vnet and existing private dns zones.
Moves the platform + created/existing zone resource Id logic to the locals.
Removed the need for a dummy value should created/existing not be required. Either the private endpoints are managing the
dns zone group, in which case an existing or created id should be present, or they are unmanaged and a null/[] is passed and ignored.

Type of Change

  • Non-module change (e.g. CI/CD, documentation, etc.)
  • Azure Verified Module updates:
    • Bugfix containing backwards compatible bug fixes
      • Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
      • The bug was found by the module author, and no one has opened an issue to report it yet.
    • Feature update backwards compatible feature updates.
    • Breaking changes.
    • Update to documentation

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • I did run all pre-commit checks

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: Triage 🔍 Maintainers need to triage still label Dec 15, 2025
@ChrisChapman-gh ChrisChapman-gh force-pushed the add-existing-private-dns-linking-for-vnet branch from 2d11761 to 93f655a Compare December 16, 2025 09:03
MinHeinA added a commit to MinHeinA/terraform-azurerm-avm-ptn-aiml-landing-zone that referenced this pull request Jan 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Triage 🔍 Maintainers need to triage still

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[AVM Module Issue]: Private DNS Zone linking

1 participant