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

dird: deprecate client Autoprune, JobRetention, and FileRetention #1425

Conversation

alaaeddineelamri
Copy link
Contributor

@alaaeddineelamri alaaeddineelamri commented Mar 28, 2023

Thank you for contributing to the Bareos Project!

Description

This PR removes client Autoprune, JobRetention, and FileRetention from the client resource in the Director configuration.

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
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

@sebsura
Copy link
Contributor

sebsura commented Apr 6, 2023

There are still references in VolumeManagement.rst.inc; e.g.:

The tools Bareos gives you to help automatically manage these problems are the following:
[File Retention (Dir->Client)]: catalog file record retention period.
[Job Retention (Dir->Client)]: catalog job record retention period.
[Auto Prune (Dir->Client)] = yes: permit the application of the above two retention periods.

Do you think we should get rid of these mentions as well to discourage further use ?

@alaaeddineelamri alaaeddineelamri force-pushed the dev/alaaeddineelamri/master/s5411-deprecate-client-autoprune branch from 095ffd9 to 4b5c179 Compare April 6, 2023 11:04
sebsura

This comment was marked as resolved.

@pstorz pstorz force-pushed the dev/alaaeddineelamri/master/s5411-deprecate-client-autoprune branch from 812ed2c to df59caa Compare April 11, 2023 10:15
@pstorz pstorz merged commit 8b46f63 into bareos:master Apr 11, 2023
@Ropnuk
Copy link

Ropnuk commented Dec 13, 2023

Hello.
how to manage the duration of storage of backups and information about them?
In Dir->pool config? Or another way?
We tried to configure "Auto Prune" parameter in pool config file, but in bconsole I don't see this parameter.

@sebsura
Copy link
Contributor

sebsura commented Dec 13, 2023

What did you enter into bconsole and what do you expect it to print ?
And to your question: yes, you should use pool->autoprune for your autopruning needs, see here: https://docs.bareos.org/Configuration/Director.html#config-Dir_Pool_AutoPrune

@sebsura
Copy link
Contributor

sebsura commented Dec 13, 2023

The best place to ask questions like these is in the user mailing list; see here https://www.bareos.com/community/join-in/ for details.

@Ropnuk
Copy link

Ropnuk commented Dec 13, 2023

In .conf file I see this:
Pool_conf
In bconsole I see this
Pool_bconsole
In bconsole there are no information about autoprune parameter.

@arogge
Copy link
Member

arogge commented Dec 13, 2023

Pool->AutoPrune is not Client->AutoPrune so this is totally unrelated to this PR.
In other news: Pool->AutoPrune = yes is the default, thus it is omitted on show pool.

@Ropnuk
Copy link

Ropnuk commented Dec 14, 2023

I understand.
Thank you for help

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.

None yet

5 participants