Bug #14023

puppet-foreman does not allow for configuration of PassengerMaxPoolSize

Added by Chris Duryee 12 months ago. Updated 3 months ago.

Status:Closed
Priority:Normal
Assigned To:Chris Duryee
Category:Foreman modules
Target version:-
Difficulty: Bugzilla link:1163452
Found in release: Pull request:https://github.com/theforeman/foreman-installer/pull/209
Story points-
Velocity based estimate-
Release1.14.0Release relationshipAuto

Description

The katello installer puppet code (not the puppet module itself but the foreman module) does not allow for configuring PassengerMaxPoolSize. It defaults to six, which is not difficult to max out, especially on heavily-used systems.

To replicate this behavior without a true load test, alter the 'index' method in /usr/share/foreman/app/controllers/api/v2/users_controller.rb to be:

     def index
        sleep(20)
        @users = resource_scope_for_index
      end

After this, you should be able to use hammer to get the user list and max out the passenger worker pool (note that this is different than the httpd worker pool).

The default value of six is OK, but it should be spelled out in the conf file that it's six instead of relying on passenger internal defaults, and it should be configurable via installer options.

Associated revisions

Revision 9df10e1b
Added by Stephen Benjamin 3 months ago

fixes #14023 - set passenger tuning options (#209)

max instances to 12 and max instances per app to 6 - this ensures
passenger has enough resources to respond to foreman and puppet
requests when one is under heavy load.

History

#1 Updated by Eric Helms 12 months ago

  • Project changed from Katello to Foreman
  • Subject changed from katello installer does not allow for configuration of PassengerMaxPoolSize to puppet-foreman does not allow for configuration of PassengerMaxPoolSize
  • Category deleted (Performance)

#2 Updated by Dominic Cleal 12 months ago

  • Project changed from Foreman to Installer

#3 Updated by The Foreman Bot 3 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman-installer/pull/209 added

#4 Updated by Stephen Benjamin 3 months ago

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

#5 Updated by Dominic Cleal 3 months ago

  • Release set to 1.14.0

#6 Updated by Dominic Cleal 3 months ago

  • Category set to Foreman modules

Also available in: Atom PDF