openstack
/
puppet-cinder
Code Issues Proposed changes

Use oslo::coordination to manage coordination parameters 

This change replaces current implementation to manage coordination
parameters by the base class in puppet-oslo. With this change
the required backend package is installed according to the backend
driver used.

Depends-on: https://review.opendev.org/791628
Change-Id: I210f207d4f05862c98a2ef6d04162880d1cfc4ae
This commit is contained in:
Takashi Kajinami 2021-05-18 10:05:02 +09:00
parent 496eb152f9
commit 9078af8559
2 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/coordination.pp
View File

@ -14,7 +14,7 @@ class cinder::coordination (
include cinder::deps
cinder_config {
'coordination/backend_url': value => $backend_url;
oslo::coordination{ 'cinder_config':
backend_url => $backend_url
}
}

8
spec/classes/cinder_coordination_spec.rb
View File

@ -4,7 +4,9 @@ describe 'cinder::coordination' do
shared_examples 'cinder::coordination' do
context 'with default parameters' do
it {
is_expected.to contain_cinder_config('coordination/backend_url').with_value('<SERVICE DEFAULT>')
is_expected.to contain_oslo__coordination('cinder_config').with(
:backend_url => '<SERVICE DEFAULT>'
)
}
end
@ -16,7 +18,9 @@ describe 'cinder::coordination' do
end
it {
is_expected.to contain_cinder_config('coordination/backend_url').with_value('etcd3+http://127.0.0.1:2379')
is_expected.to contain_oslo__coordination('cinder_config').with(
:backend_url => 'etcd3+http://127.0.0.1:2379'
)
}
end
end