From 1705095e3ad3b752aa0326fc8a0bc90b1b921ff7 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Wed, 11 Jan 2023 18:46:11 +0000 Subject: [PATCH] Bug 32612: Add worker-error to log4perl conf To test: 1 - Apply patch 2 - vim /etc/koha/sites/kohadev/log4perl.conf, Add lines below: log4perl.logger.worker = WARN, WORKER log4perl.appender.WORKER=Log::Log4perl::Appender::File log4perl.appender.WORKER.filename=/var/log/koha/kohadev/worker-error.log log4perl.appender.WORKER.mode=append log4perl.appender.WORKER.layout=PatternLayout log4perl.appender.WORKER.layout.ConversionPattern=[%d] [%p] %m %l%n log4perl.appender.WORKER.utf8=1 3 - Restart all 4 - Edit misc/background_jobs_worker.pl - my $job = Koha::BackgroundJobs->find($args->{job_id}); + my $job;# = Koha::BackgroundJobs->find($args->{job_id}); 5 - In another terminal: tail -f /var/log/koha/kohadev/koha-worker-error.log 6 - Force enqueue a job (that won't be found because of #4 perl -e 'use Koha::BackgroundJob::BatchUpdateItem; my $bg = Koha::BackgroundJob::BatchUpdateItem->new(); $bg->enqueue({ record_ids=>['888888']});' 7 - Note error in log like: [2023/01/11 19:26:10] [WARN] No job found for id=2983 main:: /kohadevbox/koha/misc/background_jobs_worker.pl (111) --- etc/log4perl.conf | 8 ++++++++ misc/background_jobs_worker.pl | 4 ++-- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/etc/log4perl.conf b/etc/log4perl.conf index a07d139d96..c01d1e67d4 100644 --- a/etc/log4perl.conf +++ b/etc/log4perl.conf @@ -14,6 +14,14 @@ log4perl.appender.OPAC.layout=PatternLayout log4perl.appender.OPAC.layout.ConversionPattern=[%d] [%p] %m %l%n log4perl.appender.OPAC.utf8=1 +log4perl.logger.worker = WARN, WORKER +log4perl.appender.WORKER=Log::Log4perl::Appender::File +log4perl.appender.WORKER.filename=__LOG_DIR__/worker-error.log +log4perl.appender.WORKER.mode=append +log4perl.appender.WORKER.layout=PatternLayout +log4perl.appender.WORKER.layout.ConversionPattern=[%d] [%p] %m %l%n +log4perl.appender.WORKER.utf8=1 + log4perl.logger.z3950 = WARN, Z3950 log4perl.appender.Z3950=Log::Log4perl::Appender::File log4perl.appender.Z3950.filename=__LOG_DIR__/z3950-error.log diff --git a/misc/background_jobs_worker.pl b/misc/background_jobs_worker.pl index aa5a2eb590..49b96d7525 100755 --- a/misc/background_jobs_worker.pl +++ b/misc/background_jobs_worker.pl @@ -94,7 +94,7 @@ while (1) { my $body = $frame->body; decode_json($body); # TODO Should this be from_json? Check utf8 flag. } catch { - Koha::Logger->get->warn(sprintf "Frame not processed - %s", $_); + Koha::Logger->get({ interface => 'worker', category => 'error' })->warn(sprintf "Frame not processed - %s", $_); } finally { $conn->ack( { frame => $frame } ); }; @@ -106,7 +106,7 @@ while (1) { my $job = Koha::BackgroundJobs->find($args->{job_id}); unless ( $job ) { - Koha::Logger->get->warn(sprintf "No job found for id=%s", $args->{job_id}); + Koha::Logger->get({ interface => 'worker', category => 'error' })->warn(sprintf "No job found for id=%s", $args->{job_id}); next; } -- 2.30.2