fix missing Install section in timer

fixes #40
TheMeier · Mar 3, 2024 · 0264db3 · 0264db3
1 parent 49fcc1b
commit 0264db3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/manifests/init.pp b/manifests/init.pp
@@ -135,15 +135,18 @@
     }) + $_service_overrides,
   }
   systemd::manage_unit { "${unit_name}_cron.timer":
-    ensure      => $file_ensure,
-    unit_entry  => delete_undef_values({
+    ensure        => $file_ensure,
+    unit_entry    => delete_undef_values({
         'Description' => $timer_description ,
     }) + $timer_unit_overrides,
-    timer_entry => delete_undef_values({
+    timer_entry   => delete_undef_values({
         'OnCalendar'      => $on_calendar,
         'OnBootSec'       => $on_boot_sec,
         'OnUnitActiveSec' => $on_unitactive_sec,
     }) + $_timer_overrides,
+    install_entry => {
+      'WantedBy' => 'timers.target',
+    },
   }
 
   service { "${unit_name}_cron.timer":

diff --git a/spec/defines/init_spec.rb b/spec/defines/init_spec.rb
@@ -38,6 +38,7 @@
         it {
           is_expected.to contain_file("/etc/systemd/system/#{title}_cron.timer")
             .with_content(%r{Description=timer for Print date cron})
+            .with_content(%r{WantedBy=timers.target})
         }
         it {
           is_expected.to contain_file("/etc/systemd/system/#{title}_cron.timer")