From ead9aaa3dd7ca48012b2b21cc930ee73c8eaa9d3 Mon Sep 17 00:00:00 2001
From: Guilhem Moulin <guilhem@fripost.org>
Date: Tue, 3 Nov 2020 03:15:10 +0100
Subject: Bacula: refactor systemd service files.

Use unit overrides on top of upstream's service files instead of
overriding entire service files.  In particular, upstream uses flag `-P`
so we don't need to use RuntimeDirectory= anymore.
---
 .../etc/systemd/system/bacula-director.service     | 27 ----------------------
 .../system/bacula-director.service.d/override.conf | 13 +++++++++++
 roles/bacula-dir/tasks/main.yml                    | 14 +++++++----
 3 files changed, 23 insertions(+), 31 deletions(-)
 delete mode 100644 roles/bacula-dir/files/etc/systemd/system/bacula-director.service
 create mode 100644 roles/bacula-dir/files/etc/systemd/system/bacula-director.service.d/override.conf

(limited to 'roles/bacula-dir')

diff --git a/roles/bacula-dir/files/etc/systemd/system/bacula-director.service b/roles/bacula-dir/files/etc/systemd/system/bacula-director.service
deleted file mode 100644
index 8b2f5ff..0000000
--- a/roles/bacula-dir/files/etc/systemd/system/bacula-director.service
+++ /dev/null
@@ -1,27 +0,0 @@
-[Unit]
-Description=Bacula Director service
-After=network.target
-
-[Service]
-Type=simple
-StandardOutput=syslog
-User=bacula
-Group=bacula
-ExecStart=/usr/sbin/bacula-dir -f -c /etc/bacula/bacula-dir.conf
-
-# Hardening
-NoNewPrivileges=yes
-PrivateDevices=yes
-ProtectHome=yes
-ProtectSystem=strict
-ReadWriteDirectories=-/var/lib/bacula
-ReadWriteDirectories=-/var/log/bacula
-RuntimeDirectory=bacula
-PrivateDevices=yes
-ProtectControlGroups=yes
-ProtectKernelModules=yes
-ProtectKernelTunables=yes
-RestrictAddressFamilies=AF_UNIX AF_INET AF_INET6
-
-[Install]
-WantedBy=multi-user.target
diff --git a/roles/bacula-dir/files/etc/systemd/system/bacula-director.service.d/override.conf b/roles/bacula-dir/files/etc/systemd/system/bacula-director.service.d/override.conf
new file mode 100644
index 0000000..f0d36c4
--- /dev/null
+++ b/roles/bacula-dir/files/etc/systemd/system/bacula-director.service.d/override.conf
@@ -0,0 +1,13 @@
+[Service]
+# Hardening
+NoNewPrivileges=yes
+PrivateDevices=yes
+ProtectHome=yes
+ProtectSystem=strict
+ReadWriteDirectories=-/var/lib/bacula
+ReadWriteDirectories=-/var/log/bacula
+PrivateDevices=yes
+ProtectControlGroups=yes
+ProtectKernelModules=yes
+ProtectKernelTunables=yes
+RestrictAddressFamilies=AF_UNIX AF_INET AF_INET6
diff --git a/roles/bacula-dir/tasks/main.yml b/roles/bacula-dir/tasks/main.yml
index 2f7ab25..2fdb35b 100644
--- a/roles/bacula-dir/tasks/main.yml
+++ b/roles/bacula-dir/tasks/main.yml
@@ -12,7 +12,7 @@
   notify:
     - Restart bacula-director
 
-# Create with:
+# Populate with:
 #   echo bconsole $(pwgen -sn 64 1) | sudo tee -a /etc/bacula/passwords-dir
 #   echo $sd-sd   $(pwgen -sn 64 1) | sudo tee -a /etc/bacula/passwords-dir
 #   echo $fd-fd   $(pwgen -sn 64 1) | sudo tee -a /etc/bacula/passwords-dir
@@ -41,9 +41,15 @@
   notify:
     - Restart bacula-director
 
-- name: Copy bacula-director.service
-  copy: src=etc/systemd/system/bacula-director.service
-        dest=/etc/systemd/system/bacula-director.service
+- name: Create /etc/systemd/system/bacula-director.service.d
+  file: path=/etc/systemd/system/bacula-director.service.d
+        state=directory
+        owner=root group=root
+        mode=0755
+
+- name: Copy bacula-director.service override
+  copy: src=etc/systemd/system/bacula-director.service.d/override.conf
+        dest=/etc/systemd/system/bacula-director.service.d/override.conf
         owner=root group=root
         mode=0644
   notify:
-- 
cgit v1.2.3