openstack
/
puppet-swift
Code Issues Proposed changes

Merge "Update default value of [filter:slo] max_manifest_size"

This commit is contained in:
Zuul 2021-12-23 14:38:34 +00:00 committed by Gerrit Code Review
parent b0552cf939 7017388604
commit 10295c4595
3 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/proxy/slo.pp
View File

@ -13,7 +13,7 @@
#
# [*max_manifest_size*]
# (Optional) Max manifest size.
# Defaults to 2097152.
# Defaults to 8388608.
#
# [*rate_limit_under_size*]
# (Optional) Rate limiting applies only to segments smaller than this size.
@ -69,7 +69,7 @@
#
class swift::proxy::slo (
$max_manifest_segments = '1000',
$max_manifest_size = '2097152',
$max_manifest_size = '8388608',
$rate_limit_under_size = $::os_service_default,
$rate_limit_after_segment = '10',
$rate_limit_segments_per_sec = '1',

5
releasenotes/notes/update-default-max_manifst_size-1077b1abd8813d7f.yaml Normal file
View File

@ -0,0 +1,5 @@
---
upgrade:
- |
Default value of ``swift::proxy::slo::max_manifest_size`` has been changed
from ``2097152`` to ``8388608``, following the change in swift itself.

2
spec/classes/swift_proxy_slo_spec.rb
View File

@ -5,7 +5,7 @@ describe 'swift::proxy::slo' do
describe "when using default parameters" do
it { is_expected.to contain_swift_proxy_config('filter:slo/use').with_value('egg:swift#slo') }
it { is_expected.to contain_swift_proxy_config('filter:slo/max_manifest_segments').with_value('1000') }
it { is_expected.to contain_swift_proxy_config('filter:slo/max_manifest_size').with_value('2097152') }
it { is_expected.to contain_swift_proxy_config('filter:slo/max_manifest_size').with_value('8388608') }
it { is_expected.to contain_swift_proxy_config('filter:slo/rate_limit_under_size').with_value('<SERVICE DEFAULT>') }
it { is_expected.to contain_swift_proxy_config('filter:slo/rate_limit_after_segment').with_value('10') }
it { is_expected.to contain_swift_proxy_config('filter:slo/rate_limit_segments_per_sec').with_value('1') }