Use workers instead of threads in wsgi by default

In most of puppet-openstack modules, we use workers instead of threads
in wsgi to realize concurrency of api process by default.

This patch makes puppet-vitrage follow that standard configuration,
so that the default setup can be more consistent among all modules.

Change-Id: Icf5c7620291fe10e2a15e2189ea144751974fdb5

manifests/wsgi/apache.pp

@@ -47,7 +47,7 @@
 #
 #   [*workers*]
 #     Number of WSGI workers to spawn.
-#     Optional. Defaults to 1
+#     Optional. Defaults to $::os_workers
 #
 #   [*priority*]
 #     (optional) The priority for the vhost.
@@ -55,7 +55,7 @@
 #
 #   [*threads*]
 #     (optional) The number of threads for the vhost.
-#     Defaults to $::os_workers
+#     Defaults to 1
 #
 #   [*access_log_file*]
 #     The log file name for the virtualhost.
@@ -107,7 +107,7 @@ class vitrage::wsgi::apache (
   $bind_host                   = undef,
   $path                        = '/',
   $ssl                         = true,
-  $workers                     = 1,
+  $workers                     = $::os_workers,
   $wsgi_process_display_name   = undef,
   $ssl_cert                    = undef,
   $ssl_key                     = undef,
@@ -116,7 +116,7 @@ class vitrage::wsgi::apache (
   $ssl_crl_path                = undef,
   $ssl_crl                     = undef,
   $ssl_certs_dir               = undef,
-  $threads                     = $::os_workers,
+  $threads                     = 1,
   $priority                    = '10',
   $access_log_file             = false,
   $access_log_format           = false,

releasenotes/notes/wsgi-use-workers-38fb5ced1f3fd979.yaml

@@ -0,0 +1,5 @@
+---
+fixes:
+  - |
+    In wsgi configuration, now workers is used to define concurrency by default
+    instead of threads, to be consistent with the other puppet modules.

spec/classes/vitrage_wsgi_apache_spec.rb

@@ -14,9 +14,9 @@ describe 'vitrage::wsgi::apache' do
         :path                        => '/',
         :servername                  => facts[:fqdn],
         :ssl                         => true,
-        :threads                     => facts[:os_workers],
+        :threads                     => 1,
         :user                        => 'vitrage',
-        :workers                     => 1,
+        :workers                     => facts[:os_workers],
         :wsgi_daemon_process         => 'vitrage',
         :wsgi_process_group          => 'vitrage',
         :wsgi_script_dir             => platform_params[:wsgi_script_path],
@@ -56,7 +56,7 @@ describe 'vitrage::wsgi::apache' do
         :path                        => '/',
         :servername                  => 'dummy.host',
         :ssl                         => false,
-        :threads                     => facts[:os_workers],
+        :threads                     => 1,
         :user                        => 'vitrage',
         :workers                     => 37,
         :wsgi_daemon_process         => 'vitrage',