[Policy] SDK validation for TypeSpec swagger#54693
Draft
zedy-wj wants to merge 5 commits intoAzure:mainfrom
Draft
[Policy] SDK validation for TypeSpec swagger#54693zedy-wj wants to merge 5 commits intoAzure:mainfrom
zedy-wj wants to merge 5 commits intoAzure:mainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Azure Policy SDK to validate against a new TypeSpec swagger specification. The primary changes include updating the commit ID and correcting input file paths in the autorest configuration, along with regenerating SDK code to use the new 2025-03-01 API version instead of the previous 2021-06-01/2022-06-01 versions.
Key Changes:
- Updated autorest.md to reference new TypeSpec-based OpenAPI specifications (2025-03-01 policy API version)
- Consolidated multiple policy-related input files into a single openapi.json file
- Added support for policy definition/set definition versions and policy tokens
- Regenerated all sample files and resource classes with updated API versions and file paths
Reviewed changes
Copilot reviewed 16 out of 145 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| autorest.md | Updated input files to use consolidated 2025-03-01 openapi.json and new commit ID; added skipped operations and request path mappings |
| Sample files (multiple) | Updated example definition paths from 2021-06-01/2022-06-01 to 2025-03-01; added new sample methods for policy versions and tokens |
| TenantResource.cs | Added new methods for listing built-in policy definition/set definition versions |
| SubscriptionResource.cs | Added methods for policy version listing and policy token acquisition |
| PolicySetDefinition/PolicyDefinition resources | Added expand parameter and version collection support; updated API versions |
| New version resources | Added new resource types for managing policy definition and set definition versions |
| PolicyAssignmentData.cs | Updated base class reference to use fully qualified name |
Generate sdk code from swagger fix variable not exist error fix some validation error fix some validation error Generate sdk code from swagger fix some validation error fix some validation error fix and run sample tests export api regenerate by latest commit update some api info
zedy-wj
force-pushed
the
wenjie-dev/policy
branch
from
7b3c9ee to
653421f
Compare
Member
|
We don't need to merge this PR because Policy will be released by a new namespace |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is the SDK validation for Azure Policy.
TSP link: Azure/azure-rest-api-specs#38509.
Changes:
PolicyAssignments_ListForResourcebecause duplicated methods were generated in the samples, issues: SDK Validation - Policy #54960.