Skip to content

Remove HDInsightContainers SDK due to HDInsightOnAKS deprecation#25580

Closed
2258093686 wants to merge 1 commit intoAzure:mainfrom
2258093686:deprecate-go-sdk-HdinsightOnAks
Closed

Remove HDInsightContainers SDK due to HDInsightOnAKS deprecation#25580
2258093686 wants to merge 1 commit intoAzure:mainfrom
2258093686:deprecate-go-sdk-HdinsightOnAks

Conversation

@2258093686
Copy link
Member

HDInsightOnAKS service was retired on January 31, 2025; so the SDK for it is now useless and can be removed.
Retirement Announcement: https://azure.microsoft.com/en-us/updates?id=azure-hdinsight-on-aks-retirement

Copilot AI review requested due to automatic review settings November 10, 2025 08:01
@github-actions github-actions bot added the Mgmt This issue is related to a management-plane library. label Nov 10, 2025
@2258093686 2258093686 marked this pull request as draft November 10, 2025 08:02
Copy link
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 pull request removes the entire armhdinsightcontainers SDK module by deleting all of its source files, including:

  • Core client implementations (operations, locations, clusters, cluster pools)
  • Model serialization code (models_serde.go with 5682 lines)
  • Type definitions (responses.go, options.go, interfaces.go)
  • Helper utilities (time_rfc3339.go, polymorphic_helpers.go)
  • Fake/test server implementations
  • Module dependencies (go.mod, go.sum)
  • Example test files

Key Changes

  • Complete removal of the HDInsight Containers ARM SDK module
  • Deletion of ~7500+ lines of generated code
  • Removal of all client implementations, models, and supporting infrastructure
  • Elimination of module dependencies and test infrastructure

Reviewed Changes

Copilot reviewed 54 out of 56 changed files in this pull request and generated no comments.

Show a summary per file
File Description
go.mod, go.sum Removed module dependencies and version constraints
models_serde.go Deleted 5682 lines of model serialization/deserialization code
operations_client.go, locations_client.go Removed API client implementations
responses.go, options.go Deleted type definitions for API responses and options
time_rfc3339.go Removed RFC3339 time handling utilities
polymorphic_helpers.go Deleted polymorphic type unmarshaling helpers
interfaces.go Removed interface definitions for polymorphic types
fake/*.go Deleted all fake server implementations for testing
*_example_test.go Removed example test files

@2258093686 2258093686 marked this pull request as ready for review November 11, 2025 01:40
@2258093686
Copy link
Member Author

The Azure SDK has been marked as deprecated.
Azure/azure-sdk#9363

@2258093686 2258093686 marked this pull request as draft November 11, 2025 08:18
@2258093686 2258093686 closed this Nov 17, 2025
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-plane library.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants