Attachment 145218 Details for Bug 32612 – Bug 32612: Add worker-error to log4perl conf

[patch] Bug 32612: Add worker-error to log4perl conf

Bug-32612-Add-worker-error-to-log4perl-conf.patch (text/plain), 3.17 KB, created by Nick Clemens (kidclamp) on 2023-01-11 19:41:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2023-01-11 19:41:56 UTC

Size: 3.17 KB

patch

obsolete

>From 1705095e3ad3b752aa0326fc8a0bc90b1b921ff7 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>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

Actions: View | Diff | Splinter Review

Attachments on bug 32612: 145218 | 145960 | 145972 | 145973 | 145974 | 145995 | 146025 | 146026 | 146027 | 146061 | 146062 | 146063 | 146064 | 146065 | 146066 | 146315 | 146353