Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upAlerts should be relabelable #1744
Comments
brian-brazil
added
the
kind/enhancement
label
Jun 15, 2016
This comment has been minimized.
This comment has been minimized.
aecolley
commented
Jul 14, 2016
|
What labels can prevent an alert from being sent to alertmanagers? I traced the path back from notifier's POST to sendAlerts in rules/manager.go but didn't find any label-related filtering. |
This comment has been minimized.
This comment has been minimized.
|
This issue is about implementing such support. |
juliusv
added
the
help wanted
label
Jul 23, 2016
fabxc
assigned
brancz
Aug 8, 2016
This comment has been minimized.
This comment has been minimized.
|
@brian-brazil is this not fixed with #1878 ? If not can you elaborate? |
This comment has been minimized.
This comment has been minimized.
|
Relabelling should be after external labels, not before. |
This comment has been minimized.
This comment has been minimized.
|
Ok that makes sense. If I understand correctly, we just need to move this before relabelling? |
This comment has been minimized.
This comment has been minimized.
|
Yes |
This comment has been minimized.
This comment has been minimized.
|
PR on it's way |
brancz
referenced this issue
Sep 27, 2016
Merged
notifier: attach external labels before relabelling #2035
juliusv
closed this
in
#2035
Sep 27, 2016
This comment has been minimized.
This comment has been minimized.
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. |
brian-brazil commentedJun 15, 2016
Before sending alerts out you should have a chance to relabel them. Use cases include having a different external label on a HA pair that you want to strip so that the alertmanager will de-dupe them, and having alerts that are only used as part of
ALERTSfor other alerts and shouldn't be sent to the alertmanager.Only labels would be relabelable, not annotations.
ALERTSis not affected.