openstack/puppet-aodh
Code Issues Proposed changes

healthcheck: Expose allowed_source_ranges

Browse Source 
... which was added to puppet-oslo recently.

Depends-on: https://review.opendev.org/905557
Change-Id: I28bfa93296cf1a1cb2f69da3e92ad86d64b08ab5
This commit is contained in:
Takashi Kajinami 2024-01-17 02:38:00 +09:00
parent 94a5e732f0
commit 39d3dcc438
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

@ -13,6 +13,11 @@
# that information back as part of a request. # that information back as part of a request.
# Defaults to $facts['os_service_default'] # Defaults to $facts['os_service_default']
# #
# [*allowed_source_ranges*]
# (Optional) A list of network addresses to limit source ip allowed to access
# healthcheck information.
# Defaults to $facts['os_service_default']
#
# [*disable_by_file_path*] # [*disable_by_file_path*]
# (Optional) Check the presence of a file to determine if an application # (Optional) Check the presence of a file to determine if an application
# is running on a port. # is running on a port.
@ -26,6 +31,7 @@
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'],
$allowed_source_ranges = $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'],
) { ) {
@ -35,6 +41,7 @@ class aodh::healthcheck (
oslo::healthcheck { 'aodh_config': oslo::healthcheck { 'aodh_config':
detailed => $detailed, detailed => $detailed,
backends => $backends, backends => $backends,
allowed_source_ranges => $allowed_source_ranges,
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,
} }

5
releasenotes/notes/healthcheck-allowed_source_ranges-9ef4907fe56656cb.yaml Normal file
View File

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

3
spec/classes/aodh_healthcheck_spec.rb
View File

@ -13,6 +13,7 @@ describe 'aodh::healthcheck' do
is_expected.to contain_oslo__healthcheck('aodh_config').with( is_expected.to contain_oslo__healthcheck('aodh_config').with(
:detailed => '<SERVICE DEFAULT>', :detailed => '<SERVICE DEFAULT>',
:backends => '<SERVICE DEFAULT>', :backends => '<SERVICE DEFAULT>',
:allowed_source_ranges => '<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>',
) )
@ -24,6 +25,7 @@ describe 'aodh::healthcheck' do
{ {
:detailed => true, :detailed => true,
:backends => ['disable_by_file'], :backends => ['disable_by_file'],
:allowed_source_ranges => ['10.0.0.0/24', '10.0.1.0/24'],
: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'],
} }
@ -33,6 +35,7 @@ describe 'aodh::healthcheck' do
is_expected.to contain_oslo__healthcheck('aodh_config').with( is_expected.to contain_oslo__healthcheck('aodh_config').with(
:detailed => true, :detailed => true,
:backends => ['disable_by_file'], :backends => ['disable_by_file'],
:allowed_source_ranges => ['10.0.0.0/24', '10.0.1.0/24'],
: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'],
) )