[AutoPR @azure-arm-servicefabricmanagedclusters]-generated-from-SDK Generation - JS-5919238#37356
Conversation
…manager/Microsoft.ServiceFabric/ServiceFabricManagedClusters/tspconfig.yaml', API Version: 2026-02-01, SDK Release Type: stable, and CommitSHA: '57a2504930224552a409d9435d523e631a6c93e2' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5919238 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.
There was a problem hiding this comment.
Pull request overview
This PR updates the Azure Service Fabric Managed Clusters SDK from version 1.0.0-beta.6 to 1.0.0, transitioning from API version 2025-10-01-preview to 2026-02-01 (stable). The changes are auto-generated from TypeSpec specifications and include removal of preview fault simulation features, improvements to API version handling, URL normalization enhancements, and various helper utilities.
Changes:
- Removed fault simulation preview features from both ManagedClusters and NodeTypes operations
- Made apiVersion optional with fallback to "2026-02-01" throughout operation files
- Added URL normalization for unreserved characters and API version injection for pagination/polling
- Updated constructor to support optional subscriptionId parameter
- Updated samples to reflect API version change and removed fault simulation examples
Reviewed changes
Copilot reviewed 244 out of 247 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tsp-location.yaml | Updated commit hash for spec generation source |
| urlTemplate.ts | Added URL normalization for unreserved characters (RFC 3986 compliance) |
| check-prop-undefined.ts | New helper to check if all properties are undefined |
| pollingHelpers.ts | Added API version parameter to polling options and URL injection logic |
| pagingHelpers.ts | Added API version and nextLinkMethod parameters for pagination |
| serviceFabricManagedClustersManagementClient.ts | Added constructor overloads to make subscriptionId optional |
| serviceFabricManagedClustersManagementContext.ts | Made apiVersion optional, removed custom API version policy |
| services/operations.ts | Applied apiVersion fallback pattern, removed status code 201 from delete |
| models/index.ts, index.ts | Removed fault simulation model exports |
| classic/*/index.ts | Removed fault simulation operations from client interfaces |
| api/*/operations.ts | Applied apiVersion ?? "2026-02-01" fallback pattern consistently |
| api/*/options.ts | Removed fault simulation operation parameter types |
| samples/* | Updated API version references, removed fault simulation samples, formatting changes |
| package.json | Version bump from 1.0.0-beta.6 to 1.0.0 |
| metadata.json | Updated API version and emitter version, removed fault simulation mappings |
| README.md | Updated API reference URL to remove preview tag |
| review/*.api.md | Removed fault simulation APIs, made apiVersion optional |
|
ci failure is because you need to run the test and push the test recordings but testing is not a self service blocker, feel free to run the test according to above guide or skip the test by using describe.skip in https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/servicefabricmanagedclusters/arm-servicefabricmanagedclusters/test/public/servicefabricmanagedclusters_operations_test.spec.ts#L19 |
|
LGTM |
…eneration - JS-5919238 (Azure#37356) Configurations: 'specification/servicefabricmanagedclusters/resource-manager/Microsoft.ServiceFabric/ServiceFabricManagedClusters/tspconfig.yaml', API Version: 2026-02-01, SDK Release Type: stable, and CommitSHA: '57a2504930224552a409d9435d523e631a6c93e2' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5919238 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=2c8e6891-a411-f111-8406-000d3a5a7db7](https://web.powerapps.com/apps/821ab569-ae60-420d-8264-d7b5d5ef734c?release-plan-id=2c8e6891-a411-f111-8406-000d3a5a7db7) **Submitted by**: iliu@microsoft.com ## Release Plan Details - Release Plan: https://aka.ms/sdk-release-planner?release-plan-id=2c8e6891-a411-f111-8406-000d3a5a7db7 Spec pull request: Azure/azure-rest-api-specs#39563 Spec API version: 2026-02-01 --------- Co-authored-by: Ivy Liu <iliu@microsoft.com>
Configurations: 'specification/servicefabricmanagedclusters/resource-manager/Microsoft.ServiceFabric/ServiceFabricManagedClusters/tspconfig.yaml', API Version: 2026-02-01, SDK Release Type: stable, and CommitSHA: '57a2504930224552a409d9435d523e631a6c93e2' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5919238 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=2c8e6891-a411-f111-8406-000d3a5a7db7 Submitted by: iliu@microsoft.com
Release Plan Details
Spec pull request: [servicefabricmanagedclusters] 2026-02-01 azure-rest-api-specs#39563
Spec API version: 2026-02-01