Skip to content

Conversation

dsapaliga
Copy link
Contributor

@dsapaliga dsapaliga commented Aug 15, 2025

Description

This is a simple fix to remove the validation check for restorable accounts when the customer uses the powershell restore command. Couple of reason for removing this:

Due to the mismatch in location format given as input vs how we validate the permissions.
Avoid timeout during this validation check for large account restores.
This change is already part of stable, updating this for powershell preview now.

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

Copy link

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

@dsapaliga dsapaliga changed the title restorable api check removed PS cosmos db restore command: Skipping client side restore validation check Aug 15, 2025
@dsapaliga dsapaliga requested review from vinhms and kavskalyan August 15, 2025 22:40
@vinhms
Copy link
Contributor

vinhms commented Aug 16, 2025

LGTM

@NoriZC NoriZC self-assigned this Aug 16, 2025
@NoriZC
Copy link
Contributor

NoriZC commented Aug 16, 2025

/azp run azure-powershell - security-tools

Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@NoriZC NoriZC merged commit 9496aa0 into Azure:Az.CosmosDB-preview Aug 18, 2025
12 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.

4 participants