Skip to content

Update azure-typespec/http-client-csharp-mgmt version to prerelease 1.0.0-alpha.20260211.1#56186

Merged
haiyuazhang merged 5 commits intoAzure:mainfrom
azure-sdk:validate-typespec-20260211.1-azure-typespec/http-client-csharp-mgmt
Feb 12, 2026
Merged

Update azure-typespec/http-client-csharp-mgmt version to prerelease 1.0.0-alpha.20260211.1#56186
haiyuazhang merged 5 commits intoAzure:mainfrom
azure-sdk:validate-typespec-20260211.1-azure-typespec/http-client-csharp-mgmt

Conversation

@azure-sdk
Copy link
Copy Markdown
Collaborator

Generated by branded - http-client-csharp-mgmt build 20260211.1
Triggered from branch: main

Copilot AI review requested due to automatic review settings February 11, 2026 09:33
@github-actions github-actions bot added the Mgmt This issue is related to a management package. label Feb 11, 2026
Copy link
Copy Markdown
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 PR updates the Azure Management Generator (http-client-csharp-mgmt) from version 1.0.0-alpha.20260210.3 to 1.0.0-alpha.20260211.1, regenerating affected SDKs with structural improvements.

Changes:

  • Updated the management generator package version and its dependencies
  • Refactored SignalR SDK to move SignalRReplicaSharedPrivateLinkResource operations from SignalRResource to SignalRReplicaResource, correcting the resource hierarchy
  • Removed locally-generated ManagedServiceIdentityType from Dynatrace SDK to use the common type from Azure.ResourceManager.Models

Reviewed changes

Copilot reviewed 1 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
eng/azure-typespec-http-client-csharp-mgmt-emitter-package.json Updated generator package version to 1.0.0-alpha.20260211.1
eng/azure-typespec-http-client-csharp-mgmt-emitter-package-lock.json Updated package lock with new version and dependency checksums
sdk/signalr/Azure.ResourceManager.SignalR/src/Generated/SignalRResource.cs Removed incorrectly placed replica shared private link resource methods
sdk/signalr/Azure.ResourceManager.SignalR/src/Generated/SignalRReplicaResource.cs Added replica shared private link resource collection and convenience methods
sdk/signalr/Azure.ResourceManager.SignalR/src/Generated/SignalRReplicaSharedPrivateLinkResourceCollection.cs Updated collection to properly parent under SignalRReplicaResource and added GetAll methods
sdk/signalr/Azure.ResourceManager.SignalR/src/Generated/SignalRReplicaSharedPrivateLinkResource.cs Updated documentation to reference correct parent resource
sdk/signalr/Azure.ResourceManager.SignalR/src/Generated/Extensions/SignalRExtensions.cs Removed obsolete extension methods for replica shared private link resources
sdk/signalr/Azure.ResourceManager.SignalR/src/Generated/Extensions/MockableSignalRResourceGroupResource.cs Removed mocking support for obsolete extension methods
sdk/signalr/Azure.ResourceManager.SignalR/src/Generated/CollectionResults/* Updated diagnostic scope names from MockableSignalRResourceGroupResource to SignalRReplicaSharedPrivateLinkResourceCollection
sdk/dynatrace/Azure.ResourceManager.Dynatrace/src/Generated/Models/ManagedServiceIdentityType.cs Deleted locally-generated type; now using Azure.ResourceManager.Models.ManagedServiceIdentityType
Files not reviewed (1)
  • eng/azure-typespec-http-client-csharp-mgmt-emitter-package-lock.json: Language not supported

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 11, 2026

API Change Check

APIView identified API level changes in this PR and created the following API reviews

Azure.ResourceManager.SignalR

@live1206
Copy link
Copy Markdown
Member

Root Cause Analysis

The Problem

Regenerated the SignalR SDK and moved SignalRReplicaSharedPrivateLinkResource
from being a child of SignalRResource to being a child of SignalRReplicaResource. This is a breaking API change.

Root Cause — PR #56180's Parent Detection Fix

PR #56180 fixed the parent resource detection in resource-detection.ts by replacing a "first match" break with a "longest prefix
match" algorithm. Here's what changed:

Before the fix (first-match): The loop would iterate the resource map and break on the first path that was a prefix. For
SignalR's 3-level resource hierarchy:

  • SignalR → /subscriptions/{...}/providers/Microsoft.SignalRService/signalR/{resourceName} (6 segments)
  • SignalRReplica → ...signalR/{resourceName}/replicas/{replicaName} (8 segments)
  • SignalRReplicaSharedPrivateLinkResource → ...replicas/{replicaName}/sharedPrivateLinkResources/{...} (10 segments)

Due to map iteration order, when finding the parent for SignalRReplicaSharedPrivateLinkResource, SignalR was encountered first
(before SignalRReplica), so it was incorrectly assigned as the parent. This was a latent bug — the grandchild was placed on the
grandparent.

After the fix (longest-prefix-match): The algorithm now correctly picks SignalRReplica (8 shared segments) over SignalR (6 shared
segments) as the parent for SignalRReplicaSharedPrivateLinkResource.

@live1206 live1206 force-pushed the validate-typespec-20260211.1-azure-typespec/http-client-csharp-mgmt branch 2 times, most recently from 896f9fc to 804cb56 Compare February 12, 2026 03:13
Copy link
Copy Markdown
Member

@ArcturusZhang ArcturusZhang left a comment

Choose a reason for hiding this comment

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

checking the CI on this commit: 81ffa81
before your backward compat changes, the CI could pass build, and it is only complaining about the api files are not updated accordingly.
Therefore these changes are not actually breaking changes, we might not have released them yet.
No need to write customization code to fix them, just need to export the api

@live1206 live1206 force-pushed the validate-typespec-20260211.1-azure-typespec/http-client-csharp-mgmt branch from 804cb56 to 81ffa81 Compare February 12, 2026 05:47
@haiyuazhang haiyuazhang merged commit d9a022e into Azure:main Feb 12, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Mgmt This issue is related to a management package.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants