Remove deprecated logging
authorTobias Urdin <tobias.urdin@binero.se>
Thu, 29 Nov 2018 08:58:25 +0000 (09:58 +0100)
committerTobias Urdin <tobias.urdin@binero.se>
Sat, 1 Dec 2018 14:58:16 +0000 (15:58 +0100)
Change-Id: I767309510ab5d8c314a6c7001255626289085d2d

manifests/init.pp
manifests/logging.pp
releasenotes/notes/remove-deprecated-logging-9a75efe3f778ff40.yaml [new file with mode: 0644]
spec/classes/ceilometer_init_spec.rb

index 85514a5cbe7d905bef7cc4efa5f97d24a7c111aa..9044af8e93c45bbafee1b68d98c8aae7564af234 100644 (file)
 #   (Optional) ensure state for package.
 #   Defaults to 'present'.
 #
-# [*debug*]
-#   (Optional) Should the daemons log debug messages.
-#   Defaults to undef.
-#
-# [*log_dir*]
-#   (Optional) Directory to which ceilometer logs are sent.
-#   If set to $::os_service_default, it will not log to any directory.
-#   Defaults to undef.
-#
-# [*use_syslog*]
-#   (Optional) Use syslog for logging
-#   Defaults to undef.
-#
-# [*use_stderr*]
-#   (Optional) Use stderr for logging
-#   Defaults to undef.
-#
-# [*log_facility*]
-#   (Optional) Syslog facility to receive log lines.
-#   Defaults to undef.
-#
 # [*default_transport_url*]
 #   (optional) A URL representing the messaging driver to use and its full
 #   configuration. Transport URLs take the form:
@@ -236,11 +215,6 @@ class ceilometer(
   $notification_topics                = ['notifications'],
   $notification_driver                = $::os_service_default,
   $package_ensure                     = 'present',
-  $debug                              = undef,
-  $log_dir                            = undef,
-  $use_syslog                         = undef,
-  $use_stderr                         = undef,
-  $log_facility                       = undef,
   $default_transport_url              = $::os_service_default,
   $rpc_response_timeout               = $::os_service_default,
   $control_exchange                   = $::os_service_default,
@@ -294,7 +268,6 @@ class ceilometer(
   }
 
   include ::ceilometer::deps
-  include ::ceilometer::logging
   include ::ceilometer::params
 
   group { 'ceilometer':
index 3c707b2d17ce98956e00dba369cc144f7f9ee2ee..4c34cc2f39785f3efb64eafccb6af18ad2d7b72f 100644 (file)
@@ -116,26 +116,14 @@ class ceilometer::logging(
 
   include ::ceilometer::deps
 
-  # NOTE(spredzy): In order to keep backward compatibility we rely on the pick function
-  # to use ceilometer::<myparam> first then ceilometer::logging::<myparam>.
-  $use_syslog_real = pick($::ceilometer::use_syslog,$use_syslog)
-  $use_stderr_real = pick($::ceilometer::use_stderr,$use_stderr)
-  $log_facility_real = pick($::ceilometer::log_facility,$log_facility)
-  if $log_dir != '' {
-    $log_dir_real = pick($::ceilometer::log_dir,$log_dir)
-  } else {
-    $log_dir_real = $log_dir
-  }
-  $debug_real = pick($::ceilometer::debug,$debug)
-
   oslo::log { 'ceilometer_config':
-    debug                         => $debug_real,
-    use_stderr                    => $use_stderr_real,
-    use_syslog                    => $use_syslog_real,
+    debug                         => $debug,
+    use_stderr                    => $use_stderr,
+    use_syslog                    => $use_syslog,
     use_json                      => $use_json,
     use_journal                   => $use_journal,
-    log_dir                       => $log_dir_real,
-    syslog_log_facility           => $log_facility_real,
+    log_dir                       => $log_dir,
+    syslog_log_facility           => $log_facility,
     logging_context_format_string => $logging_context_format_string,
     logging_default_format_string => $logging_default_format_string,
     logging_debug_format_suffix   => $logging_debug_format_suffix,
diff --git a/releasenotes/notes/remove-deprecated-logging-9a75efe3f778ff40.yaml b/releasenotes/notes/remove-deprecated-logging-9a75efe3f778ff40.yaml
new file mode 100644 (file)
index 0000000..83979a5
--- /dev/null
@@ -0,0 +1,5 @@
+---
+upgrade:
+  - |
+    The deprecated parameters use_syslog, use_stderr, log_facility, log_dir and
+    debug in the init class is now removed. Please set them in the logging class.
index 6711cf2b58e8685f86b1473ae9beb4fe9734976b..952133df59488a6a92b9a8bd67b89a1d7a60f29d 100644 (file)
@@ -7,9 +7,6 @@ describe 'ceilometer' do
       :http_timeout          => '600',
       :telemetry_secret      => 'metering-s3cr3t',
       :package_ensure        => 'present',
-      :debug                 => 'False',
-      :log_dir               => '/var/log/ceilometer',
-      :use_stderr            => 'True',
       :purge_config          => false,
       :host                  => 'foo.domain'
     }
@@ -63,7 +60,6 @@ describe 'ceilometer' do
 
   shared_examples_for 'a ceilometer base installation' do
 
-    it { is_expected.to contain_class('ceilometer::logging') }
     it { is_expected.to contain_class('ceilometer::params') }
 
     it 'configures ceilometer group' do