Honor the sendmail.smtpUser from gerrit.config on upgrade

If a review site was already existing when the "init" was run but
sendmail.smtpUser was not defined then "init" would set
sendmail.smtpUser to the username of the user running the init. This
would make gerrit use smtp authentication where, before upgrade, smtp
authentication wasn't used.

Instead of doing that, we now honor the value of sendmail.smtpUser from
gerrit.config even when it is null (not present in gerrit.config) in
order to ensure the same behavior of smtp authentication as before
upgrade.

Change-Id: I1b241484ae1d9b5ded7eaf97dbe5dd5cbe10eaf0
Signed-off-by: Sasa Zivkov <sasa.zivkov@sap.com>
1 file changed