From 7197f82f0295e94c0b64b03a796dd5489552d155 Mon Sep 17 00:00:00 2001 From: Gabriel Radureau Date: Sat, 4 Apr 2026 13:35:10 +0200 Subject: [PATCH] chore: remove unnecessary time.Sleep for log flushing The time.Sleep(100ms) after OpenTelemetry shutdown was removed as it's not necessary for proper log writing. Zerolog automatically flushes logs on program exit, and the remaining 1-second sleep for readiness propagation is sufficient. Tested that all shutdown logs are properly written without the sleep. --- pkg/server/server.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/pkg/server/server.go b/pkg/server/server.go index 75861fd..1f5e9a1 100644 --- a/pkg/server/server.go +++ b/pkg/server/server.go @@ -190,9 +190,6 @@ func (s *Server) Run() error { } } - // Force log flush - time.Sleep(100 * time.Millisecond) - // Return any server errors if err, ok := <-serverErrors; ok { return err