Notifications do not use the SMTP server if auth method is none

Bug #1558140 reported by Swann Croiset
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StackLight
Fix Released
Medium
Swann Croiset

Bug Description

Service notifications: when the 'SMTP authentication method' is 'none' the SMTP server IP is not used but use the local smtp server Postfix

Tags: nagios smtp
Revision history for this message
Simon Pasquier (simon-pasquier) wrote :

Could you create another bug report to avoid conflating the 2 issues?

Swann Croiset (swann-w)
description: updated
summary: - Nagios notifications doesn't use smtp parameters in some cases
+ Notifications do not use the SMTP server if auth method is none
Changed in lma-toolchain:
status: New → Confirmed
importance: Undecided → Medium
Swann Croiset (swann-w)
Changed in lma-toolchain:
milestone: none → 0.10.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-plugin-lma-infrastructure-alerting (master)

Fix proposed to branch: master
Review: https://review.openstack.org/341076

Changed in lma-toolchain:
assignee: LMA-Toolchain Fuel Plugins (mos-lma-toolchain) → Swann Croiset (swann-w)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-plugin-lma-infrastructure-alerting (stable/0.9)

Fix proposed to branch: stable/0.9
Review: https://review.openstack.org/341540

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-plugin-lma-infrastructure-alerting (stable/0.10)

Fix proposed to branch: stable/0.10
Review: https://review.openstack.org/344118

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-plugin-lma-infrastructure-alerting (stable/0.10)

Reviewed: https://review.openstack.org/344118
Committed: https://git.openstack.org/cgit/openstack/fuel-plugin-lma-infrastructure-alerting/commit/?id=5d153fda7e0fc3f914143b69815cc6d8ce5dfce4
Submitter: Jenkins
Branch: stable/0.10

commit 5d153fda7e0fc3f914143b69815cc6d8ce5dfce4
Author: Swann Croiset <email address hidden>
Date: Tue Jul 12 17:29:30 2016 +0200

    Fix SMTP without authentication

    Closes-bug: #1558140
    Change-Id: I1272414be40d79f2f32c937a0b6e7c852b649af5

Changed in lma-toolchain:
assignee: Swann Croiset (swann-w) → Simon Pasquier (simon-pasquier)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-plugin-lma-infrastructure-alerting (master)

Reviewed: https://review.openstack.org/341076
Committed: https://git.openstack.org/cgit/openstack/fuel-plugin-lma-infrastructure-alerting/commit/?id=3ce87b3aaca45f2c6b263ab88315d597c47a9227
Submitter: Jenkins
Branch: master

commit 3ce87b3aaca45f2c6b263ab88315d597c47a9227
Author: Swann Croiset <email address hidden>
Date: Tue Jul 12 17:29:30 2016 +0200

    Fix SMTP without authentication

    Closes-bug: #1558140
    Change-Id: I1272414be40d79f2f32c937a0b6e7c852b649af5

Changed in lma-toolchain:
status: In Progress → Fix Committed
Revision history for this message
Andrey Grebennikov (agrebennikov) wrote :

This patch is not going to work, as well as the initial approach.
In Ubuntu 14.04 mail client doesn't support -r variable.
We have to properly set "myhostname" in /etc/postfix/main.cf, according to the UI settings, and somehow we have to specify username when sending email.

# mail -s 'testmessage' <email address hidden> -r <email address hidden> < hello
mail: invalid option -- 'r'
usage: mail [-dEIinv] [-a header] [-b bcc-addr] [-c cc-addr] [-s subject] to-addr ...
       mail [-dEIiNnv] -f [name]
       mail [-dEIiNnv] [-u user]

Revision history for this message
Andrey Grebennikov (agrebennikov) wrote :

the root cause of the issue - for Some Misterious reason I got the package "bsd-mailx" installed, while manifests are clearly saying "hairloom-mailx", which was not. When I installed proper package - it replaced /etc/alternatives and now /usr/bin/mail points to the proper one.
Please take a look.

Revision history for this message
Andrey Grebennikov (agrebennikov) wrote :

I found why - the patch actually fixes it in upstream while for me in 0.9 there was no such requirement for the package.

Changed in lma-toolchain:
status: Fix Committed → Fix Released
Swann Croiset (swann-w)
Changed in lma-toolchain:
assignee: Simon Pasquier (simon-pasquier) → Swann Croiset (swann-w)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-plugin-lma-infrastructure-alerting (stable/0.9)

Change abandoned by Swann Croiset (<email address hidden>) on branch: stable/0.9
Review: https://review.openstack.org/341540
Reason: for history !

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.