Skip to content

fix: DevOps CI/CD agents pattern module DNS fix#7030

Draft
AlexanderSehr wants to merge 13 commits into
Azure:mainfrom
AlexanderSehr:users/alsehr/johnlokerse_update-cicd-agent-ptn
Draft

fix: DevOps CI/CD agents pattern module DNS fix#7030
AlexanderSehr wants to merge 13 commits into
Azure:mainfrom
AlexanderSehr:users/alsehr/johnlokerse_update-cicd-agent-ptn

Conversation

@AlexanderSehr

@AlexanderSehr AlexanderSehr commented May 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Description

Follow up to #5944
Ran into some DNS issues when bringing your own DNS setup and when using DINE policies. This pull request removes privateDnsZoneGroupName and privateDnsZoneResourceIds in module acr. These properties were not used. Updated them to the correct properties as defined in the UDT. Additionally, added the parameter dnsIsDeployedByPolicy to be able to skip the configuration of privateDnsZoneGroups.
Updated modules to the latest version.

Regenerated files to resolve conflicts

Pipeline Reference

Pipeline Reference

Pipeline
avm.ptn.dev-ops.cicd-agents-and-runners

Type of Change

  • Azure Verified Module updates:
    • Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:
    • Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
    • Breaking changes and I have bumped the MAJOR version in version.json.
    • Update to documentation
  • Update to CI Environment or utilities (Non-module affecting changes)

johnlokerse and others added 9 commits September 8, 2025 11:09
@johnlokerse
AVM-RES: Update DNS configuration parameters
cdc3630 
- Added optional parameter 'dnsIsDeployedByPolicy' to control DNS deployment by policy.
- Updated private DNS zone group logic to utilize the new parameter.
@johnlokerse
AVM-RES: Add dnsIsDeployedByPolicy parameter
5fb6992 
- Introduced a new parameter `dnsIsDeployedByPolicy` to indicate if the DNS configuration is deployed by policy.
- Updated README.md to document the new parameter.
- Modified main.json to include the new parameter in the parameters section.
@johnlokerse
AVM-RES: Update resource versions in Bicep modules
d5f2567 
- Updated API version for private DNS zone and container registry modules.
- Adjusted references to outputs for new version compatibility.
- Updated managed environment and job module versions for consistency.
@johnlokerse
Ran bicep build
f07c753
@johnlokerse
Updated changelog
4d7a058
@johnlokerse
Merge remote-tracking branch 'origin/main' into johnlokerse/update-ci…
d186c0e 
…cd-agent-ptn
@johnlokerse
Fix merge conflicts
cea7697
@johnlokerse
README fixes
ec947b6
@AlexanderSehr
Update to latest
f49992d
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs: Triage 🔍 Maintainers need to triage still label May 11, 2026
@AlexanderSehr AlexanderSehr added Type: Bug 🐛 Something isn't working Class: Resource Module 📦 This is a resource module and removed Needs: Triage 🔍 Maintainers need to triage still labels May 11, 2026
@AlexanderSehr AlexanderSehr self-assigned this May 11, 2026
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue label May 11, 2026
@AlexanderSehr AlexanderSehr mentioned this pull request May 11, 2026
Closed
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@AlexanderSehr AlexanderSehr

Labels

Class: Resource Module 📦 This is a resource module Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue Type: Bug 🐛 Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@AlexanderSehr @johnlokerse