Skip to content

[PS] migrate MachineLearningServices module to autorest v4#28818

Open
JoyerJin wants to merge 11 commits intomainfrom
joyer/machinelearningservices-migrate-v4
Open

[PS] migrate MachineLearningServices module to autorest v4#28818
JoyerJin wants to merge 11 commits intomainfrom
joyer/machinelearningservices-migrate-v4

Conversation

@JoyerJin
Copy link
Contributor

@JoyerJin JoyerJin commented Nov 6, 2025

Description

Preannouncement PR: #28844

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

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

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

@JoyerJin JoyerJin marked this pull request as ready for review November 7, 2025 10:00
Copilot AI review requested due to automatic review settings November 7, 2025 10:00
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 migrates the MachineLearningServices module to AutoRest v4, which involves regenerating documentation and updating API references across numerous cmdlet help files. The changes primarily reflect the AutoRest v4 code generation updates, including parameter type changes from strongly-typed enums to strings and modifications to cmdlet examples and output formatting.

Key Changes:

  • Updated all cmdlet documentation files to reflect AutoRest v4 API schema changes
  • Converted strongly-typed enum parameters to string types across all cmdlets
  • Added new parameter sets and parameters for improved cmdlet flexibility (e.g., WorkspaceInputObject, JsonFilePath, JsonString)

Reviewed Changes

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

File Description
All Update-*.md files Updated parameter types from enums to strings, removed "To construct" notes, added new parameter sets
All New-*.md files Updated parameter types, added identity-based and JSON-based parameter sets, improved examples
All Remove-*.md files Added new identity-based parameter sets for improved pipeline support
All Start/Stop/Restart-*.md files Added workspace identity parameter sets for better workflow integration

@Pan-Qi Pan-Qi marked this pull request as draft January 8, 2026 06:19
@Pan-Qi Pan-Qi added Contains Breaking Change This PR contains breaking change autorest v4 migration pr migrating module from generated by autorest.powershell v3 to v4 labels Jan 9, 2026
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

To the author of the pull request,
This PR was labeled "Contains Breaking Change" because breaking changes have been detected by the static analysis pipeline.

  • According to our policy, breaking changes can only take place during major release and they must be preannounced.
  • Please follow our guide on the detailed steps.
  • Required: Please fill in the task below to facilitate our contact,you will receive notifications related to breaking changes.

@isra-fel
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

Copilot AI review requested due to automatic review settings January 14, 2026 04:01
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

Copilot reviewed 212 out of 466 changed files in this pull request and generated 1 comment.

SystemDataLastModifiedByType : User
Tag : {
}
Type : M
Copy link

Copilot AI Jan 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Type property value appears to be truncated to just 'M'. This should be the full resource type string 'Microsoft.MachineLearningServices/workspaces/models' as shown in other examples in the same file.

Copilot uses AI. Check for mistakes.
@isra-fel
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@isra-fel
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@JiaSeng-v JiaSeng-v marked this pull request as ready for review January 14, 2026 05:47
Copilot AI review requested due to automatic review settings January 14, 2026 05:48
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

Copilot reviewed 212 out of 467 changed files in this pull request and generated no new comments.

@isra-fel isra-fel added this to the Az 16.0.0 (TBD) milestone Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autorest v4 migration pr migrating module from generated by autorest.powershell v3 to v4 Contains Breaking Change This PR contains breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants