From 78ed1dc7c6a13a860945c1b11e6a65153c49a5ed Mon Sep 17 00:00:00 2001 From: Guilhem Lettron Date: Wed, 27 Mar 2024 16:56:02 +0100 Subject: [PATCH] fix: init gostats to not flood log (#520) Default gostats store flood messages on stderr except if statd or if GOSTATS_LOGGING_SINK_DISABLED is set to "true". This commit init store in a unambiguous way. Signed-off-by: Guilhem Lettron --- src/service_cmd/runner/runner.go | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/src/service_cmd/runner/runner.go b/src/service_cmd/runner/runner.go index f59ce445..cf7e876e 100644 --- a/src/service_cmd/runner/runner.go +++ b/src/service_cmd/runner/runner.go @@ -38,8 +38,19 @@ type Runner struct { } func NewRunner(s settings.Settings) Runner { + + var store gostats.Store + // use statsd + if s.UseStatsd { + store = gostats.NewStore(gostats.NewTCPStatsdSink(gostats.WithStatsdHost(s.StatsdHost), gostats.WithStatsdPort(s.StatsdPort)), false) + } else { + store = gostats.NewStore(gostats.NewNullSink(), false) + } + + go store.Start(time.NewTicker(10 * time.Second)) + return Runner{ - statsManager: stats.NewStatManager(gostats.NewDefaultStore(), s), + statsManager: stats.NewStatManager(store, s), settings: s, } }