From 807ec54afdbea607df5fb1dc6dc27a3f71e404f0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Dulko?= Date: Fri, 26 May 2017 15:42:05 +0200 Subject: [PATCH] Fix some warnings from `keystone-manage doctor` This commit fixes some warnings shown by `keystone-manage doctor` command that are present in Keystone installed with default chart configuration. In particular: * Set max_toke_size=32 as this is correct value for uuid token provider. * Enable caching using memcache by setting [cache] memcache_servers option correctly in configmap-etc.yaml Change-Id: I38cc7be577e0a7cd9be715e633f3637baafcc21b Closes-Bug: 1693806 --- keystone/templates/configmap-etc.yaml | 2 +- keystone/values.yaml | 8 ++++++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/keystone/templates/configmap-etc.yaml b/keystone/templates/configmap-etc.yaml index de6c0e541b..c51751074f 100644 --- a/keystone/templates/configmap-etc.yaml +++ b/keystone/templates/configmap-etc.yaml @@ -22,7 +22,7 @@ {{- tuple "oslo_messaging" "internal" "user" "amqp" . | include "helm-toolkit.endpoints.authenticated_endpoint_uri_lookup" | set .Values.conf.keystone.default.oslo.messaging "transport_url" | quote | trunc 0 -}} {{- end -}} -{{- if empty .Values.conf.keystone.cache.oslo.cache -}} +{{- if empty .Values.conf.keystone.cache.oslo.cache.memcache_servers -}} {{- tuple "oslo_cache" "internal" "memcache" . | include "helm-toolkit.endpoints.hostname_endpoint_uri_lookup" | set .Values.conf.keystone.cache.oslo.cache "memcache_servers" | quote | trunc 0 -}} {{- end -}} diff --git a/keystone/values.yaml b/keystone/values.yaml index e7a27ef0ba..011260b172 100644 --- a/keystone/values.yaml +++ b/keystone/values.yaml @@ -128,6 +128,9 @@ conf: keystone: override: append: + default: + keystone: + max_token_size: 32 token: keystone: provider: uuid @@ -135,6 +138,11 @@ conf: oslo: db: max_retries: -1 + cache: + oslo: + cache: + enabled: true + backend: dogpile.cache.memcached # typically overriden by environmental # values, but should include all endpoints