diff --git a/manifests/cache.pp b/manifests/cache.pp index e03f35477..a20146b5f 100644 --- a/manifests/cache.pp +++ b/manifests/cache.pp @@ -203,12 +203,12 @@ # the HashClient's internal mechanisms. # Default to $facts['os_service_default'] # -# [*hashclient_retry_delay*] +# [*hashclient_retry_timeout*] # (Optional) Time in seconds that should pass between # retry attempts in the HashClient's internal mechanisms. # Default to $facts['os_service_default'] # -# [*dead_timeout*] +# [*hashclient_dead_timeout*] # (Optional) Time in seconds before attempting to add a node # back in the pool in the HashClient's internal mechanisms. # Default to $facts['os_service_default'] @@ -217,6 +217,18 @@ # (Optional) Whether to install the backend package for the cache. # Defaults to true # +# DEPRECATED PARAMETERS +# +# [*hashclient_retry_delay*] +# (Optional) Time in seconds that should pass between +# retry attempts in the HashClient's internal mechanisms. +# Default to undef +# +# [*dead_timeout*] +# (Optional) Time in seconds before attempting to add a node +# back in the pool in the HashClient's internal mechanisms. +# Default to undef +# class nova::cache ( $config_prefix = $facts['os_service_default'], $expiration_time = $facts['os_service_default'], @@ -255,9 +267,11 @@ class nova::cache ( $retry_attempts = $facts['os_service_default'], $retry_delay = $facts['os_service_default'], $hashclient_retry_attempts = $facts['os_service_default'], - $hashclient_retry_delay = $facts['os_service_default'], - $dead_timeout = $facts['os_service_default'], + $hashclient_retry_timeout = $facts['os_service_default'], + $hashclient_dead_timeout = $facts['os_service_default'], Boolean $manage_backend_package = true, + $hashclient_retry_delay = undef, + $dead_timeout = undef, ) { include nova::deps @@ -299,9 +313,11 @@ class nova::cache ( retry_attempts => $retry_attempts, retry_delay => $retry_delay, hashclient_retry_attempts => $hashclient_retry_attempts, + hashclient_retry_timeout => $hashclient_retry_timeout, + hashclient_dead_timeout => $hashclient_dead_timeout, + manage_backend_package => $manage_backend_package, hashclient_retry_delay => $hashclient_retry_delay, dead_timeout => $dead_timeout, - manage_backend_package => $manage_backend_package, } # all cache settings should be applied and all packages should be installed diff --git a/releasenotes/notes/cache-rename-hashclient-params-492c9f2d3fa0f00c.yaml b/releasenotes/notes/cache-rename-hashclient-params-492c9f2d3fa0f00c.yaml new file mode 100644 index 000000000..622037d84 --- /dev/null +++ b/releasenotes/notes/cache-rename-hashclient-params-492c9f2d3fa0f00c.yaml @@ -0,0 +1,8 @@ +--- +deprecations: + - | + The ``nova::cache::hashclient_retry_delay`` parameter was deprecated + in favor of the new ``hashclient_retry_timeout`` parameter. + - | + The ``nova::cache::dead_timeout`` parameter was deprecated in favor + of the new ``hashclient_dead_timeout`` parameter. diff --git a/spec/classes/nova_cache_spec.rb b/spec/classes/nova_cache_spec.rb index a11384d50..e37ddafc5 100644 --- a/spec/classes/nova_cache_spec.rb +++ b/spec/classes/nova_cache_spec.rb @@ -48,9 +48,11 @@ describe 'nova::cache' do :retry_attempts => '', :retry_delay => '', :hashclient_retry_attempts => '', - :hashclient_retry_delay => '', - :dead_timeout => '', + :hashclient_retry_timeout => '', + :hashclient_dead_timeout => '', :manage_backend_package => true, + :hashclient_retry_delay => nil, + :dead_timeout => nil, ) end end @@ -90,9 +92,11 @@ describe 'nova::cache' do :retry_attempts => 2, :retry_delay => 0, :hashclient_retry_attempts => 2, - :hashclient_retry_delay => 1, - :dead_timeout => 60, + :hashclient_retry_timeout => 1, + :hashclient_dead_timeout => 60, :manage_backend_package => false, + :hashclient_retry_delay => 3, + :dead_timeout => 61, } end @@ -135,9 +139,11 @@ describe 'nova::cache' do :retry_attempts => 2, :retry_delay => 0, :hashclient_retry_attempts => 2, - :hashclient_retry_delay => 1, - :dead_timeout => 60, + :hashclient_retry_timeout => 1, + :hashclient_dead_timeout => 60, :manage_backend_package => false, + :hashclient_retry_delay => 3, + :dead_timeout => 61, ) end end