Skip to content

Replace Compute local swagger with remote swagger#29101

Merged
vidai-msft merged 5 commits intoAzure:mainfrom
audreyttt:audreyt-compute-remote-swagger
Jan 27, 2026
Merged

Replace Compute local swagger with remote swagger#29101
vidai-msft merged 5 commits intoAzure:mainfrom
audreyttt:audreyt-compute-remote-swagger

Conversation

@audreyttt
Copy link
Member

Description

Compute has been generating its SDK in PowerShell using local swagger. After migrating to autorest.powershell we can now use remote swagger without the issues we were having previously.

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.

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 Compute SDK to use remote swagger files instead of local swagger files, enabled by the autorest.powershell migration. The changes consist primarily of:

  • Removing the local common.json swagger file
  • Regenerating the SDK with updated API versions and enhanced validation
  • Adding new models and operations for Gallery Scripts functionality
  • Updating API versions from 2024-03-03 to 2025-03-03 for gallery-related operations

Changes:

  • Removed local swagger file (common.json) as the SDK now uses remote swagger
  • Regenerated SDK code with additional validation (subscription ID min length checks)
  • Updated API versions for SharedGallery and CommunityGallery operations (2024-03-03 → 2025-03-03)
  • Added new Gallery Scripts and Gallery Script Versions operations and models

Reviewed changes

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

Show a summary per file
File Description
src/Compute/Compute.Management.Sdk/Rest-api-specs/common.json Removed local swagger file (no longer needed with remote swagger)
src/Compute/Compute/ChangeLog.md Updated with changelog entry describing the migration
Multiple *Operations.cs files Added SubscriptionId min length validation across all operations
SharedGallery/CommunityGallery operations Updated API versions from 2024-03-03 to 2025-03-03
Multiple Model files Updated documentation comments for resource properties (id, name, type, location, tags)
New Gallery Script models Added complete Gallery Scripts functionality with new models and operations
Operations.cs and related files Fixed return type from object to Operation
Various model files Added new properties (storageAccountStrategy, enableFips1403Encryption, instantAccess, etc.)

Copilot AI review requested due to automatic review settings January 26, 2026 22:06
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 6 out of 142 changed files in this pull request and generated no new comments.

@audreyttt
Copy link
Member Author

Tests in record mode have been run across all core Compute resources (Disk, Gallery, VMSS, VM) and all are still passing post-migration.

Copy link
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

@vidai-msft vidai-msft merged commit f0d58de into Azure:main Jan 27, 2026
17 of 18 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.

3 participants