From d82e6d3edb02dbb70652502eb3aad83596532a17 Mon Sep 17 00:00:00 2001 From: Lance Bragstad Date: Thu, 5 Mar 2020 14:32:12 -0600 Subject: [PATCH] Update the number of keystone workers We recently patched puppet-keystone and puppet-openstacklib to set KeystoneWorkers to a reasonable default given the consolidation of two keystone processes into a single keystone process (keystone-main and keystone-admin): https://review.opendev.org/#/c/702031/13 https://review.opendev.org/#/c/705041/3 This patch updates THT to ensure we're using the new variable that's dedicated to keystone's process count. This ensures there isn't a performance regression due to reduced process counts when upgrading to newer OpenStack versions because the keystone applications were consolidated in puppet-keystone. Change-Id: I47506181c38b9363b9c82c536e7c9d7d765f093f (cherry picked from commit 8c728bde8457f6571c48f9c51827e6049ab4278f) --- deployment/keystone/keystone-container-puppet.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/deployment/keystone/keystone-container-puppet.yaml b/deployment/keystone/keystone-container-puppet.yaml index 12e504e4f4..e051743caf 100644 --- a/deployment/keystone/keystone-container-puppet.yaml +++ b/deployment/keystone/keystone-container-puppet.yaml @@ -104,7 +104,7 @@ parameters: KeystoneWorkers: type: string description: Set the number of workers for keystone::wsgi::apache - default: '%{::os_workers}' + default: '%{::os_workers_keystone}' MonitoringSubscriptionKeystone: default: 'overcloud-keystone' type: string