Change Nova Conductor workers to ::os_workers

In Ieb69b5a6a50d37b0a49a8c9e57275673d60b5f10, the default
number of workers for the metadata_workers and osapi_compute_workers
was changed from $::processorcount to $::os_workers, which takes
CPU count into account but caps the number of workers for
machines with high numbers of CPUs.   The
"workers" configuration option used by Nova Conductor however
was left out of this, likely because it did not previously
default to $::processorcount.  However Nova itself defaults
all of these services to the processor count in any case,
and the number of workers for Conductor is likely the most critical
value to be limited, since it directly corresponds to pooled
database connections which when there are too many processes
results in often thousands of connections that are almost all
unused.   This patch limits Conductor workers in the same
way as that of the other worker counts.

Change-Id: I1969b555c1c8cfb686a12cad7f54ac10b1656af5
This commit is contained in:
Mike Bayer 2017-09-22 18:04:53 -04:00
parent 0469856f16
commit 9fe9b56aeb
2 changed files with 8 additions and 2 deletions

View File

@ -18,7 +18,7 @@
#
# [*workers*]
# (optional) Number of workers for OpenStack Conductor service
# Defaults to undef (i.e. parameter will not be present)
# Defaults to $::os_workers
#
# [*enable_new_services*]
# (optional) When a new service (for example "nova-compute") start up, it gets
@ -37,7 +37,7 @@ class nova::conductor(
$enabled = true,
$manage_service = true,
$ensure_package = 'present',
$workers = undef,
$workers = $::os_workers,
$enable_new_services = $::os_service_default,
# DEPREACTED PARAMETERS
$use_local = undef,

View File

@ -0,0 +1,6 @@
---
other:
- Added Nova Conductor service to the list of services where
the default number of spawned child processes for
distributing processing have had their default value changed from
::processorcount to ::os_workers.