[AutoPR azure-mgmt-dnsresolver]-generated-from-SDK Generation - Python-5425117#43279
Merged
jamesvoongms merged 4 commits intoAzure:mainfrom Oct 27, 2025
Merged
Conversation
….Network/DnsResolver/tspconfig.yaml', API Version: 2025-10-01-preview, SDK Release Type: beta, and CommitSHA: '3ec630fc62ebc03e9adf238cb2fb0b538d93ead4' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5425117 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR updates the Azure DNS Resolver management SDK to use the new Python Code Generator (formerly AutoRest Code Generator), transitioning to API version 2025-10-01-preview. The changes represent a major SDK regeneration with architectural improvements in code organization and structure.
Key changes:
- Migration from AutoRest to Python Code Generator for SDK generation
- Consolidation of operation files into a unified
_operations.pymodule - Updated API version from 2025-05-01 to 2025-10-01-preview
- Modernization of type hints (e.g.,
List[str]→list[str])
Reviewed Changes
Copilot reviewed 41 out of 137 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
_dns_resolvers_operations.py |
Removed - consolidated into unified operations module |
_dns_resolver_policy_virtual_network_links_operations.py |
Removed - consolidated into unified operations module |
_dns_resolver_policies_operations.py |
Removed - consolidated into unified operations module |
_dns_resolver_domain_lists_operations.py |
Removed - consolidated into unified operations module |
_dns_forwarding_rulesets_operations.py |
Removed - consolidated into unified operations module |
operations/__init__.py |
Updated imports to reference consolidated _operations module |
models/_patch.py |
Modernized type hints and updated copyright header format |
models/_models_py3.py |
Removed - replaced with regenerated models in new structure |
sdk/dnsresolver/azure-mgmt-dnsresolver/azure/mgmt/dnsresolver/models/_patch.py
Show resolved
Hide resolved
sdk/dnsresolver/azure-mgmt-dnsresolver/azure/mgmt/dnsresolver/models/_patch.py
Show resolved
Hide resolved
sdk/dnsresolver/azure-mgmt-dnsresolver/azure/mgmt/dnsresolver/operations/__init__.py
Show resolved
Hide resolved
Member
|
will have to regenerate the sdk after Azure/azure-rest-api-specs#38383 merged |
….Network/DnsResolver/tspconfig.yaml', API Version: 2025-10-01-preview, SDK Release Type: beta, and CommitSHA: '95bf758fc791fd2f1684dc05a4149f96df700727' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5489958 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.
Member
|
@msyyc, live test passed, please help review |
msyyc
reviewed
Oct 24, 2025
msyyc
approved these changes
Oct 27, 2025
Member
|
Thank you everyone! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Configurations: 'specification/dnsresolver/resource-manager/Microsoft.Network/DnsResolver/tspconfig.yaml', API Version: 2025-10-01-preview, SDK Release Type: beta, and CommitSHA: '3ec630fc62ebc03e9adf238cb2fb0b538d93ead4' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5425117 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release. Release plan link: https://web.powerapps.com/apps/821ab569-ae60-420d-8264-d7b5d5ef734c?release-plan-id=61053214-6a8e-f011-b4cc-0022480bb79b Submitted by: jamesvoong@microsoft.com
Release Plan Details
Spec pull request: Add 2025-10-01-preview changes to dns resolver azure-rest-api-specs#37724
Spec API version: 2025-10-01-preview