Merge pull request #1048 from bastelfreak/smee

profile::core::foreman: Make smee installation opt-out

Author: jhoblitt

site/profile/manifests/core/foreman.pp

@@ -17,12 +17,16 @@
 # @param foreman_global_parameter
 #   `foreman_global_parameter` resources to create.
 #
+# @param manage_smee
+#   Make smee installation opt-out
+#
 class profile::core::foreman (
-  Stdlib::HTTPSUrl $smee_url,
+  Optional[Stdlib::HTTPSUrl] $smee_url = undef,
   Boolean $enable_puppetdb = false,
   Optional[Hash[String, Hash]] $foreman_config = undef,
   Optional[Hash[String, Hash]] $foreman_hostgroup = undef,
   Optional[Hash[String, Hash]] $foreman_global_parameter = undef,
+  Boolean $manage_smee = true,
 ) {
   include cron
   include foreman
@@ -81,10 +85,21 @@
     }
   }
 
-  class { 'smee':
-    url  => $smee_url,
-    path => '/payload',
-    port => 8088,
+  # smee has some startup problems due to too new (?) nodejs on EL8
+  # https://gist.github.com/bastelfreak/6d0d3bc324633f04c5b1fc432e871192
+  if $manage_smee {
+    assert_type(Stdlib::HTTPSUrl, $smee_url)
+    class { 'smee':
+      url  => $smee_url,
+      path => '/payload',
+      port => 8088,
+    }
+    cron { 'smee':
+      command => '/usr/bin/systemctl restart smee > /dev/null 2>&1',
+      user    => 'root',
+      hour    => 4,
+      minute  => 42,
+    }
   }
 
   # el7 systemd is too old to support periodic graceful restarts of a service unit.
@@ -97,13 +112,6 @@
     minute  => 42,
   }
 
-  cron { 'smee':
-    command => '/usr/bin/systemctl restart smee > /dev/null 2>&1',
-    user    => 'root',
-    hour    => 4,
-    minute  => 42,
-  }
-
   package { 'hiera-eyaml':
     provider => 'puppetserver_gem',
   }