DiskRP Swagger — Add snapshotAccessState to DiskRestorePointProperties for InstantAccess - #42620
Merged
Haider Agha (haagha) merged 1 commit intoMay 6, 2026
Conversation
Added snapshotAccessState property to DiskRestorePointProperties in TypeSpec and generated OpenAPI spec for API version 2026-03-02. This property already exists on SnapshotProperties. The SnapshotAccessState enum already exists in common/models.tsp — no new types needed. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Akash Pargat (akashpargat)
requested review from
Audrey Toney (audreyttt),
Haider Agha (haagha) and
Ojo Okoka (ookoka)
as code owners
April 23, 2026 23:27
Next Steps to MergeNext steps that must be taken to merge this PR:
Comment generated by summarize-checks workflow run. |
API Change CheckAPIView identified API level changes in this PR and created the following API reviews
Comment generated by After APIView workflow run. |
Haider Agha (haagha)
added
the
WaitForARMFeedback
Gary Li (gary-x-li)
added
ARMSignedOff
Haider Agha (haagha)
merged commit May 6, 2026
a3930bc
into
Azure:compute-2026-03-02
127 of 130 checks passed
9 tasks
Haider Agha (haagha)
added a commit
that referenced
this pull request
Jun 22, 2026
* Add Disk RP api-version 2026-03-02 * Fix broken common-types paths in DiskRP.json (2026-03-02) The DiskRP.json swagger had incorrect relative paths using '../../../common-types/v1/common.json' (3 levels up) which resolves to a non-existent 'Microsoft.Compute/common-types/' path. Fixed to use '../../common-types/v1/common.json' (2 levels up) matching the correct 'Compute/common-types/' location, consistent with other DiskRP versions. * Update DiskRP.json * fix: update package tags and input files for Compute API specifications * Add snapshotAccessState to DiskRestorePointProperties for InstantAccess (#42620) (ARM Approved) Added snapshotAccessState property to DiskRestorePointProperties in TypeSpec and generated OpenAPI spec for API version 2026-03-02. This property already exists on SnapshotProperties. The SnapshotAccessState enum already exists in common/models.tsp — no new types needed. Co-authored-by: NORTHAMERICA\akashpargat <akashpargat@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Swagger changes for ConfidentialVMVersion (#42541) ( ARMSignedOff) * swagger changes for confidentialVMVersion * addressing comments * DiskRP Swagger changes -- adding two APIs support for immutable snapshot -- for 2026-03-02 (#42025) * v2 * v2 * v3 * v3 snapshot and compile * V3 fix1 * v4 fix 2 * Suppress ResourceNameRestriction for snapshot updateImmutabilityPolicy paths in DiskRP.json * Remove ineffective where: selector from snapshot ResourceNameRestriction suppression --------- Co-authored-by: Rui Zhan <rzhan@microsoft.com> Co-authored-by: Haider Agha <64601174+haagha@users.noreply.github.com> Co-authored-by: Haider Agha <Haider.Agha@microsoft.com> * Address ARM API review feedback for snapshot immutability policy Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Update immutability policy examples with response fields and async headers Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Refactor immutability policy documentation and examples for snapshots --------- Co-authored-by: Haider Agha <Haider.Agha@microsoft.com> Co-authored-by: Tong Xu (Wicresoft North America Ltd) <v-xuto@microsoft.com> Co-authored-by: Akash Pargat <akashpargat@yahoo.com> Co-authored-by: NORTHAMERICA\akashpargat <akashpargat@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: aayushkher <153165259+aayushkher@users.noreply.github.com> Co-authored-by: zhanrz88-sudo <zhanrz88@gmail.com> Co-authored-by: Rui Zhan <rzhan@microsoft.com>
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
5 participants
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.
ARM (Control Plane) API Specification Update Pull Request
Purpose of this PR
Due diligence checklist
Summary
Adds \snapshotAccessState\ (read-only) to \DiskRestorePointProperties\ for the InstantAccess Restore Point feature. This property already exists on \SnapshotProperties\ and uses the existing \SnapshotAccessState\ enum from \common/models.tsp. The DiskRP backend already computes and returns this field.
Changes