From dfcb8a72fde7fbab4e819f5e7c93ff7f78465048 Mon Sep 17 00:00:00 2001 From: David Taylor Date: Thu, 3 Feb 2022 14:24:15 +0000 Subject: [PATCH] DEV: Ensure Sidekiq warnings are logged to STDERR (#15800) The default configuration will log to STOUT, which pollutes the output of scripts/rake-tasks --- config/initializers/100-sidekiq.rb | 2 +- lib/demon/sidekiq.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/config/initializers/100-sidekiq.rb b/config/initializers/100-sidekiq.rb index a14a080307a..b309e451237 100644 --- a/config/initializers/100-sidekiq.rb +++ b/config/initializers/100-sidekiq.rb @@ -81,7 +81,7 @@ if Sidekiq.server? end end -Sidekiq.logger.level = Logger::WARN +Sidekiq.logger = Sidekiq::Logger.new(STDERR, level: Logger::WARN) class SidekiqLogsterReporter < Sidekiq::ExceptionHandler::Logger def call(ex, context = {}) diff --git a/lib/demon/sidekiq.rb b/lib/demon/sidekiq.rb index a66fed3aefc..b8d418f9287 100644 --- a/lib/demon/sidekiq.rb +++ b/lib/demon/sidekiq.rb @@ -31,7 +31,7 @@ class Demon::Sidekiq < ::Demon::Base # trouble, if STDOUT is closed in our process all sort of weird # will ensue, resetting the logger ensures it will reinit correctly # parent process is in charge of the file anyway. - Sidekiq.logger = nil + Sidekiq.logger = Sidekiq::Logger.new(STDERR, level: Logger::WARN) cli = Sidekiq::CLI.instance # Unicorn uses USR1 to indicate that log files have been rotated