Add HashClient retry options for the dogpile.cache backend
This patch specifies a set of options required to setup the HashClient retry feature of dogpile.cache cache backend. Original oslo.cache change: https://review.opendev.org/c/openstack/oslo.cache/+/824944 Co-Authored-By: Hervé Beraud <hberaud@redhat.com> Depends-On: https://review.opendev.org/826870 Change-Id: Icecc2329c7f4732ee63dbb34a1b8d9d99a30239a
This commit is contained in:
parent
f2678624ec
commit
d6327e99d7
|
@ -133,6 +133,22 @@
|
||||||
# (Optional) Number of seconds to sleep between each attempt.
|
# (Optional) Number of seconds to sleep between each attempt.
|
||||||
# Default to $::os_service_default
|
# Default to $::os_service_default
|
||||||
#
|
#
|
||||||
|
# [*hashclient_retry_attempts*]
|
||||||
|
# (Optional) Amount of times a client should be tried
|
||||||
|
# before it is marked dead and removed from the pool in
|
||||||
|
# the HashClient's internal mechanisms.
|
||||||
|
# Default to $::os_service_default
|
||||||
|
#
|
||||||
|
# [*hashclient_retry_delay*]
|
||||||
|
# (Optional) Time in seconds that should pass between
|
||||||
|
# retry attempts in the HashClient's internal mechanisms.
|
||||||
|
# Default to $::os_service_default
|
||||||
|
#
|
||||||
|
# [*dead_timeout*]
|
||||||
|
# (Optional) Time in seconds before attempting to add a node
|
||||||
|
# back in the pool in the HashClient's internal mechanisms.
|
||||||
|
# Default to $::os_service_default
|
||||||
|
#
|
||||||
class ec2api::cache (
|
class ec2api::cache (
|
||||||
$config_prefix = $::os_service_default,
|
$config_prefix = $::os_service_default,
|
||||||
$expiration_time = $::os_service_default,
|
$expiration_time = $::os_service_default,
|
||||||
|
@ -156,6 +172,9 @@ class ec2api::cache (
|
||||||
$enable_retry_client = $::os_service_default,
|
$enable_retry_client = $::os_service_default,
|
||||||
$retry_attempts = $::os_service_default,
|
$retry_attempts = $::os_service_default,
|
||||||
$retry_delay = $::os_service_default,
|
$retry_delay = $::os_service_default,
|
||||||
|
$hashclient_retry_attempts = $::os_service_default,
|
||||||
|
$hashclient_retry_delay = $::os_service_default,
|
||||||
|
$dead_timeout = $::os_service_default,
|
||||||
) {
|
) {
|
||||||
|
|
||||||
include ec2api::deps
|
include ec2api::deps
|
||||||
|
@ -183,5 +202,8 @@ class ec2api::cache (
|
||||||
enable_retry_client => $enable_retry_client,
|
enable_retry_client => $enable_retry_client,
|
||||||
retry_attempts => $retry_attempts,
|
retry_attempts => $retry_attempts,
|
||||||
retry_delay => $retry_delay,
|
retry_delay => $retry_delay,
|
||||||
|
hashclient_retry_attempts => $hashclient_retry_attempts,
|
||||||
|
hashclient_retry_delay => $hashclient_retry_delay,
|
||||||
|
dead_timeout => $dead_timeout,
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -0,0 +1,5 @@
|
||||||
|
---
|
||||||
|
features:
|
||||||
|
- |
|
||||||
|
Add options to configure pymemcache's HashClient retrying mechanisms
|
||||||
|
(dogpile.cache) backend.
|
|
@ -32,6 +32,9 @@ describe 'ec2api::cache' do
|
||||||
:enable_retry_client => '<SERVICE DEFAULT>',
|
:enable_retry_client => '<SERVICE DEFAULT>',
|
||||||
:retry_attempts => '<SERVICE DEFAULT>',
|
:retry_attempts => '<SERVICE DEFAULT>',
|
||||||
:retry_delay => '<SERVICE DEFAULT>',
|
:retry_delay => '<SERVICE DEFAULT>',
|
||||||
|
:hashclient_retry_attempts => '<SERVICE DEFAULT>',
|
||||||
|
:hashclient_retry_delay => '<SERVICE DEFAULT>',
|
||||||
|
:dead_timeout => '<SERVICE DEFAULT>',
|
||||||
:manage_backend_package => true,
|
:manage_backend_package => true,
|
||||||
)
|
)
|
||||||
end
|
end
|
||||||
|
@ -56,6 +59,9 @@ describe 'ec2api::cache' do
|
||||||
:enable_retry_client => false,
|
:enable_retry_client => false,
|
||||||
:retry_attempts => 2,
|
:retry_attempts => 2,
|
||||||
:retry_delay => 0,
|
:retry_delay => 0,
|
||||||
|
:hashclient_retry_attempts => 2,
|
||||||
|
:hashclient_retry_delay => 1,
|
||||||
|
:dead_timeout => 60,
|
||||||
:manage_backend_package => false,
|
:manage_backend_package => false,
|
||||||
}
|
}
|
||||||
end
|
end
|
||||||
|
@ -83,6 +89,9 @@ describe 'ec2api::cache' do
|
||||||
:enable_retry_client => false,
|
:enable_retry_client => false,
|
||||||
:retry_attempts => 2,
|
:retry_attempts => 2,
|
||||||
:retry_delay => 0,
|
:retry_delay => 0,
|
||||||
|
:hashclient_retry_attempts => 2,
|
||||||
|
:hashclient_retry_delay => 1,
|
||||||
|
:dead_timeout => 60,
|
||||||
:manage_backend_package => false,
|
:manage_backend_package => false,
|
||||||
)
|
)
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in New Issue