openstack
/
puppet-ceilometer
Code Issues Proposed changes

Merge "Add support for [DEFAULT] max_parallel_requests"

This commit is contained in:
Zuul 2021-11-09 08:33:05 +00:00 committed by Gerrit Code Review
parent 31b5d7f00b b1ebc891f1
commit 2ce16ad4b8
3 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
manifests/init.pp
View File

@ -9,6 +9,11 @@
# (Optional) Timeout seconds for HTTP requests. # (Optional) Timeout seconds for HTTP requests.
# Defaults to 600. # Defaults to 600.
# #
# [*max_parallel_requests*]
# (Optional) Maximum number of parallel requests for services to handle at
# the same time.
# Defaults to $::os_service_default
#
# [*telemetry_secret*] # [*telemetry_secret*]
# (Required) Secret key for signing messages. # (Required) Secret key for signing messages.
# #
@ -264,6 +269,7 @@
# #
class ceilometer( class ceilometer(
$http_timeout = '600', $http_timeout = '600',
$max_parallel_requests = $::os_service_default,
$telemetry_secret = false, $telemetry_secret = false,
$notification_topics = ['notifications'], $notification_topics = ['notifications'],
$notification_driver = $::os_service_default, $notification_driver = $::os_service_default,
@ -392,6 +398,7 @@ will be removed in a future release.')
# Once we got here, we can act as an honey badger on the rpc used. # Once we got here, we can act as an honey badger on the rpc used.
ceilometer_config { ceilometer_config {
'DEFAULT/http_timeout' : value => $http_timeout; 'DEFAULT/http_timeout' : value => $http_timeout;
'DEFAULT/max_parallel_requests' : value => $max_parallel_requests;
'DEFAULT/host' : value => $host; 'DEFAULT/host' : value => $host;
'publisher/telemetry_secret' : value => $telemetry_secret, secret => true; 'publisher/telemetry_secret' : value => $telemetry_secret, secret => true;
'hardware/readonly_user_name' : value => $snmpd_readonly_username; 'hardware/readonly_user_name' : value => $snmpd_readonly_username;

4
releasenotes/notes/max_parallel_requests-ef9a035e9ecf0951.yaml Normal file
View File

@ -0,0 +1,4 @@
---
features:
- |
The new ``ceilometer::max_parallel_requests`` parameter has been added.

5
spec/classes/ceilometer_init_spec.rb
View File

@ -5,6 +5,7 @@ describe 'ceilometer' do
let :params do let :params do
{ {
:http_timeout => '600', :http_timeout => '600',
:max_parallel_requests => 64,
:telemetry_secret => 'metering-s3cr3t', :telemetry_secret => 'metering-s3cr3t',
:package_ensure => 'present', :package_ensure => 'present',
:purge_config => false, :purge_config => false,
@ -24,6 +25,10 @@ describe 'ceilometer' do
is_expected.to contain_ceilometer_config('DEFAULT/http_timeout').with_value(params[:http_timeout]) is_expected.to contain_ceilometer_config('DEFAULT/http_timeout').with_value(params[:http_timeout])
end end
it 'configures max_parallel_requests' do
is_expected.to contain_ceilometer_config('DEFAULT/max_parallel_requests').with_value(params[:max_parallel_requests])
end
it 'configures host name' do it 'configures host name' do
is_expected.to contain_ceilometer_config('DEFAULT/host').with_value(params[:host]) is_expected.to contain_ceilometer_config('DEFAULT/host').with_value(params[:host])
end end