From fb069191df63c5c90627a6c6fe1b8d23eb126ed36dd9c1ea089511c756164076 Mon Sep 17 00:00:00 2001
From: Bernhard Wiedemann <bwiedemann@suse.com>
Date: Fri, 13 Jan 2023 20:48:18 +0000
Subject: [PATCH] Accepting request 1058011 from
 home:bmwiedemann:branches:server:monitoring

Drop harmful chown in %post scripts (boo#1155078)

OBS-URL: https://build.opensuse.org/request/show/1058011
OBS-URL: https://build.opensuse.org/package/show/server:monitoring/munin?expand=0&rev=75
---
 munin.changes |  5 +++++
 munin.spec    | 18 ++++++------------
 2 files changed, 11 insertions(+), 12 deletions(-)

diff --git a/munin.changes b/munin.changes
index b6e11c4..80d2e9b 100644
--- a/munin.changes
+++ b/munin.changes
@@ -1,3 +1,8 @@
+-------------------------------------------------------------------
+Thu Jan 12 13:16:28 UTC 2023 - Bernhard Wiedemann <bwiedemann@suse.com>
+
+- Drop harmful chown in %post scripts (boo#1155078)
+
 -------------------------------------------------------------------
 Thu Dec 15 17:20:35 UTC 2022 - Carsten Ziepke <kieltux@gmail.com>
 
diff --git a/munin.spec b/munin.spec
index 3fa85ad..b07ca5a 100644
--- a/munin.spec
+++ b/munin.spec
@@ -228,12 +228,6 @@ getent passwd munin > /dev/null || %{_sbindir}/useradd -r -c "munin monitoring"
 %endif
 
 %post
-chown -R munin:munin %{htmldir}
-chown -R munin:munin %{dbdir}
-chmod 755 %{dbdir}
-touch %{logdir}/munin-graph.log %{logdir}/munin-html.log %{logdir}/munin-nagios.log %{logdir}/munin-limits.log %{logdir}/munin-update.log
-chown munin:munin %{logdir}/*
-chown root:root %{logdir}/munin-node.log* >/dev/null 2>&1 || true
 %tmpfiles_create munin.conf
 %service_add_post munin-cgi-graph.service
 %service_add_post munin-cgi-html.service
@@ -273,12 +267,6 @@ getent passwd munin > /dev/null || %{_sbindir}/useradd -r -c "munin monitoring"
 if [ $1 = 1 ]; then
 %{_sbindir}/munin-node-configure --shell | sh
 fi
-chown -R munin:munin %{dbdir}
-chmod 755 %{dbdir}
-touch %{logdir}/munin-node.log
-chown munin:munin %{logdir}/*
-chown root:root %{logdir}/munin-node.log*
-chown -R nobody:nobody %{dbdir}/plugin-state/* >/dev/null 2>&1
 %tmpfiles_create munin-node.conf
 %service_add_post munin-node.service
 %firewalld_reload
@@ -368,6 +356,11 @@ chown -R nobody:nobody %{dbdir}/plugin-state/* >/dev/null 2>&1
 %{_mandir}/man8/munin.8%{?ext_man}
 %attr(0750, munin, munin) %dir %{logdir}
 %attr(0755, munin, munin) %dir %{dbdir}
+%ghost %attr(0644, munin, munin) %{logdir}/munin-graph.log
+%ghost %attr(0644, munin, munin) %{logdir}/munin-html.log
+%ghost %attr(0644, munin, munin) %{logdir}/munin-nagios.log
+%ghost %attr(0644, munin, munin) %{logdir}/munin-limits.log
+%ghost %attr(0644, munin, munin) %{logdir}/munin-update.log
 %ghost /run/munin
 
 %files node
@@ -452,6 +445,7 @@ chown -R nobody:nobody %{dbdir}/plugin-state/* >/dev/null 2>&1
 %attr(0750, munin, munin) %dir %{logdir}
 %attr(0755, munin, munin) %dir %{dbdir}
 %attr(0775, nobody, nobody) %dir %{dbdir}/plugin-state
+%ghost %{logdir}/munin-node.log
 %ghost /run/munin
 %dir %{_prefix}/lib/firewalld
 %dir %{_prefix}/lib/firewalld/services