From 0fedc4399676f1cb2c40e83f567a055b3e64b2d2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Petr=20Men=C5=A1=C3=ADk?= Date: Fri, 5 Sep 2025 15:21:19 +0200 Subject: [PATCH] logrotate: skip if empty and remove old variants Many variants are never built anymore. Clean actions to just those still shipped. But do not trigger named reload when named.run file is empty. That is common on freeipa installation, where configuration changes logging to put it elsewhere. named reload is disruptive because how bind-dyndb-ldap behaves during reloads. Avoid unnecessary reloads with visible service disruption. Keep named-pkcs11 reload variant. Resolves: RHEL-113960 --- bind.spec | 5 ++++- named.logrotate | 4 +--- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/bind.spec b/bind.spec index a7d1ae9..07a8476 100644 --- a/bind.spec +++ b/bind.spec @@ -80,7 +80,7 @@ License: MPL-2.0 AND ISC AND MIT AND BSD-3-Clause AND BSD-2-Clause # Before rebasing bind, ensure bind-dyndb-ldap is ready to be rebuild and use side-tag with it. # Updating just bind will cause freeipa-dns-server package to be uninstallable. Version: 9.18.33 -Release: 7%{?dist} +Release: 8%{?dist} Epoch: 32 Url: https://www.isc.org/downloads/bind/ # @@ -923,6 +923,9 @@ fi; %endif %changelog +* Fri Sep 12 2025 Petr Menšík <> - 32:9.18.33-8 +- logrotate: skip if empty and remove old variants (RHEL-113942) + * Wed Jul 09 2025 Petr Menšík - 32:9.18.33-7 - Add runtime tunable limit by environment NAMED_MAXADDITIONAL (RHEL-84006) diff --git a/named.logrotate b/named.logrotate index 5df448f..5934e52 100644 --- a/named.logrotate +++ b/named.logrotate @@ -2,11 +2,9 @@ missingok su named named create 0644 named named + notifempty postrotate /usr/bin/systemctl reload named.service > /dev/null 2>&1 || true /usr/bin/systemctl reload named-chroot.service > /dev/null 2>&1 || true - /usr/bin/systemctl reload named-sdb.service > /dev/null 2>&1 || true - /usr/bin/systemctl reload named-sdb-chroot.service > /dev/null 2>&1 || true - /usr/bin/systemctl reload named-pkcs11.service > /dev/null 2>&1 || true endscript }