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.

Sign up for GitHub

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

Jump to bottom

Improve systemd units #498

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Improve systemd units #498

wants to merge 3 commits into from

Commits on Nov 12, 2022

  1. contrib: systemd: btrbk.timer: run hourly 

    btrbk supports hourly backups, so it makes sense for the timer to run hourly,
too.

Signed-off-by: Christoph Anton Mitterer <[email protected]>
    @calestyo
    calestyo committed Nov 12, 2022
    Configuration menu
    Copy the full SHA
    87eeefa View commit details
    Browse the repository at this point in the history

  2. contrib: systemd: run units only with existing config file 

    It does not make sense for the timer or the service to be started (the latter
when done so manually) when neither of the two default config files exists.

Signed-off-by: Christoph Anton Mitterer <[email protected]>
    @calestyo
    calestyo committed Nov 12, 2022
    Configuration menu
    Copy the full SHA
    c9d95a6 View commit details
    Browse the repository at this point in the history

  3. contrib: systemd: btrbk.timer: use default systemd AccuracySec 

    systemd’s `AccuracySec` is mainly there in order to allow it to coalescing
wake-ups for multiple timers.
It’s default of `1min` minute should be enough for this.

Using its `RandomizedDelaySec` in order to spread load from e.g. multiple
sources that would all perform backups to one target, wouldn’t really make much
sense either.
Either, the value would need to be quite large, thereby making backups/snapshots
too wobbly, or it wouldn’t be effective as at least the backups typically take
quite some time.

Signed-off-by: Christoph Anton Mitterer <[email protected]>
    @calestyo
    calestyo committed Nov 12, 2022
    Configuration menu
    Copy the full SHA
    eb1d9e4 View commit details
    Browse the repository at this point in the history