openstack/puppet-zaqar
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: I62acc833d4589017bc3c3c54550b038911572bc6
This commit is contained in:
Takashi Kajinami 2025-04-16 21:01:49 +09:00
parent 7ec4db9e61
commit 6ccb4a4dfc
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 $facts['os_service_default']
#
class zaqar::logging(
$use_syslog = $facts['os_service_default'],
$use_json = $facts['os_service_default'],
@ -127,8 +121,6 @@ class zaqar::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 zaqar::deps
@ -152,7 +144,6 @@ class zaqar::logging(
syslog_log_facility => $log_facility,
log_dir => $log_dir,
log_file => $log_file,
watch_log_file => $watch_log_file,
debug => $debug,
}

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

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

2
spec/classes/zaqar_logging_spec.rb
View File

@ -30,7 +30,6 @@ describe 'zaqar::logging' do
:log_facility => 'LOG_FOO',
:log_dir => '/var/log',
:log_file => '/var/log/zaqar.log',
:watch_log_file => true,
:debug => true,
}
end
@ -72,7 +71,6 @@ describe 'zaqar::logging' do
:syslog_log_facility => 'LOG_FOO',
:log_dir => '/var/log',
:log_file => '/var/log/zaqar.log',
:watch_log_file => true,
:debug => true,
)
end