Skip to content

ML ext Release 2.40.1 - Adding integration tests for DT, preview feature and command update t…#9408

Merged
kairu-ms merged 4 commits intoAzure:mainfrom
kshitij-microsoft:cliFixMain
Nov 11, 2025
Merged

ML ext Release 2.40.1 - Adding integration tests for DT, preview feature and command update t…#9408
kairu-ms merged 4 commits intoAzure:mainfrom
kshitij-microsoft:cliFixMain

Conversation

@kshitij-microsoft
Copy link
Copy Markdown
Member

…o show


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

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
  • My extension version conforms to the Extension version schema

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Copilot AI review requested due to automatic review settings November 10, 2025 05:57
@azure-client-tools-bot-prd
Copy link
Copy Markdown

azure-client-tools-bot-prd bot commented Nov 10, 2025

❌Azure CLI Extensions Breaking Change Test
❌machinelearningservices
rule cmd_name rule_message suggest_message
1002 - CmdRemove ml deployment-template get cmd ml deployment-template get removed please confirm cmd ml deployment-template get removed
⚠️ 1003 - CmdPropAdd ml deployment-template archive cmd ml deployment-template archive added property is_preview
⚠️ 1003 - CmdPropAdd ml deployment-template create cmd ml deployment-template create added property is_preview
⚠️ 1003 - CmdPropAdd ml deployment-template list cmd ml deployment-template list added property is_preview
⚠️ 1003 - CmdPropAdd ml deployment-template restore cmd ml deployment-template restore added property is_preview
⚠️ 1001 - CmdAdd ml deployment-template show cmd ml deployment-template show added
⚠️ 1003 - CmdPropAdd ml deployment-template update cmd ml deployment-template update added property is_preview

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

Hi @kshitij-microsoft,
Please write the description of changes which can be perceived by customers into HISTORY.rst.
If you want to release a new extension version, please update the version in setup.py as well.

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Nov 10, 2025

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

@github-actions
Copy link
Copy Markdown
Contributor

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>

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 10, 2025

Hi @kshitij-microsoft

⚠️ Release Requirements

Module: machinelearningservices

  • Please log updates into to src/machinelearningservices/HISTORY.rst
  • ⚠️ Please update VERSION to be 3.0.0 in src/machinelearningservices/setup.py

Notes

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

This PR introduces a new deployment template management feature for Azure Machine Learning services, including comprehensive test coverage and CLI command integration. The main changes include renaming the get command to show (aligning with Azure CLI conventions), marking the feature as preview, and adding extensive test scenarios.

Key changes:

  • Renamed az ml deployment-template get to az ml deployment-template show for consistency with Azure CLI standards
  • Added is_preview=True flag to the deployment-template command group
  • Added comprehensive test suite with 12 test scenarios covering CRUD operations and edge cases
  • Added test configuration files and recordings for scenario-based testing

Reviewed Changes

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

Show a summary per file
File Description
manual/commands.py Renamed command from "get" to "show" and marked command group as preview
manual/_params/_deployment_template_params.py Updated argument context from "get" to "show"
manual/_help/_deployment_template_help.py Updated help text to use "show" instead of "get"
manual/custom/deployment_template.py Updated warning message to reference "show" command
tests/latest/test_deployment_template_scenarios.py Added comprehensive test suite with 12 test cases
tests/test_configs/deployment_template/deployment_template_basic.yaml Added test configuration file for deployment templates
tests/latest/recordings/*.yaml Added 12 test recording files for playback testing

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Nov 10, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 2 pipeline(s).

@kshitij-microsoft
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Commenter does not have sufficient privileges for PR 9408 in repo Azure/azure-cli-extensions

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Nov 10, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 2 pipeline(s).

@kshitij-microsoft kshitij-microsoft changed the title [Do not merge] Adding integration tests for DT, preview feature and command update t… Release - Adding integration tests for DT, preview feature and command update t… Nov 11, 2025
@github-actions github-actions bot added the release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. label Nov 11, 2025
@kshitij-microsoft kshitij-microsoft changed the title Release - Adding integration tests for DT, preview feature and command update t… ML ext Release 2.40.1 - Adding integration tests for DT, preview feature and command update t… Nov 11, 2025
@wangzelin007
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 2 pipeline(s).

@wangzelin007 wangzelin007 added skip-cal-version and removed release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. labels Nov 11, 2025
@kairu-ms kairu-ms merged commit 9565ba3 into Azure:main Nov 11, 2025
35 of 36 checks passed
@azclibot
Copy link
Copy Markdown
Collaborator

[Release] Update index.json for extension [ machinelearningservices-2.40.1 ] : https://dev.azure.com/msazure/One/_build/results?buildId=143323779&view=results

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants