Merge "Add support for [DEFAULT] api_workers"

This commit is contained in:
Zuul 2022-01-10 19:27:25 +00:00 committed by Gerrit Code Review
commit 2530f039ca
3 changed files with 16 additions and 3 deletions

View File

@ -22,6 +22,10 @@
# (optional) The port to bind to.
# Defaults to $::os_service_default
#
# [*api_workers*]
# (optional) Number of separate worker process for service.
# Defaults to $::os_workers
#
# [*package_ensure*]
# (Optional) Ensure state for package.
# Defaults to 'present'
@ -32,6 +36,7 @@ class tacker::server(
$auth_strategy = 'keystone',
$bind_host = $::os_service_default,
$bind_port = $::os_service_default,
$api_workers = $::os_workers,
$package_ensure = 'present',
) {
@ -52,6 +57,7 @@ class tacker::server(
tacker_config {
'DEFAULT/bind_host' : value => $bind_host;
'DEFAULT/bind_port' : value => $bind_port;
'DEFAULT/api_workers': value => $api_workers;
}
if $manage_service {

View File

@ -0,0 +1,4 @@
---
features:
- |
The new ``tacker::server::api_workers`` parameter has been added.

View File

@ -26,6 +26,7 @@ describe 'tacker::server' do
it 'configures api' do
is_expected.to contain_tacker_config('DEFAULT/bind_host').with_value( params[:bind_host] )
is_expected.to contain_tacker_config('DEFAULT/bind_port').with_value( params[:bind_port] )
is_expected.to contain_tacker_config('DEFAULT/api_workers').with_value(4)
end
[{:enabled => true}, {:enabled => false}].each do |param_hash|
@ -59,7 +60,9 @@ describe 'tacker::server' do
}).each do |os,facts|
context "on #{os}" do
let (:facts) do
facts.merge!(OSDefaults.get_facts())
facts.merge!(OSDefaults.get_facts(
:os_workers => 4
))
end
let(:platform_params) do