diff --git a/manifests/init.pp b/manifests/init.pp index 0330de81d..d4eead320 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -404,13 +404,13 @@ # (optional) The number of worker processes to serve the admin eventlet application. # This option is deprecated along with eventlet and will be removed in M. # This setting has no affect when using WSGI. -# Defaults to max($::processorcount, 2) +# Defaults to $::os_workers # # [*public_workers*] # (optional) The number of worker processes to serve the public eventlet application. # This option is deprecated along with eventlet and will be removed in M. # This setting has no affect when using WSGI. -# Defaults to max($::processorcount, 2) +# Defaults to $::os_workers # # [*sync_db*] # (Optional) Run db sync on the node. @@ -749,8 +749,8 @@ class keystone( $enable_proxy_headers_parsing = $::os_service_default, $purge_config = false, # DEPRECATED PARAMETERS - $admin_workers = max($::processorcount, 2), - $public_workers = max($::processorcount, 2), + $admin_workers = $::os_workers, + $public_workers = $::os_workers, $service_provider = undef, $verbose = undef, $enable_pki_setup = undef, diff --git a/manifests/wsgi/apache.pp b/manifests/wsgi/apache.pp index b0396913b..703104d90 100644 --- a/manifests/wsgi/apache.pp +++ b/manifests/wsgi/apache.pp @@ -95,7 +95,7 @@ # # [*threads*] # (optional) The number of threads for the vhost. -# Defaults to $::processorcount +# Defaults to $::os_workers # # [*wsgi_application_group*] # (optional) The application group of the WSGI script. @@ -188,7 +188,7 @@ class keystone::wsgi::apache ( $ssl_crl_path = undef, $ssl_crl = undef, $ssl_certs_dir = undef, - $threads = $::processorcount, + $threads = $::os_workers, $priority = '10', $wsgi_application_group = '%{GLOBAL}', $wsgi_pass_authorization = 'On', diff --git a/releasenotes/notes/os_workers_for_worker_count-50c1f496bf4dc954.yaml b/releasenotes/notes/os_workers_for_worker_count-50c1f496bf4dc954.yaml new file mode 100644 index 000000000..5616f0d7e --- /dev/null +++ b/releasenotes/notes/os_workers_for_worker_count-50c1f496bf4dc954.yaml @@ -0,0 +1,5 @@ +--- +other: + - Parameters that control the number of spawned child processes for + distributing processing have had their default value changed from + ::processorcount to ::os_workers. diff --git a/spec/classes/keystone_db_postgresql_spec.rb b/spec/classes/keystone_db_postgresql_spec.rb index 04a1e619c..2f6b7e39f 100644 --- a/spec/classes/keystone_db_postgresql_spec.rb +++ b/spec/classes/keystone_db_postgresql_spec.rb @@ -30,7 +30,7 @@ describe 'keystone::db::postgresql' do context "on #{os}" do let (:facts) do facts.merge(OSDefaults.get_facts({ - :processorcount => 8, + :os_workers => 8, :concat_basedir => '/var/lib/puppet/concat' })) end diff --git a/spec/classes/keystone_roles_admin_spec.rb b/spec/classes/keystone_roles_admin_spec.rb index 30a42f4f5..8d35a38bf 100644 --- a/spec/classes/keystone_roles_admin_spec.rb +++ b/spec/classes/keystone_roles_admin_spec.rb @@ -11,7 +11,7 @@ describe 'keystone::roles::admin' do :osfamily => 'Debian', :operatingsystem => 'Debian', :operatingsystemrelease => '7.0', - :processorcount => '1' + :os_workers => 1, }) end diff --git a/spec/classes/keystone_spec.rb b/spec/classes/keystone_spec.rb index 6f10e9fd1..a372e54c4 100644 --- a/spec/classes/keystone_spec.rb +++ b/spec/classes/keystone_spec.rb @@ -4,7 +4,6 @@ describe 'keystone' do let :global_facts do { - :processorcount => 42, :concat_basedir => '/var/lib/puppet/concat', :fqdn => 'some.host.tld' } @@ -15,7 +14,6 @@ describe 'keystone' do :osfamily => 'Debian', :operatingsystem => 'Debian', :operatingsystemrelease => '7.0', - :processorcount => '1' })) end diff --git a/spec/classes/keystone_wsgi_apache_spec.rb b/spec/classes/keystone_wsgi_apache_spec.rb index 6076267f4..e0263baba 100644 --- a/spec/classes/keystone_wsgi_apache_spec.rb +++ b/spec/classes/keystone_wsgi_apache_spec.rb @@ -4,7 +4,7 @@ describe 'keystone::wsgi::apache' do let :global_facts do { - :processorcount => 42, + :os_workers => 8, :concat_basedir => '/var/lib/puppet/concat', :fqdn => 'some.host.tld' }