Skip to content

setting the LRO response's FinalResult and using clientName to fix naming issues - #47153

Open
iwrinshow wants to merge 10 commits into
Azure:mainfrom
iwrinshow:fixesforsdk
Open

iwrinshow wants to merge 10 commits into
Azure:mainfrom
iwrinshow:fixesforsdk

Conversation

@iwrinshow

Copy link
Copy Markdown
Member

Choose a PR Template

Switch to "Preview" on this description then select one of the choices below.

Click here to open a PR for a Data Plane API.

Click here to open a PR for a Control Plane (ARM) API.

Click here to open a PR for only SDK configuration.

@github-actions

github-actions Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ This PR is in purview of the ARM review (label: ARMReview). This PR must get ARMSignedOff label from an ARM reviewer.
    This PR is not ready for ARM review (label: NotReadyForARMReview). This PR will not be reviewed by ARM until relevant problems are fixed. Consult the rest of this Next Steps to Merge comment for details.
    Once the blocking problems are addressed, add to the PR a comment with contents /azp run. Automation will re-evaluate this PR and if everything looks good, it will add WaitForARMFeedback label which will put this PR on the ARM review queue.
    For details of the ARM review, see aka.ms/azsdk/pr-arm-review
  • ❌ This PR is NotReadyForARMReview because it has the BreakingChangeReviewRequired label.
  • ❌ This PR has at least one breaking change (label: BreakingChangeReviewRequired).
    To unblock this PR, follow the process at aka.ms/brch.
  • ❌ The required check named Swagger BreakingChange has failed. To unblock this PR, follow the process at aka.ms/brch.


Comment generated by summarize-checks workflow run.

@github-actions

github-actions Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Contributor readiness: required access

Warning

The access issues below can prevent PR approvals from counting or block Azure DevOps pipeline runs.

User / area Access issue PR impact
🟡 iwshow@microsoft.com Author/committer account unavailable; add or verify this email on a GitHub account. Affects 8 commits: 7d38560, a45af1c, 69f3403, 3792d5b, bf28b5b, e319b64, 68413ce, 8f46af7. Not determined.
🟡 singhprab@microsoft.com Author/committer account unavailable; add or verify this email on a GitHub account. Affects 1 commit: 0775253. Not determined.

Internal contributors: setup / renew required access. Recheck: /azsdk check-access.

Contributors with verified access (1)

Public Azure membership and repository write access verified.

User Roles
iwrinshow PR author, commit author

Non-blocking report; external fork contributions are allowed. GitHub review rules still apply.

@github-actions github-actions Bot added ARMReview resource-manager TypeSpec Authored with TypeSpec WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required ARMAutoSignedOff-Test ARMAutoSignedOff-Trivial ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review and removed WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Oct 9, 2026
@github-actions

github-actions Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

API Change Check

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

Language API Review for Package
Go sdk/resourcemanager/recoveryservices/armrecoveryservicesbackup
JavaScript @azure/arm-recoveryservicesbackup
Python azure-mgmt-recoveryservicesbackup
Java com.azure.resourcemanager:azure-resourcemanager-recoveryservicesbackup
Swagger Microsoft.RecoveryServices-RecoveryServicesBackup

Comment generated by After APIView workflow run.

@github-actions github-actions Bot added BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required and removed ARMAutoSignedOff-Test labels Oct 12, 2026
@github-actions github-actions Bot added NotReadyForARMReview ARMAutoSignedOff-Test and removed ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review labels Oct 12, 2026
Prabhkirat Singh and others added 5 commits October 12, 2026 13:01
…LRO initiator

The operation result endpoint declared ArmLroLocationHeader on its 202
response. That decorator carries @pollingLocation/@finalLocation, which tells
the SDK emitters that the operation *starts* a long running operation. The
result is that the Java emitter generates a poller for the very endpoint that
is the poll target, and because a GET has no final result metadata of its own
the poller is typed PollResult<Void>. RPExtendedInfoResourceListInner is
generated but never referenced, so the feature returns nothing in Java.

The Location header is still declared and still returned, so the REST contract
is unchanged. The only generated swagger difference is the header description.
Polling semantics continue to come from the getRPExtendedInfo POST, which
carries final-state-via: location and final-state-schema.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: c58056c6-6882-48a3-bc36-04b13e2305cc
…tendedinfo-result-lro

Model getRPExtendedInfoOperationResult as a plain GET rather than an LRO initiator
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.

2 participants