From d6327e99d786fc923fad4184bb3774060050f22d Mon Sep 17 00:00:00 2001 From: Takashi Kajinami Date: Wed, 16 Feb 2022 11:40:46 +0900 Subject: [PATCH] Add HashClient retry options for the dogpile.cache backend MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 Depends-On: https://review.opendev.org/826870 Change-Id: Icecc2329c7f4732ee63dbb34a1b8d9d99a30239a --- manifests/cache.pp | 22 +++++++++++++++++++ ...client_retry_options-cb26062940c71f69.yaml | 5 +++++ spec/classes/ec2api_cache_spec.rb | 9 ++++++++ 3 files changed, 36 insertions(+) create mode 100644 releasenotes/notes/add_cache_hashclient_retry_options-cb26062940c71f69.yaml diff --git a/manifests/cache.pp b/manifests/cache.pp index 8865dcb..598a3a6 100644 --- a/manifests/cache.pp +++ b/manifests/cache.pp @@ -133,6 +133,22 @@ # (Optional) Number of seconds to sleep between each attempt. # 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 ( $config_prefix = $::os_service_default, $expiration_time = $::os_service_default, @@ -156,6 +172,9 @@ class ec2api::cache ( $enable_retry_client = $::os_service_default, $retry_attempts = $::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 @@ -183,5 +202,8 @@ class ec2api::cache ( enable_retry_client => $enable_retry_client, retry_attempts => $retry_attempts, retry_delay => $retry_delay, + hashclient_retry_attempts => $hashclient_retry_attempts, + hashclient_retry_delay => $hashclient_retry_delay, + dead_timeout => $dead_timeout, } } diff --git a/releasenotes/notes/add_cache_hashclient_retry_options-cb26062940c71f69.yaml b/releasenotes/notes/add_cache_hashclient_retry_options-cb26062940c71f69.yaml new file mode 100644 index 0000000..116e436 --- /dev/null +++ b/releasenotes/notes/add_cache_hashclient_retry_options-cb26062940c71f69.yaml @@ -0,0 +1,5 @@ +--- +features: + - | + Add options to configure pymemcache's HashClient retrying mechanisms + (dogpile.cache) backend. diff --git a/spec/classes/ec2api_cache_spec.rb b/spec/classes/ec2api_cache_spec.rb index 27244d3..deaf964 100644 --- a/spec/classes/ec2api_cache_spec.rb +++ b/spec/classes/ec2api_cache_spec.rb @@ -32,6 +32,9 @@ describe 'ec2api::cache' do :enable_retry_client => '', :retry_attempts => '', :retry_delay => '', + :hashclient_retry_attempts => '', + :hashclient_retry_delay => '', + :dead_timeout => '', :manage_backend_package => true, ) end @@ -56,6 +59,9 @@ describe 'ec2api::cache' do :enable_retry_client => false, :retry_attempts => 2, :retry_delay => 0, + :hashclient_retry_attempts => 2, + :hashclient_retry_delay => 1, + :dead_timeout => 60, :manage_backend_package => false, } end @@ -83,6 +89,9 @@ describe 'ec2api::cache' do :enable_retry_client => false, :retry_attempts => 2, :retry_delay => 0, + :hashclient_retry_attempts => 2, + :hashclient_retry_delay => 1, + :dead_timeout => 60, :manage_backend_package => false, ) end