Skip to content

Generate Compute SDK using autorest.powershell#28976

Merged
vidai-msft merged 24 commits intoAzure:mainfrom
audreyttt:audrey/autorest-migration
Jan 23, 2026
Merged

Generate Compute SDK using autorest.powershell#28976
vidai-msft merged 24 commits intoAzure:mainfrom
audreyttt:audrey/autorest-migration

Conversation

@audreyttt
Copy link
Copy Markdown
Member

@audreyttt audreyttt commented Dec 12, 2025

Description

Uses autorest powershell to generate SDK and replace the deprecated Track 1 SDK

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

Copilot AI review requested due to automatic review settings December 12, 2025 23:34
@azure-client-tools-bot-prd
Copy link
Copy Markdown

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

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.

Copilot wasn't able to review any files in this pull request.

@audreyttt audreyttt marked this pull request as ready for review December 22, 2025 05:47
Copilot AI review requested due to automatic review settings December 22, 2025 05:47
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

Copilot reviewed 12 out of 848 changed files in this pull request and generated 1 comment.

@github-actions
Copy link
Copy Markdown

This PR was labeled "needs-revision" because it has unresolved review comments or CI failures.
Please resolve all open review comments and make sure all CI checks are green. Refer to our guide to troubleshoot common CI failures.

@isra-fel
Copy link
Copy Markdown
Member

isra-fel commented Jan 9, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

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

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

@audreyttt
Copy link
Copy Markdown
Member Author

Please update the ChangeLog.md

I believe no Changelog edits are needed because we are just updating the SDK generation method and not adding anything new, but let me know if I misunderstood

@vidai-msft
Copy link
Copy Markdown
Contributor

@audreyttt Change log is required because a new package should be released. Additionally, I noticed the casing of some property names has been changed. Can you please confirm if these properties are part of the output objects? If so, it will cause potential breaking changes to the customers if they reference the classes directly using C# code. The recommended way is to use directives to rename the properties.

Copilot AI review requested due to automatic review settings January 13, 2026 16:43
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

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

Copilot AI review requested due to automatic review settings January 13, 2026 18:32
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

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

Copilot AI review requested due to automatic review settings January 20, 2026 05:54
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

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

Copy link
Copy Markdown
Contributor

@vidai-msft vidai-msft left a comment

Choose a reason for hiding this comment

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

LGTM

@audreyttt
Copy link
Copy Markdown
Member Author

All tests are passing in Playback mode post-migration and a few tests per critical resource group (VM, VMSS, Gallery, Disk) were run in Record mode and are confirmed to be working post-migration as well.

@vidai-msft vidai-msft merged commit 070064c into Azure:main Jan 23, 2026
12 checks passed
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.

5 participants