From 2ee44729da778040830b28d90e3c90fcefb7e47b Mon Sep 17 00:00:00 2001
From: Marcus Furlong <furlongm@gmail.com>
Date: Thu, 14 Apr 2016 05:35:31 +0000
Subject: [PATCH] add missing num_engine_workers parameter

This option exists for heat::api but is missing for heat::engine.
This commit adds the missing num_engine_workers parameter.

Change-Id: I69565c13b732ef74531a516aac8a368597411856
---
 manifests/engine.pp              | 6 ++++++
 spec/classes/heat_engine_spec.rb | 2 ++
 2 files changed, 8 insertions(+)

diff --git a/manifests/engine.pp b/manifests/engine.pp
index a771aa0e..64ed802c 100644
--- a/manifests/engine.pp
+++ b/manifests/engine.pp
@@ -89,6 +89,10 @@
 #   (Optional) Maximum resources allowed per top-level stack.
 #   Defaults to $::os_service_default
 #
+# [*num_engine_workers*]
+#   (Optional) The number of workers to spawn.
+#   Defaults to $::os_service_default.
+#
 class heat::engine (
   $auth_encryption_key,
   $package_ensure                                  = 'present',
@@ -106,6 +110,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,
 ) {
 
   include ::heat::deps
@@ -159,5 +164,6 @@ class heat::engine (
     'DEFAULT/max_resources_per_stack':                         value => $max_resources_per_stack;
     'DEFAULT/instance_connection_https_validate_certificates': value => $instance_connection_https_validate_certificates;
     'DEFAULT/instance_connection_is_secure':                   value => $instance_connection_is_secure;
+    'DEFAULT/num_engine_workers':                              value => $num_engine_workers;
   }
 }
diff --git a/spec/classes/heat_engine_spec.rb b/spec/classes/heat_engine_spec.rb
index 7f501489..1d5a1f46 100644
--- a/spec/classes/heat_engine_spec.rb
+++ b/spec/classes/heat_engine_spec.rb
@@ -32,6 +32,7 @@ describe 'heat::engine' do
         :deferred_auth_method          => 'trusts',
         :default_software_config_transport   => 'POLL_SERVER_CFN',
         :default_deployment_signal_transport => 'CFN_SIGNAL',
+        :num_engine_workers            => '4',
       }
     ].each do |new_params|
       describe 'when #{param_set == {} ? "using default" : "specifying"} parameters'
@@ -72,6 +73,7 @@ describe 'heat::engine' do
       it { is_expected.to contain_heat_config('DEFAULT/instance_connection_is_secure').with_value('<SERVICE DEFAULT>') }
       it { is_expected.to contain_heat_config('DEFAULT/instance_connection_https_validate_certificates').with_value('<SERVICE DEFAULT>') }
       it { is_expected.to contain_heat_config('DEFAULT/max_resources_per_stack').with_value('<SERVICE DEFAULT>') }
+      it { is_expected.to contain_heat_config('DEFAULT/num_engine_workers').with_value( expected_params[:num_engine_workers] ) }
     end
 
     context 'with disabled service managing' do