Skip to content

[HDInsight] Update hdinsight api-version to 2025-01-15-preview#32273

Merged
kairu-ms merged 7 commits intoAzure:devfrom
2258093686:hdi-cli-sdk-2025-01-15-preview
Oct 22, 2025
Merged

[HDInsight] Update hdinsight api-version to 2025-01-15-preview#32273
kairu-ms merged 7 commits intoAzure:devfrom
2258093686:hdi-cli-sdk-2025-01-15-preview

Conversation

@2258093686
Copy link
Copy Markdown
Member

@2258093686 2258093686 commented Oct 17, 2025

Related command

az hdinsight create
az hdinsight credentials update
az hdinsight credentials show

Description

  • Update API version to 2025-01-15-preview.
  • Add parameters --entra-user-identity and --entra-user-full-info to az hdinsight create to support creating Entra-enabled clusters.
  • Add support for creating clusters with WASB + MSI (Managed Identity) authentication in az hdinsight create.
  • Add two new commands to update/get cluster credentials.

Testing Guide

  1. create entra cluster by --entra-user-identity
az hdinsight create {other params}  --entra-user-identity "00000000-0000-0000-0000-000000000000" 
az hdinsight create {other params}  --entra-user-identity "00000000-0000-0000-0000-000000000000" "test@example.com"  
  1. create entra cluster by --entra-user-full-info
az hdinsight create {other params}  --entra-user-full-info @path/EntraUserFullInfoConfig.json
  1. create cluster with WASB + MSI authentication
az hdinsight create {other params} --storage-account-name mystorageaccount --storage-account-managed-identity MyMSI
  1. update entra cluster's entra user
az hdinsight credentials update -g  -n {cluster} --resource-group {rg} --entra-user-identity "00000000-0000-0000-0000-000000000000" "test@example.com"  
  1. show current cluster credentials
az hdinsight credentials show -g  -n {cluster} --resource-group {rg} 

History Notes

[HDInsight] az hdinsight create: Support creating Entra-enabled clusters and creating clusters with WASB + MSI
[HDInsight] az hdinsight credentials update: Update cluster credentials
[HDInsight] az hdinsight credentials show: Show current cluster credentials


This checklist is used to make sure that common guidelines for a pull request are followed.

Copilot AI review requested due to automatic review settings October 17, 2025 01:38
@2258093686 2258093686 requested a review from kairu-ms as a code owner October 17, 2025 01:38
@azure-client-tools-bot-prd
Copy link
Copy Markdown

azure-client-tools-bot-prd bot commented Oct 17, 2025

️✔️AzureCLI-FullTest
️✔️acr
️✔️latest
️✔️3.12
️✔️3.13
️✔️acs
️✔️latest
️✔️3.12
️✔️3.13
️✔️advisor
️✔️latest
️✔️3.12
️✔️3.13
️✔️ams
️✔️latest
️✔️3.12
️✔️3.13
️✔️apim
️✔️latest
️✔️3.12
️✔️3.13
️✔️appconfig
️✔️latest
️✔️3.12
️✔️3.13
️✔️appservice
️✔️latest
️✔️3.12
️✔️3.13
️✔️aro
️✔️latest
️✔️3.12
️✔️3.13
️✔️backup
️✔️latest
️✔️3.12
️✔️3.13
️✔️batch
️✔️latest
️✔️3.12
️✔️3.13
️✔️batchai
️✔️latest
️✔️3.12
️✔️3.13
️✔️billing
️✔️latest
️✔️3.12
️✔️3.13
️✔️botservice
️✔️latest
️✔️3.12
️✔️3.13
️✔️cdn
️✔️latest
️✔️3.12
️✔️3.13
️✔️cloud
️✔️latest
️✔️3.12
️✔️3.13
️✔️cognitiveservices
️✔️latest
️✔️3.12
️✔️3.13
️✔️compute_recommender
️✔️latest
️✔️3.12
️✔️3.13
️✔️computefleet
️✔️latest
️✔️3.12
️✔️3.13
️✔️config
️✔️latest
️✔️3.12
️✔️3.13
️✔️configure
️✔️latest
️✔️3.12
️✔️3.13
️✔️consumption
️✔️latest
️✔️3.12
️✔️3.13
️✔️container
️✔️latest
️✔️3.12
️✔️3.13
️✔️containerapp
️✔️latest
️✔️3.12
️✔️3.13
️✔️core
️✔️latest
️✔️3.12
️✔️3.13
️✔️cosmosdb
️✔️latest
️✔️3.12
️✔️3.13
️✔️databoxedge
️✔️latest
️✔️3.12
️✔️3.13
️✔️dls
️✔️latest
️✔️3.12
️✔️3.13
️✔️dms
️✔️latest
️✔️3.12
️✔️3.13
️✔️eventgrid
️✔️latest
️✔️3.12
️✔️3.13
️✔️eventhubs
️✔️latest
️✔️3.12
️✔️3.13
️✔️feedback
️✔️latest
️✔️3.12
️✔️3.13
️✔️find
️✔️latest
️✔️3.12
️✔️3.13
️✔️hdinsight
️✔️latest
️✔️3.12
️✔️3.13
️✔️identity
️✔️latest
️✔️3.12
️✔️3.13
️✔️iot
️✔️latest
️✔️3.12
️✔️3.13
️✔️keyvault
️✔️latest
️✔️3.12
️✔️3.13
️✔️lab
️✔️latest
️✔️3.12
️✔️3.13
️✔️managedservices
️✔️latest
️✔️3.12
️✔️3.13
️✔️maps
️✔️latest
️✔️3.12
️✔️3.13
️✔️marketplaceordering
️✔️latest
️✔️3.12
️✔️3.13
️✔️monitor
️✔️latest
️✔️3.12
️✔️3.13
️✔️mysql
️✔️latest
️✔️3.12
️✔️3.13
️✔️netappfiles
️✔️latest
️✔️3.12
️✔️3.13
️✔️network
️✔️latest
️✔️3.12
️✔️3.13
️✔️policyinsights
️✔️latest
️✔️3.12
️✔️3.13
️✔️privatedns
️✔️latest
️✔️3.12
️✔️3.13
️✔️profile
️✔️latest
️✔️3.12
️✔️3.13
️✔️rdbms
️✔️latest
️✔️3.12
️✔️3.13
️✔️redis
️✔️latest
️✔️3.12
️✔️3.13
️✔️relay
️✔️latest
️✔️3.12
️✔️3.13
️✔️resource
️✔️latest
️✔️3.12
️✔️3.13
️✔️role
️✔️latest
️✔️3.12
️✔️3.13
️✔️search
️✔️latest
️✔️3.12
️✔️3.13
️✔️security
️✔️latest
️✔️3.12
️✔️3.13
️✔️servicebus
️✔️latest
️✔️3.12
️✔️3.13
️✔️serviceconnector
️✔️latest
️✔️3.12
️✔️3.13
️✔️servicefabric
️✔️latest
️✔️3.12
️✔️3.13
️✔️signalr
️✔️latest
️✔️3.12
️✔️3.13
️✔️sql
️✔️latest
️✔️3.12
️✔️3.13
️✔️sqlvm
️✔️latest
️✔️3.12
️✔️3.13
️✔️storage
️✔️latest
️✔️3.12
️✔️3.13
️✔️synapse
️✔️latest
️✔️3.12
️✔️3.13
️✔️telemetry
️✔️latest
️✔️3.12
️✔️3.13
️✔️util
️✔️latest
️✔️3.12
️✔️3.13
️✔️vm
️✔️latest
️✔️3.12
️✔️3.13

@azure-client-tools-bot-prd
Copy link
Copy Markdown

azure-client-tools-bot-prd bot commented Oct 17, 2025

⚠️AzureCLI-BreakingChangeTest
⚠️hdinsight
rule cmd_name rule_message suggest_message
⚠️ 1006 - ParaAdd hdinsight create cmd hdinsight create added parameter entra_user_full_info
⚠️ 1006 - ParaAdd hdinsight create cmd hdinsight create added parameter entra_user_identity
⚠️ 1011 - SubgroupAdd hdinsight credentials sub group hdinsight credentials added

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Oct 17, 2025

Thank you for your contribution! We will review the pull request and get back to you soon.

@github-actions
Copy link
Copy Markdown

The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR.

Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions).
After that please run the following commands to enable git hooks:

pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>

Copy link
Copy Markdown
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

Updates the HDInsight module to use a new API version (2025-01-15-preview) and adds Entra ID user authentication support for cluster creation and credential management.

  • Updates azure-mgmt-hdinsight dependency from 9.0.0b3 to 9.1.0b2
  • Adds new utility functions for Entra ID user validation and WASB storage detection
  • Introduces new command group for credential management with show and update operations

Reviewed Changes

Copilot reviewed 12 out of 37 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
setup.py and requirements files Updates HDInsight SDK dependency version across platforms
util.py Adds utility functions for storage account type detection and Entra user information processing
custom.py Implements cluster creation with Entra authentication and credential update functionality
commands.py Registers new credential management command group
_params.py and _help.py Defines parameters and help documentation for Entra authentication features
test files Updates test cases with new authentication scenarios and revised expectations

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/azure-cli/azure/cli/command_modules/hdinsight/custom.py
Comment thread src/azure-cli/azure/cli/command_modules/hdinsight/_help.py
Comment thread src/azure-cli/azure/cli/command_modules/hdinsight/_help.py
Comment thread src/azure-cli/azure/cli/command_modules/hdinsight/commands.py Outdated
@2258093686 2258093686 marked this pull request as draft October 17, 2025 02:04
@yonzhan yonzhan requested a review from jsntcy October 17, 2025 03:28
@2258093686 2258093686 changed the title {HDInsight} new api version 2025-01-15-preview [HDInsight] Update hdinsight api-version to 2025-01-15-preview Oct 20, 2025
@2258093686 2258093686 marked this pull request as ready for review October 21, 2025 07:00
@kairu-ms
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@2258093686
Copy link
Copy Markdown
Member Author

The PR description has been modified according to the error prompts.

@2258093686
Copy link
Copy Markdown
Member Author

Hi, @kairu-ms , Could you please trigger /azp run again to rerun the checks?

@kairu-ms
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@kairu-ms kairu-ms merged commit ecf19d1 into Azure:dev Oct 22, 2025
48 checks passed
@bebound bebound mentioned this pull request Oct 28, 2025
@bebound
Copy link
Copy Markdown
Contributor

bebound commented Oct 28, 2025

Please create a replacer to mask the credential in PR: #32340

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.

7 participants