Skip to content

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
Deprecation package: https://pypi.org/project/azure-mgmt-hdinsightcontainers/1.0.0b4/

Copilot AI review requested due to automatic review settings November 13, 2025 08:23
Copilot finished reviewing on behalf of 2258093686 November 13, 2025 08:25
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 azure-mgmt-hdinsightcontainers SDK following the retirement of the Azure HDInsight on AKS service on January 31, 2025. The removal is comprehensive and appropriate given the service deprecation.

Key changes:

  • Complete removal of the azure-mgmt-hdinsightcontainers package and all its components
  • Removal from CI/CD configuration (ci.yml)
  • Deletion of all source code, tests, samples, and configuration files

Reviewed Changes

Copilot reviewed 122 out of 123 changed files in this pull request and generated no comments.

File Description
sdk/hdinsight/ci.yml Removed azure-mgmt-hdinsightcontainers from CI build artifacts
sdk/hdinsight/azure-mgmt-hdinsightcontainers/* Complete deletion of SDK package including all source files, tests, samples, and configuration

The removal is clean and complete, with all related files being deleted. This is the appropriate action following the service retirement announcement.

@2258093686 2258093686 changed the title Remove HDInsightContainers SDK due to HDInsightOnAKS deprecation Remove HDInsightContainers SDK Code due to HDInsightOnAKS deprecation Nov 17, 2025
@2258093686
Copy link
Member Author

Hi, @ChenxiJiang333 , Could you please help review this PR?

@2258093686
Copy link
Member Author

Hi, @msyyc , @scbedd , @lmazuel , Could you please help review this PR? Thanks

@2258093686 2258093686 merged commit cff4c23 into Azure:main Nov 20, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants