openstack
/
puppet-cinder
Code Issues Proposed changes

Fix the default of cinder::api::ratelimits_factory 

The cinder::api class introduces RateLimitingMiddleware when
the ratelimits parameter is true. However the current default of
ratelimits_factory still points to v1 API code which was already
removed, and this results in an invalid definition.

This change ensures that it points to v2 API code which still exists.

Closes-Bug: #1941860
Change-Id: I97d14585ebb28166c95755169df0ea5433adb3e8
This commit is contained in:
Takashi Kajinami 2021-09-01 08:42:56 +09:00
parent d0c1bfd14e
commit 3111f64dd2
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/api.pp
View File

@ -30,7 +30,7 @@
#
# [*ratelimits_factory*]
# (optional) Factory to use for ratelimiting
# Defaults to 'cinder.api.v1.limits:RateLimitingMiddleware.factory'
# Defaults to 'cinder.api.v2.limits:RateLimitingMiddleware.factory'
#
# [*default_volume_type*]
# (optional) default volume type to use.
@ -126,7 +126,7 @@ class cinder::api (
$ratelimits = $::os_service_default,
$default_volume_type = $::os_service_default,
$ratelimits_factory =
'cinder.api.v1.limits:RateLimitingMiddleware.factory',
'cinder.api.v2.limits:RateLimitingMiddleware.factory',
$validate = false,
$sync_db = true,
$public_endpoint = $::os_service_default,