diff --git a/pkg/services/screenshot/screenshot.go b/pkg/services/screenshot/screenshot.go index 47e29fdccdb..58241961106 100644 --- a/pkg/services/screenshot/screenshot.go +++ b/pkg/services/screenshot/screenshot.go @@ -132,22 +132,22 @@ func (s *HeadlessScreenshotService) Take(ctx context.Context, opts ScreenshotOpt return nil, fmt.Errorf("failed to take screenshot: %w", err) } - defer s.successes.Inc() + s.successes.Inc() screenshot := Screenshot{Path: result.FilePath} return &screenshot, nil } func (s *HeadlessScreenshotService) instrumentError(err error) { if errors.Is(err, dashboards.ErrDashboardNotFound) { - defer s.failures.With(prometheus.Labels{ + s.failures.With(prometheus.Labels{ "reason": "dashboard_not_found", }).Inc() } else if errors.Is(err, context.Canceled) { - defer s.failures.With(prometheus.Labels{ + s.failures.With(prometheus.Labels{ "reason": "context_canceled", }).Inc() } else { - defer s.failures.With(prometheus.Labels{ + s.failures.With(prometheus.Labels{ "reason": "error", }).Inc() }