Project

General

Profile

« Previous | Next » 

Revision d84fb8ed

Added by Evgeni Golov over 2 years ago

Fixes #33511 - configure redis before dynflow workers

sometimes dynflow workers are started before redis, resulting in ugly
errors in the logs until redis is available

View differences:

manifests/config.pp
} else {
include redis
$dynflow_redis_url = "redis://localhost:${redis::port}/6"
Class['redis'] -> Service <| tag == 'foreman::dynflow::worker' |>
}
file { '/etc/foreman/dynflow':
spec/classes/foreman_spec.rb
# 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('Class[redis]') }
it { is_expected.to contain_service('dynflow-sidekiq@worker-1').with_ensure('running').with_enable(true).that_requires('Class[redis]') }
# settings
it { should contain_class('foreman::settings').that_requires('Class[foreman::database]') }

Also available in: Unified diff