Disable removing utmp by default

This breaks various scripts such as runlevel and hence invoke-rc.d

2 files changed, 15 insertions, 5 deletions

diff --git a/README b/README
index 777aae7..9d9d8ef 100644
--- a/README
+++ b/README
@@ -20,15 +20,25 @@ You can set the following parameters for the loginrecords class.
 If you just include the class loginrecords this will disable all
 loginlogs.
 
-$disable_btmp, $disable_utmp, $disable_wtmp
--------------------------------------------
+$disable_btmp, $disable_wtmp
+---------------------------
 
-Default: /var/log/btmp, /var/run/utmp and /var/log/wtmp are ensured to
-be absent.
+Default: /var/log/btmp and /var/log/wtmp are ensured to be absent.
 
 These variables, when set to a false, non-empty value, have these
 files created and their logging enabled again.
 
+$disable_utmp
+-------------
+
+Default: /var/run/utmp is not removed.
+
+WARNING: Removing utmp removes necessary information for binaries such
+as runlevel. This might break other scripts that are relaying on the
+output of this binary or other scripts depending on their output. For
+example it is known that puppet will not anymore be able to idempodently
+manage runlevels. -> https://projects.puppetlabs.com/issues/5409
+
 $disable_faillog
 ----------------
 
diff --git a/manifests/init.pp b/manifests/init.pp
index 9c0a880..fe65eec 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -2,7 +2,7 @@ class loginrecords(
     $disable_btmp = true,
     $disable_faillog = true,
     $disable_lastlog = true,
-    $disable_utmp = true,
+    $disable_utmp = false,
     $disable_wtmp = true
 ){
     # Include main class