Bug #18422

Expose the foreman-tasks pool size via config/installer

Added by Ivan Necas 11 months ago. Updated 3 months 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 3 months ago

Fixes #18422 - Allow configuring Dynflow pool size

Revision 04391025
Added by Adam Růžička 3 months ago

Fixes #18422 - Expose dynflow worker pool size in config

History

#1 Updated by Ivan Necas 11 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 4 months 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 3 months ago

  • Project changed from foreman-tasks to Foreman

#4 Updated by The Foreman Bot 3 months ago

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

#5 Updated by Anonymous 3 months ago

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

#6 Updated by Adam Ruzicka 3 months 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 3 months ago

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

#8 Updated by The Foreman Bot 3 months ago

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

Also available in: Atom PDF