Switch heat-engine workers to use os_workers_heat_engine

Previously the default value for the heat-engine workers setting was set
to $::os_service_default. We usually use our specific facts for the
worker settings so this change switches it to use the
$::os_worker_heat_engine fact.

Change-Id: I6a8d745b76afca1b1958816fa10a3f1dd30face6
Related-Bug: #1763671
This commit is contained in:
Alex Schultz 2018-04-13 08:06:58 -06:00 committed by Emilien Macchi
parent bbb53148b1
commit 8adb596e33
3 changed files with 14 additions and 3 deletions

View File

@ -103,7 +103,7 @@
#
# [*num_engine_workers*]
# (Optional) The number of workers to spawn.
# Defaults to $::os_service_default.
# Defaults to $::os_workers_heat_engine
#
# [*convergence_engine*]
# (Optional) Enables engine with convergence architecture.
@ -147,7 +147,7 @@ class heat::engine (
$instance_connection_is_secure = $::os_service_default,
$instance_connection_https_validate_certificates = $::os_service_default,
$max_resources_per_stack = $::os_service_default,
$num_engine_workers = $::os_service_default,
$num_engine_workers = $::os_workers_heat_engine,
$convergence_engine = $::os_service_default,
$reauthentication_auth_method = $::os_service_default,
$environment_dir = $::os_service_default,

View File

@ -0,0 +1,11 @@
---
fixes:
- |
Previously the number of heat engine workers was set to $::os_service_default
which would use the default from python. Usually this means it will use
the number of processors in the system. We have found that this is not an
ideal default for large number of cpu systems. We have a provided fact
specific to heat engine named $::os_workers_heat_engine. We have changed
the default value for the heat engine works to use this fact. This
fact will cap the value at 24 but usually is the larger of
(number of procs / 2) or 4.

View File

@ -139,7 +139,7 @@ describe 'heat::engine' do
}).each do |os,facts|
context "on #{os}" do
let (:facts) do
facts.merge!(OSDefaults.get_facts())
facts.merge!(OSDefaults.get_facts({:os_workers_heat_engine => 2}))
end
let :platform_params do