From 7017388604a1d1be50efb7868be26f6c48f065c2 Mon Sep 17 00:00:00 2001 From: Takashi Kajinami Date: Sun, 19 Dec 2021 23:45:05 +0900 Subject: [PATCH] Update default value of [filter:slo] max_manifest_size Default value of [filter:slo] max_manifest_size was changed in swift[1] but puppet-swift still hard-codes the old default. This change updates the default value to use the consistent value. [1] 11bf9e45884aa5122e76a97cf62062d7b0bdfe54 Change-Id: Iba68f63dcf5dc401cbbf2d396530a66157ad8fce --- manifests/proxy/slo.pp | 4 ++-- .../update-default-max_manifst_size-1077b1abd8813d7f.yaml | 5 +++++ spec/classes/swift_proxy_slo_spec.rb | 2 +- 3 files changed, 8 insertions(+), 3 deletions(-) create mode 100644 releasenotes/notes/update-default-max_manifst_size-1077b1abd8813d7f.yaml diff --git a/manifests/proxy/slo.pp b/manifests/proxy/slo.pp index 93b14ea0..bd7c31e5 100644 --- a/manifests/proxy/slo.pp +++ b/manifests/proxy/slo.pp @@ -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', diff --git a/releasenotes/notes/update-default-max_manifst_size-1077b1abd8813d7f.yaml b/releasenotes/notes/update-default-max_manifst_size-1077b1abd8813d7f.yaml new file mode 100644 index 00000000..58b5a887 --- /dev/null +++ b/releasenotes/notes/update-default-max_manifst_size-1077b1abd8813d7f.yaml @@ -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. diff --git a/spec/classes/swift_proxy_slo_spec.rb b/spec/classes/swift_proxy_slo_spec.rb index 6db03f19..cb27ee50 100644 --- a/spec/classes/swift_proxy_slo_spec.rb +++ b/spec/classes/swift_proxy_slo_spec.rb @@ -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('') } 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') }