Fix linting messages

- fix truthy values to proper booleans
- fix too long lines: wrapped
- fix indentation
- fix amount of preceding whitespace
- fix argument_spec syntax: borg_retention_policy as type dict with specific options
This commit is contained in:
Max Fuxjäger 2025-04-28 10:47:35 +02:00 committed by Manu
parent 320975170d
commit e2f8e75155
4 changed files with 44 additions and 39 deletions

View File

@ -25,7 +25,7 @@ argument_specs:
borg_venv_path: borg_venv_path:
type: str type: str
required: false required: false
description: Path to store the venv for borg(backup) and borgmatic. description: Path to store the venv for borg(backup) and borgmatic.
borg_install_method: borg_install_method:
type: str type: str
required: false required: false
@ -115,34 +115,37 @@ argument_specs:
type: bool type: bool
description: Bypass Borg error about a repository that has been moved. description: Bypass Borg error about a repository that has been moved.
borg_retention_policy: borg_retention_policy:
keep_secondly: type: dict
type: int description: Define the pruning policy
required: false options:
description: Number of secondly archives to keep. keep_secondly:
keep_minutely: type: int
type: int required: false
required: false description: Number of secondly archives to keep.
description: Number of minutely archives to keep. keep_minutely:
keep_hourly: type: int
type: int required: false
required: false description: Number of minutely archives to keep.
description: Number of hourly archives to keep. keep_hourly:
keep_daily: type: int
type: int required: false
required: false description: Number of hourly archives to keep.
description: Number of daily archives to keep. keep_daily:
keep_weekly: type: int
type: int required: false
required: false description: Number of daily archives to keep.
description: Number of weekly archives to keep. keep_weekly:
keep_monthly: type: int
type: int required: false
required: false description: Number of weekly archives to keep.
description: Number of monthly archives to keep. keep_monthly:
keep_yearly: type: int
type: int required: false
required: false description: Number of monthly archives to keep.
description: Number of yearly archives to keep. keep_yearly:
type: int
required: false
description: Number of yearly archives to keep.
borgmatic_checks: borgmatic_checks:
type: list type: list
required: false required: false

View File

@ -26,6 +26,6 @@
user: "{{ borg_user }}" user: "{{ borg_user }}"
cron_file: "{{ borgmatic_timer_cron_name }}" cron_file: "{{ borgmatic_timer_cron_name }}"
name: PATH name: PATH
env: yes env: true
value: /usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin value: /usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin
... ...

View File

@ -61,5 +61,7 @@
- name: Show hints - name: Show hints
when: "'backup_init_repo' not in ansible_run_tags" when: "'backup_init_repo' not in ansible_run_tags"
ansible.builtin.debug: ansible.builtin.debug:
msg: "Attention: Since the repo was not initialized automatically, the systemd service (borgmatic.service) and the timer (borgmatic.timer) are not activated." msg: >-
Attention: Since the repo was not initialized automatically,
the systemd service (borgmatic.service) and the timer (borgmatic.timer) are not activated.
... ...

View File

@ -4,14 +4,14 @@
- name: Check if EPEL repo is enabled, if installation from distro is requested - name: Check if EPEL repo is enabled, if installation from distro is requested
when: borg_require_epel when: borg_require_epel
block: block:
- name: Get list of installed packages - name: Get list of installed packages
ansible.builtin.package_facts: ansible.builtin.package_facts:
manager: auto manager: auto
- name: Ensure EPEL is enabled - name: Ensure EPEL is enabled
ansible.builtin.assert: ansible.builtin.assert:
that: that:
- "'epel-release' in ansible_facts.packages" - "'epel-release' in ansible_facts.packages"
fail_msg: Need EPEL repo to install via distro package. fail_msg: Need EPEL repo to install via distro package.
- name: Install borgmatic and borg via distribution package manager - name: Install borgmatic and borg via distribution package manager
ansible.builtin.package: ansible.builtin.package: