Skip to content

Comments

[AutoPR azure-mgmt-paloaltonetworksngfw]-generated-from-SDK Generation - Python-5516768#43697

Merged
Prakriti-nith merged 10 commits intoAzure:mainfrom
azure-sdk:sdkauto/azure-mgmt-paloaltonetworksngfw-5516768
Nov 13, 2025
Merged

[AutoPR azure-mgmt-paloaltonetworksngfw]-generated-from-SDK Generation - Python-5516768#43697
Prakriti-nith merged 10 commits intoAzure:mainfrom
azure-sdk:sdkauto/azure-mgmt-paloaltonetworksngfw-5516768

Conversation

@azure-sdk
Copy link
Collaborator

@azure-sdk azure-sdk commented Oct 30, 2025

Configurations: 'specification/paloaltonetworks/PaloAltoNetworks.Management/tspconfig.yaml', API Version: 2025-10-08, SDK Release Type: stable, and CommitSHA: '8e3e3baa2523e17becb3d7032eb909aba12b7f2c' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5516768 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=6d9c662a-75b5-f011-bbd3-6045bd08111b Submitted by: prakgupta@microsoft.com

Release Plan Details

…gement/tspconfig.yaml', API Version: 2025-10-08, SDK Release Type: stable, and CommitSHA: '8e3e3baa2523e17becb3d7032eb909aba12b7f2c' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5516768 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.
@Prakriti-nith Prakriti-nith marked this pull request as ready for review October 30, 2025 11:04
Copilot AI review requested due to automatic review settings October 30, 2025 11:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This is an auto-generated SDK update for the azure-mgmt-paloaltonetworksngfw Python SDK, updating from SDK Generation - Python-5516768 with API version 2025-10-08. The PR updates the SDK to use a new code generator (Microsoft (R) Python Code Generator instead of AutoRest Code Generator) and includes significant structural changes to the codebase.

Key changes include:

  • Complete regeneration with new Python Code Generator
  • Removal of many existing operation files and consolidation into unified operations module
  • Updated models, enums, and type annotations throughout
  • Migration from List to list type annotations for Python 3.9+ compatibility

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
Multiple operations files Entire files removed as part of SDK regeneration consolidation
init.py files Updated imports to use unified operations module and new type system
models/_enums.py Updated with new enum values and improved documentation
models/init.py Consolidated imports and removed redundant ListResult classes
_patch.py files Updated copyright headers and changed List to list type annotations
Multiple aio operations Removed async operation files as part of consolidation
init.py (mgmt level) Added type ignore comment for pkgutil extend_path

@ChenxiJiang333
Copy link
Member

@msyyc, live test passed, please help review pr.

@Prakriti-nith
Copy link
Member

@ChenxiJiang333 @msyyc One question - this PR was created after migrating the swagger to typespec, so there are many changes. Are these changes expected and looks okay?

@ChenxiJiang333
Copy link
Member

ChenxiJiang333 commented Oct 31, 2025

@ChenxiJiang333 @msyyc One question - this PR was created after migrating the swagger to typespec, so there are many changes. Are these changes expected and looks okay?

For the breaking changes variables moved under property properties, they are expected according to the model migration change guide that flatten is not supported after migration.

And for the breaking changes parameter changed from positional_or_keyword to keyword_only, they are also expected by the operation migration change guide that query and header signatures are going to be changed from positional to keyword-only.

@Prakriti-nith
Copy link
Member

@msyyc @ChenxiJiang333 Is this PR good to be merged?

@msyyc
Copy link
Member

msyyc commented Nov 4, 2025

@msyyc @ChenxiJiang333 Is this PR good to be merged?

Waiting for confirmation from Python SDK architect @johanste

@Prakriti-nith
Copy link
Member

@johanste @msyyc Is there any update on this PR?

@msyyc
Copy link
Member

msyyc commented Nov 12, 2025

@johanste @msyyc Is there any update on this PR?

This time we will still release from swagger. @ChenxiJiang333 Let us regenerate from swagger.

@ChenxiJiang333
Copy link
Member

@Prakriti-nith, pr ready, feel free to merge and release

@Prakriti-nith Prakriti-nith merged commit 4f07ea0 into Azure:main Nov 13, 2025
20 checks passed
msyyc added a commit that referenced this pull request Nov 25, 2025
…n - Python-5516768 (#43697)

* Configurations:  'specification/paloaltonetworks/PaloAltoNetworks.Management/tspconfig.yaml', API Version: 2025-10-08, SDK Release Type: stable, and CommitSHA: '8e3e3baa2523e17becb3d7032eb909aba12b7f2c' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5516768 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.

* add live test decorator

* update changelog

* update testcases

* Update CHANGELOG.md

* Update CHANGELOG.md

* Configurations:  'specification/paloaltonetworks/resource-manager/readme.md', API Version: 2025-10-08, SDK Release Type: stable, and CommitSHA: 'ea59de0de01a84fdea241f85e3cb518c0339c9ba' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5562526 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.

* update changelog

* Update CHANGELOG.md

* Update version from 2.0.0 to 1.1.0

---------

Co-authored-by: Yuchao Yan <yuchaoyan@microsoft.com>
Co-authored-by: ChenxiJiang333 <v-chenjiang@microsoft.com>
Co-authored-by: ChenxiJiang333 <119990644+ChenxiJiang333@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants