From d585e8a17d6c5a55883bb5dde8079346ec0a3813 Mon Sep 17 00:00:00 2001 From: Nathan Kinder Date: Mon, 3 Jun 2019 15:27:55 +0200 Subject: [PATCH] Don't require memcached_node_ips when deploying keystone The keystone module always expects memcached_node_ips to be defined in hieradata, the value of which is used to configure mod_auth_openidc when OpenID Connect is enabled for federation. In some cases, such as when using a trimmed down custom Controller role for development purposes, memcached may not even be deployed. This will result in memcached_node_ips not being set, which causes a deployment failure. This patch defaults memcached_ips to an empty list, which allows a deployment of keystone to succeed even when memcached is not being deployed. Change-Id: If44b6d11f8c41c96bd823c3e38bacdc08034986d --- manifests/profile/base/keystone.pp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/manifests/profile/base/keystone.pp b/manifests/profile/base/keystone.pp index d9ed40b42..9dd02e781 100644 --- a/manifests/profile/base/keystone.pp +++ b/manifests/profile/base/keystone.pp @@ -190,7 +190,7 @@ class tripleo::profile::base::keystone ( $keystone_enable_member = hiera('keystone_enable_member', false), $keystone_federation_enabled = hiera('keystone_federation_enabled', false), $keystone_openidc_enabled = hiera('keystone_openidc_enabled', false), - $memcached_ips = hiera('memcached_node_ips') + $memcached_ips = hiera('memcached_node_ips', []) ) { if $::hostname == downcase($bootstrap_node) { $sync_db = true