Project

General

Profile

Bug #27552

foreman-proxy-log JOURNAL option should install foreman-proxy-journald

Added by Ewoud Kohl van Wijngaarden about 1 month ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Category:
Foreman modules
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1738784

Description of problem:

The foreman-proxy-journald package is not installed when you run installer with this option:

--foreman-proxy-log JOURNAL

but it is required to make use of logging to journald.

Version-Release number of selected component (if applicable):

~]# rpm -q satellite-installer
satellite-installer-6.6.0.19-1.beta.el7sat.noarch

How reproducible:

Steps to Reproduce:
1. satellite-installer --foreman-proxy-log JOURNAL
2. rpm -q foreman-proxy-journald

Actual results:

~]# rpm -q foreman-proxy-journald
package foreman-proxy-journald is not installed

Expected results:
~]# rpm -q foreman-proxy-journald
foreman-proxy-journald-1.22.0.1-1.el7sat.noarch

Additional info:

https://bugzilla.redhat.com/show_bug.cgi?id=1163020#c18

Associated revisions

Revision ecaaa169 (diff)
Added by Ewoud Kohl van Wijngaarden about 1 month ago

Fixes #27552 - Fix journald logging

fde8d9b9547b1dc50b6010616b4aaf5bde51fbf4 attempted to implement
installing foreman-proxy-journald when the logging was set to JOURNAL.
The check was comparing JOURNAL to JOURNALD.

This patch corrects it and adds both unit and acceptance tests.

History

#1 Updated by The Foreman Bot about 1 month ago

  • Assignee set to Ewoud Kohl van Wijngaarden
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/puppet-foreman_proxy/pull/531 added

#2 Updated by The Foreman Bot about 1 month ago

  • Fixed in Releases 1.24.0 added

#3 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Status changed from Ready For Testing to Closed

#4 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Triaged changed from No to Yes
  • Category set to Foreman modules
  • Fixed in Releases 1.22.1 added

Also available in: Atom PDF