Skip to content

docs: catalog maintenance improvements#1379

Merged
pstorz merged 3 commits intobareos:masterfrom
bruno-at-bareos:dev/bruno/master/docs-catalog-maintenance-improvements
Mar 2, 2023
Merged

docs: catalog maintenance improvements#1379
pstorz merged 3 commits intobareos:masterfrom
bruno-at-bareos:dev/bruno/master/docs-catalog-maintenance-improvements

Conversation

@bruno-at-bareos
Copy link
Contributor

@bruno-at-bareos bruno-at-bareos commented Feb 14, 2023

This PR aims to improve our catalog maintenance chapter with

  • document free space needs
  • document how to provide postgresql.service with PGDG package
  • document how to edit PGDG postgresqlXX.service to provide an alias for bareos-dir.service

OP#5339 OP#5390

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)

Make sure you check/merge the PR using devtools/pr-tool to have some simple automated checks run and a proper changelog record added.

General
  • Is the PR title usable as CHANGELOG entry?
  • Purpose of the PR is understood
  • Commit descriptions are understandable and well formatted
  • Check backport line

@bruno-at-bareos bruno-at-bareos added documentation This PR changes only documentation requires backport to 22 labels Feb 14, 2023
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.

I think the whole paragraph "Setting Retention Periods" can be removed, as we have disabled the job and file retention by default, and do not suggest to use it at all.

We can talk directly about my remarks!

@bruno-at-bareos
Copy link
Contributor Author

@pstorz the retention section has been removed and your requests have been included now, so that PR is ready for review, whatever kind of flue J has.

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.

Good work, thank you!

@pstorz pstorz self-requested a review February 28, 2023 09:36
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.

Good work, please see small comments.

@bruno-at-bareos
Copy link
Contributor Author

Suggestions applied, and one reworded.

@pstorz pstorz force-pushed the dev/bruno/master/docs-catalog-maintenance-improvements branch from b79b14c to 46c1c0f Compare March 2, 2023 17:29
@pstorz pstorz mentioned this pull request Mar 2, 2023
10 tasks
bruno-at-bareos and others added 3 commits March 2, 2023 18:35
- document free space needs
- document how to provide postgresql.service with PGDG package
- document how to require after PGDG postgresql for bareos-dir.service

Signed-off-by: Bruno Friedmann <bruno.friedmann@bareos.com>
Signed-off-by: Bruno Friedmann <bruno.friedmann@bareos.com>
@pstorz pstorz force-pushed the dev/bruno/master/docs-catalog-maintenance-improvements branch from e8f24c2 to 5bc274f Compare March 2, 2023 17:35
@pstorz pstorz merged commit a1a7797 into bareos:master Mar 2, 2023
@bruno-at-bareos bruno-at-bareos deleted the dev/bruno/master/docs-catalog-maintenance-improvements branch June 12, 2023 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation This PR changes only documentation requires backport to 22

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants