openstack/puppet-vitrage
Code Issues Proposed changes

Rename persistor config opt

Browse Source 
vitrage has renamed 'persistor/persist_events' config
option in [1], similar change is required in puppet-vitrage.

[1] https://review.openstack.org/#/c/530650

Change-Id: I140dddd7baae00b11cbb50f3cf2d1a3b6b7e162b
This commit is contained in:
yatin 2018-01-16 15:29:48 +05:30
parent f687aa3909
commit 5e6e68140d
3 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/persistor.pp
View File

@ -36,7 +36,7 @@ class vitrage::persistor (
}
vitrage_config {
'persistor/persist_events': value => $enabled;
'persistency/enable_persistency': value => $enabled;
}
service { 'vitrage-persistor':

8
releasenotes/notes/rename_persistor_config_params-2898e5dd42fdd5f3.yaml Normal file
View File

@ -0,0 +1,8 @@
---
other:
- |
vitrage has renamed persistor config option in
https://review.openstack.org/#/c/530650. So rename of
config option 'persistor/persist_events' to
'persistency/enable_persistency' is required in
puppet-vitrage.

4
spec/classes/vitrage_persistor_spec.rb
View File

@ -13,7 +13,7 @@ describe 'vitrage::persistor' do
it { is_expected.to contain_class('vitrage::params') }
it 'configures vitrage.conf' do
is_expected.to contain_vitrage_config('persistor/persist_events').with_value('true')
is_expected.to contain_vitrage_config('persistency/enable_persistency').with_value('true')
end
it 'installs vitrage-persistor package' do
@ -53,7 +53,7 @@ describe 'vitrage::persistor' do
)
end
it 'configures vitrage.conf' do
is_expected.to contain_vitrage_config('persistor/persist_events').with_value('false')
is_expected.to contain_vitrage_config('persistency/enable_persistency').with_value('false')
end
end