diff --git a/manifests/coordination.pp b/manifests/coordination.pp index d4528a9f..db1852fe 100644 --- a/manifests/coordination.pp +++ b/manifests/coordination.pp @@ -18,15 +18,27 @@ # has changed. # Defaults to $facts['os_service_default'] # +# [*manage_backend_package*] +# (Optional) Whether to install the backend package. +# Defaults to true. +# +# [*package_ensure*] +# (Optional) ensure state for package. +# Defaults to 'present' +# class designate::coordination ( - $backend_url = $facts['os_service_default'], - $heartbeat_interval = $facts['os_service_default'], - $run_watchers_interval = $facts['os_service_default'], + $backend_url = $facts['os_service_default'], + $heartbeat_interval = $facts['os_service_default'], + $run_watchers_interval = $facts['os_service_default'], + Boolean $manage_backend_package = true, + Stdlib::Ensure::Package $package_ensure = present, ) { include designate::deps oslo::coordination { 'designate_config': - backend_url => $backend_url, + backend_url => $backend_url, + manage_backend_package => $manage_backend_package, + package_ensure => $package_ensure, } designate_config { diff --git a/releasenotes/notes/coordination-package-params-0dab58d520940fee.yaml b/releasenotes/notes/coordination-package-params-0dab58d520940fee.yaml new file mode 100644 index 00000000..256ccca3 --- /dev/null +++ b/releasenotes/notes/coordination-package-params-0dab58d520940fee.yaml @@ -0,0 +1,8 @@ +--- +features: + - | + The following parameters have been added to the ``designate::coordination`` + class. + + - ``manage_backend_package`` + - ``package_ensure`` diff --git a/spec/classes/designate_coordination_spec.rb b/spec/classes/designate_coordination_spec.rb index e7782a41..d592b920 100644 --- a/spec/classes/designate_coordination_spec.rb +++ b/spec/classes/designate_coordination_spec.rb @@ -5,7 +5,9 @@ describe 'designate::coordination' do context 'with default parameters' do it { is_expected.to contain_oslo__coordination('designate_config').with( - :backend_url => '' + :backend_url => '', + :manage_backend_package => true, + :package_ensure => 'present', ) is_expected.to contain_designate_config('coordination/heartbeat_interval').with_value('') is_expected.to contain_designate_config('coordination/run_watchers_interval').with_value('') @@ -15,15 +17,19 @@ describe 'designate::coordination' do context 'with specified parameters' do let :params do { - :backend_url => 'etcd3+http://127.0.0.1:2379', - :heartbeat_interval => 5.0, - :run_watchers_interval => 10.0, + :backend_url => 'etcd3+http://127.0.0.1:2379', + :heartbeat_interval => 5.0, + :run_watchers_interval => 10.0, + :manage_backend_package => false, + :package_ensure => 'latest', } end it { is_expected.to contain_oslo__coordination('designate_config').with( - :backend_url => 'etcd3+http://127.0.0.1:2379' + :backend_url => 'etcd3+http://127.0.0.1:2379', + :manage_backend_package => false, + :package_ensure => 'latest', ) is_expected.to contain_designate_config('coordination/heartbeat_interval').with_value(5.0) is_expected.to contain_designate_config('coordination/run_watchers_interval').with_value(10.0)