Conversation
…view API migration (#29308) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Yabo Hu <yabhu@microsoft.com>
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
There was a problem hiding this comment.
Pull request overview
This PR merges release content toward the Az 15.5.0 release, including DnsResolver module preannouncements for upcoming breaking changes tied to a future API migration.
Changes:
- Added DnsResolver release notes describing planned breaking changes for a future API version migration.
- Added AutoRest README directives intended to emit breaking-change annotations for affected cmdlets.
- Updated generated DnsResolver cmdlet metadata to include a breaking-change attribute for
New-AzDnsResolverDomainList.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/DnsResolver/DnsResolver/ChangeLog.md | Adds upcoming-release notes preannouncing breaking changes and a planned new bulk upload cmdlet. |
| src/DnsResolver/DnsResolver.Autorest/README.md | Adds AutoRest directives for preview/breaking-change messaging for specific cmdlets. |
| generated/DnsResolver/DnsResolver.Autorest/generated/cmdlets/NewAzDnsResolverDomainList_CreateExpanded.cs | Adds CmdletBreakingChange attribute for the New-AzDnsResolverDomainList expanded variant. |
| # Breaking change: ActionBlockResponseCode parameter removed in 2025-10-01-preview | ||
| - where: | ||
| verb: New|Update | ||
| subject: PolicyDnsSecurityRule | ||
| set: |
There was a problem hiding this comment.
The where.subject: PolicyDnsSecurityRule selector does not match the actual generated cmdlet subject (DnsResolverPolicyDnsSecurityRule), so these breaking-change directives (and the existing preview-announcement selector that also uses PolicyDnsSecurityRule) won't apply to New-AzDnsResolverPolicyDnsSecurityRule / Update-AzDnsResolverPolicyDnsSecurityRule. Update the subject matcher (e.g., DnsResolverPolicyDnsSecurityRule, or a regex that matches the generated subject) so the intended breaking-change annotations are emitted on those cmdlets.
| * Added announcement for upcoming breaking changes as part of the 2025-10-01-preview API migration: | ||
| - Removed parameter `ActionBlockResponseCode` from `New-AzDnsResolverPolicyDnsSecurityRule` and `Update-AzDnsResolverPolicyDnsSecurityRule`. The block response code is no longer configurable. | ||
| - Parameter `DnsResolverDomainList` is no longer mandatory on `New-AzDnsResolverPolicyDnsSecurityRule`. DNS security rules will support managed domain lists as an alternative to user-defined domain lists. | ||
| - Parameter `Domain` is no longer mandatory on `New-AzDnsResolverDomainList`. Domain lists will support bulk upload via the new cmdlet `Invoke-AzDnsResolverBulkDnsResolverDomainList`, which will be introduced in the upcoming release. |
There was a problem hiding this comment.
The changelog mentions that Invoke-AzDnsResolverBulkDnsResolverDomainList “will be introduced in the upcoming release”, but there’s no such cmdlet implementation in this PR/repo (only this mention). To avoid misleading users, consider rephrasing to reference the specific planned Az/module version (e.g., Az v16.0.0 / module 2.0.0) or “a future release”, consistent with the breaking-change effective date in the AutoRest config.
| - Parameter `Domain` is no longer mandatory on `New-AzDnsResolverDomainList`. Domain lists will support bulk upload via the new cmdlet `Invoke-AzDnsResolverBulkDnsResolverDomainList`, which will be introduced in the upcoming release. | |
| - Parameter `Domain` is no longer mandatory on `New-AzDnsResolverDomainList`. Domain lists will support bulk upload via the new cmdlet `Invoke-AzDnsResolverBulkDnsResolverDomainList`, which is planned for a future release. |
Description
Merge release-2026-04-07 to main. Merge it after final rc is done!
Checklist