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

Graphs in Alerts #1633

Closed
aaranmcguire opened this Issue May 13, 2016 · 6 comments

Comments

Projects
None yet
3 participants
@aaranmcguire
Copy link

aaranmcguire commented May 13, 2016

Hi,

Apologies if this has already been mentioned, or should be in prometheus/alertmanager. I wondered if its posable to embed graphs within the alerts (emails,slack,etc). I've seen this done in Bosun and it makes it really nice when getting alerts as you can visualise the issue (e.g. CPU has just spiked, Disk space is low).

If it were to be implemented I'd want a way to have a different query (Like Boson) so I could get a longer time period.

Is this something thats posable? is someone already working on it? (or has it been turned down?)

I'm new to this project so don't think I have enough experience to implement it but I'd be willing to try if this concept was accepted.

@grobie

This comment has been minimized.

Copy link
Member

grobie commented May 13, 2016

This sounds like an issue you should open in the alertmanager repo instead. The only thing which is available right now is the generator URL. If you click that the prometheus graph page for the alert condition will open.

@brian-brazil

This comment has been minimized.

Copy link
Member

brian-brazil commented May 13, 2016

Alerts are just a set of strings, if you want to do this I'd suggest plumbing through an img, iframe or A tag that'll be displayed by which ever notification method you're using. Another option is to include the expression(s) you want graphed as an annotation and plumb that through to something that can expand it.

From a general architecture/sanity standpoint sending actual images with Prometheus is unlikely to ever happen, and similarly I'd expect Alertmanager will not gain the features to display or expand them as that gets into messy reliability, security and UI stuff.

@aaranmcguire

This comment has been minimized.

Copy link
Author

aaranmcguire commented May 13, 2016

@brian-brazil Thats a shame, Although a URL may work it would of been nice to do something like this:

@brian-brazil

This comment has been minimized.

Copy link
Member

brian-brazil commented May 13, 2016

The way I see it, a single alert definition could easily create 10K alerts. At 10KB each that's 100MB. That's a lot of data to be both regularly sending to the alertmanager, and even sending from the Prometheus as it'd take up a gigabit NIC for a second. This is not a safe feature to offer in a naive fashion.

@aaranmcguire

This comment has been minimized.

Copy link
Author

aaranmcguire commented May 13, 2016

@brian-brazil That makes sense.. I'll close this off then.

@lock

This comment has been minimized.

Copy link

lock bot commented Mar 24, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Mar 24, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.