Eli Mesika has submitted this change and it was merged. Change subject: tools: Removes option MAIL_PORT_SSL ...................................................................... tools: Removes option MAIL_PORT_SSL Removes option MAIL_PORT_SSL. IMO this option is redundant since currently MAIL_PORT or MAIL_PORT_SSL are used based on SSL turned on/off. So it's easier to specify port number only in one option (MAIL_PORT). Change-Id: I76cd77d7f7227e0ae81467189c06c8841ff7f775 Bug-Url: https://bugzilla.redhat.com/1020920 Signed-off-by: Martin Perina <mperina@redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/NotificationProperties.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/sender/mail/JavaMailSender.java M backend/manager/tools/src/test/resources/conf/notifier-mail-test-plain.conf M backend/manager/tools/src/test/resources/conf/notifier-mail-test-secured.conf M packaging/services/ovirt-engine-notifier/ovirt-engine-notifier.conf.in 5 files changed, 4 insertions(+), 10 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22295 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I76cd77d7f7227e0ae81467189c06c8841ff7f775 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Martin Peřina <mperina@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com> Gerrit-Reviewer: mooli tayer <mtayer@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server