Bug 1528852

Summary: SMTP server missing in rhn.conf man page
Product: Red Hat Satellite 5 Reporter: Candace Sheremeta <cshereme>
Component: Configuration ManagementAssignee: Jiří Dostál <jdostal>
Status: CLOSED WONTFIX QA Contact: Red Hat Satellite QA List <satqe-list>
Severity: low Docs Contact:
Priority: unspecified    
Version: 580CC: jdostal, rdrazny, tlestach
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-06-01 13:13:59 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1582457    

Description Candace Sheremeta 2017-12-24 15:14:58 UTC
[RFE] Add variable for SMTP server to rhn.conf file and description in rhn.conf man page

Comment 2 Tomas Lestach 2018-01-02 12:56:19 UTC
java.smtp_server is already a valid configuration variable:

# grep -B2 -r java.smtp_server /usr/share/rhn/config-defaults/
/usr/share/rhn/config-defaults/rhn_java.conf-# Hostname/IP of an SMTP server for sending notifications and traceback
/usr/share/rhn/config-defaults/rhn_java.conf-# alerts to Satellite users
/usr/share/rhn/config-defaults/rhn_java.conf:java.smtp_server = localhost

and can be overridden in rhn.conf anytime.

If this was added to the rhn.conf file, it would just be a duplication of the rhn_java.conf entry.


All the parameters in /usr/share/rhn/config-defaults/rhn*.conf can be overridden in rhn.conf. They're simply not listed in rhn.conf as most users use the defaults anyway and we try to keep the file as short as possible.

We however may consider adding the config param(s) to the man page

Comment 3 Candace Sheremeta 2018-01-02 14:42:56 UTC
removed edits from rhn.conf file per comment 2

Comment 4 Jiří Dostál 2018-01-03 09:19:25 UTC
At the current state of PR, I've merged your commit as eb58d2346858984ae0ca0c42c4e98cdaf6cd6049