From 985c48951b42810cbfcf3a077c8773a8787fa6d2 Mon Sep 17 00:00:00 2001 From: Sergey Kolekonov Date: Fri, 7 Aug 2015 15:44:01 +0300 Subject: [PATCH] Add an ability to manage use_stderr parameter When logging is already set up, it can be useful to disable this option to avoid logs duplication (collected by Upstart, for example) Change-Id: I6c3e33e8ac3b018941eb0a1c584701c4483a3a11 Closes-bug: #1482564 --- manifests/init.pp | 5 +++++ spec/classes/ceilometer_init_spec.rb | 5 +++++ 2 files changed, 10 insertions(+) diff --git a/manifests/init.pp b/manifests/init.pp index c44c18c..0f8d28a 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -33,6 +33,9 @@ # [*use_syslog*] # (optional) Use syslog for logging # Defaults to false +# [*use_stderr*] +# (optional) Use stderr for logging +# Defaults to true # [*log_facility*] # (optional) Syslog facility to receive log lines. # Defaults to 'LOG_USER' @@ -115,6 +118,7 @@ class ceilometer( $log_dir = '/var/log/ceilometer', $verbose = false, $use_syslog = false, + $use_stderr = true, $log_facility = 'LOG_USER', $rpc_backend = 'rabbit', $rabbit_host = '127.0.0.1', @@ -297,6 +301,7 @@ class ceilometer( 'publisher/metering_secret' : value => $metering_secret, secret => true; 'DEFAULT/debug' : value => $debug; 'DEFAULT/verbose' : value => $verbose; + 'DEFAULT/use_stderr' : value => $use_stderr; 'DEFAULT/notification_topics' : value => join($notification_topics, ','); 'database/event_time_to_live' : value => $event_time_to_live; 'database/metering_time_to_live' : value => $metering_time_to_live; diff --git a/spec/classes/ceilometer_init_spec.rb b/spec/classes/ceilometer_init_spec.rb index 60fd3f0..ceb1bbe 100644 --- a/spec/classes/ceilometer_init_spec.rb +++ b/spec/classes/ceilometer_init_spec.rb @@ -12,6 +12,7 @@ describe 'ceilometer' do :debug => 'False', :log_dir => '/var/log/ceilometer', :verbose => 'False', + :use_stderr => 'True', } end @@ -145,6 +146,10 @@ describe 'ceilometer' do is_expected.to contain_ceilometer_config('DEFAULT/verbose').with_value( params[:verbose] ) end + it 'configures use_stderr option' do + is_expected.to contain_ceilometer_config('DEFAULT/use_stderr').with_value( params[:use_stderr] ) + end + it 'configures logging directory by default' do is_expected.to contain_ceilometer_config('DEFAULT/log_dir').with_value( params[:log_dir] ) end -- 2.45.2