From 6d107a7f41ec4ad0e3c225f3cf2bb05b3ddedbce Mon Sep 17 00:00:00 2001 From: "Grot (@grafanabot)" <43478413+grafanabot@users.noreply.github.com> Date: Tue, 31 May 2022 09:07:57 -0400 Subject: [PATCH] Alerting: Add Go error message to warning log for screenshots. (#49870) (#49917) Makes debugging problems with alert screenshotting easier. (cherry picked from commit 56f40bd41377b9242a32e28c97a99073fa237809) Co-authored-by: Joe Blubaugh --- pkg/services/ngalert/state/manager.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/pkg/services/ngalert/state/manager.go b/pkg/services/ngalert/state/manager.go index 5937a793780..42760d73f14 100644 --- a/pkg/services/ngalert/state/manager.go +++ b/pkg/services/ngalert/state/manager.go @@ -247,10 +247,11 @@ func (st *Manager) setNextState(ctx context.Context, alertRule *ngModels.AlertRu err := st.maybeTakeScreenshot(ctx, alertRule, currentState, oldState) if err != nil { - st.log.Warn("Error generating a screenshot for an alert instance.", + st.log.Warn("failed to generate a screenshot for an alert instance", "alert_rule", alertRule.UID, "dashboard", alertRule.DashboardUID, - "panel", alertRule.PanelID) + "panel", alertRule.PanelID, + "err", err) } st.set(currentState)