Bug #12134

Debian package missing lsb-release dependency

Added by Adam Cécile over 2 years ago. Updated over 2 years ago.

Status:Closed
Priority:Normal
Assigned To:Adam Cécile
Category:Debian/Ubuntu
Target version:-
Difficulty:trivial Bugzilla link:
Found in release:1.9.2 Pull request:https://github.com/theforeman/foreman-packaging/pull/843
Story points-
Velocity based estimate-
Release1.9.3Release relationshipAuto

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 over 2 years ago

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

History

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

  • Assigned To set to Adam Cécile
  • Release set to 1.9.3

#5 Updated by Dominic Cleal over 2 years ago

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

Also available in: Atom PDF