Skip to content

Conversation

@ArthurMa1978
Copy link
Member

Fixes #53906

Contributing to the Azure SDK

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see this section.

Copilot AI review requested due to automatic review settings December 20, 2025 05:42
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 PR migrates the WorkloadOrchestration SDK from the old Autorest-based generator to the new Management Plane Generator (MPG) based on TypeSpec. The changes update attribute syntax, remove obsolete dependencies, and add custom model files required for the new generator.

  • Updates CodeGenModel attributes to CodeGenType with fully qualified namespace
  • Removes Azure.Core using directive and IncludeAutorestDependency project property
  • Adds custom base classes and model factory methods required by the new generator

Reviewed changes

Copilot reviewed 7 out of 484 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
EdgeTargetPatchProperties.cs Updates code generation attribute to use new MPG syntax and removes unused Azure.Core import
EdgeSolutionTemplatePatchProperties.cs Updates code generation attribute to use new MPG syntax and removes unused Azure.Core import
EdgeJobStepStatistics.cs Adds new abstract base class for job step statistics with protected constructor
EdgeJobContent.cs Adds new abstract base class for job content parameters with protected constructor
EdgeContextPatchProperties.cs Updates code generation attribute to use new MPG syntax and removes unused Azure.Core import
ArmWorkloadOrchestrationModelFactory.cs Adds model factory with helper methods for creating EdgeSolutionTemplateProperties and EdgeConfigTemplateProperties
Azure.ResourceManager.WorkloadOrchestration.csproj Removes obsolete IncludeAutorestDependency property

@github-actions
Copy link

github-actions bot commented Dec 20, 2025

API Change Check

APIView identified API level changes in this PR and created the following API reviews

Azure.ResourceManager.WorkloadOrchestration

@ArthurMa1978 ArthurMa1978 merged commit 52b71eb into main Dec 22, 2025
21 checks passed
@ArthurMa1978 ArthurMa1978 deleted the mgmt-workloadorchestration branch December 22, 2025 08:06
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.

MPG migration - workloadorchestration

3 participants