Log, hour and minute support at nodo::subsystem::apt

diff --git a/manifests/subsystem/apt.pp b/manifests/subsystem/apt.pp
index dffd42ab95d7d1b431f202680c83dbcb345e14f0..748941388b3928e62d820621d97c40f6b8b00b4f 100644 (file)
--- a/manifests/subsystem/apt.pp
+++ b/manifests/subsystem/apt.pp
@@ -1,6 +1,8 @@
 class nodo::subsystem::apt(
   $ensure       = present,
   $auto_upgrade = present,
+  $hour         = 2,
+  $minute       = 0,
 ) {
   file { '/etc/apt/sources.list':
     ensure  => $ensure ? {
@@ -13,15 +15,17 @@ class nodo::subsystem::apt(
     content => template("nodo/apt/${::operatingsystem}.sources.list.erb"),
   }
 
+  # We have /var/log/dpkg.log, so we do not need to rotate /var/log/upgrade.log
+  $log     = ">> /var/log/upgrade.log 2>&1"
   $apt     = '/usr/bin/apt-get'
-  $command = "${apt} update && ${apt} dist-upgrade -y && ${apt} autoremove -y && ${apt} clean"
+  $command = "${apt} update ${log} && ${apt} dist-upgrade -y ${log} && ${apt} autoremove -y ${log} && ${apt} clean ${log}"
 
   cron { 'nodo-apt-auto-upgrade':
     ensure      => $auto_upgrade,
     command     => $command,
     environment => 'PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin',
     user        => 'root',
-    hour        => 2,
-    minute      => 0,
+    hour        => $hour,
+    minute      => $minute,
   }
 }