diff options
author | Stefan Berg <stefan.k.berg@ericsson.com> | 2016-11-16 08:40:40 +0000 |
---|---|---|
committer | Gerrit Code Review <gerrit@opnfv.org> | 2016-11-16 08:40:40 +0000 |
commit | da5ed062277da74d394202a97ffa9344fe88ea2a (patch) | |
tree | e71e707c1584e94f74925400dd59fcc87b5428ee /build/f_repos/patch/fuel-library | |
parent | d4b0f79649da3ad7a3f3f59d3b92ea230f5b1826 (diff) | |
parent | 9f8aff6636e0d86834178b461802f3e7cb95189d (diff) |
Merge "Uplift to Fuel 10.0/Newton"
Diffstat (limited to 'build/f_repos/patch/fuel-library')
4 files changed, 0 insertions, 624 deletions
diff --git a/build/f_repos/patch/fuel-library/0001-Reduce-ceilometer-memory-usage.patch b/build/f_repos/patch/fuel-library/0001-Reduce-ceilometer-memory-usage.patch deleted file mode 100644 index e51e70b64..000000000 --- a/build/f_repos/patch/fuel-library/0001-Reduce-ceilometer-memory-usage.patch +++ /dev/null @@ -1,26 +0,0 @@ -From: Michael Polenchuk <mpolenchuk@mirantis.com> -Date: Thu, 8 Sep 2016 19:25:14 +0300 -Subject: [PATCH] Reduce ceilometer memory usage - -Change-Id: I8b1a97bd710c859a3543d1aed8226313f35f4f10 ---- - .../puppet/openstack_tasks/manifests/ceilometer/controller.pp | 7 +++++++ - 1 file changed, 7 insertions(+) - -diff --git a/deployment/puppet/openstack_tasks/manifests/ceilometer/controller.pp b/deployment/puppet/openstack_tasks/manifests/ceilometer/controller.pp -index 2c63ff0..1dda109 100644 ---- a/deployment/puppet/openstack_tasks/manifests/ceilometer/controller.pp -+++ b/deployment/puppet/openstack_tasks/manifests/ceilometer/controller.pp -@@ -234,5 +234,12 @@ class openstack_tasks::ceilometer::controller { - Ceilometer_config<| title == 'oslo_messaging_rabbit/kombu_compression' |> { value => $kombu_compression } - } - } -+ -+ # Reduce memory usage -+ ceilometer_config { -+ 'DEFAULT/executor_thread_pool_size': value => ceiling($service_workers*0.5); -+ 'notification/pipeline_processing_queues': value => ceiling($service_workers*1.5); -+ 'oslo_messaging_rabbit/rabbit_qos_prefetch_count': value => 5; -+ } - } - } diff --git a/build/f_repos/patch/fuel-library/0002-Disable-token-revoke-to-increase-keystone-performanc.patch b/build/f_repos/patch/fuel-library/0002-Disable-token-revoke-to-increase-keystone-performanc.patch deleted file mode 100644 index d4baa9c1e..000000000 --- a/build/f_repos/patch/fuel-library/0002-Disable-token-revoke-to-increase-keystone-performanc.patch +++ /dev/null @@ -1,40 +0,0 @@ -From: iberezovskiy <iberezovskiy@mirantis.com> -Date: Mon, 19 Sep 2016 12:35:05 +0300 -Subject: [PATCH] Disable token revoke to increase keystone performance - -Change-Id: I4df816369093908ad1eac372f38c92155efbe8e0 -Closes-bug: #1625077 ---- - deployment/puppet/openstack_tasks/manifests/keystone/keystone.pp | 2 ++ - tests/noop/spec/hosts/keystone/keystone_spec.rb | 5 +++++ - 2 files changed, 7 insertions(+) - -diff --git a/deployment/puppet/openstack_tasks/manifests/keystone/keystone.pp b/deployment/puppet/openstack_tasks/manifests/keystone/keystone.pp -index 3162287..ba9d7df 100644 ---- a/deployment/puppet/openstack_tasks/manifests/keystone/keystone.pp -+++ b/deployment/puppet/openstack_tasks/manifests/keystone/keystone.pp -@@ -327,6 +327,8 @@ class openstack_tasks::keystone::keystone { - memcache_pool_unused_timeout => '60', - cache_memcache_servers => $memcache_servers, - policy_driver => 'keystone.policy.backends.sql.Policy', -+ # Set revoke_by_id to false according to LP #1625077 -+ revoke_by_id => false, - } - - Package<| title == 'keystone'|> ~> Service<| title == 'keystone'|> -diff --git a/tests/noop/spec/hosts/keystone/keystone_spec.rb b/tests/noop/spec/hosts/keystone/keystone_spec.rb -index b29e691..3c0f847 100644 ---- a/tests/noop/spec/hosts/keystone/keystone_spec.rb -+++ b/tests/noop/spec/hosts/keystone/keystone_spec.rb -@@ -176,6 +176,11 @@ describe manifest do - should contain_class('keystone').with('sync_db' => primary_controller) - end - -+ it 'should declare keystone class with revoke_by_id set to false' do -+ # Set revoke_by_id to false according to LP #1625077 -+ should contain_class('keystone').with('revoke_by_id' => false) -+ end -+ - it 'should configure keystone with paramters' do - should contain_keystone_config('token/caching').with(:value => 'false') - should contain_keystone_config('cache/enabled').with(:value => 'true') diff --git a/build/f_repos/patch/fuel-library/0003-Rework-processorcount-limits-to-os_workers.patch b/build/f_repos/patch/fuel-library/0003-Rework-processorcount-limits-to-os_workers.patch deleted file mode 100644 index cc961222c..000000000 --- a/build/f_repos/patch/fuel-library/0003-Rework-processorcount-limits-to-os_workers.patch +++ /dev/null @@ -1,432 +0,0 @@ -From feb6582408ed13ec34e0f6450af121f5614d5dfb Mon Sep 17 00:00:00 2001 -From: Bogdan Dobrelya <bdobrelia@mirantis.com> -Date: Fri, 30 Sep 2016 12:03:34 +0200 -Subject: [PATCH] Rework processorcount limits to os_workers - -* reduce workers_max default limits of a 16 to the $::os_workers, which is limited by [2, 8] interval. -* reduce workers/pool size/overflow limited by F($::processorcount) to F($::os_workers) -* change number of keystone threads to 1 - -Closes-bug: #1629238 -Closes-Bug: #1628534 - -Change-Id: I5cc4d70b902eeaa1c9cf42911606eba13dd84aa2 -Signed-off-by: Bogdan Dobrelya <bdobrelia@mirantis.com> ---- - deployment/puppet/openstack/manifests/cinder.pp | 2 +- - .../manifests/ceilometer/controller.pp | 2 +- - .../openstack_tasks/manifests/glance/glance.pp | 2 +- - .../openstack_tasks/manifests/horizon/horizon.pp | 2 +- - .../manifests/ironic/ironic_compute.pp | 4 ++-- - .../openstack_tasks/manifests/keystone/keystone.pp | 4 ++-- - .../manifests/openstack_cinder/openstack_cinder.pp | 6 +++--- - .../openstack_controller/openstack_controller.pp | 6 +++--- - .../manifests/openstack_network/agents/metadata.pp | 2 +- - .../manifests/openstack_network/server_config.pp | 2 +- - .../openstack_tasks/manifests/roles/cinder.pp | 4 ++-- - .../openstack_tasks/manifests/sahara/sahara.pp | 4 ++-- - .../openstack_tasks/manifests/swift/parts/proxy.pp | 4 ++-- - .../manifests/swift/proxy_storage.pp | 2 +- - .../puppet/osnailyfacter/lib/facter/os_workers.rb | 20 ++++++++++++++++++++ - .../puppet/osnailyfacter/manifests/apache_mpm.pp | 2 +- - .../osnailyfacter/manifests/globals/globals.pp | 8 ++++---- - .../spec/classes/osnailyfacter_atop_spec.rb | 2 ++ - .../noop/spec/hosts/ironic/ironic-compute_spec.rb | 6 +++--- - tests/noop/spec/hosts/keystone/keystone_spec.rb | 2 ++ - .../openstack-controller_spec.rb | 6 +++--- - tests/noop/spec/hosts/sahara/sahara_spec.rb | 6 +++--- - 22 files changed, 61 insertions(+), 37 deletions(-) - create mode 100644 deployment/puppet/osnailyfacter/lib/facter/os_workers.rb - -diff --git a/deployment/puppet/openstack/manifests/cinder.pp b/deployment/puppet/openstack/manifests/cinder.pp -index 072252e..3cdf077 100644 ---- a/deployment/puppet/openstack/manifests/cinder.pp -+++ b/deployment/puppet/openstack/manifests/cinder.pp -@@ -44,7 +44,7 @@ class openstack::cinder( - $keystone_user = 'cinder', - $region = 'RegionOne', - $notification_driver = undef, -- $service_workers = $::processorcount, -+ $service_workers = $::os_workers, - $vmware_host_ip = '10.10.10.10', - $vmware_host_username = 'administrator@vsphere.local', - $vmware_host_password = 'password', -diff --git a/deployment/puppet/openstack_tasks/manifests/ceilometer/controller.pp b/deployment/puppet/openstack_tasks/manifests/ceilometer/controller.pp -index 2c63ff0..d0ce24d 100644 ---- a/deployment/puppet/openstack_tasks/manifests/ceilometer/controller.pp -+++ b/deployment/puppet/openstack_tasks/manifests/ceilometer/controller.pp -@@ -37,7 +37,7 @@ class openstack_tasks::ceilometer::controller { - $service_endpoint = hiera('service_endpoint', $management_vip) - $ha_mode = pick($ceilometer_hash['ha_mode'], true) - $ssl_hash = hiera_hash('use_ssl', {}) -- $workers_max = hiera('workers_max', 16) -+ $workers_max = hiera('workers_max', $::os_workers) - $service_workers = pick($ceilometer_hash['workers'], - min(max($::processorcount, 2), $workers_max)) - -diff --git a/deployment/puppet/openstack_tasks/manifests/glance/glance.pp b/deployment/puppet/openstack_tasks/manifests/glance/glance.pp -index ab7cc13..99153a0 100644 ---- a/deployment/puppet/openstack_tasks/manifests/glance/glance.pp -+++ b/deployment/puppet/openstack_tasks/manifests/glance/glance.pp -@@ -22,7 +22,7 @@ class openstack_tasks::glance::glance { - $max_overflow = hiera('max_overflow') - $ceilometer_hash = hiera_hash('ceilometer', {}) - $region = hiera('region','RegionOne') -- $workers_max = hiera('workers_max', 16) -+ $workers_max = hiera('workers_max', $::os_workers) - $service_workers = pick($glance_hash['glance_workers'], - min(max($::processorcount, 2), $workers_max)) - $ironic_hash = hiera_hash('ironic', {}) -diff --git a/deployment/puppet/openstack_tasks/manifests/horizon/horizon.pp b/deployment/puppet/openstack_tasks/manifests/horizon/horizon.pp -index ca1ef45..a52106f 100644 ---- a/deployment/puppet/openstack_tasks/manifests/horizon/horizon.pp -+++ b/deployment/puppet/openstack_tasks/manifests/horizon/horizon.pp -@@ -119,7 +119,7 @@ class openstack_tasks::horizon::horizon { - $wsgi_processes = 2 - $wsgi_threads = 9 - } else { -- $wsgi_processes = $::processorcount -+ $wsgi_processes = $::os_workers - $wsgi_threads = 15 - } - -diff --git a/deployment/puppet/openstack_tasks/manifests/ironic/ironic_compute.pp b/deployment/puppet/openstack_tasks/manifests/ironic/ironic_compute.pp -index 940c7a3..322e703 100644 ---- a/deployment/puppet/openstack_tasks/manifests/ironic/ironic_compute.pp -+++ b/deployment/puppet/openstack_tasks/manifests/ironic/ironic_compute.pp -@@ -35,8 +35,8 @@ class openstack_tasks::ironic::ironic_compute { - $db_name = pick($nova_hash['db_name'], 'nova') - $db_password = pick($nova_hash['db_password'], 'nova') - -- $max_pool_size = hiera('max_pool_size', min($::processorcount * 5 + 0, 30 + 0)) -- $max_overflow = hiera('max_overflow', min($::processorcount * 5 + 0, 60 + 0)) -+ $max_pool_size = hiera('max_pool_size', min($::os_workers * 5 + 0, 30 + 0)) -+ $max_overflow = hiera('max_overflow', min($::os_workers * 5 + 0, 60 + 0)) - $idle_timeout = hiera('idle_timeout', '3600') - $max_retries = hiera('max_retries', '-1') - -diff --git a/deployment/puppet/openstack_tasks/manifests/keystone/keystone.pp b/deployment/puppet/openstack_tasks/manifests/keystone/keystone.pp -index 3162287..8146548 100644 ---- a/deployment/puppet/openstack_tasks/manifests/keystone/keystone.pp -+++ b/deployment/puppet/openstack_tasks/manifests/keystone/keystone.pp -@@ -30,7 +30,7 @@ class openstack_tasks::keystone::keystone { - $syslog_log_facility = hiera('syslog_log_facility_keystone') - $rabbit_hash = hiera_hash('rabbit', {}) - $neutron_user_password = hiera('neutron_user_password', false) -- $workers_max = hiera('workers_max', 16) -+ $workers_max = hiera('workers_max', $::os_workers) - $service_workers = pick($keystone_hash['workers'], - min(max($::processorcount, 2), $workers_max)) - $default_log_levels = hiera_hash('default_log_levels') -@@ -151,7 +151,7 @@ class openstack_tasks::keystone::keystone { - - class { '::keystone::wsgi::apache': - priority => '05', -- threads => 3, -+ threads => 1, - workers => min($::processorcount, 6), - ssl => $ssl, - vhost_custom_fragment => $vhost_limit_request_field_size, -diff --git a/deployment/puppet/openstack_tasks/manifests/openstack_cinder/openstack_cinder.pp b/deployment/puppet/openstack_tasks/manifests/openstack_cinder/openstack_cinder.pp -index a298e70..cc4c2b3 100644 ---- a/deployment/puppet/openstack_tasks/manifests/openstack_cinder/openstack_cinder.pp -+++ b/deployment/puppet/openstack_tasks/manifests/openstack_cinder/openstack_cinder.pp -@@ -12,7 +12,7 @@ class openstack_tasks::openstack_cinder::openstack_cinder { - $sahara_hash = hiera_hash('sahara', {}) - $rabbit_hash = hiera_hash('rabbit', {}) - $service_endpoint = hiera('service_endpoint') -- $workers_max = hiera('workers_max', 16) -+ $workers_max = hiera('workers_max', $::os_workers) - $service_workers = pick($cinder_hash['workers'], min(max($::processorcount, 2), $workers_max)) - $cinder_user_password = $cinder_hash[user_password] - $keystone_user = pick($cinder_hash['user'], 'cinder') -@@ -98,8 +98,8 @@ class openstack_tasks::openstack_cinder::openstack_cinder { - } - - # SQLAlchemy backend configuration -- $max_pool_size = min($::processorcount * 5 + 0, 30 + 0) -- $max_overflow = min($::processorcount * 5 + 0, 60 + 0) -+ $max_pool_size = min($::os_workers * 5 + 0, 30 + 0) -+ $max_overflow = min($::os_workers * 5 + 0, 60 + 0) - $max_retries = '-1' - $idle_timeout = '3600' - -diff --git a/deployment/puppet/openstack_tasks/manifests/openstack_controller/openstack_controller.pp b/deployment/puppet/openstack_tasks/manifests/openstack_controller/openstack_controller.pp -index f8915c1..dbc7a9e 100644 ---- a/deployment/puppet/openstack_tasks/manifests/openstack_controller/openstack_controller.pp -+++ b/deployment/puppet/openstack_tasks/manifests/openstack_controller/openstack_controller.pp -@@ -56,7 +56,7 @@ class openstack_tasks::openstack_controller::openstack_controller { - $keystone_user = pick($nova_hash['user'], 'nova') - $keystone_tenant = pick($nova_hash['tenant'], 'services') - $region = hiera('region', 'RegionOne') -- $workers_max = hiera('workers_max', 16) -+ $workers_max = hiera('workers_max', $::os_workers) - $service_workers = pick($nova_hash['workers'], - min(max($::processorcount, 2), $workers_max)) - $compute_nodes = get_nodes_hash_by_roles($network_metadata, ['compute']) -@@ -127,8 +127,8 @@ class openstack_tasks::openstack_controller::openstack_controller { - }) - - # SQLAlchemy backend configuration -- $max_pool_size = hiera('max_pool_size', min($::processorcount * 5 + 0, 30 + 0)) -- $max_overflow = hiera('max_overflow', min($::processorcount * 5 + 0, 60 + 0)) -+ $max_pool_size = hiera('max_pool_size', min($::os_workers * 5 + 0, 30 + 0)) -+ $max_overflow = hiera('max_overflow', min($::os_workers * 5 + 0, 60 + 0)) - $idle_timeout = hiera('idle_timeout', '3600') - $max_retries = hiera('max_retries', '-1') - -diff --git a/deployment/puppet/openstack_tasks/manifests/openstack_network/agents/metadata.pp b/deployment/puppet/openstack_tasks/manifests/openstack_network/agents/metadata.pp -index 98acddb..9efcb5a 100644 ---- a/deployment/puppet/openstack_tasks/manifests/openstack_network/agents/metadata.pp -+++ b/deployment/puppet/openstack_tasks/manifests/openstack_network/agents/metadata.pp -@@ -10,7 +10,7 @@ class openstack_tasks::openstack_network::agents::metadata { - $neutron_advanced_config = hiera_hash('neutron_advanced_configuration', { }) - $neutron_config = hiera_hash('neutron_config') - $dvr = pick($neutron_advanced_config['neutron_dvr'], false) -- $workers_max = hiera('workers_max', 16) -+ $workers_max = hiera('workers_max', $::os_workers) - - if $compute { - $metadata_workers = pick($neutron_config['workers'], -diff --git a/deployment/puppet/openstack_tasks/manifests/openstack_network/server_config.pp b/deployment/puppet/openstack_tasks/manifests/openstack_network/server_config.pp -index 414eedf..03d03a8 100644 ---- a/deployment/puppet/openstack_tasks/manifests/openstack_network/server_config.pp -+++ b/deployment/puppet/openstack_tasks/manifests/openstack_network/server_config.pp -@@ -86,7 +86,7 @@ class openstack_tasks::openstack_network::server_config { - $nova_admin_auth_url = "${admin_auth_protocol}://${admin_auth_endpoint}:35357/" - $nova_url = "${nova_internal_protocol}://${nova_internal_endpoint}:8774/v2" - -- $workers_max = hiera('workers_max', 16) -+ $workers_max = hiera('workers_max', $::os_workers) - $service_workers = pick($neutron_config['workers'], min(max($::processorcount, 1), $workers_max)) - - $neutron_advanced_config = hiera_hash('neutron_advanced_configuration', { }) -diff --git a/deployment/puppet/openstack_tasks/manifests/roles/cinder.pp b/deployment/puppet/openstack_tasks/manifests/roles/cinder.pp -index f2d6c72..c41dae6 100644 ---- a/deployment/puppet/openstack_tasks/manifests/roles/cinder.pp -+++ b/deployment/puppet/openstack_tasks/manifests/roles/cinder.pp -@@ -86,8 +86,8 @@ class openstack_tasks::roles::cinder { - } - - # SQLAlchemy backend configuration -- $max_pool_size = min($::processorcount * 5 + 0, 30 + 0) -- $max_overflow = min($::processorcount * 5 + 0, 60 + 0) -+ $max_pool_size = min($::os_workers * 5 + 0, 30 + 0) -+ $max_overflow = min($::os_workers * 5 + 0, 60 + 0) - $max_retries = '-1' - $idle_timeout = '3600' - -diff --git a/deployment/puppet/openstack_tasks/manifests/sahara/sahara.pp b/deployment/puppet/openstack_tasks/manifests/sahara/sahara.pp -index f6d50c6..30278ee 100644 ---- a/deployment/puppet/openstack_tasks/manifests/sahara/sahara.pp -+++ b/deployment/puppet/openstack_tasks/manifests/sahara/sahara.pp -@@ -52,8 +52,8 @@ class openstack_tasks::sahara::sahara { - $sahara_user = pick($sahara_hash['user'], 'sahara') - $sahara_password = pick($sahara_hash['user_password']) - $tenant = pick($sahara_hash['tenant'], 'services') -- $max_pool_size = min($::processorcount * 5 + 0, 30 + 0) -- $max_overflow = min($::processorcount * 5 + 0, 60 + 0) -+ $max_pool_size = min($::os_workers * 5 + 0, 30 + 0) -+ $max_overflow = min($::os_workers * 5 + 0, 60 + 0) - $max_retries = '-1' - $idle_timeout = '3600' - -diff --git a/deployment/puppet/openstack_tasks/manifests/swift/parts/proxy.pp b/deployment/puppet/openstack_tasks/manifests/swift/parts/proxy.pp -index eb06244..52c2b3a 100644 ---- a/deployment/puppet/openstack_tasks/manifests/swift/parts/proxy.pp -+++ b/deployment/puppet/openstack_tasks/manifests/swift/parts/proxy.pp -@@ -26,7 +26,7 @@ class openstack_tasks::swift::parts::proxy ( - 'account_quotas', - 'slo', - 'proxy-server'], -- $proxy_workers = $::processorcount, -+ $proxy_workers = $::os_workers, - $proxy_port = '8080', - $proxy_allow_account_management = true, - $proxy_account_autocreate = true, -@@ -225,4 +225,4 @@ class openstack_tasks::swift::parts::proxy ( - anchor { 'openstack_tasks_proxy_end' :} - - } --} -\ No newline at end of file -+} -diff --git a/deployment/puppet/openstack_tasks/manifests/swift/proxy_storage.pp b/deployment/puppet/openstack_tasks/manifests/swift/proxy_storage.pp -index 66fcd21..4608b67 100644 ---- a/deployment/puppet/openstack_tasks/manifests/swift/proxy_storage.pp -+++ b/deployment/puppet/openstack_tasks/manifests/swift/proxy_storage.pp -@@ -30,7 +30,7 @@ class openstack_tasks::swift::proxy_storage { - $keystone_user = pick($swift_hash['user'], 'swift') - $keystone_password = pick($swift_hash['user_password'], 'passsword') - $keystone_tenant = pick($swift_hash['tenant'], 'services') -- $workers_max = hiera('workers_max', 16) -+ $workers_max = hiera('workers_max', $::os_workers) - $service_workers = pick($swift_hash['workers'], min(max($::processorcount, 2), $workers_max)) - $ssl_hash = hiera_hash('use_ssl', {}) - $rabbit_hash = hiera_hash('rabbit') -diff --git a/deployment/puppet/osnailyfacter/lib/facter/os_workers.rb b/deployment/puppet/osnailyfacter/lib/facter/os_workers.rb -new file mode 100644 -index 0000000..404fb90 ---- /dev/null -+++ b/deployment/puppet/osnailyfacter/lib/facter/os_workers.rb -@@ -0,0 +1,20 @@ -+# -+# We've found that using $::processorcount for workers/threads can lead to -+# unexpected memory or process counts for people deploying on baremetal or -+# if they have large number of cpus. This fact allows us to tweak the formula -+# used to determine number of workers in a single place but use it across all -+# modules. -+# -+# The value for os_workers is max between '(<# processors> / 4)' and '2' with -+# a cap of 8. -+# -+# This fact can be overloaded by an external fact from /etc/factor/facts.d if -+# a user would like to provide their own default value. -+# -+Facter.add(:os_workers) do -+ has_weight 100 -+ setcode do -+ processors = Facter.value('processorcount') -+ [ [ (processors.to_i / 4), 2 ].max, 8 ].min -+ end -+end -diff --git a/deployment/puppet/osnailyfacter/manifests/apache_mpm.pp b/deployment/puppet/osnailyfacter/manifests/apache_mpm.pp -index 9d0d4d5..9652064 100644 ---- a/deployment/puppet/osnailyfacter/manifests/apache_mpm.pp -+++ b/deployment/puppet/osnailyfacter/manifests/apache_mpm.pp -@@ -11,7 +11,7 @@ class osnailyfacter::apache_mpm { - if $::processorcount <= 2 { - $startservers = 2 - } else { -- $startservers = $::processorcount -+ $startservers = $::os_workers - } - - $maxrequestsperchild = 0 -diff --git a/deployment/puppet/osnailyfacter/manifests/globals/globals.pp b/deployment/puppet/osnailyfacter/manifests/globals/globals.pp -index 854d297..c4d1cf7 100644 ---- a/deployment/puppet/osnailyfacter/manifests/globals/globals.pp -+++ b/deployment/puppet/osnailyfacter/manifests/globals/globals.pp -@@ -323,15 +323,15 @@ class osnailyfacter::globals::globals { - # More services share the same node, more RAM ratio should be given to the workers. - # The default value assumes there are 20 different types of workers limited by 100Mb each. - $workers_ratio = hiera('workers_ratio', 2000) -- # Defines the maximum allowed number of workers for each service by RAM limits. Cannot exceed the value of 30. -- $workers_max = inline_template("<%= [(@memorysize_mb.to_i / @workers_ratio.to_i).floor + 1, 30].min %>") -+ # Defines the maximum allowed number of workers for each service by RAM limits. Cannot exceed the value of 10. -+ $workers_max = inline_template("<%= [(@memorysize_mb.to_i / @workers_ratio.to_i).floor + 1, 10].min %>") - - $node_name_prefix_for_messaging = hiera('node_name_prefix_for_messaging', 'messaging-') - - # MySQL and SQLAlchemy backend configuration - $custom_mysql_setup_class = hiera('custom_mysql_setup_class', 'galera') -- $max_pool_size = hiera('max_pool_size', min($::processorcount * 5 + 0, 30 + 0)) -- $max_overflow = hiera('max_overflow', min($::processorcount * 5 + 0, 60 + 0)) -+ $max_pool_size = hiera('max_pool_size', min($::os_workers * 5 + 0, 30 + 0)) -+ $max_overflow = hiera('max_overflow', min($::os_workers * 5 + 0, 60 + 0)) - $max_retries = hiera('max_retries', '-1') - $idle_timeout = hiera('idle_timeout','3600') - $nova_db_password = $nova_hash['db_password'] -diff --git a/deployment/puppet/osnailyfacter/spec/classes/osnailyfacter_atop_spec.rb b/deployment/puppet/osnailyfacter/spec/classes/osnailyfacter_atop_spec.rb -index ed57325..1d27e87 100644 ---- a/deployment/puppet/osnailyfacter/spec/classes/osnailyfacter_atop_spec.rb -+++ b/deployment/puppet/osnailyfacter/spec/classes/osnailyfacter_atop_spec.rb -@@ -73,6 +73,7 @@ describe 'osnailyfacter::atop' do - :osfamily => 'Debian', - :operatingsystem => 'Debian', - :processorcount => 2, -+ :os_workers => 2, - :memorysize_mb => 4096, - } - end -@@ -86,6 +87,7 @@ describe 'osnailyfacter::atop' do - :osfamily => 'RedHat', - :operatingsystem => 'RedHat', - :processorcount => 2, -+ :os_workers => 2, - :memorysize_mb => 4096, - } - end -diff --git a/tests/noop/spec/hosts/ironic/ironic-compute_spec.rb b/tests/noop/spec/hosts/ironic/ironic-compute_spec.rb -index 4712298..f7c7512 100644 ---- a/tests/noop/spec/hosts/ironic/ironic-compute_spec.rb -+++ b/tests/noop/spec/hosts/ironic/ironic-compute_spec.rb -@@ -122,11 +122,11 @@ describe manifest do - else - extra_params = '?charset=utf8' - end -- facts[:processorcount] = 10 -- max_overflow = Noop.hiera 'max_overflow', [facts[:processorcount] * 5 + 0, 60 + 0].min -+ facts[:os_workers] = 8 -+ max_overflow = Noop.hiera 'max_overflow', [facts[:os_workers] * 5 + 0, 60 + 0].min - idle_timeout = Noop.hiera 'idle_timeout', '3600' - max_retries = Noop.hiera 'max_retries', '-1' -- max_pool_size = Noop.hiera 'max_pool_size', [facts[:processorcount] * 5 + 0, 30 + 0].min -+ max_pool_size = Noop.hiera 'max_pool_size', [facts[:os_workers] * 5 + 0, 30 + 0].min - - should contain_class('nova').with( - :database_connection => "mysql://#{nova_db_user}:#{nova_db_password}@#{database_vip}/#{nova_db_name}#{extra_params}", -diff --git a/tests/noop/spec/hosts/keystone/keystone_spec.rb b/tests/noop/spec/hosts/keystone/keystone_spec.rb -index b29e691..c258104 100644 ---- a/tests/noop/spec/hosts/keystone/keystone_spec.rb -+++ b/tests/noop/spec/hosts/keystone/keystone_spec.rb -@@ -220,6 +220,7 @@ describe manifest do - - it 'should declare keystone::wsgi::apache class with 6 processes and 3 threads on 48 CPU system' do - facts[:processorcount] = 48 -+ facts[:os_workers] = 8 - should contain_class('keystone::wsgi::apache').with( - 'threads' => '3', - 'workers' => '6' -@@ -228,6 +229,7 @@ describe manifest do - - it 'should declare keystone::wsgi::apache class with 1 process and 3 threads on 1 CPU system' do - facts[:processorcount] = 1 -+ facts[:os_workers] = 2 - should contain_class('keystone::wsgi::apache').with( - 'threads' => '3', - 'workers' => '1' -diff --git a/tests/noop/spec/hosts/openstack-controller/openstack-controller_spec.rb b/tests/noop/spec/hosts/openstack-controller/openstack-controller_spec.rb -index f5f273b..efd2acb 100644 ---- a/tests/noop/spec/hosts/openstack-controller/openstack-controller_spec.rb -+++ b/tests/noop/spec/hosts/openstack-controller/openstack-controller_spec.rb -@@ -208,9 +208,9 @@ describe manifest do - end - - it 'should configure nova with the basics' do -- facts[:processorcount] = 10 -- max_pool_size = Noop.hiera 'max_pool_size', [facts[:processorcount] * 5 + 0, 30 + 0].min -- max_overflow = Noop.hiera 'max_overflow', [facts[:processorcount] * 5 + 0, 60 + 0].min -+ facts[:os_workers] = 10 -+ max_pool_size = Noop.hiera 'max_pool_size', [facts[:os_workers] * 5 + 0, 30 + 0].min -+ max_overflow = Noop.hiera 'max_overflow', [facts[:os_workers] * 5 + 0, 60 + 0].min - idle_timeout = Noop.hiera 'idle_timeout', '3600' - max_retries = Noop.hiera 'max_retries', '-1' - -diff --git a/tests/noop/spec/hosts/sahara/sahara_spec.rb b/tests/noop/spec/hosts/sahara/sahara_spec.rb -index 2d15c23..2d07680 100644 ---- a/tests/noop/spec/hosts/sahara/sahara_spec.rb -+++ b/tests/noop/spec/hosts/sahara/sahara_spec.rb -@@ -91,7 +91,7 @@ describe manifest do - enable = Noop.hiera_structure('sahara/enabled') - context 'if sahara is enabled', :if => enable do - it 'should declare sahara class correctly' do -- facts[:processorcount] = 10 -+ facts[:os_workers] = 8 - sahara_plugins = %w(ambari cdh mapr spark vanilla) - sahara_user = Noop.hiera_structure('sahara/user', 'sahara') - sahara_password = Noop.hiera_structure('sahara/user_password') -@@ -101,8 +101,8 @@ describe manifest do - db_name = Noop.hiera_structure('sahara/db_name', 'sahara') - db_password = Noop.hiera_structure('sahara/db_password') - db_host = Noop.hiera_structure('sahara/db_host', database_vip) -- max_pool_size =[facts[:processorcount] * 5 + 0, 30 + 0].min -- max_overflow = [facts[:processorcount] * 5 + 0, 60 + 0].min -+ max_pool_size =[facts[:os_workers] * 5 + 0, 30 + 0].min -+ max_overflow = [facts[:os_workers] * 5 + 0, 60 + 0].min - max_retries = '-1' - idle_timeout = '3600' - read_timeout = '60' --- -1.7.9.5 - diff --git a/build/f_repos/patch/fuel-library/0004-Shift-public-ping-checker-into-deployment-stage.patch b/build/f_repos/patch/fuel-library/0004-Shift-public-ping-checker-into-deployment-stage.patch deleted file mode 100644 index fd929323f..000000000 --- a/build/f_repos/patch/fuel-library/0004-Shift-public-ping-checker-into-deployment-stage.patch +++ /dev/null @@ -1,126 +0,0 @@ -From 60dbecc548b4a81b99d50feac96864ed42d9af4b Mon Sep 17 00:00:00 2001 -From: Michael Polenchuk <mpolenchuk@mirantis.com> -Date: Wed, 26 Oct 2016 15:26:11 +0300 -Subject: [PATCH] Shift public ping checker into deployment stage - -Avoid public vip failover at the end of post-deployment -by moving ping_checker resource into deployment stage. -Also pingd constraint has been changed to be based on score. -If all the nodes failed to ping the default gw, the resource -ain't stopped and remains on the last node where it was. - -Change-Id: I666fbbbe98d000c66f97cf4751f304a2da9aad67 ---- - .../puppet/cluster/manifests/virtual_ip_ping.pp | 10 +++---- - .../puppet/osnailyfacter/modular/astute/tasks.yaml | 15 ---------- - .../osnailyfacter/modular/virtual_ips/tasks.yaml | 30 ++++++++++++++++++++ - 3 files changed, 35 insertions(+), 20 deletions(-) - -diff --git a/deployment/puppet/cluster/manifests/virtual_ip_ping.pp b/deployment/puppet/cluster/manifests/virtual_ip_ping.pp -index 3e49c09..7642422 100644 ---- a/deployment/puppet/cluster/manifests/virtual_ip_ping.pp -+++ b/deployment/puppet/cluster/manifests/virtual_ip_ping.pp -@@ -11,7 +11,7 @@ define cluster::virtual_ip_ping ( - $parameters = { - 'host_list' => $host_list, - 'multiplier' => '1000', -- 'dampen' => '30s', -+ 'dampen' => '45s', - 'timeout' => '3s', - } - $operations = { -@@ -41,16 +41,16 @@ define cluster::virtual_ip_ping ( - primitive => $vip_name, - rules => [ - { -- 'score' => '-inf', -+ 'score' => '50', - 'expressions' => [ - { - 'attribute' => "pingd", -- 'operation' => 'not_defined', -+ 'operation' => 'defined', - }, - { - 'attribute' => "pingd", -- 'operation'=>'lte', -- 'value' => '0', -+ 'operation'=>'gte', -+ 'value' => '1', - }, - ], - }, -diff --git a/deployment/puppet/osnailyfacter/modular/astute/tasks.yaml b/deployment/puppet/osnailyfacter/modular/astute/tasks.yaml -index 5e3e6ee..83242c4 100644 ---- a/deployment/puppet/osnailyfacter/modular/astute/tasks.yaml -+++ b/deployment/puppet/osnailyfacter/modular/astute/tasks.yaml -@@ -10,7 +10,6 @@ - cross-depends: - - name: dns-client - - name: ntp-server -- - name: public_vip_ping - parameters: - cmd: ruby /etc/puppet/modules/osnailyfacter/modular/astute/enable_quorum.rb - timeout: 180 -@@ -110,20 +109,6 @@ - timeout: 180 - cwd: / - --- id: public_vip_ping -- type: puppet -- version: 2.1.0 -- role: [primary-controller, controller] -- requires: [post_deployment_start] -- required_for: [post_deployment_end] -- condition: -- yaql_exp: "changed($.network_scheme) or changed($.get('run_ping_checker'))" -- parameters: -- puppet_manifest: /etc/puppet/modules/osnailyfacter/modular/virtual_ips/public_vip_ping.pp -- puppet_modules: /etc/puppet/modules -- timeout: 3600 -- cwd: / -- - - id: configure_default_route - type: puppet - version: 2.1.0 -diff --git a/deployment/puppet/osnailyfacter/modular/virtual_ips/tasks.yaml b/deployment/puppet/osnailyfacter/modular/virtual_ips/tasks.yaml -index 00d4306..7420099 100644 ---- a/deployment/puppet/osnailyfacter/modular/virtual_ips/tasks.yaml -+++ b/deployment/puppet/osnailyfacter/modular/virtual_ips/tasks.yaml -@@ -34,3 +34,33 @@ - cmd: ruby /etc/puppet/modules/osnailyfacter/modular/virtual_ips/conntrackd_pre.rb - test_post: - cmd: ruby /etc/puppet/modules/osnailyfacter/modular/virtual_ips/conntrackd_post.rb -+ -+- id: primary_public_vip_ping -+ type: puppet -+ version: 2.1.0 -+ groups: [primary-controller] -+ requires: [virtual_ips] -+ required_for: [primary-cluster-haproxy] -+ condition: -+ yaql_exp: &pub_viping "changed($.network_scheme) or changed($.get('run_ping_checker'))" -+ parameters: -+ puppet_manifest: /etc/puppet/modules/osnailyfacter/modular/virtual_ips/public_vip_ping.pp -+ puppet_modules: /etc/puppet/modules -+ timeout: 300 -+ cwd: / -+ -+- id: public_vip_ping -+ type: puppet -+ version: 2.1.0 -+ groups: [controller] -+ requires: [virtual_ips] -+ required_for: [deploy_end] -+ condition: -+ yaql_exp: *pub_viping -+ cross-depends: -+ - name: primary_public_vip_ping -+ parameters: -+ puppet_manifest: /etc/puppet/modules/osnailyfacter/modular/virtual_ips/public_vip_ping.pp -+ puppet_modules: /etc/puppet/modules -+ timeout: 300 -+ cwd: / --- -1.7.9.5 - |