Bug #12134

Debian package missing lsb-release dependency

Added by Adam Cécile almost 3 years ago. Updated 7 days ago.

Status:Closed
Priority:Normal
Assignee:Adam Cécile
Category:Debian/Ubuntu
Target version:1.9.3
Difficulty:trivial Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link: Found in Releases:Foreman - 1.9.2
Pull request:https://github.com/theforeman/foreman-packaging/pull/843

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
Added by Adam Cecile almost 3 years ago

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

History

#1 Updated by Michael Moll almost 3 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 almost 3 years ago

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

#4 Updated by Dominic Cleal almost 3 years ago

  • Assignee set to Adam Cécile
  • Legacy Backlogs Release (now unused) set to 91

#5 Updated by Dominic Cleal almost 3 years ago

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

Also available in: Atom PDF