Skip to content
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

[backup] Fix on demand backup #17367

Merged
merged 2 commits into from
Mar 23, 2021
Merged

[backup] Fix on demand backup #17367

merged 2 commits into from
Mar 23, 2021

Conversation

hiaga
Copy link
Member

@hiaga hiaga commented Mar 19, 2021

Description

fixing on demand backups for SQL workloads - modifying when a user should or shouldn't give the value for retain-until

Testing Guide

az backup protection backup-now -v vaultName -g rgName--backup-management-type AzureWorkload --workload-type MSSQL -c "containerName" -i "itemName" --backup-type "CopyOnlyFull/Full" --retain-until 06-04-2021 --debug

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


This checklist is used to make sure that common guidelines for a pull request are followed.

@hiaga hiaga requested a review from fengzhou-msft as a code owner March 19, 2021 11:12
@hiaga hiaga force-pushed the hiaga/SQLBackup branch from 7529702 to 8798c2e Compare March 21, 2021 05:15
@hiaga
Copy link
Member Author

hiaga commented Mar 21, 2021

@yungezz , @fengzhou-msft, @houk-ms : can you please review this PR to fix the on demand backup for SQL and HANA workloads.

.gitignore Outdated Show resolved Hide resolved
@hiaga
Copy link
Member Author

hiaga commented Mar 22, 2021

@qwordy : I have made the change as suggested, pls review and merge.

@qwordy
Copy link
Member

qwordy commented Mar 23, 2021

@qwordy : I have made the change as suggested, pls review and merge.

@fengzhou-msft Code owner review required. Please review and merge.

@hiaga hiaga force-pushed the hiaga/SQLBackup branch from b6e5ba7 to 1bab713 Compare March 23, 2021 02:18
@fengzhou-msft fengzhou-msft changed the title [az backup] fixing on demand backup [backup] fixing on demand backup Mar 23, 2021
@fengzhou-msft fengzhou-msft changed the title [backup] fixing on demand backup [backup] Fix on demand backup Mar 23, 2021
@fengzhou-msft fengzhou-msft merged commit 8fa70cf into Azure:dev Mar 23, 2021
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.

4 participants