Skip to content

Commit

Permalink
Fixes #38022 - Do not use undefined setting
Browse files Browse the repository at this point in the history
Sometimes the production.log is filled with a message: "Setting
remote_execution_workers_pool_size has no definition, please define it
before using"

This message does not appear to be a blocker to any of the existing
functionalities, however; it is misleading while troubleshooting the
issues.
  • Loading branch information
adamlazik1 committed Nov 20, 2024
1 parent c4e8f66 commit 2574bf5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/foreman_remote_execution/engine.rb
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ class Engine < ::Rails::Engine

initializer 'foreman_remote_execution.require_dynflow', :before => 'foreman_tasks.initialize_dynflow' do |app|
ForemanTasks.dynflow.require!
ForemanTasks.dynflow.config.queues.add(DYNFLOW_QUEUE, :pool_size => Setting['remote_execution_workers_pool_size']) if Setting.table_exists? rescue(false)
ForemanTasks.dynflow.config.queues.add(DYNFLOW_QUEUE, :pool_size => Setting['remote_execution_workers_pool_size']) if Setting.find_by :name => 'remote_execution_workers_pool_size' rescue(false)
ForemanTasks.dynflow.config.eager_load_paths << File.join(ForemanRemoteExecution::Engine.root, 'app/lib/actions')
end

Expand Down

0 comments on commit 2574bf5

Please sign in to comment.