From 7928f602c7acf650ec652df5ffce8022a0dc57e5 Mon Sep 17 00:00:00 2001
From: Matthieu Patou <mat@matws.net>
Date: Fri, 28 Oct 2022 08:27:28 -0700
Subject: [PATCH] feat: allow customization of a postrotate script

You need to signal to bind to reload log files after rotating them,
doing `rndc reconfig` is a good way of doing so.

This change allows you to actually have that in the generated logrotate
configuration
---
 bind/files/debian/logrotate_bind | 6 ++++++
 pillar.example                   | 3 +++
 2 files changed, 9 insertions(+)

diff --git a/bind/files/debian/logrotate_bind b/bind/files/debian/logrotate_bind
index 405fc4ea..52b10d98 100644
--- a/bind/files/debian/logrotate_bind
+++ b/bind/files/debian/logrotate_bind
@@ -2,12 +2,18 @@
 {%- set user = salt['pillar.get']('bind:config:user', map.user) %}
 {%- set group = salt['pillar.get']('bind:config:group', map.group) %}
 {%- set mode = salt['pillar.get']('bind:config:log_mode', map.log_mode) %}
+{%- set postrotate = salt['pillar.get']('bind:logrotate:postrotate_script', "") %}
 {{ map.log_dir }}/*.log {
     rotate 7
     daily
     missingok
     notifempty
     sharedscripts
+    {%- if postrotate != "" %}
+    postrotate
+      {{ postrotate }}
+    endscript
+    {%- endif %}
     copytruncate
     compress
     create {{ mode }} {{ user }} {{ group }}
diff --git a/pillar.example b/pillar.example
index 4f292070..2d8a38d6 100644
--- a/pillar.example
+++ b/pillar.example
@@ -9,6 +9,9 @@
 ### Overrides for the defaults specified by ###
 ### map.jinja                               ###
 bind:
+  logrotate:
+    postrotate_script: >
+      rndc reconfig &>/dev/null || true
   lookup:
     pkgs:
       - bind                                      # Need to install