WIP

Author: ekohl

manifests/apache.pp

@@ -62,6 +62,4 @@
       path        => '/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin',
     }
   }
-
-  Class['apache::service'] -> Class['foreman::apache']
 }

manifests/init.pp

@@ -336,18 +336,23 @@
   include foreman::install
   include foreman::config
   include foreman::database
-  contain foreman::service
+  include foreman::service
+
+  anchor { 'foreman::running': # lint:ignore:anchor_resource
+  }
 
   Anchor <| title == 'foreman::repo' |> ~> Class['foreman::install']
   Class['foreman::install'] ~> Class['foreman::config', 'foreman::service']
   Class['foreman::config'] ~> Class['foreman::database', 'foreman::service']
   Class['foreman::database'] ~> Class['foreman::service']
-  Class['foreman::service'] -> Foreman_smartproxy <| base_url == $foreman_url |>
+  Class['foreman::service'] -> Anchor['foreman::running']
+  Anchor['foreman::running'] -> Foreman_smartproxy <| base_url == $foreman_url |>
 
   if $apache {
-    contain foreman::apache
+    include foreman::apache
 
     Class['foreman::config', 'foreman::database'] -> Class['foreman::apache']
+    Class['foreman::apache', 'apache::service'] -> Class['foreman::running']
     if $ipa_authentication and $keycloak {
       fail("${facts['networking']['hostname']}: External authentication via IPA and Keycloak are mutually exclusive.")
     }