diff --git a/dispatch/dispatch.go b/dispatch/dispatch.go index 6883786dca..c16ad068d7 100644 --- a/dispatch/dispatch.go +++ b/dispatch/dispatch.go @@ -350,7 +350,7 @@ func (d *Dispatcher) processAlert(alert *types.Alert, route *Route) { go ag.run(func(ctx context.Context, alerts ...*types.Alert) bool { _, _, err := d.stage.Exec(ctx, d.logger, alerts...) if err != nil { - logger := d.logger.With("num_alerts", len(alerts), "err", err) + logger := d.logger.With("aggrGroup", ag.GroupKey(), "num_alerts", len(alerts), "err", err) if errors.Is(ctx.Err(), context.Canceled) { // It is expected for the context to be canceled on // configuration reload or shutdown. In this case, the diff --git a/notify/webhook/webhook.go b/notify/webhook/webhook.go index 41b9f497dc..e94361234f 100644 --- a/notify/webhook/webhook.go +++ b/notify/webhook/webhook.go @@ -87,7 +87,7 @@ func (n *Notifier) Notify(ctx context.Context, alerts ...*types.Alert) (bool, er n.logger.Error("error extracting group key", "err", err) } - // @tjhop: should we debug log the key here like most other Notify() implementations? + n.logger.Debug("extracted group key", "key", groupKey.String()) msg := &Message{ Version: "4",