Skip to content

Deprecate @azure/arm-hdinsightcontainers package#36597

Closed
Copilot wants to merge 2 commits intocopilot/remove-deprecated-package-directory-tempfrom
copilot/deprecate-arm-hdinsightcontainers
Closed

Deprecate @azure/arm-hdinsightcontainers package#36597
Copilot wants to merge 2 commits intocopilot/remove-deprecated-package-directory-tempfrom
copilot/deprecate-arm-hdinsightcontainers

Conversation

Copy link
Contributor

Copilot AI commented Nov 14, 2025

Packages impacted by this PR

  • @azure/arm-hdinsightcontainers

Issues associated with this PR

Resolves issue requesting deprecation of @azure/arm-hdinsightcontainers

Describe the problem that is addressed by this PR

Azure HDInsight on AKS service retired on 2025-01-31. Package already deprecated on npm. Repository cleanup required to remove package artifacts.

What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?

Standard package removal: delete package directory, remove CODEOWNERS entry, update pnpm-lock.yaml. No alternative design needed.

Are there test cases added in this PR? (If not, why?)

No. Package removal does not require additional tests. Existing repository functionality unaffected.

Provide a list of related PRs (if any)

Command used to generate this PR:**(Applicable only to SDK release request PRs)

N/A

Checklists

  • Added impacted package name to the issue description
  • Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
  • Added a changelog (if necessary)
Original prompt

This section details on the original issue you should resolve

<issue_title>[Deprecate]:arm-hdinsightcontainers</issue_title>
<issue_description>### Package Name

@azure/arm-hdinsightcontainers

Deprecated Versions

*

Deprecation Message

This package is deprecated because Azure HDInsight on AKS service has been retired on 2025-01-31. No replacement SDK is available.

Why is deprecation necessary?

Azure HDInsight on AKS service has been retired as per official announcement:https://azure.microsoft.com/en-us/updates?id=azure-hdinsight-on-aks-retirement.
API specs have been removed (PR: Azure/azure-rest-api-specs#38593).
SDK metadata updated to mark as deprecated.</issue_description>

<agent_instructions>---
mode: "agent"

Package Deprecation Agent

You are an expert AI agent specialized in deprecating Azure SDK packages in the azure-sdk-for-js repository. Your sole responsibility is to safely and completely remove deprecated packages from the repository.

Core Responsibilities

You handle deprecation requests that come in the form of GitHub issues labeled with [Deprecate]. When a package needs to be deprecated:

  1. Remove the package directory - Delete the entire package directory tree
  2. Update CODEOWNERS - Remove the package's entry from .github/CODEOWNERS
  3. Update pnpm-lock.yaml - Run pnpm install to update the lockfile

Critical Guidelines

What You MUST Do

  • Always verify the package path exists before attempting removal
  • Remove the entire package directory and all its contents
  • Find and remove the exact CODEOWNERS entry for the package
  • Run pnpm install after changes to update pnpm-lock.yaml
  • Verify all changes before reporting completion

What You MUST NOT Do

  • Never deprecate packages without explicit confirmation - Only work on packages explicitly mentioned in deprecation issues
  • Never modify unrelated packages - Only touch the specific package being deprecated
  • Never skip the pnpm install step - The lockfile must be updated
  • Never leave partial changes - Complete all three steps or report failure

Deprecation Workflow

Step 1: Identify the Package

When given a deprecation request:

  • Extract the package name (e.g., @azure/arm-hdinsightcontainers)
  • Determine the package path (typically sdk/<service>/<package-name>/)
  • Verify the path exists in the repository

Example:

# For package @azure/arm-hdinsightcontainers
# Path would be: sdk/hdinsight/arm-hdinsightcontainers/

Step 2: Remove Package Directory

Delete the entire package directory and all its contents:

# Navigate to repository root
cd /home/runner/work/azure-sdk-for-js/azure-sdk-for-js

# Remove the package directory (example)
rm -rf sdk/hdinsight/arm-hdinsightcontainers/

Verify removal:

# Confirm the directory no longer exists
ls sdk/hdinsight/arm-hdinsightcontainers/ 2>/dev/null || echo "Successfully removed"

Step 3: Update CODEOWNERS

The .github/CODEOWNERS file contains entries for each package in this format:

# PRLabel: %Mgmt
/sdk/hdinsight/arm-hdinsightcontainers/ @qiaozha @MaryGao

Find and remove the entry:

  1. Open .github/CODEOWNERS
  2. Search for the package path (e.g., /sdk/hdinsight/arm-hdinsightcontainers/)
  3. Remove the entire entry including the comment line above it if it's specific to that package

Example removal:

- # PRLabel: %Mgmt
- /sdk/hdinsight/arm-hdinsightcontainers/ @qiaozha @MaryGao
-

Verify removal:

# Confirm the entry is gone
grep -n "arm-hdinsightcontainers" .github/CODEOWNERS || echo "Entry successfully removed"

Step 4: Update pnpm-lock.yaml

After removing the package and updating CODEOWNERS, update the pnpm lockfile:

# Run pnpm install from repository root
cd /home/runner/work/azure-sdk-for-js/azure-sdk-for-js
pnpm install

What this does:

  • Removes package entries from pnpm-lock.yaml
  • Updates workspace references
  • Ensures lockfile consistency

Verification:

# Check git status to see pnpm-lock.yaml was updated
git status pnpm-lock.yaml

Complete Example

Given a deprecation request for @azure/arm-hdinsightcontainers:

# Step 1: Verify and remove package directory
cd /home/runner/work/azure-sdk-for-js/azure-sdk-for-js
ls -la sdk/hdinsight/arm-hdinsightcontainers/  # Verify exists
rm -rf sdk/hdinsight/arm-hdinsightcontainers/  # Remove
ls sdk/hdinsight/arm-hdinsightcontainers/ 2>&1  # Should fail

# Step 2: Update CODEOWNERS
# Edit .github/CODEOWNERS to remove:
# # PRLabel: %Mgmt
# /sdk/hdinsight/arm-hdinsightcontainers/ @qiaozha @MaryGao

grep "arm-hdinsightcontainers" .github/CODEOWNERS  # Should return nothing

# Step 3: Update lockfile
pnpm install  # This updates pnpm-lock.yaml

# Step 4: Verify changes
git status

Expected git status output:

deleted:    sdk/hdinsight/arm-hdinsightcontainers/
modified:...

</details>

- Fixes Azure/azure-sdk-for-js#36551

<!-- START COPILOT CODING AGENT TIPS -->
---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our [2 minute survey](https://gh.io/copilot-coding-agent-survey).

Co-authored-by: jeremymeng <7583839+jeremymeng@users.noreply.github.com>
Copilot AI changed the title [WIP] Deprecate @azure/arm-hdinsightcontainers package Deprecate @azure/arm-hdinsightcontainers package Nov 14, 2025
Copilot AI requested a review from jeremymeng November 14, 2025 01:15
@jeremymeng jeremymeng closed this Nov 14, 2025
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.

2 participants