From c08634706b714d65299e6f8b3cb62aabaa985158 Mon Sep 17 00:00:00 2001
From: ZhongShengping <chdzsp@163.com>
Date: Tue, 12 Mar 2019 10:37:40 +0800
Subject: [PATCH] Add log_file parameter

Change-Id: I32bb5ff447420710194136da6f36e2418269268d
Closes-Bug: #1819417
---
 manifests/logging.pp                                        | 6 ++++++
 .../notes/add-log_file-for-logging-ab7202a308cd6c2b.yaml    | 4 ++++
 spec/classes/vitrage_logging_spec.rb                        | 3 +++
 3 files changed, 13 insertions(+)
 create mode 100644 releasenotes/notes/add-log_file-for-logging-ab7202a308cd6c2b.yaml

diff --git a/manifests/logging.pp b/manifests/logging.pp
index 1e1354e..849643c 100644
--- a/manifests/logging.pp
+++ b/manifests/logging.pp
@@ -35,6 +35,10 @@
 #   any directory.
 #   Defaults to '/var/log/vitrage'.
 #
+# [*log_file*]
+#   (Optional) File where logs should be stored.
+#   Defaults to $::os_service_default
+#
 # [*logging_context_format_string*]
 #   (Optional) Format string to use for log messages with context.
 #   Defaults to $::os_service_default
@@ -102,6 +106,7 @@ class vitrage::logging(
   $use_stderr                    = $::os_service_default,
   $log_facility                  = $::os_service_default,
   $log_dir                       = '/var/log/vitrage',
+  $log_file                      = $::os_service_default,
   $debug                         = $::os_service_default,
   $logging_context_format_string = $::os_service_default,
   $logging_default_format_string = $::os_service_default,
@@ -125,6 +130,7 @@ class vitrage::logging(
     use_journal                   => $use_journal,
     use_stderr                    => $use_stderr,
     log_dir                       => $log_dir,
+    log_file                      => $log_file,
     syslog_log_facility           => $log_facility,
     logging_context_format_string => $logging_context_format_string,
     logging_default_format_string => $logging_default_format_string,
diff --git a/releasenotes/notes/add-log_file-for-logging-ab7202a308cd6c2b.yaml b/releasenotes/notes/add-log_file-for-logging-ab7202a308cd6c2b.yaml
new file mode 100644
index 0000000..e40d4c8
--- /dev/null
+++ b/releasenotes/notes/add-log_file-for-logging-ab7202a308cd6c2b.yaml
@@ -0,0 +1,4 @@
+---
+features:
+  - |
+    Add the log_file option for logging.
diff --git a/spec/classes/vitrage_logging_spec.rb b/spec/classes/vitrage_logging_spec.rb
index b29f7ac..51e0ac8 100644
--- a/spec/classes/vitrage_logging_spec.rb
+++ b/spec/classes/vitrage_logging_spec.rb
@@ -29,6 +29,7 @@ describe 'vitrage::logging' do
      :use_stderr                     => false,
      :log_facility                   => 'LOG_FOO',
      :log_dir                        => '/var/log',
+     :log_file                       => '/var/log/vitrage.log',
      :debug                          => true,
     }
   end
@@ -64,6 +65,7 @@ describe 'vitrage::logging' do
         :use_stderr          => '<SERVICE DEFAULT>',
         :syslog_log_facility => '<SERVICE DEFAULT>',
         :log_dir             => '/var/log/vitrage',
+        :log_file            => '<SERVICE DEFAULT>',
         :debug               => '<SERVICE DEFAULT>',
       )
     end
@@ -78,6 +80,7 @@ describe 'vitrage::logging' do
         :use_stderr          => false,
         :syslog_log_facility => 'LOG_FOO',
         :log_dir             => '/var/log',
+        :log_file            => '/var/log/vitrage.log',
         :debug               => true,
       )
     end