From b544127d9ac0da0c8c6f2c95ce752c0cfc537b68 Mon Sep 17 00:00:00 2001 From: Dimitar Dimitrov Date: Fri, 4 Oct 2024 12:19:12 +0200 Subject: [PATCH] ruler: correct logging of alert name The error messages we see in the embedded ruler in Mimir right now are contain `alert="unsupported value type"` and `data="unsupported value type"` ``` ts=2024-10-04T09:59:56.900015691Z caller=alerting.go:384 level=warn component=ruler insight=true user=XXX alert="unsupported value type" msg="Expanding alert template failed" err="error executing template __alert_service_mesh_down_alert: template: __alert_service_mesh_down_alert:1:139: executing \"__alert_service_mesh_down_alert\" at <.registry_name>: can't evaluate field registry_name in type struct { Labels map[string]string; ExternalLabels map[string]string; ExternalURL string; Value interface {} }" data="unsupported value type" ``` Signed-off-by: Dimitar Dimitrov --- rules/alerting.go | 2 +- rules/manager.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/rules/alerting.go b/rules/alerting.go index 2dc0917dce..4fc90d6139 100644 --- a/rules/alerting.go +++ b/rules/alerting.go @@ -381,7 +381,7 @@ func (r *AlertingRule) Eval(ctx context.Context, queryOffset time.Duration, ts t result, err := tmpl.Expand() if err != nil { result = fmt.Sprintf("", err) - level.Warn(r.logger).Log("msg", "Expanding alert template failed", "err", err, "data", tmplData) + level.Warn(r.logger).Log("msg", "Expanding alert template failed", "err", err, "data", fmt.Sprintf("%#v", tmplData)) } return result } diff --git a/rules/manager.go b/rules/manager.go index 9e5b33fbc9..e316a249fa 100644 --- a/rules/manager.go +++ b/rules/manager.go @@ -323,7 +323,7 @@ func (m *Manager) LoadGroups( externalLabels, externalURL, m.restored, - log.With(m.logger, "alert", r.Alert), + log.With(m.logger, "alert", r.Alert.Value), )) continue }