Skip to content

Commit

Permalink
Update param docs and tests for WORKER_TTL feature
Browse files Browse the repository at this point in the history
  • Loading branch information
wbclark committed Sep 30, 2021
1 parent 3eebc60 commit eff37b8
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 12 deletions.
3 changes: 2 additions & 1 deletion manifests/init.pp
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,8 @@
# be done incrementally with benchmarking at each step to determine an optimal value for your deployment.
#
# @param worker_ttl
# The number of seconds before a pulpcore worker should be considered lost.
# The number of seconds before a pulpcore worker should be considered lost. If undefined, pulpcore will use its default setting which is 30 seconds as of
# pulpcore version 3.14. You should not need to modify this setting unless the application reports workers timing out while they are busy completing tasks.
#
# @param use_rq_tasking_system
# Use the older RQ workers tasking system instead of the newer PostgreSQL tasking system introduced in Pulpcore 3.14.
Expand Down
4 changes: 0 additions & 4 deletions spec/acceptance/basic_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -76,10 +76,6 @@ class { 'pulpcore':

describe command("PULP_SETTINGS=/etc/pulp/settings.py pulpcore-manager diffsettings") do
its(:stdout) { is_expected.to match(/^USE_NEW_WORKER_TYPE = True/) }
its(:exit_status) { is_expected.to eq 0 }
end

describe command("PULP_SETTINGS=/etc/pulp/settings.py pulpcore-manager diffsettings") do
its(:stdout) { is_expected.to match(/^WORKER_TTL = 30/) }
its(:exit_status) { is_expected.to eq 0 }
end
Expand Down
10 changes: 3 additions & 7 deletions spec/classes/pulpcore_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -514,14 +514,10 @@
end
end

context 'can change the worker_ttl to 60' do
let :params do
{
worker_ttl: 60
}
end
context 'with parameter worker_ttl = 60' do
let :params { worker_ttl: 60 }

it do
it 'configures pulpcore with setting WORKER_TTL = 60' do
is_expected.to contain_concat__fragment('base')
.with_content(%r{\sWORKER_TTL = 60})
end
Expand Down

0 comments on commit eff37b8

Please sign in to comment.