From f67be989cc81e8838f5a254e114287e9a7c9245d Mon Sep 17 00:00:00 2001 From: Saibotk Date: Fri, 29 Oct 2021 23:46:10 +0200 Subject: [PATCH] lvm_self_backup: Option to configure user/group for status dir This allows to give the telegraf container access to the backup status flags --- roles/lvm_self_backup/defaults/main.yml | 5 +++++ roles/lvm_self_backup/tasks/main.yml | 4 ++-- roles/lvm_self_backup/templates/backup-lvm.sh | 2 ++ 3 files changed, 9 insertions(+), 2 deletions(-) diff --git a/roles/lvm_self_backup/defaults/main.yml b/roles/lvm_self_backup/defaults/main.yml index 81b64f3..6411197 100644 --- a/roles/lvm_self_backup/defaults/main.yml +++ b/roles/lvm_self_backup/defaults/main.yml @@ -21,6 +21,11 @@ # The directory to put the status flag in backup_status_directory: "/monitoring" +# Status directory user & group id +# This can be used for example to give telegraf access to the status files. +backup_status_directory_uid: 100 +backup_status_directory_gid: 101 + # The LVM volumes to backup # Available fields: # vg_name - The volume group that the volume resides in diff --git a/roles/lvm_self_backup/tasks/main.yml b/roles/lvm_self_backup/tasks/main.yml index cfff9c8..d64f04d 100644 --- a/roles/lvm_self_backup/tasks/main.yml +++ b/roles/lvm_self_backup/tasks/main.yml @@ -34,8 +34,8 @@ path: "{{ item }}" state: directory mode: '0700' - owner: 'root' - group: 'root' + owner: "{{ backup_status_directory_uid }}" + group: "{{ backup_status_directory_gid }}" recurse: true with_items: - "{{ backup_status_directory }}" diff --git a/roles/lvm_self_backup/templates/backup-lvm.sh b/roles/lvm_self_backup/templates/backup-lvm.sh index 152bb7a..1e6efb5 100644 --- a/roles/lvm_self_backup/templates/backup-lvm.sh +++ b/roles/lvm_self_backup/templates/backup-lvm.sh @@ -84,4 +84,6 @@ fi echo $success > {{ backup_status_directory }}/backup.status +chown {{ backup_status_directory_uid }}:{{ backup_status_directory_gid }} {{ backup_status_directory }}/backup.status + exit $success