stackforge/puppet-openstack-cloud
Code Issues Proposed changes

Neutron / Metadata: improve agent scalability

Browse Source 
we need by default to set the metadata backlog value and the workers
value. Inspired from
19:34: https://www.youtube.com/watch?v=AF9r_VQrcJ0 and
https://review.openstack.org/#/c/95372

Signed-off-by: Emilien Macchi <emilien.macchi@enovance.com>
This commit is contained in:
Emilien Macchi 2014-06-08 16:10:26 +02:00
parent 846b5cd18b
commit e730145dc2
2 changed files with 26 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
manifests/network/metadata.pp
View File

@ -38,7 +38,13 @@ class cloud::network::metadata(
metadata_ip => $nova_metadata_server, metadata_ip => $nova_metadata_server,
auth_url => "${ks_keystone_admin_proto}://${ks_keystone_admin_host}:${ks_keystone_admin_port}/v2.0", auth_url => "${ks_keystone_admin_proto}://${ks_keystone_admin_host}:${ks_keystone_admin_port}/v2.0",
auth_password => $ks_neutron_password, auth_password => $ks_neutron_password,
auth_region => $auth_region auth_region => $auth_region,
metadata_workers => $::processorcount
}
# TODO(EmilienM) need to be deleted hen https://review.openstack.org/98633 got merged
neutron_metadata_agent_config {
'DEFAULT/metadata_backlog': value => '4096';
} }
} }

10
spec/classes/cloud_network_metadata_spec.rb
View File

@ -92,14 +92,17 @@ describe 'cloud::network::metadata' do
:metadata_ip => '10.0.0.1', :metadata_ip => '10.0.0.1',
:auth_url => 'http://10.0.0.1:35357/v2.0', :auth_url => 'http://10.0.0.1:35357/v2.0',
:auth_password => 'secrete', :auth_password => 'secrete',
:auth_region => 'MyRegion' :auth_region => 'MyRegion',
:metadata_workers => '8'
) )
should contain_neutron_metadata_agent_config('DEFAULT/metadata_backlog').with(:value => '4096')
end end
end end
context 'on Debian platforms' do context 'on Debian platforms' do
let :facts do let :facts do
{ :osfamily => 'Debian' } { :osfamily => 'Debian',
:processorcount => '8' }
end end
let :platform_params do let :platform_params do
@ -111,7 +114,8 @@ describe 'cloud::network::metadata' do
context 'on RedHat platforms' do context 'on RedHat platforms' do
let :facts do let :facts do
{ :osfamily => 'RedHat' } { :osfamily => 'RedHat',
:processorcount => '8' }
end end
let :platform_params do let :platform_params do