-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Add UUID To Alert #2332
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
Add UUID To Alert #2332
Conversation
linclaus
commented
Jul 28, 2020
- add UUID attribute to each alert, so that we can locate them
Signed-off-by: Simon Pasquier <spasquie@redhat.com> Signed-off-by: linclaus <linclaus@outlook.com>
Signed-off-by: Simon Pasquier <spasquie@redhat.com> Signed-off-by: linclaus <linclaus@outlook.com>
Signed-off-by: Simon Pasquier <spasquie@redhat.com> Signed-off-by: linclaus <linclaus@outlook.com>
|
That would not work in a HA setup. |
Can you outline the reasons? |
|
Because when you have mulitple Alertmanager instances, each of them will generate a different UUID value while the alerts are the same. I'm not sure what you're trying to accomplish here. |
|
And I think that alerts also offer a fingerprint now. |
|
Alerts are already uniquely identified by their labels. |
Fingerprint can only distinguish between alerts with different labels, but what if I want to distinguish each alert that has been sent? |
So we will receive multiple alerts from one Alertmanager route when there have multiple Alertmanager instances? From the recipient's point of view, I want to record and trace all the alerts I received. |
|
Hello from the Alertmanager bug scrub. This change is stale so it will be closed. If you would like to continue this work, please open a new pull request. |