-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[RecoveryServices.Backup]Added support for restoring multiple files/folders in AzureFileShare #11373
[RecoveryServices.Backup]Added support for restoring multiple files/folders in AzureFileShare #11373
Conversation
Can one of the admins verify this patch? |
57a5133
to
06d089c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, please see my inline comments. Besides, could you add an example to restore multiple items? Thanks
...Services/RecoveryServices.Backup/Cmdlets/Restore/RestoreAzureRMRecoveryServicesBackupItem.cs
Outdated
Show resolved
Hide resolved
/azp run azure-powershell - windows-powershell |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Hi @sambitratha , as #11378 was merged, it introduced some conflicts which blocks automatic merging. Could you resolve the conflicts? Thanks |
8f46948
to
35fdf5e
Compare
Hi @isra-fel i have force-pushed the branch with conflicts resolved. Please approve |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM; changing the target branch to release branch
Description
Added support for restoring multiple files/folders in AzureFileShare
Design review: https://github.com/Azure/azure-powershell-cmdlet-review-pr/issues/508
Checklist
CONTRIBUTING.md
ChangeLog.md
file(s) has been updated:ChangeLog.md
file can be found atsrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
## Upcoming Release
header -- no new version header should be added