openstack/puppet-swift
Code Issues Proposed changes

storage: Add support for db_preallocation

Browse Source 
Change-Id: I6f9f46b2c00e29d108d9f943b54e2205844ef973
This commit is contained in:
Takashi Kajinami 2024-08-14 18:44:29 +09:00
parent ebc1224ee1
commit 5b6630e97b
3 changed files with 15 additions and 0 deletions

8
manifests/storage/server.pp

@ -79,6 +79,11 @@
# all times. This option affects only <type>-server processes.
# Defaults to $facts['os_service_default'].
#
# [*db_preallocation*]
# (optional) Preallocate disk space with SQLite database to decrease
# fragmentation.
# Defaults to $facts['os_service_default'].
#
# [*servers_per_port*]
# (optional) Spawn multiple servers per device on different ports.
# Make object-server run this many worker processes per unique port of
@ -296,6 +301,7 @@ define swift::storage::server(
$disable_fallocate = $facts['os_service_default'],
$fallocate_reserve = $facts['os_service_default'],
$server_fallocate_reserve = $facts['os_service_default'],
$db_preallocation = $facts['os_service_default'],
$servers_per_port = $facts['os_service_default'],
$user = undef,
$workers = $facts['os_workers'],
@ -480,6 +486,7 @@ define swift::storage::server(
case $type {
'account': {
$type_opts = {
'DEFAULT/db_preallocation' => {'value' => $db_preallocation},
# account-server
# account-auditor
# account-replicator
@ -511,6 +518,7 @@ define swift::storage::server(
}
'container': {
$type_opts = {
'DEFAULT/db_preallocation' => {'value' => $db_preallocation},
'DEFAULT/allowed_sync_hosts' => {'value' => join($::swift::storage::container::allowed_sync_hosts, ',')},
# container-server
# container-auditor

5
releasenotes/notes/db_preallocation-4226028636c46c4d.yaml Normal file

@ -0,0 +1,5 @@
---
features:
- |
The new ``swift::storage::server::db_preallocation`` parameter has been
added.

2
spec/defines/swift_storage_server_spec.rb

@ -98,6 +98,7 @@ describe 'swift::storage::server' do
is_expected.to contain_swift_account_config('DEFAULT/log_statsd_sample_rate_factor').with_value('<SERVICE DEFAULT>')
is_expected.to contain_swift_account_config('DEFAULT/log_statsd_metric_prefix').with_value('<SERVICE DEFAULT>')
is_expected.to contain_swift_account_config('DEFAULT/db_preallocation').with_value('<SERVICE DEFAULT>')
is_expected.to contain_swift_account_config('account-auditor/').with_ensure('present')
is_expected.to contain_swift_account_config('account-auditor/log_name').with_ensure('absent')
is_expected.to contain_swift_account_config('account-replicator/').with_ensure('present')
@ -287,6 +288,7 @@ describe 'swift::storage::server' do
is_expected.to contain_swift_container_config('DEFAULT/log_statsd_sample_rate_factor').with_value('<SERVICE DEFAULT>')
is_expected.to contain_swift_container_config('DEFAULT/log_statsd_metric_prefix').with_value('<SERVICE DEFAULT>')
is_expected.to contain_swift_container_config('DEFAULT/db_preallocation').with_value('<SERVICE DEFAULT>')
is_expected.to contain_swift_container_config('DEFAULT/allowed_sync_hosts').with_value('127.0.0.1')
is_expected.to contain_swift_container_config('container-auditor/').with_ensure('present')
is_expected.to contain_swift_container_config('container-auditor/log_name').with_ensure('absent')