Skip to content
This repository has been archived by the owner on Jul 22, 2020. It is now read-only.

custom alerts grouping #107

Closed
trigan-d opened this issue May 30, 2017 · 6 comments
Closed

custom alerts grouping #107

trigan-d opened this issue May 30, 2017 · 6 comments
Assignees
Labels

Comments

@trigan-d
Copy link

Hi. Thanks for the nice tool!
We (Upwork) have just one major problem with Unsee.
In our AlertManager setup we are grouping the same alerts by different criteria in order to provide different email notifications. Specifically, we have a lot of microservices maintained by several teams. Every team maintains more than one service. Our alerts have both "service" and "team" labels. And AlertManager groups the alerts by service for daily email notification and groups the same alerts by team for weekly digest. So we have 2 nodes in our routing tree.
But Unsee groups all alerts by team only! It seems to be the first group met in AlertManager response, and Unsee just uses it. It is not convenient for us. We'd rather prefer to have the alerts grouped by service in Unsee. But unfortunately this is not configurable at all.
Have you ever considered storing all alerts received from AlertManager as a plain list without any grouping, and then allowing the user to define any "groupBy" condition he want? Perfectly if in UI directly, but in config file is OK too. It'd be just cool, at least for us.
Thanks in advance for any answer!

@prymitive
Copy link
Contributor

An early unsee internal prototype was doing grouping on its own, but we decided to just use Alertmanager groups so that the structure from notifications reflected what you see on the dashboard. This way you didn't need to search for the alert on the dashboard and you have a single source of truth for grouping logic, so I'd rather avoid pushing more logic/config down to unsee layer.
But we do use more real-time notification services than email and it sounds like your case isn't covered well, so we should try to improve that. It seems that a good first step would be to look into supporting multiple routing trees and maybe allowing user to select one of them with a filter (e.g. @tree=email-daily). I'll look into that.

@prymitive prymitive self-assigned this May 30, 2017
@prymitive prymitive added the bug label May 30, 2017
@trigan-d
Copy link
Author

Thanks a lot! We'll be waiting for it.

@prymitive
Copy link
Contributor

I looked into this a bit and it will require some bigger changes. I hope to get a PR working next week.

@prymitive
Copy link
Contributor

There's a WIP receiver branch which handles multiple receivers, it's incomplete but will show all alerts and you can pass @receiver filter, let me know if that helps with your issue @trigan-d

@trigan-d
Copy link
Author

trigan-d commented Jun 8, 2017

@prymitive , thanks a lot! It is completely OK for our needs! Now we are waiting for a release :)

@prymitive
Copy link
Contributor

We're releasing it internally, I'll let it run for a few days to ensure there are no bugs and if all goes well I'll tag a new release next week.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants