Skip to content

docs: mssql: after PITR restore, a full backup is required#1235

Merged
pstorz merged 2 commits intobareos:masterfrom
bruno-at-bareos:dev/bruno/docs-mssql-pitr-restore-need-full-backup-afterwards
Aug 30, 2022
Merged

docs: mssql: after PITR restore, a full backup is required#1235
pstorz merged 2 commits intobareos:masterfrom
bruno-at-bareos:dev/bruno/docs-mssql-pitr-restore-need-full-backup-afterwards

Conversation

@bruno-at-bareos
Copy link
Contributor

@bruno-at-bareos bruno-at-bareos commented Aug 23, 2022 •
edited
Loading

This PR will improve the appendix/howto 3rd database MSSQL.
In case of PITR the backup chain is broken, so a new full or diff backup is needed afterwards.

OP#4943

Please check

  • Short description and the purpose of this PR is present above this paragraph
  • Your name is present in the AUTHORS file (optional)

If you have any questions or problems, please give a comment in the PR.

Helpful documentation and best practices

Checklist for the reviewer of the PR (will be processed by the Bareos team)

General
  • PR name is meaningful
  • Purpose of the PR is understood
  • Separate commit for this PR in the CHANGELOG.md, PR number referenced is same
  • Commit descriptions are understandable and well formatted
  • If backport: add original PR number and target branch at top of this file: Backport of PR#000 to bareos-2x
Source code quality
  • Source code changes are understandable
  • Variable and function names are meaningful
  • Code comments are correct (logically and spelling)
  • Required documentation changes are present and part of the PR
  • bareos-check-sources --since-merge does not report any problems
  • git status should not report modifications in the source tree after building and testing
Tests
  • Decision taken that a test is required (if not, then remove this paragraph)
  • The choice of the type of test (unit test or systemtest) is reasonable
  • Testname matches exactly what is being tested
  • On a fail, output of the test leads quickly to the origin of the fault

@bruno-at-bareos bruno-at-bareos added documentation This PR changes only documentation requires backport to 21 labels Aug 23, 2022
@bruno-at-bareos bruno-at-bareos requested a review from pstorz August 25, 2022 11:21
@pstorz pstorz changed the title docs: mssql pitr restore need full backup afterwards docs: mssql: after PITR restore, a full backup is required Aug 29, 2022
pstorz
pstorz requested changes Aug 29, 2022
View reviewed changes
Copy link
Member

@pstorz pstorz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments

@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/docs-mssql-pitr-restore-need-full-backup-afterwards branch from 8406bd8 to b0fedfe Compare August 29, 2022 13:37
@bruno-at-bareos bruno-at-bareos requested a review from pstorz August 29, 2022 13:37
bruno-at-bareos and others added 2 commits August 30, 2022 18:57
@bruno-at-bareos @pstorz
docs: Appendix/Howto mssql pitr warning
3e502a7 
- add a warning needed full or diff after a pitr.

Signed-off-by: Bruno Friedmann <bruno.friedmann@bareos.com>
Co-authored-by: Philipp Storz <philipp.storz@bareos.com>
@bruno-at-bareos @pstorz
update CHANGELOG.md for PR#1235
6a80f37 
Signed-off-by: Bruno Friedmann <bruno.friedmann@bareos.com>
@pstorz pstorz force-pushed the dev/bruno/docs-mssql-pitr-restore-need-full-backup-afterwards branch from b0fedfe to 6a80f37 Compare August 30, 2022 16:57
@pstorz pstorz merged commit b34f2d1 into bareos:master Aug 30, 2022
@bruno-at-bareos bruno-at-bareos mentioned this pull request Aug 31, 2022
Merged
13 tasks
@bruno-at-bareos bruno-at-bareos deleted the dev/bruno/docs-mssql-pitr-restore-need-full-backup-afterwards branch November 15, 2022 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pstorz pstorz pstorz approved these changes

Assignees

@pstorz pstorz

Labels

documentation This PR changes only documentation requires backport to 21

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bruno-at-bareos @pstorz