[AutoPR azure-resourcemanager-servicegroups]-generated-from-SDK Generation - Java-6712453 - #50168
Conversation
…Management/ServiceGroups/tspconfig.yaml', SDK Release Type: stable, and CommitSHA: '4cd7e2ac392a5d75c6a29fee14e93d2146364e79' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=6712453 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.
|
Azure Pipelines: Successfully started running 1 pipeline(s). 35 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
There was a problem hiding this comment.
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 Service Groups management SDK to a newer API version (2026-08-01), introducing an Operations surface area and new models, while removing the previous listAncestors API and related types.
Changes:
- Bump API version to
2026-08-01and release version to1.0.0. - Add Operations client/models (
Operation,OperationDisplay,Origin,ActionType,OperationListResult) andServiceGroupAttributes. - Remove
ServiceGroups.listAncestors*APIs andServiceGroupCollectionResponse(+ related samples/tests), update samples/docs accordingly.
Reviewed changes
Copilot reviewed 36 out of 55 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/servicegroups/azure-resourcemanager-servicegroups/tsp-location.yaml | Updates the swagger/spec commit reference used to generate the SDK. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/ServiceGroupsListAncestorsWithResponseMockTests.java | Removes mock test for the removed listAncestorsWithResponse API. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/ServiceGroupsGetWithResponseMockTests.java | Updates mock payload/assertions to include new attributes shape. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/ServiceGroupPropertiesTests.java | Adds coverage for ServiceGroupProperties.attributes. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/ServiceGroupInnerTests.java | Updates serialization/deserialization tests to include attributes. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/ServiceGroupCollectionResponseInnerTests.java | Removes tests tied to removed ServiceGroupCollectionResponseInner. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/ServiceGroupAttributesTests.java | Adds tests for the new ServiceGroupAttributes model. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/ResourceProvidersUpdateServiceGroupMockTests.java | Updates update mock test to include attributes. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/ResourceProvidersCreateOrUpdateServiceGroupMockTests.java | Updates create/update mock test to include attributes. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/ParentServiceGroupPropertiesTests.java | Updates test data for parent properties. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/OperationsListMockTests.java | Adds mocking test scaffold for Operations list API. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/OperationListResultTests.java | Adds a model test for OperationListResult. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/OperationInnerTests.java | Adds a model test for OperationInner. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/test/java/com/azure/resourcemanager/servicegroups/generated/OperationDisplayTests.java | Adds a model test for OperationDisplay. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/samples/java/com/azure/resourcemanager/servicegroups/generated/ServiceGroupsListAncestorsSamples.java | Removes sample for removed ListAncestors API. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/samples/java/com/azure/resourcemanager/servicegroups/generated/ServiceGroupsGetSamples.java | Updates sample metadata to the new API version. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/samples/java/com/azure/resourcemanager/servicegroups/generated/ResourceProviderUpdateServiceGroupSamples.java | Updates patch sample to include new ServiceGroupAttributes. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/samples/java/com/azure/resourcemanager/servicegroups/generated/ResourceProviderDeleteServiceGroupSamples.java | Updates sample metadata to the new API version. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/samples/java/com/azure/resourcemanager/servicegroups/generated/ResourceProviderCreateOrUpdateServiceGroupSamples.java | Updates put sample to include new ServiceGroupAttributes and api-version. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/samples/java/com/azure/resourcemanager/servicegroups/generated/OperationsListSamples.java | Adds a new sample for listing Operations. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/resources/META-INF/native-image/com.azure.resourcemanager/azure-resourcemanager-servicegroups/proxy-config.json | Adds Operations proxy service for native-image reflection/proxy config. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/resources/META-INF/azure-resourcemanager-servicegroups_metadata.json | Updates api version, adds Operations definitions/models, and generated file list. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/module-info.java | Opens implementation models for JSON deserialization in Java modules. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/package-info.java | Removes detailed package documentation text. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/models/package-info.java | Removes detailed models package documentation text. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/models/ServiceGroups.java | Removes listAncestors* methods from public surface area. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/models/ServiceGroupProperties.java | Adds ServiceGroupAttributes attributes and JSON read/write support. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/models/ServiceGroupCollectionResponse.java | Removes obsolete ServiceGroupCollectionResponse type. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/models/ServiceGroupAttributes.java | Adds new model to represent service group attributes (e.g., criticality). |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/models/Origin.java | Adds extensible enum for operation origin. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/models/Operations.java | Adds resource collection interface for listing operations. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/models/OperationDisplay.java | Adds model for operation display info. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/models/Operation.java | Adds public client-side representation of an operation. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/models/ActionType.java | Adds extensible enum for operation action type. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/implementation/package-info.java | Removes detailed implementation package documentation text. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/implementation/ServiceGroupsManagementClientImpl.java | Adds OperationsClient initialization + bumps apiVersion constant. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/implementation/ServiceGroupsImpl.java | Removes ancestor listing implementation. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/implementation/ServiceGroupsClientImpl.java | Removes REST proxy methods and client methods for ancestor listing. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/implementation/ServiceGroupCollectionResponseImpl.java | Removes impl for the removed ServiceGroupCollectionResponse. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/implementation/OperationsImpl.java | Adds high-level Operations resource collection implementation. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/implementation/OperationsClientImpl.java | Adds generated REST client implementation for Operations list paging. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/implementation/OperationImpl.java | Adds client-side wrapper implementation for Operation. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/fluent/package-info.java | Removes detailed fluent package documentation text. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/fluent/models/package-info.java | Removes detailed fluent models package documentation text. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/fluent/models/ServiceGroupCollectionResponseInner.java | Replaces service-group collection inner model with operations list result model (see comment). |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/fluent/models/OperationInner.java | Adds inner model for Operations payload. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/fluent/ServiceGroupsManagementClient.java | Exposes Operations client from the fluent management client. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/fluent/ServiceGroupsClient.java | Removes ancestor listing methods from fluent surface. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/fluent/OperationsClient.java | Adds fluent Operations client interface. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/ServiceGroupsManager.java | Adds operations() entry point for Operations resource collection. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/pom.xml | Bumps version to 1.0.0, updates description/api-version, enables revapi. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/SAMPLE.md | Adds Operations section/sample; removes ServiceGroups ListAncestors sample. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/README.md | Updates version and api-version in README. |
| sdk/servicegroups/azure-resourcemanager-servicegroups/CHANGELOG.md | Documents 1.0.0 release, breaking changes, and added features/models. |
| eng/versioning/version_client.txt | Updates the tracked artifact version to 1.0.0. |
Suppressed comments (1)
sdk/servicegroups/azure-resourcemanager-servicegroups/src/main/java/com/azure/resourcemanager/servicegroups/package-info.java:1
- The package-level Javadoc previously documented what the Service Groups RP provides; that information has been removed, leaving a very minimal description. Consider restoring the higher-level description (even in shortened form) so generated/reference docs retain useful context for SDK consumers.
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
Automated management AutoPR review
Warning
Informational
|
Configurations: 'specification/management/resource-manager/Microsoft.Management/ServiceGroups/tspconfig.yaml', SDK Release Type: stable, and CommitSHA: '4cd7e2ac392a5d75c6a29fee14e93d2146364e79' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=6712453 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=35595 Submitted by: meghanagupta@microsoft.com