From a1ae65f32306a035f561b73f5dfb13077afe45fd Mon Sep 17 00:00:00 2001
From: ZhongShengping <chdzsp@163.com>
Date: Tue, 14 Sep 2021 16:02:09 +0800
Subject: [PATCH] Add watch_log_file option

Add support for Using logging handler designed to watch file system.

Change-Id: I2feecb96c4b859584ead37155269cb33d742e9d2
Closes-Bug: #1943212
---
 manifests/logging.pp                            |  6 ++++++
 ..._watch_log_file-option-aa4ff24a35c6459b.yaml |  4 ++++
 spec/classes/heat_logging_spec.rb               | 17 ++++++++++-------
 3 files changed, 20 insertions(+), 7 deletions(-)
 create mode 100644 releasenotes/notes/add_watch_log_file-option-aa4ff24a35c6459b.yaml

diff --git a/manifests/logging.pp b/manifests/logging.pp
index 24c57adf..5cbc5218 100644
--- a/manifests/logging.pp
+++ b/manifests/logging.pp
@@ -37,6 +37,10 @@
 #   (Optional) File where logs should be stored.
 #   Defaults to $::os_service_default
 #
+# [*watch_log_file*]
+#   (Optional) Uses logging handler designed to watch file system (boolean value).
+#   Defaults to $::os_service_default
+#
 # [*logging_context_format_string*]
 #   (Optional) Format string to use for log messages with context.
 #   Defaults to $::os_service_default
@@ -105,6 +109,7 @@ class heat::logging(
   $log_facility                  = $::os_service_default,
   $log_dir                       = '/var/log/heat',
   $log_file                      = $::os_service_default,
+  $watch_log_file                = $::os_service_default,
   $debug                         = $::os_service_default,
   $logging_context_format_string = $::os_service_default,
   $logging_default_format_string = $::os_service_default,
@@ -127,6 +132,7 @@ class heat::logging(
     log_date_format               => $log_date_format,
     log_dir                       => $log_dir,
     log_file                      => $log_file,
+    watch_log_file                => $watch_log_file,
     use_syslog                    => $use_syslog,
     use_json                      => $use_json,
     use_journal                   => $use_journal,
diff --git a/releasenotes/notes/add_watch_log_file-option-aa4ff24a35c6459b.yaml b/releasenotes/notes/add_watch_log_file-option-aa4ff24a35c6459b.yaml
new file mode 100644
index 00000000..7c40384a
--- /dev/null
+++ b/releasenotes/notes/add_watch_log_file-option-aa4ff24a35c6459b.yaml
@@ -0,0 +1,4 @@
+---
+features:
+  - |
+    The new ``heat::logging::watch_log_file`` parameter has been added.
diff --git a/spec/classes/heat_logging_spec.rb b/spec/classes/heat_logging_spec.rb
index 24a7c61c..ba7cd85b 100644
--- a/spec/classes/heat_logging_spec.rb
+++ b/spec/classes/heat_logging_spec.rb
@@ -30,6 +30,7 @@ describe 'heat::logging' do
      :log_facility                   => 'LOG_FOO',
      :log_dir                        => '/var/log',
      :log_file                       => '/var/log/heat/heat.log',
+     :watch_log_file                 => true,
      :debug                          => true,
     }
   end
@@ -59,13 +60,14 @@ describe 'heat::logging' do
   shared_examples 'basic default logging settings' do
     it 'configures heat logging settings with default values' do
       is_expected.to contain_oslo__log('heat_config').with(
-        :use_syslog  => '<SERVICE DEFAULT>',
-        :use_json    => '<SERVICE DEFAULT>',
-        :use_journal => '<SERVICE DEFAULT>',
-        :use_stderr  => '<SERVICE DEFAULT>',
-        :log_dir     => '/var/log/heat',
-        :log_file    => '<SERVICE DEFAULT>',
-        :debug       => '<SERVICE DEFAULT>',
+        :use_syslog     => '<SERVICE DEFAULT>',
+        :use_json       => '<SERVICE DEFAULT>',
+        :use_journal    => '<SERVICE DEFAULT>',
+        :use_stderr     => '<SERVICE DEFAULT>',
+        :log_dir        => '/var/log/heat',
+        :log_file       => '<SERVICE DEFAULT>',
+        :watch_log_file => '<SERVICE DEFAULT>',
+        :debug          => '<SERVICE DEFAULT>',
       )
     end
   end
@@ -80,6 +82,7 @@ describe 'heat::logging' do
         :syslog_log_facility => 'LOG_FOO',
         :log_dir             => '/var/log',
         :log_file            => '/var/log/heat/heat.log',
+        :watch_log_file      => true,
         :debug               => true,
       )
     end