openstack/puppet-ceilometer
Code Issues Proposed changes

Require valid polling_interval

Browse Source 
Change-Id: Idd60111074d918c826cb51287e640b58deca62bd
Signed-off-by: Takashi Kajinami <kajinamit@oss.nttdata.com>
This commit is contained in:
Takashi Kajinami
2025-09-27 23:04:35 +09:00
parent 0022e86fd1
commit c81cca723c
2 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/agent/polling.pp
View File

@@ -142,7 +142,7 @@ class ceilometer::agent::polling (
$resource_update_interval = $facts['os_service_default'],
$resource_cache_expiry = $facts['os_service_default'],
Boolean $manage_polling = false,
$polling_interval = 300,
Integer[1] $polling_interval = 300,
Array[String[1]] $polling_meters = $ceilometer::params::polling_meters,
Optional[Hash] $polling_config = undef,
$cfg_file = $facts['os_service_default'],

8
spec/classes/ceilometer_agent_polling_spec.rb
View File

@@ -246,9 +246,11 @@ sources:
context 'with polling and basic custom settings' do
before do
params.merge!( :manage_polling => true,
:polling_interval => 30,
:polling_meters => ['meter1', 'meter2'] )
params.merge!({
:manage_polling => true,
:polling_interval => 30,
:polling_meters => ['meter1', 'meter2']
})
end
it { should contain_file('polling').with(