openstack/puppet-tripleo
Code Issues Proposed changes

Don't require memcached_node_ips when deploying keystone

Browse Source 
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
This commit is contained in:
Nathan Kinder
2019-06-03 15:27:55 +02:00
parent 27c01401d0
commit d585e8a17d
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/profile/base/keystone.pp
View File

@@ -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