openstack/puppet-zaqar
Code Issues Proposed changes

Add support for [cache] backend_expiration_time

Browse Source 
Depends-on: https://review.opendev.org/933978
Change-Id: I65379644da9a41f1c4b5b0d67d2d9dd69d0694de
This commit is contained in:
Takashi Kajinami 2024-11-03 09:26:54 +09:00
parent 02c15edeeb
commit 1035997c1b
3 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
manifests/cache.pp
View File

@ -18,6 +18,11 @@
# (integer value) # (integer value)
# Defaults to $facts['os_service_default'] # Defaults to $facts['os_service_default']
# #
# [*backend_expiration_time*]
# (Optional) Expiration time in cache backend to purge expired records
# automatically.
# Defaults to $facts['os_service_default']
#
# [*backend*] # [*backend*]
# (Optional) Dogpile.cache backend module. It is recommended that # (Optional) Dogpile.cache backend module. It is recommended that
# Memcache with pooling (oslo_cache.memcache_pool) or Redis # Memcache with pooling (oslo_cache.memcache_pool) or Redis
@ -215,6 +220,7 @@
class zaqar::cache ( class zaqar::cache (
$config_prefix = $facts['os_service_default'], $config_prefix = $facts['os_service_default'],
$expiration_time = $facts['os_service_default'], $expiration_time = $facts['os_service_default'],
$backend_expiration_time = $facts['os_service_default'],
$backend = $facts['os_service_default'], $backend = $facts['os_service_default'],
$backend_argument = $facts['os_service_default'], $backend_argument = $facts['os_service_default'],
$proxies = $facts['os_service_default'], $proxies = $facts['os_service_default'],
@ -259,6 +265,7 @@ class zaqar::cache (
oslo::cache { 'zaqar_config': oslo::cache { 'zaqar_config':
config_prefix => $config_prefix, config_prefix => $config_prefix,
expiration_time => $expiration_time, expiration_time => $expiration_time,
backend_expiration_time => $backend_expiration_time,
backend => $backend, backend => $backend,
backend_argument => $backend_argument, backend_argument => $backend_argument,
proxies => $proxies, proxies => $proxies,

4
releasenotes/notes/cache-backend-expiration-time-46609285b7b759fe.yaml Normal file
View File

@ -0,0 +1,4 @@
---
features:
- |
The new ``zaqar::cache::backend_expiration_time`` parameter has been added.

3
spec/classes/zaqar_cache_spec.rb
View File

@ -13,6 +13,7 @@ describe 'zaqar::cache' do
is_expected.to contain_oslo__cache('zaqar_config').with( is_expected.to contain_oslo__cache('zaqar_config').with(
:config_prefix => '<SERVICE DEFAULT>', :config_prefix => '<SERVICE DEFAULT>',
:expiration_time => '<SERVICE DEFAULT>', :expiration_time => '<SERVICE DEFAULT>',
:backend_expiration_time => '<SERVICE DEFAULT>',
:backend => '<SERVICE DEFAULT>', :backend => '<SERVICE DEFAULT>',
:backend_argument => '<SERVICE DEFAULT>', :backend_argument => '<SERVICE DEFAULT>',
:proxies => '<SERVICE DEFAULT>', :proxies => '<SERVICE DEFAULT>',
@ -59,6 +60,7 @@ describe 'zaqar::cache' do
{ {
:config_prefix => 'prefix', :config_prefix => 'prefix',
:expiration_time => 3600, :expiration_time => 3600,
:backend_expiration_time => 7200,
:backend => 'oslo_cache.memcache_pool', :backend => 'oslo_cache.memcache_pool',
:proxies => ['proxy01:8888', 'proxy02:8888'], :proxies => ['proxy01:8888', 'proxy02:8888'],
:enabled => true, :enabled => true,
@ -98,6 +100,7 @@ describe 'zaqar::cache' do
is_expected.to contain_oslo__cache('zaqar_config').with( is_expected.to contain_oslo__cache('zaqar_config').with(
:config_prefix => 'prefix', :config_prefix => 'prefix',
:expiration_time => 3600, :expiration_time => 3600,
:backend_expiration_time => 7200,
:backend => 'oslo_cache.memcache_pool', :backend => 'oslo_cache.memcache_pool',
:backend_argument => '<SERVICE DEFAULT>', :backend_argument => '<SERVICE DEFAULT>',
:proxies => ['proxy01:8888', 'proxy02:8888'], :proxies => ['proxy01:8888', 'proxy02:8888'],