Bug #18422

Expose the foreman-tasks pool size via config/installer

Added by Ivan Necas 9 months ago. Updated about 1 month ago.

Status:Closed
Priority:Normal
Assigned To:Adam Ruzicka
Category:-
Target version:Team Ivan Iteration 21
Difficulty: Bugzilla link:1420397
Found in release: Pull request:https://github.com/theforeman/puppet-foreman/pull/598, https://github.com/theforeman/foreman/pull/4897, https://github.com/theforeman/puppet-foreman/pull/596
Story points-
Velocity based estimate-

Description

Currently, we use a pool of 5 workers in the foreman-tasks to handle the incoming requests. At large scale, this might not be enough. For start, we should be
at least able to set the number of works via config file/installer

As a workaround, one would need to edit `pool_size` in initialize method of

/opt/theforeman/tfm/root/usr/share/gems/gems/foreman-tasks-*/lib/foreman_tasks/dynflow/configuration.rb

and after changing the size, restart the foreman-tasks service

Associated revisions

Revision be9c5b93
Added by Adam Růžička about 1 month ago

Fixes #18422 - Allow configuring Dynflow pool size

Revision 04391025
Added by Adam Růžička about 1 month ago

Fixes #18422 - Expose dynflow worker pool size in config

History

#1 Updated by Ivan Necas 9 months ago

  • Subject changed from Expose the foreman-tasks pool size via config/installer to Expose the foreman-tasks pool size via config/installer
  • Target version set to Team Ivan Iteration 10

#2 Updated by The Foreman Bot about 1 month ago

  • Status changed from New to Ready For Testing
  • Assigned To set to Adam Ruzicka
  • Pull request https://github.com/theforeman/foreman-tasks/pull/284 added

#3 Updated by Ivan Necas about 1 month ago

  • Project changed from foreman-tasks to Foreman

#4 Updated by The Foreman Bot about 1 month ago

  • Pull request https://github.com/theforeman/foreman/pull/4897 added

#5 Updated by Anonymous about 1 month ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#6 Updated by Adam Ruzicka about 1 month ago

  • Pull request https://github.com/theforeman/puppet-foreman/pull/596 added

The Foreman PR was not merged yet and for some reason redmine won't let me reopen this issue

#7 Updated by Adam Ruzicka about 1 month ago

  • Target version changed from Team Ivan Iteration 10 to Team Ivan Iteration 21

#8 Updated by The Foreman Bot about 1 month ago

  • Pull request https://github.com/theforeman/puppet-foreman/pull/598 added

Also available in: Atom PDF