From b2954993591b09b8924128b8b797a71751c10bed Mon Sep 17 00:00:00 2001 From: Bogdan Dobrelya Date: Wed, 8 Dec 2021 17:49:28 +0100 Subject: [PATCH] [Train-Only] Fix hiera condition for cache_backend There is a condition misplaced in hiera config for cache_backend. That set the caching backend to dogpile.cache.null. In the master/wallaby, the hiera's cache_backend misconfig had been fixed by https://review.opendev.org/q/topic:simplify_tht We cannot backport that for T,U and V, so let's provide only related fixes there. Change-Id: Ia595d1e636c8d74dd811f240626363481a1a741c Related: rhbz#1981915 Signed-off-by: Bogdan Dobrelya --- deployment/heat/heat-base-puppet.yaml | 7 ++++--- deployment/keystone/keystone-container-puppet.yaml | 9 +++++---- deployment/nova/nova-base-puppet.yaml | 7 ++++--- 3 files changed, 13 insertions(+), 10 deletions(-) diff --git a/deployment/heat/heat-base-puppet.yaml b/deployment/heat/heat-base-puppet.yaml index 037a56a4d4..df00403281 100644 --- a/deployment/heat/heat-base-puppet.yaml +++ b/deployment/heat/heat-base-puppet.yaml @@ -211,7 +211,8 @@ outputs: heat::cache::enabled: {get_param: EnableCache} heat::cache::tls_enabled: {get_param: MemcachedTLS} heat::cache::resource_finder_caching: false - if: + heat::cache::backend: + if: - tls_cache_enabled - - heat::cache::backend: 'dogpile.cache.pymemcache' - - heat::cache::backend: 'dogpile.cache.memcached' + - 'dogpile.cache.pymemcache' + - 'dogpile.cache.memcached' diff --git a/deployment/keystone/keystone-container-puppet.yaml b/deployment/keystone/keystone-container-puppet.yaml index 38d0b807ed..200c3c303d 100644 --- a/deployment/keystone/keystone-container-puppet.yaml +++ b/deployment/keystone/keystone-container-puppet.yaml @@ -501,10 +501,11 @@ outputs: - keystone::cache_enabled: {get_param: EnableCache} keystone::tls_enabled: {get_param: MemcachedTLS} - if: - - tls_cache_enabled - - keystone::cache_backend: 'dogpile.cache.pymemcache' - - keystone::cache_backend: 'dogpile.cache.memcached' + keystone::cache_backend: + if: + - tls_cache_enabled + - 'dogpile.cache.pymemcache' + - 'dogpile.cache.memcached' - if: - keystone_federation_enabled diff --git a/deployment/nova/nova-base-puppet.yaml b/deployment/nova/nova-base-puppet.yaml index 3b9531ebe7..28eb5ecaa7 100644 --- a/deployment/nova/nova-base-puppet.yaml +++ b/deployment/nova/nova-base-puppet.yaml @@ -353,10 +353,11 @@ outputs: - get_attr: [RoleParametersValue, value] - nova::cache::enabled: {get_param: EnableCache} nova::cache::tls_enabled: {get_param: MemcachedTLS} - if: + nova::cache::backend: + if: - tls_cache_enabled - - nova::cache::backend: 'dogpile.cache.pymemcache' - - nova::cache::backend: 'dogpile.cache.memcached' + - 'dogpile.cache.pymemcache' + - 'dogpile.cache.memcached' - if: - compute_upgrade_level_empty - {}