[AutoPR azure-mgmt-paloaltonetworksngfw]-generated-from-SDK Generation - Python-5512246 #43683
Conversation
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
…gement/tspconfig.yaml', API Version: 2025-10-08, SDK Release Type: beta, and CommitSHA: 'bef8b5a67ca1bd7320fd44d0f8e95dc4bae3f6ae' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5512246 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 pull request represents a major SDK regeneration for the Azure Palo Alto Networks NGFW management SDK, updating from the AutoRest-based generation to the new Python Code Generator. The PR migrates API version from 2023-09-01 to 2025-10-08 and restructures the codebase to consolidate operations into a single operations module while maintaining backwards compatibility.
Key changes include:
- Migration from AutoRest Code Generator to Python Code Generator
- API version upgrade to 2025-10-08
- Removal of multiple individual operation files and consolidation into a single
_operations.pymodule - Addition of new models and operations (MetricsObjectFirewallOperations, PaloAltoNetworksCloudngfwOperationsOperations)
Reviewed Changes
Copilot reviewed 44 out of 284 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| operations/init.py | Updates imports to use consolidated operations module and adds new operation classes |
| models/_patch.py | Updates copyright header and modernizes type annotations |
| models/_enums.py | Updates generated code attribution and adds new enum types |
| models/init.py | Restructures imports to use consolidated models module and adds new model types |
| aio/operations/_patch.py | Updates copyright header and modernizes type annotations |
| Multiple operations files | Complete removal of individual operation files as they're consolidated |
Member
https://teams.microsoft.com/l/message/19:meeting_MTJiYzZmNDctZjA0OS00MjBkLTgyNWEtOTM5YTYyMjVmMzk5@thread.v2/1761811482305?context=%7B%22contextType%22%3A%22chat%22%7D
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 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/paloaltonetworks/PaloAltoNetworks.Management/tspconfig.yaml', API Version: 2025-10-08, SDK Release Type: beta, and CommitSHA: 'bef8b5a67ca1bd7320fd44d0f8e95dc4bae3f6ae' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5512246 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.