From 57f338f8dbce4918b729a25baea187d9a2a96ee5 Mon Sep 17 00:00:00 2001 From: Pradeep Kilambi Date: Wed, 19 Apr 2017 10:58:05 -0400 Subject: [PATCH] Add metric_processing_delay param Expose this param to tweak the delay between processing metrics in gnocchi. Change-Id: I5de5283d1b14e0bba63d6d9a440611914ba86ca4 (cherry picked from commit 315733a00c12fdc6586094394852f7e7d05c3eca) --- manifests/storage.pp | 12 +++++++++--- .../notes/add-metric-delay-c4b5db22171d9165.yaml | 3 +++ spec/classes/gnocchi_storage_spec.rb | 6 +++++- 3 files changed, 17 insertions(+), 4 deletions(-) create mode 100644 releasenotes/notes/add-metric-delay-c4b5db22171d9165.yaml diff --git a/manifests/storage.pp b/manifests/storage.pp index 2b7bf348..7ff9947e 100644 --- a/manifests/storage.pp +++ b/manifests/storage.pp @@ -24,10 +24,15 @@ # (optional) The url to use for distributed group membership coordination. # Defaults to $::os_service_default. # +# [*metric_processing_delay*] +# (optional) Delay between processng metrics +# Defaults to $::os_service_default. +# class gnocchi::storage( - $package_ensure = 'present', - $coordination_url = $::os_service_default, + $package_ensure = 'present', + $coordination_url = $::os_service_default, + $metric_processing_delay = $::os_service_default, ) inherits gnocchi::params { package { 'gnocchi-carbonara': @@ -37,7 +42,8 @@ class gnocchi::storage( } gnocchi_config { - 'storage/coordination_url' : value => $coordination_url; + 'storage/coordination_url' : value => $coordination_url; + 'storage/metric_processing_delay' : value => $metric_processing_delay; } } diff --git a/releasenotes/notes/add-metric-delay-c4b5db22171d9165.yaml b/releasenotes/notes/add-metric-delay-c4b5db22171d9165.yaml new file mode 100644 index 00000000..419aef34 --- /dev/null +++ b/releasenotes/notes/add-metric-delay-c4b5db22171d9165.yaml @@ -0,0 +1,3 @@ +--- +fixes: + - Expose metric processing delay param. diff --git a/spec/classes/gnocchi_storage_spec.rb b/spec/classes/gnocchi_storage_spec.rb index eb949ba3..25cd4dfa 100644 --- a/spec/classes/gnocchi_storage_spec.rb +++ b/spec/classes/gnocchi_storage_spec.rb @@ -20,11 +20,15 @@ describe 'gnocchi::storage' do context 'with coordination' do before do - params.merge!({ :coordination_url => 'redis://localhost:6379' }) + params.merge!({ + :coordination_url => 'redis://localhost:6379', + :metric_processing_delay => 30, + }) end it 'configures backend_url' do is_expected.to contain_gnocchi_config('storage/coordination_url').with_value('redis://localhost:6379') + is_expected.to contain_gnocchi_config('storage/metric_processing_delay').with_value(30) end end end