openstack/puppet-keystone
Code Issues Proposed changes

Merge "Add support for [healthcheck] enable_by_file_paths"

Browse Source
This commit is contained in:
Zuul 2024-07-25 15:01:36 +00:00 committed by Gerrit Code Review
commit 19e9fcc6b7
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.
# 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 keystone::healthcheck (
$detailed = $facts['os_service_default'],
$backends = $facts['os_service_default'],
@ -39,6 +44,7 @@ class keystone::healthcheck (
$ignore_proxied_requests = $facts['os_service_default'],
$disable_by_file_path = $facts['os_service_default'],
$disable_by_file_paths = $facts['os_service_default'],
$enable_by_file_paths = $facts['os_service_default'],
) {
include keystone::deps
@ -50,5 +56,6 @@ class keystone::healthcheck (
ignore_proxied_requests => $ignore_proxied_requests,
disable_by_file_path => $disable_by_file_path,
disable_by_file_paths => $disable_by_file_paths,
enable_by_file_paths => $enable_by_file_paths,
}
}

5
releasenotes/notes/hc-enable_by_file_paths-99e9b04853dbdb6a.yaml Normal file
View File

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

3
spec/classes/keystone_healthcheck_spec.rb
View File

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