Accepting request 1127772 from systemsmanagement

OBS-URL: https://build.opensuse.org/request/show/1127772
OBS-URL: https://build.opensuse.org/package/show/openSUSE:Factory/ipmiutil?expand=0&rev=25

ipmiutil.changes

@@ -1,3 +1,8 @@
+-------------------------------------------------------------------
+Fri Nov 17 02:36:26 UTC 2023 - Martin Schreiner <martin.schreiner@suse.com>
+
+- Migrate from cron to systemd timers.
+
 -------------------------------------------------------------------
 Tue Sep 12 15:07:27 UTC 2023 - Andrea Manzini <andrea.manzini@suse.com>
 

ipmiutil.spec

@@ -30,6 +30,8 @@ Patch0:         harden_ipmi_port.service.patch
 Patch1:         harden_ipmiutil_asy.service.patch
 Patch2:         harden_ipmiutil_evt.service.patch
 Patch3:         harden_ipmiutil_wdt.service.patch
+Source1:        ipmiutil_checksel.service
+Source2:        ipmiutil_checksel.timer
 BuildRequires:  autoconf
 BuildRequires:  automake
 BuildRequires:  gcc
@@ -83,6 +85,8 @@ autoreconf -fiv
 
 %install
 %make_install
+install -D -m 0644 %{SOURCE1} %{buildroot}/%{_unitdir}/ipmiutil_checksel.service
+install -D -m 0644 %{SOURCE2} %{buildroot}/%{_unitdir}/ipmiutil_checksel.timer
 
 %files
 %dir %{_datadir}/%{name}
@@ -120,6 +124,8 @@ autoreconf -fiv
 %{_unitdir}/ipmiutil_asy.service
 %{_unitdir}/ipmiutil_wdt.service
 %{_unitdir}/ipmi_port.service
+%{_unitdir}/ipmiutil_checksel.service
+%{_unitdir}/ipmiutil_checksel.timer
 %{_datadir}/%{name}/ipmiutil.pre
 %{_datadir}/%{name}/ipmiutil.setup
 %{_datadir}/%{name}/ipmi_if.sh
@@ -192,7 +198,7 @@ then
    vardir=%{_var}/lib/%{name}
    scr_dir=%{_datadir}/%{name}
 
-%service_add_post ipmi_port.service ipmiutil_evt.service ipmiutil_asy.service ipmiutil_wdt.service
+%service_add_post ipmi_port.service ipmiutil_evt.service ipmiutil_asy.service ipmiutil_wdt.service ipmiutil_checksel.service ipmiutil_checksel.timer
 
    # Test whether an IPMI interface is known to the motherboard
    IPMIret=1
@@ -211,9 +217,7 @@ then
          cp -f %{scr_dir}/ipmi.init.basic  %{_initddir}/ipmi
       fi
       # If IPMI is enabled, automate managing the IPMI SEL
-      if [ -d %{_sysconfdir}/cron.daily ]; then
-         cp -f %{_datadir}/%{name}/checksel %{_sysconfdir}/cron.daily
-      fi
+      systemctl enable ipmiutil_checksel.timer
       # IPMI_IS_ENABLED, so enable services, but only if Red Hat
       if [ -f %{_sysconfdir}/redhat-release ]; then
          if [ -x /bin/systemctl ]; then
@@ -246,9 +250,7 @@ else
     IPMIret=1
     %{_sbindir}/dmidecode |grep -q IPMI && IPMIret=0
     if [ $IPMIret -eq 0 ]; then
-      if [ -d %{_sysconfdir}/cron.daily ]; then
-         cp -f %{_datadir}/%{name}/checksel %{_sysconfdir}/cron.daily
-      fi
+      systemctl enable ipmiutil_checksel.timer
     fi
    fi
 fi
@@ -257,15 +259,12 @@ fi
 # before uninstall,  $1 = 1 if rpm -U, $1 = 0 if rpm -e
 if [ "$1" = "0" ]
 then
-%service_del_preun ipmi_port.service ipmiutil_evt.service ipmiutil_asy.service ipmiutil_wdt.service
-   if [ -f %{_sysconfdir}/cron.daily/checksel ]; then
-	rm -f %{_sysconfdir}/cron.daily/checksel
-   fi
+%service_del_preun ipmi_port.service ipmiutil_evt.service ipmiutil_asy.service ipmiutil_wdt.service ipmiutil_checksel.service ipmiutil_checksel.timer
 fi
 
 %postun
 # after uninstall,  $1 = 1 if update, $1 = 0 if rpm -e
 /sbin/ldconfig
-%service_del_postun ipmi_port.service ipmiutil_evt.service ipmiutil_asy.service ipmiutil_wdt.service
+%service_del_postun ipmi_port.service ipmiutil_evt.service ipmiutil_asy.service ipmiutil_wdt.service ipmiutil_checksel.service ipmiutil_checksel.timer
 
 %changelog

ipmiutil_checksel.service

@@ -0,0 +1,16 @@
+[Unit]
+Description=Automatic execution of ipmiutil's checksel
+
+[Service]
+# added automatically, for details please see
+# https://en.opensuse.org/openSUSE:Security_Features#Systemd_hardening_effort
+ProtectSystem=full
+ProtectHome=true
+ProtectHostname=true
+ProtectKernelTunables=true
+ProtectKernelModules=true
+ProtectControlGroups=true
+RestrictRealtime=true
+# end of automatic additions 
+Type=oneshot
+ExecStart=/usr/share/ipmiutil/checksel

ipmiutil_checksel.timer

@@ -0,0 +1,9 @@
+[Unit]
+Description=Automatic execution of ipmiutil's checksel
+
+[Timer]
+OnCalendar=Daily
+Persistent=true
+
+[Install]
+WantedBy=timers.target