openstack/puppet-designate
Code Issues Proposed changes

coordination: Expose parameters for package management

Browse Source 
Change-Id: I2932e6166f0d2c26f343d81daaf607c69c977723
Signed-off-by: Takashi Kajinami <kajinamit@oss.nttdata.com>
This commit is contained in:
Takashi Kajinami
2025-10-02 14:44:05 +09:00
parent c994ad8a9c
commit 69243bf89e
3 changed files with 35 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
manifests/coordination.pp
View File

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

8
releasenotes/notes/coordination-package-params-0dab58d520940fee.yaml Normal file
View File

@@ -0,0 +1,8 @@
---
features:
- |
The following parameters have been added to the ``designate::coordination``
class.
- ``manage_backend_package``
- ``package_ensure``

16
spec/classes/designate_coordination_spec.rb
View File

@@ -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 => '<SERVICE DEFAULT>'
:backend_url => '<SERVICE DEFAULT>',
:manage_backend_package => true,
:package_ensure => 'present',
)
is_expected.to contain_designate_config('coordination/heartbeat_interval').with_value('<SERVICE DEFAULT>')
is_expected.to contain_designate_config('coordination/run_watchers_interval').with_value('<SERVICE DEFAULT>')
@@ -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)