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

Support alertmanager 0.15.x #267

Closed
giganteous opened this issue Jul 19, 2018 · 5 comments
Closed

Support alertmanager 0.15.x #267

giganteous opened this issue Jul 19, 2018 · 5 comments

Comments

@giganteous
Copy link
Contributor

I'm using unsee against AM 0.15.1, and a few things look weird.
Can I verify myself if the new alertmanager is the culprit, or assist in debugging it?

The exact problem I'm experiencing: the alertname doesn't show in the header of an "incident block", it's promoted (or demoted) to the listing inside the incident block.

@djvl
Copy link

djvl commented Aug 29, 2018

I experienced the same and rolled back to 0.14.0..

@prymitive
Copy link
Contributor

@giganteous / @djvl are you running Alertmanager via official docker images by any chance? If so there was a change in the path for the config file used in that docker image, if you updated image but didn't change but are still mounting the config under old path you might get default grouping, which might be related to what you're seeing.
I don't think that 0.15 introduced any API changes.

@terinjokes / @bobrik
It doesn't look like anyone is maintaining this repo after I left cloudflare and it's a shame to see people raising issues and not getting any feedback.
Can I suggest to add a note to the top of the readme file warning people that it's no longer maintained? Or maybe even disable github issues?

@giganteous
Copy link
Contributor Author

@prymitive Thanks for your reply, and the update on the maintenance of this project.

I've tried reproducting with the new 1.15.2, and the problem is now gone (still a problem on 1.15.1). @djvl can you confirm this?

@djvl
Copy link

djvl commented Aug 31, 2018

I'm using an RPM install of alertmanager. After some digging it appears the default for 'group_by' has changed recently (v0.15.x), see: prometheus/alertmanager#1288
So, when setting the following in 'alertmanager.yml' the old behavior is restored.

route:
  group_by: ['alertname']

@prymitive I switched to your repo and it looks good! Thanks for mentioning it.

@giganteous
Copy link
Contributor Author

Cheers!

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

No branches or pull requests

3 participants