Bug #27552
foreman-proxy-log JOURNAL option should install foreman-proxy-journald
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:
Related issues
Associated revisions
History
#1
Updated by The Foreman Bot over 3 years 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 over 3 years ago
- Fixed in Releases 1.24.0 added
#3
Updated by Ewoud Kohl van Wijngaarden over 3 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset puppet-foreman_proxy|ecaaa16984a9c35de16e741b1745aa4bc151e787.
#4
Updated by Ewoud Kohl van Wijngaarden over 3 years ago
- Triaged changed from No to Yes
- Category set to Foreman modules
- Fixed in Releases 1.22.1 added
#5
Updated by Ewoud Kohl van Wijngaarden over 3 years ago
- Has duplicate Bug #27551: foreman-proxy-log JOURNAL option should install foreman-proxy-journald added
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.