From c09978f9cd9f5eebde21c4061d92490167e72dd2 Mon Sep 17 00:00:00 2001 From: james pickett Date: Wed, 7 Feb 2024 13:10:12 -0800 Subject: [PATCH] dont set slogger where not used, change rungroup leveling --- ee/desktop/user/notify/notify_darwin.go | 4 +--- ee/desktop/user/notify/notify_windows.go | 4 +--- pkg/rungroup/rungroup.go | 2 +- 3 files changed, 3 insertions(+), 7 deletions(-) diff --git a/ee/desktop/user/notify/notify_darwin.go b/ee/desktop/user/notify/notify_darwin.go index ee8e92242..ee64bbd0b 100644 --- a/ee/desktop/user/notify/notify_darwin.go +++ b/ee/desktop/user/notify/notify_darwin.go @@ -24,13 +24,11 @@ import ( ) type macNotifier struct { - slogger *slog.Logger interrupt chan struct{} } -func NewDesktopNotifier(slogger *slog.Logger, _ string) *macNotifier { +func NewDesktopNotifier(_ *slog.Logger, _ string) *macNotifier { return &macNotifier{ - slogger: slogger.With("component", "desktop_notifier"), interrupt: make(chan struct{}), } } diff --git a/ee/desktop/user/notify/notify_windows.go b/ee/desktop/user/notify/notify_windows.go index 508e75b33..fe2828f11 100644 --- a/ee/desktop/user/notify/notify_windows.go +++ b/ee/desktop/user/notify/notify_windows.go @@ -11,14 +11,12 @@ import ( type windowsNotifier struct { iconFilepath string - slogger *slog.Logger interrupt chan struct{} } -func NewDesktopNotifier(slogger *slog.Logger, iconFilepath string) *windowsNotifier { +func NewDesktopNotifier(_ *slog.Logger, iconFilepath string) *windowsNotifier { return &windowsNotifier{ iconFilepath: iconFilepath, - slogger: slogger.With("component", "desktop_notifier"), interrupt: make(chan struct{}), } } diff --git a/pkg/rungroup/rungroup.go b/pkg/rungroup/rungroup.go index 20a9f6657..b674d4ac9 100644 --- a/pkg/rungroup/rungroup.go +++ b/pkg/rungroup/rungroup.go @@ -77,7 +77,7 @@ func (g *Group) Run() error { // Wait for the first actor to stop. initialActorErr := <-errors - g.slogger.Log(context.TODO(), slog.LevelDebug, + g.slogger.Log(context.TODO(), slog.LevelInfo, "received interrupt error from first actor -- shutting down other actors", "err", initialActorErr, )