Skip to content

[AutoPR azure-mgmt-edgezones]-generated-from-SDK Generation - Python-6639987 - #48348

Open
azure-sdk-automation[bot] wants to merge 3 commits into
mainfrom
sdkauto/azure-mgmt-edgezones-6635035
Open

[AutoPR azure-mgmt-edgezones]-generated-from-SDK Generation - Python-6639987#48348
azure-sdk-automation[bot] wants to merge 3 commits into
mainfrom
sdkauto/azure-mgmt-edgezones-6635035

Conversation

@azure-sdk-automation

@azure-sdk-automation azure-sdk-automation Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Configurations: 'specification/edgezones/resource-manager/Microsoft.EdgeZones/EdgeZones/tspconfig.yaml', API Version: 2026-10-01, SDK Release Type: stable, and CommitSHA: 'e3d848eb1635fe15a89663daa425dd1619df1f37' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=6639987 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://azsdk-releaseplan-dashboard-hveph5aqhhcfhtgu.westus-01.azurewebsites.net/?releaseplan=35683 Submitted by: mertsahan@microsoft.com

Copilot AI review requested due to automatic review settings July 29, 2026 17:38
@azure-sdk-automation azure-sdk-automation Bot added the auto-release When merged to main, release pipelines for this PR's changed packages trigger automatically. label Jul 29, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
9 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot could not run the full agentic suite for this review because it was automatically requested on a bot-authored pull request. Request a review from Copilot under Reviewers to retry with the full agentic suite. Improved support for bot-authored pull requests is coming soon.

Updates the azure-mgmt-edgezones Python package to a newer API/spec generation, modernizes packaging to pyproject.toml, and refreshes generated client/model/runtime code (including cloud-aware ARM endpoint selection).

Changes:

  • Bump API version to 2026-10-01, update generated operations/models, and refresh samples.
  • Migrate packaging metadata from setup.py/sdk_packaging.toml to pyproject.toml and mark package as stable (1.0.0).
  • Introduce cloud endpoint selection (cloud_setting + ARM endpoints resolution) in sync/async clients and configuration.

Reviewed changes

Copilot reviewed 36 out of 36 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
sdk/edgezones/azure-mgmt-edgezones/tsp-location.yaml Updates the source spec commit reference for generation.
sdk/edgezones/azure-mgmt-edgezones/setup.py Removes legacy setuptools setup script in favor of pyproject-based builds.
sdk/edgezones/azure-mgmt-edgezones/sdk_packaging.toml Removes standalone packaging config (moved into pyproject.toml).
sdk/edgezones/azure-mgmt-edgezones/pyproject.toml Adds PEP 517/621 packaging config, dependencies, and packaging metadata.
sdk/edgezones/azure-mgmt-edgezones/generated_samples/operations_list.py Updates sample metadata to new API version.
sdk/edgezones/azure-mgmt-edgezones/generated_samples/extended_zones_unregister.py Updates sample metadata to new API version.
sdk/edgezones/azure-mgmt-edgezones/generated_samples/extended_zones_register.py Updates sample metadata to new API version.
sdk/edgezones/azure-mgmt-edgezones/generated_samples/extended_zones_list_by_subscription.py Updates sample metadata to new API version.
sdk/edgezones/azure-mgmt-edgezones/generated_samples/extended_zones_get.py Updates sample metadata to new API version.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/operations/_patch.py Updates patch scaffold and typing style.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/operations/_operations.py Updates operation implementations, paging types, API version, and streaming/decompress behavior.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/models/_patch.py Updates patch scaffold and typing style.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/models/_models.py Switches models to new base (_utils.model_base.Model) and updates typing.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/models/_enums.py Fixes enum docstrings punctuation.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/aio/operations/_patch.py Updates async patch scaffold and typing style.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/aio/operations/_operations.py Updates async operation implementations, paging types, and streaming/decompress behavior.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/aio/_patch.py Updates async patch scaffold and typing style.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/aio/_configuration.py Adds cloud_setting and updates API version defaults/docs for async config.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/aio/_client.py Adds cloud-aware ARM endpoint resolution and dynamic Self import for async client.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/_version.py Promotes version from beta to stable 1.0.0.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/_utils/init.py Adds generated _utils package marker.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/_serialization.py Modernizes serialization/deserialization utilities and adds duration/fast-path logic.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/_patch.py Updates patch scaffold and typing style.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/_model_base.py Major refactor/expansion of model base, XML handling, defaults, and (de)serialization behavior.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/_configuration.py Adds cloud_setting and updates API version defaults/docs for sync config.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/_client.py Adds cloud-aware ARM endpoint resolution and dynamic Self import for sync client.
sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/init.py Adds typing ignore for namespace package path extension.
sdk/edgezones/azure-mgmt-edgezones/azure/init.py Adds typing ignore for namespace package path extension.
sdk/edgezones/azure-mgmt-edgezones/apiview-properties.json Updates APIView mapping keys and adds cross-language version marker.
sdk/edgezones/azure-mgmt-edgezones/api.metadata.yml Adds API metadata output from tooling.
sdk/edgezones/azure-mgmt-edgezones/api.md Adds generated API surface documentation.
sdk/edgezones/azure-mgmt-edgezones/_metadata.json Adds new generation metadata including API versions and emitter details.
sdk/edgezones/azure-mgmt-edgezones/_meta.json Removes legacy generation metadata file.
sdk/edgezones/azure-mgmt-edgezones/README.md Updates Python support and authentication terminology/link.
sdk/edgezones/azure-mgmt-edgezones/MANIFEST.in Updates included files list and recursive include patterns.
sdk/edgezones/azure-mgmt-edgezones/CHANGELOG.md Adds 1.0.0 entry documenting cloud_setting parameter.
Comments suppressed due to low confidence (1)

sdk/edgezones/azure-mgmt-edgezones/generated_samples/operations_list.py:1

  • The x-ms-original-file annotation references 2026-10-01, which is in the future relative to the current release date in this PR. If the service API version is expected to be available only later, consider validating that the SDK should ship against it now (or update the annotation/API version to the latest currently published spec).

Comment thread sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/_client.py
Comment thread sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/_client.py
Comment thread sdk/edgezones/azure-mgmt-edgezones/CHANGELOG.md Outdated
@ChenxiJiang333

Copy link
Copy Markdown
Member

Hi, the new api version 2026-10-01 seems not deployed on portal yet, we recommend making sure the api version to be deployed before releasing the SDK, though we won't block the release in self-serve release flow.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 959ec833-f8bd-4471-9624-d74d90df8663
Copilot AI review requested due to automatic review settings July 30, 2026 02:58

@ChenxiJiang333 ChenxiJiang333 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi @mert-s, we have approved the PR. You can move on to merge and release though the action suggestion in Release Plan.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 38 out of 38 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

sdk/edgezones/azure-mgmt-edgezones/CHANGELOG.md:7

  • The 1.0.0 notes omit the new requires-python = ">=3.10" constraint. The previous package supported Python 3.8+, so users on 3.8/3.9 need this compatibility change called out in the release history.
  - Client `EdgeZonesMgmtClient` added parameter `cloud_setting` in method `__init__`

…dgeZones/EdgeZones/tspconfig.yaml', API Version: 2026-10-01, SDK Release Type: stable, and CommitSHA: 'e3d848eb1635fe15a89663daa425dd1619df1f37' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=6639987 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.
Copilot AI review requested due to automatic review settings July 30, 2026 14:03
@azure-sdk-automation azure-sdk-automation Bot changed the title [AutoPR azure-mgmt-edgezones]-generated-from-SDK Generation - Python-6635035 [AutoPR azure-mgmt-edgezones]-generated-from-SDK Generation - Python-6639987 Jul 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 38 out of 38 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (3)

sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/_configuration.py:38

  • The docstring says the default api_version is None and that None is a known value, but the implementation defaults to the string "2026-10-01" and stores it as str. Please align docs and implementation (either document the actual default string and remove None, or update typing/logic to accept Optional[str] and handle None consistently across request builders and self._config.api_version).
    :keyword api_version: The API version to use for this operation. Known values are "2026-10-01"
     and None. Default value is None. If not set, the operation's default API version will be used.
     Note that overriding this default value may result in unsupported behavior.

sdk/edgezones/azure-mgmt-edgezones/azure/mgmt/edgezones/_configuration.py:50

  • The docstring says the default api_version is None and that None is a known value, but the implementation defaults to the string "2026-10-01" and stores it as str. Please align docs and implementation (either document the actual default string and remove None, or update typing/logic to accept Optional[str] and handle None consistently across request builders and self._config.api_version).
        api_version: str = kwargs.pop("api_version", "2026-10-01")

sdk/edgezones/azure-mgmt-edgezones/MANIFEST.in:7

  • This PR replaces _meta.json with _metadata.json, but MANIFEST.in no longer includes either file. If downstream tooling or release validation expects the metadata file in sdists, it won’t be present. Consider adding an explicit include _metadata.json entry (and any other required metadata files) to keep packaging behavior consistent.
include *.md
include LICENSE
include azure/mgmt/edgezones/py.typed
recursive-include tests *.py
recursive-include samples *.py *.md
include azure/__init__.py
include azure/mgmt/__init__.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-release When merged to main, release pipelines for this PR's changed packages trigger automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants