Skip to content

Commit

Permalink
Fixes #34625 - Make execution plan cleaner more aggressive
Browse files Browse the repository at this point in the history
It will now wake up every half an hour and remove all execution plans
which have been stopped for at least half an hour. Half an hour should
give Foreman enough time to check up on the proxy in case the callback
from proxy fails, while not being too high at the same time.
  • Loading branch information
adamruzicka authored and ezr-ondrej committed Mar 21, 2022
1 parent a809129 commit 215f45e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion lib/smart_proxy_dynflow/plugin.rb
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ class Plugin < Proxy::Plugin
settings_file "dynflow.yml"
requires :foreman_proxy, ">= 1.16.0"
default_settings :console_auth => true,
:execution_plan_cleaner_age => 60 * 60 * 24
:execution_plan_cleaner_age => 60 * 30
plugin :dynflow, Proxy::Dynflow::VERSION

capability(proc { self.available_operations })
Expand Down
4 changes: 2 additions & 2 deletions settings.d/dynflow.yml.example
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,5 @@
# :console_auth: true

# Maximum age of execution plans to keep before having them cleaned
# by the execution plan cleaner (in seconds), defaults to 24 hours
# :execution_plan_cleaner_age: 86400
# by the execution plan cleaner (in seconds), defaults to 30 minutes
# :execution_plan_cleaner_age: 1800

0 comments on commit 215f45e

Please sign in to comment.