From 2367cb9461c4f13b0edcb4263e8c9381be0eaed1 Mon Sep 17 00:00:00 2001
From: Takashi Kajinami <kajinamit@oss.nttdata.com>
Date: Wed, 16 Apr 2025 20:45:48 +0900
Subject: [PATCH] Remove watch_log_file

The option was deprecated during the 2024.2 cycle[1]. Processes may
fail to start when it is set to True[2] so supporting it is no longer
useful.

[1] 31603ac6dacd0b47ec9f02d99bfdd0a7b683119c
[2] b977cb9becbeaaa67bb663bd8db5aa6846e495b9

Change-Id: Ia54b48da9f692d67c4d08a3170417bd636d41e2d
---
 manifests/logging.pp                                     | 9 ---------
 .../notes/remove-watch_log_file-164da5017252a969.yaml    | 4 ++++
 spec/classes/ceilometer_logging_spec.rb                  | 2 --
 3 files changed, 4 insertions(+), 11 deletions(-)
 create mode 100644 releasenotes/notes/remove-watch_log_file-164da5017252a969.yaml

diff --git a/manifests/logging.pp b/manifests/logging.pp
index f5306659..e68fc119 100644
--- a/manifests/logging.pp
+++ b/manifests/logging.pp
@@ -97,12 +97,6 @@
 #   Defaults to $facts['os_service_default'].
 #   Example: 'Y-%m-%d %H:%M:%S'
 #
-# DEPRECATED PARAMETERS
-#
-# [*watch_log_file*]
-#   (Optional) Uses logging handler designed to watch file system (boolean value).
-#   Defaults to undef
-#
 class ceilometer::logging(
   $use_syslog                    = $facts['os_service_default'],
   $use_json                      = $facts['os_service_default'],
@@ -123,8 +117,6 @@ class ceilometer::logging(
   $instance_format               = $facts['os_service_default'],
   $instance_uuid_format          = $facts['os_service_default'],
   $log_date_format               = $facts['os_service_default'],
-  # DEPRECATED PARAMETERS
-  $watch_log_file                = undef,
 ) {
 
   include ceilometer::deps
@@ -137,7 +129,6 @@ class ceilometer::logging(
     use_journal                   => $use_journal,
     log_dir                       => $log_dir,
     log_file                      => $log_file,
-    watch_log_file                => $watch_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/remove-watch_log_file-164da5017252a969.yaml b/releasenotes/notes/remove-watch_log_file-164da5017252a969.yaml
new file mode 100644
index 00000000..211676d6
--- /dev/null
+++ b/releasenotes/notes/remove-watch_log_file-164da5017252a969.yaml
@@ -0,0 +1,4 @@
+---
+upgrade:
+  - |
+    The ``ceilometer::logging::watch_log_file`` parameter has been removed.
diff --git a/spec/classes/ceilometer_logging_spec.rb b/spec/classes/ceilometer_logging_spec.rb
index 468bbde9..3f0fcd2a 100644
--- a/spec/classes/ceilometer_logging_spec.rb
+++ b/spec/classes/ceilometer_logging_spec.rb
@@ -30,7 +30,6 @@ describe 'ceilometer::logging' do
      :log_facility                   => 'LOG_FOO',
      :log_dir                        => '/var/log',
      :log_file                       => '/var/log/ceilometer/ceilometer.log',
-     :watch_log_file                 => true,
      :debug                          => true,
     }
   end
@@ -82,7 +81,6 @@ describe 'ceilometer::logging' do
         :syslog_log_facility => 'LOG_FOO',
         :log_dir             => '/var/log',
         :log_file            => '/var/log/ceilometer/ceilometer.log',
-        :watch_log_file      => true,
         :debug               => true,
       )
     end