openstack
/
puppet-vitrage
Code Issues Proposed changes

Merge "Rename persistor config opt"

This commit is contained in:
Zuul 2018-01-18 02:11:57 +00:00 committed by Gerrit Code Review
parent 87187f3f0e 5e6e68140d
commit 85b2177e77
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