openstack/puppet-aodh
Code Issues Proposed changes

Add support for [healthcheck] enable_by_file_paths

Browse Source 
Depends-on: https://review.opendev.org/923648
Change-Id: Ib84c504c1634c9f965539247b5f5516e39f45620
This commit is contained in:
Takashi Kajinami 2024-07-21 20:21:37 +09:00
parent 6cdb315944
commit d9656f8fdf
3 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
manifests/healthcheck.pp
View File

@ -32,6 +32,11 @@
# is running on a port. Expects a "port:path" list of strings. # is running on a port. Expects a "port:path" list of strings.
# Defaults to $facts['os_service_default'] # Defaults to $facts['os_service_default']
# #
# [*enable_by_file_paths*]
# (Optional) Check the presence of files. Used by EnableByFilesHealthcheck
# plugin.
# Defaults to $facts['os_service_default']
#
class aodh::healthcheck ( class aodh::healthcheck (
$detailed = $facts['os_service_default'], $detailed = $facts['os_service_default'],
$backends = $facts['os_service_default'], $backends = $facts['os_service_default'],
@ -39,6 +44,7 @@ class aodh::healthcheck (
$ignore_proxied_requests = $facts['os_service_default'], $ignore_proxied_requests = $facts['os_service_default'],
$disable_by_file_path = $facts['os_service_default'], $disable_by_file_path = $facts['os_service_default'],
$disable_by_file_paths = $facts['os_service_default'], $disable_by_file_paths = $facts['os_service_default'],
$enable_by_file_paths = $facts['os_service_default'],
) { ) {
include aodh::deps include aodh::deps
@ -50,5 +56,6 @@ class aodh::healthcheck (
ignore_proxied_requests => $ignore_proxied_requests, ignore_proxied_requests => $ignore_proxied_requests,
disable_by_file_path => $disable_by_file_path, disable_by_file_path => $disable_by_file_path,
disable_by_file_paths => $disable_by_file_paths, disable_by_file_paths => $disable_by_file_paths,
enable_by_file_paths => $enable_by_file_paths,
} }
} }

5
releasenotes/notes/hc-enable_by_file_paths-17fc4a38a2e0e5a1.yaml Normal file
View File

@ -0,0 +1,5 @@
---
features:
- |
The new ``aodh::healthcheck::enable_by_file_paths`` parameter has been
added.

3
spec/classes/aodh_healthcheck_spec.rb
View File

@ -17,6 +17,7 @@ describe 'aodh::healthcheck' do
:ignore_proxied_requests => '<SERVICE DEFAULT>', :ignore_proxied_requests => '<SERVICE DEFAULT>',
:disable_by_file_path => '<SERVICE DEFAULT>', :disable_by_file_path => '<SERVICE DEFAULT>',
:disable_by_file_paths => '<SERVICE DEFAULT>', :disable_by_file_paths => '<SERVICE DEFAULT>',
:enable_by_file_paths => '<SERVICE DEFAULT>',
) )
end end
end end
@ -30,6 +31,7 @@ describe 'aodh::healthcheck' do
:ignore_proxied_requests => false, :ignore_proxied_requests => false,
:disable_by_file_path => '/etc/aodh/healthcheck/disabled', :disable_by_file_path => '/etc/aodh/healthcheck/disabled',
:disable_by_file_paths => ['8042:/etc/aodh/healthcheck/disabled'], :disable_by_file_paths => ['8042:/etc/aodh/healthcheck/disabled'],
:enable_by_file_paths => ['/etc/aodh/healthcheck/enabled'],
} }
end end
@ -41,6 +43,7 @@ describe 'aodh::healthcheck' do
:ignore_proxied_requests => false, :ignore_proxied_requests => false,
:disable_by_file_path => '/etc/aodh/healthcheck/disabled', :disable_by_file_path => '/etc/aodh/healthcheck/disabled',
:disable_by_file_paths => ['8042:/etc/aodh/healthcheck/disabled'], :disable_by_file_paths => ['8042:/etc/aodh/healthcheck/disabled'],
:enable_by_file_paths => ['/etc/aodh/healthcheck/enabled'],
) )
end end
end end