openstack/puppet-ceilometer
Code Issues Proposed changes

Do not hardcode default of [DEFAULT] http_timeout

Browse Source 
Currently the ceilometer::http_timeout parameter defaults to 600 but
this equals to the default value defined in ceilometer itself.
This change removes that hardcode in puppet-ceilometer and just let
the service default used.

Change-Id: Idbd14e267b8759f7313a1a74db03f47e69a899aa
This commit is contained in:
Takashi Kajinami
2021-11-08 23:19:26 +09:00
parent b1ebc891f1
commit ed000c0cda
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/init.pp
View File

@@ -7,7 +7,7 @@
#
# [*http_timeout*]
# (Optional) Timeout seconds for HTTP requests.
# Defaults to 600.
# Defaults to $::os_service_default
#
# [*max_parallel_requests*]
# (Optional) Maximum number of parallel requests for services to handle at
@@ -268,7 +268,7 @@
# Defaults to undef.
#
class ceilometer(
$http_timeout = '600',
$http_timeout = $::os_service_default,
$max_parallel_requests = $::os_service_default,
$telemetry_secret = false,
$notification_topics = ['notifications'],