Bug #13405: Passenger doesn't start with missing PassengerRoot setting under pl-apache 1.8.0 on EL - Installer - Foreman

Actions

Copy link

Bug #13405

closed

Passenger doesn't start with missing PassengerRoot setting under pl-apache 1.8.0 on EL

Added by Dominic Cleal almost 9 years ago. Updated over 6 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Dominic Cleal

Category:

External modules

Target version:

1.11.0

Difficulty:

Triaged:

Bugzilla link:

Pull request:

https://github.com/puppetlabs/puppetlabs-apache/pull/1345, https://github.com/theforeman/foreman-installer/pull/164, https://github.com/Katello/katello-installer/pull/292, https://github.com/theforeman/foreman-installer/pull/166, https://github.com/Katello/katello-installer/pull/299

Fixed in Releases:

Found in Releases:

Red Hat JIRA:

Description

Passenger is failing to start on EL6/EL7 on nightly builds after the release of puppetlabs-apache 1.8.0.

httpd's error log says:

[Wed Jan 27 08:37:21.705232 2016] [passenger:error] [pid 5789] *** Passenger could not be initialized because of this error: The 'PassengerRoot' configuration option is not specified. This option is required, so please specify it. TIP: The correct value for this option w
as given to you by 'passenger-install-apache2-module'.

/etc/httpd/conf.modules.d/passenger.conf contains:

LoadModule passenger_module modules/mod_passenger.so

On previous releases it wasn't templated at all, allowing the default from the package to come through, which on our packages is something like:

&lt;IfModule mod_passenger.c&gt;
   PassengerRoot /usr/share/gems/gems/passenger-4.0.18/lib/phusion_passenger/locations.ini
   PassengerRuby /usr/bin/ruby
&lt;/IfModule&gt;

Actions

Copy link

Updated by Dominic Cleal almost 9 years ago

Status changed from New to Assigned
Assignee set to Dominic Cleal

https://github.com/puppetlabs/puppetlabs-apache/commit/2a6085929cba0ca4be759ea4d514143dced11dac changed the module to manage /etc/httpd/conf.modules.d/{passenger,passenger_extra}.conf rather than conf.d.

The mod_passenger package installs:

/etc/httpd/conf.modules.d/passenger.conf with LoadModule
/etc/httpd/conf.d/passenger.conf with PassengerRoot

The module used to reconfigure it to have:

/etc/httpd/conf.modules.d/passenger.conf with LoadModule (I think probably unchanged)
/etc/httpd/conf.d/passenger.conf with PassengerRoot (file {} resource but unchanged)
/etc/httpd/conf.d/passenger_extra.conf with any apache::mod::passenger settings
/etc/httpd/conf.d/zpassenger.load with LoadModule

It now configures:

/etc/httpd/conf.modules.d/passenger.conf with LoadModule (probably now matched by the file {} resource)
/etc/httpd/conf.modules.d/passenger_extra.conf with any apache::mod::passenger settings
/etc/httpd/conf.modules.d/zpassenger.load with LoadModule
/etc/httpd/conf.d/passenger.conf is purged

It probably should configure:

/etc/httpd/conf.modules.d/passenger_extra.conf with any apache::mod::passenger settings
/etc/httpd/conf.modules.d/zpassenger.load with LoadModule
/etc/httpd/conf.d/passenger.conf with PassengerRoot (file {} resource but unchanged)

Actions

Copy link

Updated by Dominic Cleal almost 9 years ago

Pull request https://github.com/puppetlabs/puppetlabs-apache/pull/1345 added

Actions

Copy link

Updated by Dominic Cleal almost 9 years ago

Pull request https://github.com/theforeman/foreman-installer/pull/164 added

pl-apache PR merged, and foreman-installer set to the 1.8.x branch for now. It'll need pinning properly to a release (either 1.7.x or 1.8.1) for 1.11.0.

Actions

Copy link