Expose use_json logging option

It enables JSON-formatted logging from oslo.log.

Change-Id: Ifc742e16748195c3cb699ac1e1dc6a36b3861d02

diff --git a/manifests/logging.pp b/manifests/logging.pp
index 748b91c7ddab984d7f2058a8667c0361e00645a4..a77f4b828533dd9a9f27acaf2e2b33bf10af271c 100644 (file)
--- a/manifests/logging.pp
+++ b/manifests/logging.pp
@@ -12,6 +12,10 @@
 #   (Optional) Use syslog for logging.
 #   Defaults to $::os_service_default.
 #
+# [*use_json*]
+#   (Optional) Use json for logging.
+#   Defaults to $::os_service_default.
+#
 # [*use_stderr*]
 #   (Optional) Use stderr for logging
 #   Defaults to $::os_service_default.
@@ -87,6 +91,7 @@
 #
 class ceilometer::logging(
   $use_syslog                    = $::os_service_default,
+  $use_json                      = $::os_service_default,
   $use_stderr                    = $::os_service_default,
   $log_facility                  = $::os_service_default,
   $log_dir                       = '/var/log/ceilometer',
@@ -122,6 +127,7 @@ class ceilometer::logging(
     debug                         => $debug_real,
     use_stderr                    => $use_stderr_real,
     use_syslog                    => $use_syslog_real,
+    use_json                      => $use_json,
     log_dir                       => $log_dir_real,
     syslog_log_facility           => $log_facility_real,
     logging_context_format_string => $logging_context_format_string,

diff --git a/releasenotes/notes/log-json-19a16ad82e22fd82.yaml b/releasenotes/notes/log-json-19a16ad82e22fd82.yaml
new file mode 100644 (file)
index 0000000..41c0413
--- /dev/null
+++ b/releasenotes/notes/log-json-19a16ad82e22fd82.yaml
@@ -0,0 +1,4 @@
+---
+features:
+  - |
+    Expose use_json logging parameter, which enables JSON formatted logging.

diff --git a/spec/classes/ceilometer_logging_spec.rb b/spec/classes/ceilometer_logging_spec.rb
index 8e7c7229bd8eb00c3bda57208b4e36539b91b428..fdcbb010dfa2e2fa1ecfd1533b1b21624a87287d 100644 (file)
--- a/spec/classes/ceilometer_logging_spec.rb
+++ b/spec/classes/ceilometer_logging_spec.rb
@@ -24,6 +24,7 @@ describe 'ceilometer::logging' do
      :instance_uuid_format => '[instance: %(uuid)s] ',
      :log_date_format => '%Y-%m-%d %H:%M:%S',
      :use_syslog => true,
+     :use_json => true,
      :use_stderr => false,
      :log_facility => 'LOG_FOO',
      :log_dir => '/var/log',
@@ -57,6 +58,7 @@ describe 'ceilometer::logging' do
     it 'configures ceilometer logging settings with default values' do
       is_expected.to contain_oslo__log('ceilometer_config').with(
         :use_syslog          => '<SERVICE DEFAULT>',
+        :use_json            => '<SERVICE DEFAULT>',
         :use_stderr          => '<SERVICE DEFAULT>',
         :syslog_log_facility => '<SERVICE DEFAULT>',
         :log_dir             => '/var/log/ceilometer',
@@ -69,6 +71,7 @@ describe 'ceilometer::logging' do
     it 'configures ceilometer logging settings with non-default values' do
       is_expected.to contain_oslo__log('ceilometer_config').with(
         :use_syslog          => true,
+        :use_json            => true,
         :use_stderr          => false,
         :syslog_log_facility => 'LOG_FOO',
         :log_dir             => '/var/log',