Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better error when SMTP is not configured #8093

Closed
mattttt opened this issue Apr 11, 2017 · 0 comments
Closed

Better error when SMTP is not configured #8093

mattttt opened this issue Apr 11, 2017 · 0 comments
Labels
area/alerting/notifications Issues when sending alert notifications area/alerting Grafana Alerting
Milestone

Comments

@mattttt
Copy link
Contributor

mattttt commented Apr 11, 2017

When adding an Email Notification Channel and SMTP is not configured, the interface throws this error:

screenshot 2017-04-11 16 24 54

This is not a helpful msg, and doesnt give the user any next steps to try and address the issue. The Grafana logs say:

EROR[04-11|16:24:51] Failed to send alert notifications       logger=context userId=1 orgId=1 uname=admin error="Grafana mailing/smtp options not configured, contact your Grafana admin"

however unless you're looking at the server logs, you'd never know it.

Proposed solution

When configuring alerts, do a check to see if SMTP is configured. If not, show a warning box within the form flow stating "Grafana mailing/SMTP options have not been configured for your instance. Ask your Grafana admin to set this up."

@mattttt mattttt added area/alerting Grafana Alerting area/alerting/notifications Issues when sending alert notifications labels Apr 11, 2017
@mattttt mattttt changed the title Better Error when SMTP not configured Better error when SMTP is not configured Apr 11, 2017
@torkelo torkelo added this to the 4.3.0 milestone Apr 12, 2017
daniellee added a commit that referenced this issue May 10, 2017
ref #8277, ref #8250, ref #8262, ref #8165, ref #8093, ref #8056, ref #8043, ref #7970, ref #7914, ref #7864, ref #7750, ref #7740, ref #7697, ref #7619, ref #5619, ref #4030, ref #5278, ref #3302, ref #2524
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/alerting/notifications Issues when sending alert notifications area/alerting Grafana Alerting
Projects
None yet
Development

No branches or pull requests

2 participants