Remove HDInsightContainers SDK due to HDInsightOnAKS deprecation#36519
Closed
2258093686 wants to merge 6 commits intoAzure:mainfrom
Closed
Remove HDInsightContainers SDK due to HDInsightOnAKS deprecation#365192258093686 wants to merge 6 commits intoAzure:mainfrom
2258093686 wants to merge 6 commits intoAzure:mainfrom
Conversation
2258093686
requested review from
a team,
MaryGao,
jeremymeng,
qiaozha and
xirzec
as code owners
github-actions
bot
added
the
Mgmt
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR removes the arm-hdinsightcontainers package from the HDInsight SDK. The changes completely delete all source files, tests, and configuration for this package, while also updating the CI configuration to no longer build or test it.
Key changes:
- Complete removal of the
arm-hdinsightcontainerspackage source code and tests - Updates to CI pipeline configuration to exclude the removed package
- Removal of all TypeScript configuration files for the package
Reviewed Changes
Copilot reviewed 150 out of 153 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/hdinsight/ci.mgmt.yml | Removes arm-hdinsightcontainers from CI triggers and artifacts |
| sdk/hdinsight/arm-hdinsightcontainers/vitest.config.ts | Deletes Vitest configuration file |
| sdk/hdinsight/arm-hdinsightcontainers/vitest.browser.config.ts | Deletes browser Vitest configuration |
| sdk/hdinsight/arm-hdinsightcontainers/tsconfig.*.json | Deletes all TypeScript configuration files |
| sdk/hdinsight/arm-hdinsightcontainers/test/*.spec.ts | Deletes all test files |
| sdk/hdinsight/arm-hdinsightcontainers/src/**/*.ts | Deletes all source code files including operations, models, and entry point |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
…2258093686/azure-sdk-for-js into deprecate-js-sdk-HdinsightOnAks
Member
Author
|
The Azure SDK has been marked as deprecated. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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