openstack/puppet-cinder
Code Issues Proposed changes

Merge "Use oslo::coordination to manage coordination parameters"

Browse Source
This commit is contained in:
Zuul 2021-06-28 14:16:53 +00:00 committed by Gerrit Code Review
commit d9ccac4f21
2 changed files with 8 additions and 4 deletions

4
manifests/coordination.pp

@ -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

@ -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