openstack/fuel-library
Code Issues Proposed changes

Set metadata_workers for nova

Browse Source 
Earlier metadata_workers option was set to processorcount
fact by default, but with upstream patch [1]
it will be switched to os_workers fact,
so in order to keep the same amount of
nova metadata workers we need to set it directly
in openstack_controller task.

[1] https://review.openstack.org/#/c/383690/

Change-Id: I95bc3d7b5f4c71fc9829b0a58d246ac4ea2fe903
Closes-Bug: #1667626
This commit is contained in:
Mykyta Karpin 2017-02-24 12:47:51 +02:00
parent 5d15b1eb23
commit 4d111c2190
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
deployment/puppet/openstack_tasks/manifests/openstack_controller/openstack_controller.pp
View File

@ -263,6 +263,7 @@ class openstack_tasks::openstack_controller::openstack_controller {
ratelimits => $nova_rate_limits_string,
neutron_metadata_proxy_shared_secret => $neutron_metadata_proxy_secret,
osapi_compute_workers => $service_workers,
metadata_workers => $service_workers,
sync_db => $primary_controller,
sync_db_api => $primary_controller,
fping_path => $fping_path,

2
tests/noop/spec/hosts/openstack-controller/openstack-controller_spec.rb
View File

@ -135,7 +135,7 @@ describe manifest do
# TODO All this stuff should be moved to shared examples controller* tests.
it 'should declare correct workers for systems with 4 processess on 4 CPU & 32G system' do
it 'should declare correct workers for systems with 4 processes on 4 CPU & 32G system' do
should contain_class('nova::api').with(
'osapi_compute_workers' => '4',
'metadata_workers' => '4'