Skip to content

Remove unnecessary HDInsight C# scope suppressions#44450

Merged
ArcturusZhang merged 2 commits into
Azure:mainfrom
ArcturusZhang:arcturus/hdinsight-unhide-actions
Jul 7, 2026
Merged

Remove unnecessary HDInsight C# scope suppressions#44450
ArcturusZhang merged 2 commits into
Azure:mainfrom
ArcturusZhang:arcturus/hdinsight-unhide-actions

Conversation

@ArcturusZhang

@ArcturusZhang ArcturusZhang commented Jul 6, 2026

Copy link
Copy Markdown
Member

Summary

Removes unnecessary HDInsight C# scope suppressions that forced several action/sub-operation endpoints out of SDK generation.

The HDInsight management generator can now keep these endpoints on the existing cluster/application resources without producing phantom resource types, so the SDK no longer needs raw-HTTP custom code for them.

Changes

  • Removes broad C# @@scope(..., "!csharp") suppressions for HDInsight action/sub-operation endpoints.
  • Keeps LocationsOperationGroup.getAzureAsyncOperationStatus scoped out because it is an LRO polling endpoint and should not be exposed as an SDK API.
  • Adds C# client-name customizations needed to preserve the existing .NET SDK public API names and parameter names after regeneration.

SDK impact

The paired .NET SDK PR regenerates Azure.ResourceManager.HDInsight and preserves the existing public API surface while moving formerly custom raw-HTTP implementations into generated code.

Links

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

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

Next Steps to Merge

✅ All automated merging requirements have been met! To get your PR merged, see aka.ms/azsdk/specreview/merge.

Comment generated by summarize-checks workflow run.

@github-actions github-actions Bot added resource-manager TypeSpec Authored with TypeSpec labels Jul 6, 2026
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

API Change Check

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

Language API Review for Package
TypeSpec Microsoft.HDInsight
Python azure-mgmt-hdinsight
C# Azure.ResourceManager.HDInsight
Java com.azure.resourcemanager:azure-resourcemanager-hdinsight

Comment generated by After APIView workflow run.

@ArcturusZhang ArcturusZhang marked this pull request as ready for review July 6, 2026 09:30
@github-actions github-actions Bot added ARMReview WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required ARMAutoSignedOff-Trivial ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review and removed WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Jul 6, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@ArcturusZhang ArcturusZhang added the PublishToCustomers Acknowledgement the changes will be published to Azure customers. label Jul 7, 2026
@ArcturusZhang ArcturusZhang merged commit fd1b45a into Azure:main Jul 7, 2026
77 checks passed
@ArcturusZhang ArcturusZhang deleted the arcturus/hdinsight-unhide-actions branch July 7, 2026 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ARMAutoSignedOff-Trivial ARMReview ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review HDInsight PublishToCustomers Acknowledgement the changes will be published to Azure customers. resource-manager TypeSpec Authored with TypeSpec

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants