openstack/puppet-watcher
Code Issues Proposed changes

Remove watch_log_file

Browse Source 
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: Idbb6775f858b6fc8e920f61a080a95290cae1af3
This commit is contained in:
Takashi Kajinami 2025-04-16 21:01:15 +09:00
parent 018b250459
commit c53c9de78d
3 changed files with 4 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
manifests/logging.pp
View File

@ -101,12 +101,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 watcher::logging(
$use_syslog = $facts['os_service_default'],
$use_json = $facts['os_service_default'],
@ -127,8 +121,6 @@ class watcher::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 watcher::deps
@ -140,7 +132,6 @@ class watcher::logging(
use_journal => $use_journal,
log_dir => $log_dir,
log_file => $log_file,
watch_log_file => $watch_log_file,
debug => $debug,
logging_context_format_string => $logging_context_format_string,
logging_default_format_string => $logging_default_format_string,

4
releasenotes/notes/remove-watch_log_file-ed46ae46b102b317.yaml Normal file
View File

@ -0,0 +1,4 @@
---
upgrade:
- |
The ``watcher::logging::watch_log_file`` parameter has been removed.

2
spec/classes/watcher_logging_spec.rb
View File

@ -30,7 +30,6 @@ describe 'watcher::logging' do
:syslog_log_facility => 'LOG_FOO',
:log_dir => '/var/log',
:log_file => '/var/tmp/watcher_random.log',
:watch_log_file => true,
:debug => true,
}
end
@ -82,7 +81,6 @@ describe 'watcher::logging' do
:syslog_log_facility => 'LOG_FOO',
:log_dir => '/var/log',
:log_file => '/var/tmp/watcher_random.log',
:watch_log_file => true,
:debug => true,
)
end