From 76d3d9e9c1e81058b9811d62f3a1e57612d964af Mon Sep 17 00:00:00 2001 From: Takashi Kajinami Date: Thu, 2 Oct 2025 14:30:14 +0900 Subject: [PATCH] coordination: Expose parameters for package management Change-Id: I78b811af8ef82f1a6e4b2a9cc024901ca4e1f7bb Signed-off-by: Takashi Kajinami --- manifests/coordination.pp | 22 ++++++++++++++----- ...ation-package-params-489e7b37d649fd92.yaml | 8 +++++++ spec/classes/aodh_coordination_spec.rb | 18 ++++++++++----- 3 files changed, 37 insertions(+), 11 deletions(-) create mode 100644 releasenotes/notes/coordination-package-params-489e7b37d649fd92.yaml diff --git a/manifests/coordination.pp b/manifests/coordination.pp index 5265beba..77e6468c 100644 --- a/manifests/coordination.pp +++ b/manifests/coordination.pp @@ -23,16 +23,28 @@ # group # 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 aodh::coordination ( - $backend_url = $facts['os_service_default'], - $heartbeat_interval = $facts['os_service_default'], - $retry_backoff = $facts['os_service_default'], - $max_retry_interval = $facts['os_service_default'], + $backend_url = $facts['os_service_default'], + $heartbeat_interval = $facts['os_service_default'], + $retry_backoff = $facts['os_service_default'], + $max_retry_interval = $facts['os_service_default'], + Boolean $manage_backend_package = true, + Stdlib::Ensure::Package $package_ensure = present, ) { include aodh::deps oslo::coordination { 'aodh_config': - backend_url => $backend_url, + backend_url => $backend_url, + manage_backend_package => $manage_backend_package, + package_ensure => $package_ensure, } aodh_config { diff --git a/releasenotes/notes/coordination-package-params-489e7b37d649fd92.yaml b/releasenotes/notes/coordination-package-params-489e7b37d649fd92.yaml new file mode 100644 index 00000000..a85bdcd3 --- /dev/null +++ b/releasenotes/notes/coordination-package-params-489e7b37d649fd92.yaml @@ -0,0 +1,8 @@ +--- +features: + - | + The following parameters have been added to the ``aodh::coordination`` + class. + + - ``manage_backend_package`` + - ``package_ensure`` diff --git a/spec/classes/aodh_coordination_spec.rb b/spec/classes/aodh_coordination_spec.rb index d55e60a2..342cfd52 100644 --- a/spec/classes/aodh_coordination_spec.rb +++ b/spec/classes/aodh_coordination_spec.rb @@ -5,7 +5,9 @@ describe 'aodh::coordination' do context 'with default parameters' do it { is_expected.to contain_oslo__coordination('aodh_config').with( - :backend_url => '' + :backend_url => '', + :manage_backend_package => true, + :package_ensure => 'present', ) is_expected.to contain_aodh_config('coordination/heartbeat_interval').with_value('') is_expected.to contain_aodh_config('coordination/retry_backoff').with_value('') @@ -16,16 +18,20 @@ describe 'aodh::coordination' do context 'with specified parameters' do let :params do { - :backend_url => 'etcd3+http://127.0.0.1:2379', - :heartbeat_interval => 1, - :retry_backoff => 1, - :max_retry_interval => 30, + :backend_url => 'etcd3+http://127.0.0.1:2379', + :heartbeat_interval => 1, + :retry_backoff => 1, + :max_retry_interval => 30, + :manage_backend_package => false, + :package_ensure => 'latest', } end it { is_expected.to contain_oslo__coordination('aodh_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_aodh_config('coordination/heartbeat_interval').with_value(1) is_expected.to contain_aodh_config('coordination/retry_backoff').with_value(1)