]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Add support for [DEFAULT] max_parallel_requests
authorTakashi Kajinami <tkajinam@redhat.com>
Mon, 8 Nov 2021 14:18:22 +0000 (23:18 +0900)
committerTakashi Kajinami <tkajinam@redhat.com>
Mon, 8 Nov 2021 14:18:22 +0000 (23:18 +0900)
Change-Id: I8fc892301c06d9ff0cb4a687c02c35fcb6942d11

manifests/init.pp
releasenotes/notes/max_parallel_requests-ef9a035e9ecf0951.yaml [new file with mode: 0644]
spec/classes/ceilometer_init_spec.rb

index c4d514f5524f8386d52c53b72ec87dfc0fa66a7b..e46ade2dc4fefefbc2b5f6ef3e271867624c495a 100644 (file)
@@ -9,6 +9,11 @@
 #   (Optional) Timeout seconds for HTTP requests.
 #   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*]
 #  (Required)  Secret key for signing messages.
 #
 #
 class ceilometer(
   $http_timeout                       = '600',
+  $max_parallel_requests              = $::os_service_default,
   $telemetry_secret                   = false,
   $notification_topics                = ['notifications'],
   $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.
   ceilometer_config {
     'DEFAULT/http_timeout'                : value => $http_timeout;
+    'DEFAULT/max_parallel_requests'       : value => $max_parallel_requests;
     'DEFAULT/host'                        : value => $host;
     'publisher/telemetry_secret'          : value => $telemetry_secret, secret => true;
     'hardware/readonly_user_name'         : value => $snmpd_readonly_username;
diff --git a/releasenotes/notes/max_parallel_requests-ef9a035e9ecf0951.yaml b/releasenotes/notes/max_parallel_requests-ef9a035e9ecf0951.yaml
new file mode 100644 (file)
index 0000000..ca9997e
--- /dev/null
@@ -0,0 +1,4 @@
+---
+features:
+  - |
+    The new ``ceilometer::max_parallel_requests`` parameter has been added.
index 1b0834354eceaccf1a7557e12cff5fee61a59a59..da1305e652c40c83692eaa3d2aa2ff47c71d138b 100644 (file)
@@ -5,6 +5,7 @@ describe 'ceilometer' do
   let :params do
     {
       :http_timeout          => '600',
+      :max_parallel_requests => 64,
       :telemetry_secret      => 'metering-s3cr3t',
       :package_ensure        => 'present',
       :purge_config          => false,
@@ -24,6 +25,10 @@ describe 'ceilometer' do
       is_expected.to contain_ceilometer_config('DEFAULT/http_timeout').with_value(params[:http_timeout])
     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
       is_expected.to contain_ceilometer_config('DEFAULT/host').with_value(params[:host])
     end