Project

General

Profile

Bug #14023

puppet-foreman does not allow for configuration of PassengerMaxPoolSize

Added by Chris Duryee about 3 years ago. Updated 10 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Foreman modules
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

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 (diff)
Added by Stephen Benjamin over 2 years 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 about 3 years 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 about 3 years ago

  • Project changed from Foreman to Installer

#3 Updated by The Foreman Bot over 2 years 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 over 2 years ago

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

#5 Updated by Dominic Cleal over 2 years ago

  • Legacy Backlogs Release (now unused) set to 189

#6 Updated by Dominic Cleal over 2 years ago

  • Category set to Foreman modules

Also available in: Atom PDF