configure redis before dynflow workers

sometimes dynflow workers are started before redis, resulting in ugly errors in the logs until redis is available
theforeman · Sep 17, 2021 · c43d0ef · c43d0ef
1 parent d5c3c9f
commit c43d0ef
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/manifests/config.pp b/manifests/config.pp
@@ -12,6 +12,7 @@
     } else {
       include redis
       $dynflow_redis_url = "redis://localhost:${redis::port}/6"
+      Class['redis'] -> Service <| tag == 'foreman::dynflow::worker' |>
     }
 
     file { '/etc/foreman/dynflow':

diff --git a/spec/classes/foreman_spec.rb b/spec/classes/foreman_spec.rb
@@ -149,8 +149,8 @@
         # service
         it { should contain_class('foreman::service') }
         it { should contain_service('foreman') }
-        it { is_expected.to contain_service('dynflow-sidekiq@orchestrator').with_ensure('running').with_enable(true) }
-        it { is_expected.to contain_service('dynflow-sidekiq@worker-1').with_ensure('running').with_enable(true) }
+        it { is_expected.to contain_service('dynflow-sidekiq@orchestrator').with_ensure('running').with_enable(true).that_requires('Service[redis]') }
+        it { is_expected.to contain_service('dynflow-sidekiq@worker-1').with_ensure('running').with_enable(true).that_requires('Service[redis]') }
 
         # settings
         it { should contain_class('foreman::settings').that_requires('Class[foreman::database]') }