Project

General

Profile

Actions

Bug #12134

closed

Debian package missing lsb-release dependency

Added by Adam Cécile over 8 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Debian/Ubuntu
Target version:
Difficulty:
trivial
Triaged:
Fixed in Releases:
Found in Releases:

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

Actions #1

Updated by Anonymous over 8 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

Actions #2

Updated by The Foreman Bot over 8 years ago

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

Updated by Dominic Cleal over 8 years ago

  • Assignee set to Adam Cécile
  • translation missing: en.field_release set to 91
Actions #5

Updated by Dominic Cleal over 8 years ago

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

Also available in: Atom PDF