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

removed edits from rhn.conf file per comment 2

At the current state of PR, I've merged your commit as eb58d2346858984ae0ca0c42c4e98cdaf6cd6049