Fix package dependencies for base installation

diff --git a/modules/base_installation/manifests/cronie.pp b/modules/base_installation/manifests/cronie.pp
index 3558d9f06334e51bc5f235249c284199c45e7fae..ca3eb2a4548193e9a8c1e0b611690cef6a98aa15 100644 (file)
--- a/modules/base_installation/manifests/cronie.pp
+++ b/modules/base_installation/manifests/cronie.pp
@@ -1,6 +1,10 @@
 class base_installation::cronie inherits base_installation {
   ensure_packages(['cronie'])
 
+  Package["cronie"] -> Cron::Job <| |>
+  Package["cronie"] -> Cron::Job::Multiple <| |>
+  Package["cronie"] -> Cron::Daily <| |>
+
   unless empty($base_installation::puppet_code_path) {
     file { 'post-hook':
       ensure  => file,

diff --git a/modules/base_installation/manifests/grub.pp b/modules/base_installation/manifests/grub.pp
index 0a96aa7d7b1020f53b096e9fcc971b325c8dde96..208b745317046426445d384b5516593713ba0acc 100644 (file)
--- a/modules/base_installation/manifests/grub.pp
+++ b/modules/base_installation/manifests/grub.pp
@@ -9,9 +9,10 @@ class base_installation::grub inherits base_installation {
   # }
 
   file_line { "/etc/default/grub#GRUB_CMDLINE_LINUX":
-    path  => "/etc/default/grub",
-    line  => 'GRUB_CMDLINE_LINUX=" console=tty0 console=ttyS0,115200"',
-    match => '^GRUB_CMDLINE_LINUX='
+    path    => "/etc/default/grub",
+    line    => 'GRUB_CMDLINE_LINUX=" console=tty0 console=ttyS0,115200"',
+    match   => '^GRUB_CMDLINE_LINUX=',
+    require => Package["grub"],
   }
 
   # exec { 'update GRUB config':

diff --git a/modules/base_installation/manifests/services.pp b/modules/base_installation/manifests/services.pp
index 1659f97f69380fef2b9885e660bc23640e5d6547..f24b55f8cf79b875062f29d6b078c59bb322cb86 100644 (file)
--- a/modules/base_installation/manifests/services.pp
+++ b/modules/base_installation/manifests/services.pp
@@ -7,13 +7,15 @@ class base_installation::services inherits base_installation {
   }
 
   service { "sshd":
-    ensure => $ensure,
-    enable => true,
+    ensure  => $ensure,
+    enable  => true,
+    require => Package["openssh"],
   }
 
   service { "atd":
-    ensure => $ensure,
-    enable => true
+    ensure  => $ensure,
+    enable  => true,
+    require => Package["at"],
   }
 
   service { "systemd-networkd":
@@ -32,8 +34,9 @@ class base_installation::services inherits base_installation {
   }
 
   service { "cronie":
-    ensure => $ensure,
-    enable => true,
+    ensure  => $ensure,
+    enable  => true,
+    require => Package["cronie"],
   }
 
   file { '/etc/systemd/system/getty@tty1.service.d/':

diff --git a/modules/base_installation/manifests/ssh.pp b/modules/base_installation/manifests/ssh.pp
index 43769e9a5454bfb52bd5531a5b72cd815d76c60f..d6317d5b7cf0a79f2bcab6cc59494c694eb19fec 100644 (file)
--- a/modules/base_installation/manifests/ssh.pp
+++ b/modules/base_installation/manifests/ssh.pp
@@ -1,13 +1,14 @@
 class base_installation::ssh inherits base_installation {
   class { 'ssh::server':
      storeconfigs_enabled => false,
-     options => {
+     options              => {
         'AcceptEnv'                       => undef,
         'X11Forwarding'                   => 'yes',
         'PrintMotd'                       => 'no',
         'ChallengeResponseAuthentication' => 'no',
         'Subsystem'                       => 'sftp /usr/lib/openssh/sftp-server',
-     }
+     },
+     require              => Package["openssh"]
   }
 
   contain "ssh::server"

diff --git a/modules/base_installation/manifests/users.pp b/modules/base_installation/manifests/users.pp
index d0ac449a55a6a4c747212e9cd68f8637d80f0cec..1ce5da76e3c0997d24d7648ff4e6e7083daef52f 100644 (file)
--- a/modules/base_installation/manifests/users.pp
+++ b/modules/base_installation/manifests/users.pp
@@ -15,7 +15,8 @@ class base_installation::users (
 
   sudo::conf { 'wheel':
     priority => 10,
-    content  => "%wheel ALL=(ALL) ALL"
+    content  => "%wheel ALL=(ALL) ALL",
+    require  => Package["sudo"],
   }
 
   contain "sudo"