Bug 1449697 - Persist passenger customisations from the perf & scaling tuning guide
Summary: Persist passenger customisations from the perf & scaling tuning guide
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Documentation
Version: 6.2.9
Hardware: All
OS: Linux
medium
medium
Target Milestone: Unspecified
Assignee: Pradeep Kumar Surisetty
QA Contact: satellite-doc-list
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-05-10 13:15 UTC by Julio Entrena Perez
Modified: 2023-10-06 17:37 UTC (History)
17 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-11-06 21:17:17 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
2016 - Performance & Scale Tuning of Satellite 6.2 and Capsules (748.67 KB, application/pdf)
2017-05-10 13:15 UTC, Julio Entrena Perez
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1312098 0 high CLOSED [RFE] Satellite 6 should ship with larger default configurations to support large client installs 2023-10-06 17:31:56 UTC
Red Hat Bugzilla 1443138 0 medium CLOSED Customizations from the perf & scale tuning guide are wiped on every upgrade 2023-10-06 17:37:02 UTC

Internal Links: 1312098 1443138

Description Julio Entrena Perez 2017-05-10 13:15:33 UTC
Created attachment 1277617 [details]
2016 - Performance & Scale Tuning of Satellite 6.2 and Capsules

Description of problem:
Page 17 of the performance and scale tuning guide for Satellite 6.2 available at https://access.redhat.com/articles/2626101 suggests to customise the following settings in the following files:

- File /etc/httpd/conf.d/05-foreman-ssl.conf
  PassengerMaxPoolSize
  PassengerMaxRequestQueueSize
  PassengerStatThrottleRate
  PassengerMaxRequests

- File /etc/httpd/conf.d/25-puppet.conf
  PassengerMinInstances
  PassengerStartTimeout
  PassengerMaxPreloaderIdleTime
  PassengerMaxRequests
  PassengerPreStart

Currently when the installer runs, for example when upgrading to the next minor release, these customisations are wiped and replaced with the default values.

There does not seem to be any way of persisting these changes across executions of the installer in file /etc/foreman-installer/custom-hiera.yaml .

This results in minor upgrades bringing back previous issues if customers aren't vigilant and re-apply the changes after each upgrade.

We should provide a way to persist these changes in /etc/foreman-installer/custom-hiera.yaml

Version-Release number of selected component (if applicable):
foreman-installer-1.11.0.15-2.el7sat

How reproducible:
Always

Steps to Reproduce:
1. Customise the settings as per above
2. Run "satellite-installer --scenario satellite --upgrade"
3. Check for the previously applied customisations

Actual results:
Customisations are gone.

Expected results:
Customisations are there after the execution of the installer.

Additional info:

Comment 6 Eric Helms 2017-07-12 15:46:45 UTC
This is a breakdown of how each item can be set, and which items can be set globally and which can be set per vhost. To begin with, the one value that cannot be set at all currently for the puppet vhost:

PassengerMaxPreloaderIdleTime

For puppet vhost:

  PassengerMinInstances: can be set with --puppet-passenger-min-instances
  PassengerPreStart: can be set with --puppet-passenger-pre-start


Globally, that is, it affects both Foreman and Puppet vhosts, the following are set by default in hiera and can be customized:

  PassengerMaxPoolSize: set to 12 by default
  PassengerMaxRequestQueueSize: set to 250 by default
  PassengerStatThrottleRate: set to 120 by default
  PassengerMaxRequests: set to 10000 by default

The following have to be set by the user using a custom hiera config:

  PassengerStartTimeout


Given PassengerMaxPreloaderIdleTime has no current way to set it, I would ask what effect it has on the tuning. Further, I would recommend splitting that out to its own BZ and consider the rest of whats laid out here to be done and part of this BZ.

Comment 16 Mike McCune 2018-11-06 21:17:17 UTC
This is resolved and able to be set in custom-heira.yaml in 6.3 and later


Note You need to log in before you can comment on or make changes to this bug.