Generate Compute SDK using autorest.powershell

[Audrey Toney (audreyttt)]

Description

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

Mandatory Checklist

• Please choose the target release of Azure PowerShell. (⚠️Target release is a different concept from API readiness. Please click below links for details.)

  • General release
  • Public preview
  • Private preview
  • Engineering build
  • No need for a release

• Check this box to confirm: I have read the Submitting Changes section of CONTRIBUTING.md and reviewed the following information:

• 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]

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

[Copilot]

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

[Copilot]

Pull request overview

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

[github-actions]

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.

[Yeming Liu (isra-fel)]

/azp run

[azure-pipelines]

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]

Pull request overview

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

[Audrey Toney (audreyttt)]

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

[Vincent Dai (vidai-msft)]

Audrey Toney (@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]

Pull request overview

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

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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

[Vincent Dai (vidai-msft)]

LGTM

[Audrey Toney (audreyttt)]

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.