Bug #12134: Debian package missing lsb-release dependency - Packaging - Foreman

Bug #12134

Debian package missing lsb-release dependency

Added by Adam Cécile over 7 years ago. Updated over 4 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Adam Cécile

Category:

Debian/Ubuntu

Target version:

1.9.3

Difficulty:

trivial

Triaged:

Bugzilla link:

Pull request:

https://github.com/theforeman/foreman-packaging/pull/843

Fixed in Releases:

Found in Releases:

Foreman - 1.9.2

Red Hat JIRA:

Description

Hello,

foreman-installer package should depends on lsb-release: https://packages.debian.org/search?keywords=lsb-release
This package provide lsb_release command used by puppet fact to identify the running OS.
Without this package, it fails to detect Debian passenger installation correctly and creates a broken /etc/apache2/mods-available/passenger.conf file which leads too the following error in Apache main log:

[Mon Oct 12 06:31:45.387116 2015] [passenger:error] [pid 26721:tid 139750078269312] *** Passenger could not be initialized because of this error: Unable to start the Phusion Passenger watchdog because its executable (/usr/buildout/agents/PassengerWatchdog) does not exist. This probably means that your Phusion Passenger installation is broken or incomplete, or that your 'PassengerRoot' directive is set to the wrong value. Please reinstall Phusion Passenger or fix your 'PassengerRoot' directive, whichever is applicable. To learn how to fix 'PassengerRoot', please read https://www.phusionpassenger.com/documentation/Users%20guide%20Apache.html#PassengerRoot

Associated revisions

Revision 1a1530c1 (diff)
Added by Adam Cecile over 7 years ago

Add lsb-release dep for LSB facts (fixes #12134)

History

#1 Updated by Anonymous over 7 years ago

while we did do some effort to not use lsb facts in our modules I see that puppetlabs-{apache,apt,mysql,postgresql} are using these esp. for Debian/Ubuntu related things, so I guess it's just fine to add the dependency. I'll open a PR once I have the time (in a few days maybe), otherwise you can do that yourself: https://github.com/theforeman/foreman-packaging/blob/deb/develop/debian/*/foreman-installer/control

#2 Updated by The Foreman Bot over 7 years ago

Status changed from New to Ready For Testing
Pull request https://github.com/theforeman/foreman-packaging/pull/843 added
Pull request deleted ()