add parameter to overwrite/add wsgi process options

Add parameter to apache_wsgi to allow overwrite
and/or add additional wsgi process options.

This possibility was added to openstacklib
with Change-Id: I41914ce3361988d5db1695f09d21209772fdf548
lease enter the commit message for your changes. Lines starting

Change-Id: I477745910b2f03e75d5faa280825c5ca45daae27
This commit is contained in:
ZhongShengping 2017-10-23 14:09:27 +08:00 committed by zhongshengping
parent 4eb6e9b75f
commit 0ef4004e85
3 changed files with 108 additions and 87 deletions

View File

@ -83,6 +83,14 @@
# apache::vhost ssl parameters.
# Optional. Default to apache::vhost 'ssl_*' defaults.
#
# [*custom_wsgi_process_options*]
# (optional) gives you the oportunity to add custom process options or to
# overwrite the default options for the WSGI main process.
# eg. to use a virtual python environment for the WSGI process
# you could set it to:
# { python-path => '/my/python/virtualenv' }
# Defaults to {}
#
# == Dependencies
#
# requires Class['apache'] & Class['vitrage']
@ -94,25 +102,26 @@
# class { 'vitrage::wsgi::apache': }
#
class vitrage::wsgi::apache (
$servername = $::fqdn,
$port = 8999,
$bind_host = undef,
$path = '/',
$ssl = true,
$workers = 1,
$wsgi_process_display_name = undef,
$ssl_cert = undef,
$ssl_key = undef,
$ssl_chain = undef,
$ssl_ca = undef,
$ssl_crl_path = undef,
$ssl_crl = undef,
$ssl_certs_dir = undef,
$threads = $::os_workers,
$priority = '10',
$access_log_file = false,
$access_log_format = false,
$error_log_file = undef,
$servername = $::fqdn,
$port = 8999,
$bind_host = undef,
$path = '/',
$ssl = true,
$workers = 1,
$wsgi_process_display_name = undef,
$ssl_cert = undef,
$ssl_key = undef,
$ssl_chain = undef,
$ssl_ca = undef,
$ssl_crl_path = undef,
$ssl_crl = undef,
$ssl_certs_dir = undef,
$threads = $::os_workers,
$priority = '10',
$access_log_file = false,
$access_log_format = false,
$error_log_file = undef,
$custom_wsgi_process_options = {},
) {
include ::vitrage::deps
@ -124,31 +133,32 @@ class vitrage::wsgi::apache (
}
::openstacklib::wsgi::apache { 'vitrage_wsgi':
bind_host => $bind_host,
bind_port => $port,
group => 'vitrage',
path => $path,
priority => $priority,
servername => $servername,
ssl => $ssl,
ssl_ca => $ssl_ca,
ssl_cert => $ssl_cert,
ssl_certs_dir => $ssl_certs_dir,
ssl_chain => $ssl_chain,
ssl_crl => $ssl_crl,
ssl_crl_path => $ssl_crl_path,
ssl_key => $ssl_key,
threads => $threads,
user => 'vitrage',
workers => $workers,
wsgi_daemon_process => 'vitrage',
wsgi_process_group => 'vitrage',
wsgi_process_display_name => $wsgi_process_display_name,
wsgi_script_dir => $::vitrage::params::vitrage_wsgi_script_path,
wsgi_script_file => 'app',
wsgi_script_source => $::vitrage::params::vitrage_wsgi_script_source,
access_log_file => $access_log_file,
access_log_format => $access_log_format,
error_log_file => $error_log_file,
bind_host => $bind_host,
bind_port => $port,
group => 'vitrage',
path => $path,
priority => $priority,
servername => $servername,
ssl => $ssl,
ssl_ca => $ssl_ca,
ssl_cert => $ssl_cert,
ssl_certs_dir => $ssl_certs_dir,
ssl_chain => $ssl_chain,
ssl_crl => $ssl_crl,
ssl_crl_path => $ssl_crl_path,
ssl_key => $ssl_key,
threads => $threads,
user => 'vitrage',
workers => $workers,
wsgi_daemon_process => 'vitrage',
wsgi_process_group => 'vitrage',
wsgi_process_display_name => $wsgi_process_display_name,
wsgi_script_dir => $::vitrage::params::vitrage_wsgi_script_path,
wsgi_script_file => 'app',
wsgi_script_source => $::vitrage::params::vitrage_wsgi_script_source,
access_log_file => $access_log_file,
access_log_format => $access_log_format,
error_log_file => $error_log_file,
custom_wsgi_process_options => $custom_wsgi_process_options,
}
}

View File

@ -0,0 +1,4 @@
---
features:
- Add parameter to apache_wsgi to allow overwrite
and/or add additional wsgi process options.

View File

@ -9,36 +9,40 @@ describe 'vitrage::wsgi::apache' do
it { is_expected.to contain_class('apache::mod::wsgi') }
it { is_expected.to contain_class('apache::mod::ssl') }
it { is_expected.to contain_openstacklib__wsgi__apache('vitrage_wsgi').with(
:bind_port => 8999,
:group => 'vitrage',
:path => '/',
:servername => facts[:fqdn],
:ssl => true,
:threads => facts[:os_workers],
:user => 'vitrage',
:workers => 1,
:wsgi_daemon_process => 'vitrage',
:wsgi_process_group => 'vitrage',
:wsgi_script_dir => platform_params[:wsgi_script_path],
:wsgi_script_file => 'app',
:wsgi_script_source => platform_params[:wsgi_script_source],
:access_log_file => false,
:access_log_format => false,
:bind_port => 8999,
:group => 'vitrage',
:path => '/',
:servername => facts[:fqdn],
:ssl => true,
:threads => facts[:os_workers],
:user => 'vitrage',
:workers => 1,
:wsgi_daemon_process => 'vitrage',
:wsgi_process_group => 'vitrage',
:wsgi_script_dir => platform_params[:wsgi_script_path],
:wsgi_script_file => 'app',
:wsgi_script_source => platform_params[:wsgi_script_source],
:access_log_file => false,
:access_log_format => false,
:custom_wsgi_process_options => {},
)}
end
describe 'when overriding parameters using different ports' do
let :params do
{
:servername => 'dummy.host',
:bind_host => '10.42.51.1',
:port => 12345,
:ssl => false,
:wsgi_process_display_name => 'vitrage',
:workers => 37,
:access_log_file => '/var/log/httpd/access_log',
:access_log_format => 'some format',
:error_log_file => '/var/log/httpd/error_log'
:servername => 'dummy.host',
:bind_host => '10.42.51.1',
:port => 12345,
:ssl => false,
:wsgi_process_display_name => 'vitrage',
:workers => 37,
:access_log_file => '/var/log/httpd/access_log',
:access_log_format => 'some format',
:error_log_file => '/var/log/httpd/error_log',
:custom_wsgi_process_options => {
'python_path' => '/my/python/path',
},
}
end
it { is_expected.to contain_class('vitrage::params') }
@ -46,24 +50,27 @@ describe 'vitrage::wsgi::apache' do
it { is_expected.to contain_class('apache::mod::wsgi') }
it { is_expected.to_not contain_class('apache::mod::ssl') }
it { is_expected.to contain_openstacklib__wsgi__apache('vitrage_wsgi').with(
:bind_host => '10.42.51.1',
:bind_port => 12345,
:group => 'vitrage',
:path => '/',
:servername => 'dummy.host',
:ssl => false,
:threads => facts[:os_workers],
:user => 'vitrage',
:workers => 37,
:wsgi_daemon_process => 'vitrage',
:wsgi_process_display_name => 'vitrage',
:wsgi_process_group => 'vitrage',
:wsgi_script_dir => platform_params[:wsgi_script_path],
:wsgi_script_file => 'app',
:wsgi_script_source => platform_params[:wsgi_script_source],
:access_log_file => '/var/log/httpd/access_log',
:access_log_format => 'some format',
:error_log_file => '/var/log/httpd/error_log',
:bind_host => '10.42.51.1',
:bind_port => 12345,
:group => 'vitrage',
:path => '/',
:servername => 'dummy.host',
:ssl => false,
:threads => facts[:os_workers],
:user => 'vitrage',
:workers => 37,
:wsgi_daemon_process => 'vitrage',
:wsgi_process_display_name => 'vitrage',
:wsgi_process_group => 'vitrage',
:wsgi_script_dir => platform_params[:wsgi_script_path],
:wsgi_script_file => 'app',
:wsgi_script_source => platform_params[:wsgi_script_source],
:access_log_file => '/var/log/httpd/access_log',
:access_log_format => 'some format',
:error_log_file => '/var/log/httpd/error_log',
:custom_wsgi_process_options => {
'python_path' => '/my/python/path',
},
)}
end
end