From 0eb885bded94b5e5a952e61872e00654dcf1a99a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=81ukasz=20Mierzwa?= Date: Mon, 27 Mar 2017 19:40:04 -0700 Subject: [PATCH] Compare timestamps as is when sorting Remove rounding of timestamps when sorting alert list, not sure why it was added, it's not needed --- models/models.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/models/models.go b/models/models.go index 3035b8faa..a60adbb3a 100644 --- a/models/models.go +++ b/models/models.go @@ -67,9 +67,8 @@ func (a UnseeAlertList) Swap(i, j int) { a[i], a[j] = a[j], a[i] } func (a UnseeAlertList) Less(i, j int) bool { - // compare timestamps rounded up to 2s, subsecond accuracy is lost to keep - // ordering stable even with small time drift - return a[i].StartsAt.Round(2 * time.Second).After(a[j].StartsAt.Round(2 * time.Second)) + // compare timestamps, if equal compare labels + return a[i].StartsAt.After(a[j].StartsAt) } // UnseeAlertGroup is vanilla Alertmanager group, but alerts are flattened