From 6b57d23220fdc5c97f3ecd45766aa520cb338ffc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mat=C4=9Bj=20Grabovsk=C3=BD?= Date: Tue, 8 Jun 2021 14:25:34 +0200 Subject: [PATCH] retrace: Fix faf db backup job The job was broken before because the backup directory (`/srv/faf/db-backup`) might not have existed. Also, factor out some magic values into variables. --- roles/abrt/faf-post/defaults/main.yml | 6 ++++++ roles/abrt/faf-post/tasks/main.yml | 9 +++++++++ 2 files changed, 15 insertions(+) diff --git a/roles/abrt/faf-post/defaults/main.yml b/roles/abrt/faf-post/defaults/main.yml index 61d58e18e0..f16bf90d57 100644 --- a/roles/abrt/faf-post/defaults/main.yml +++ b/roles/abrt/faf-post/defaults/main.yml @@ -4,3 +4,9 @@ archive_age: '+14' # Delete deferred reports deferred_age: '+7' + +# Directory for saving FAF database backups. +faf_backup_dir: /srv/faf/db-backup + +# Delete FAF database backups older than this period. +faf_old_backup_age: '+7' diff --git a/roles/abrt/faf-post/tasks/main.yml b/roles/abrt/faf-post/tasks/main.yml index 6fe73db7ab..167758208c 100644 --- a/roles/abrt/faf-post/tasks/main.yml +++ b/roles/abrt/faf-post/tasks/main.yml @@ -16,6 +16,15 @@ tags: add_repo loop: "{{ faf_repos }}" +- name: Ensure directory for FAF database backups exists + file: + path: "{{ faf_backup_dir }}" + state: directory + owner: faf + group: faf + mode: 0750 + when: env != 'staging' + - import_tasks: cron.yml tags: cron