openstack/puppet-mistral
Code Issues Proposed changes

Move evaluation_interval and older_than to engine

Browse Source 
The evaluation_interval and older_than settings apply
to the Mistral engine process (not executor). So it makes
sense to manage them within engine.pp.

Change-Id: I79ffd637ebb8094c27f42270b2c14ee77c861297
This commit is contained in:
Dan Prince
2016-05-10 15:01:11 -04:00
parent 6c82c0aaa9
commit 9b995058ff
4 changed files with 46 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
spec/classes/mistral_engine_spec.rb
View File

@@ -8,7 +8,9 @@ describe 'mistral::engine' do
:host => 'foo_host',
:topic => 'foo_topic',
:version => '1.0',
:execution_field_size_limit_kb => '1234'}
:execution_field_size_limit_kb => '1234',
:evaluation_interval => 1234,
:older_than => 60}
end
shared_examples_for 'mistral-engine' do
@@ -21,6 +23,8 @@ describe 'mistral::engine' do
it { is_expected.to contain_mistral_config('engine/topic').with_value( params[:topic] ) }
it { is_expected.to contain_mistral_config('engine/version').with_value( params[:version] ) }
it { is_expected.to contain_mistral_config('engine/execution_field_size_limit_kb').with_value( params[:execution_field_size_limit_kb] ) }
it { is_expected.to contain_mistral_config('execution_expiration_policy/evaluation_interval').with_value( params[:evaluation_interval] ) }
it { is_expected.to contain_mistral_config('execution_expiration_policy/older_than').with_value( params[:older_than] ) }
end