[Bug 13413] New: Koha::Log - an attempt to have sane logging in Koha
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Bug ID: 13413 Summary: Koha::Log - an attempt to have sane logging in Koha Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: gmcharlt@gmail.com Reporter: srdjan@catalyst.net.nz QA Contact: testopia@bugs.koha-community.org Koha is painfully lacking logging infrastructure. This is an attempt to retrofit one. It is based on Log::Contextual rather than Log::Log4Perl for the sake of simplicity and efficiency. This is a discussion bite. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Srdjan Jankovic <srdjan@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #1 from Srdjan Jankovic <srdjan@catalyst.net.nz> --- Created attachment 34188 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34188&action=edit bug_13413: Koha::Log - logging for Koha -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #2 from Srdjan Jankovic <srdjan@catalyst.net.nz> --- Created attachment 34189 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34189&action=edit bug_13413: use Koha::Log with syslog sink instead of syslog directly in Sipserver.pm This way we can sift out debugs -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #3 from Srdjan Jankovic <srdjan@catalyst.net.nz> --- Created attachment 34190 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34190&action=edit bug_13413: Koha::Log in longoverdue.pl -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #4 from Srdjan Jankovic <srdjan@catalyst.net.nz> --- There are two patches to give an example how to use Koha::Log -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=8190 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #5 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Agreed we need logging, and I'm fairly happy with the implimentation here.. However, I'd like some clear comparisons drawn to the work corried out in bug 8190 and how this patch may or may not solve the issues raised there. I don't want this bug to end up again in a long 'In Discussion' dead end, but it would be good to know why you feel this is a better aproach so we can sell it to the community as a whole :) Thanks for the work so far Srdjan -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #6 from Srdjan Jankovic <srdjan@catalyst.net.nz> --- * For some reason when I started this work, I did not see bug 8190 when I searched. Horrible omission, no excuses. * bug 8190 is a bit wider, and this replaces only one part of it (the logging module itself). If this is chosen, bug 8190 can easily be reworked to use this logger instead of its own implementation. Now comparison: * Log::LogLite seems to be limited to logging to a file only. To me it is a serious limitation, so if we wanted to extend it to support multiple sinks, we'd need to roll our sleeves. Nothing bad in that, but still. An example would be logging to the standard sink (a file or syslog) and sending error emails, eg "log me at info level, but email me errors". * What really attracted me to Log::Contextual was the use of code blocks to log, so complex debugging can be left in without any performance penalty (as stated in the pod). Although some may argue it is confusing. * There are some fancy stuff one could do (temp logger etc), but that should not be taken in account when making decision, basic stuff is more important. I suggest some people go over this implementation of Koha::Log, and bug 8190 implementation of Koha::Utils::Logger, and decide which one is a better fit. There are also other interesting points for discussion, but no point bringing those up before we decide if tis is a go or no go. In that respect this patch is intentionally minimal, and leaves hairy issues where/how to introduce it in the web app out. Having said that, it was implemented with intention to be able to fit both in central point (Context), the individual scripts and library modules at the same time - ie some best practices should be drawn, but nothing will break if you use it in anger. The starting point was that I needed a cron job logger, and I put some more effort in hope it may find wider use. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #7 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Thanks for your quick reply, and good explanation Srdjan. I think I agree that I prefer the logging capabilities given using Log::Contextual, and love the code blocks for efficiency. I'de love to see this as a starting point for moving towards the more expansive implementation as conceived in bug 8190.. I'm going to test a bit now. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@biblibre.co | |m -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Robin Sheat <robin@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |robin@catalyst.net.nz --- Comment #8 from Robin Sheat <robin@catalyst.net.nz> --- (In reply to Martin Renvoize from comment #7)
Thanks for your quick reply, and good explanation Srdjan.
I think I agree that I prefer the logging capabilities given using Log::Contextual, and love the code blocks for efficiency.
I'de love to see this as a starting point for moving towards the more expansive implementation as conceived in bug 8190.. I'm going to test a bit now.
Martin, did you have any joy? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #9 from Jonathan Druart <jonathan.druart@biblibre.com> --- Could you provide an example of how we could use this module from others Koha modules (C4::*, Koha::*) and scripts (module/*.pl)? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #10 from Srdjan Jankovic <srdjan@catalyst.net.nz> --- There is a number of examples in the POD (patch 1). Also, patches 2 and 3 are supposed to be working implementations. Can you please have a look and tell me if anything needs expanding/reformulating/improving, happy to do that. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #11 from Srdjan Jankovic <srdjan@catalyst.net.nz> --- USAGE and ADVANCED USAGE sections in POD -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #12 from Jonathan Druart <jonathan.druart@biblibre.com> --- Actually this patch is stuck in NSO without any test plan. Since I really would like to see a logging module integrated into Koha, I just try to make this moving forward. So let's test this lazily: $ git bz apply 13413 # ok Try to use the module: diff --git a/C4/Context.pm b/C4/Context.pm index c5f92ee..9a88758 100644 --- a/C4/Context.pm +++ b/C4/Context.pm @@ -107,6 +107,8 @@ use DateTime::TimeZone; use Module::Load::Conditional qw(can_load); use Carp; +use Koha::Log qw(:log with_debug); + =head1 NAME C4::Context - Maintain and manipulate the context of a Koha script @@ -564,6 +566,7 @@ sub preference { } $sysprefs{lc $var} = $value; + log_debug{"pref $var = $value"}; return $value; } diff --git a/mainpage.pl b/mainpage.pl index ab3a7ab..635a233 100755 --- a/mainpage.pl +++ b/mainpage.pl @@ -30,6 +30,8 @@ use C4::Suggestions qw/CountSuggestion/; use C4::Tags qw/get_count_by_tag_status/; use Koha::Borrower::Modifications; +use Koha::Log qw(:log with_debug); + my $query = new CGI; my ( $template, $loggedinuser, $cookie, $flags ) = get_template_and_user( @@ -42,6 +44,8 @@ my ( $template, $loggedinuser, $cookie, $flags ) = get_template_and_user( } ); +log_debug{'User on the mainpage!'}; + my $homebranch; if (C4::Context->userenv) { $homebranch = C4::Context->userenv->{'branch'}; $ tail -f koha_logs Go on the mainpage. boom, module is missing ok let's install it (no mention in the commit messages) $ sudo apt-get install liblog-contextual-perl Go on the mainpage, nothing in the logs. Ha, debug no defined, let's replace with log_info Go on the mainpage. The messages appear in the logs \o/ So the question is: How can I configure a log level for all the application? perldoc Koha/Log.pm mentions 3 levels: debug, info, warn. What about others? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #13 from Jonathan Druart <jonathan.druart@biblibre.com> --- Did you read the discussion on bug 8190? There are a lot of ideas on it. The way to use it is: my $logger = C4::Context->logger(); $logger->info("OPAC: Search for $query"); You can have a look at the tests for more information. All the job is mainly done in the constructor (Koha::Utils::Logger->new) and the write method. And a syspref controls the log level. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|gmcharlt@gmail.com |srdjan@catalyst.net.nz -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #14 from Srdjan Jankovic <srdjan@catalyst.net.nz> --- (In reply to Jonathan Druart from comment #13)
Did you read the discussion on bug 8190? There are a lot of ideas on it.
The way to use it is:
my $logger = C4::Context->logger(); $logger->info("OPAC: Search for $query");
Yes, and that is precisely what I find inflexible. One size fits all is offered. My approach was use it where/when you need it the way you need it. Log to file/syslog/email/wherever depending on your needs. So the answer is - you don't. Eg you configure it in the psgi script, or in a cron job. Or in a module, if you need something special. My initial pains were SIP and some cron jobs. I do understand that majority may prefer having it already there in context with some less flexibility, in which case we'll pick the other one and go with that. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Srdjan Jankovic <srdjan@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #34188|0 |1 is obsolete| | --- Comment #15 from Srdjan Jankovic <srdjan@catalyst.net.nz> --- Created attachment 36259 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36259&action=edit bug_13413: Koha::Log - logging for Koha -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Srdjan Jankovic <srdjan@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #34189|0 |1 is obsolete| | --- Comment #16 from Srdjan Jankovic <srdjan@catalyst.net.nz> --- Created attachment 36260 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36260&action=edit bug_13413: use Koha::Log with syslog sink instead of syslog directly in Sipserver.pm This way we can sift out debugs -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Srdjan Jankovic <srdjan@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #34190|0 |1 is obsolete| | --- Comment #17 from Srdjan Jankovic <srdjan@catalyst.net.nz> --- Created attachment 36261 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36261&action=edit bug_13413: Koha::Log in longoverdue.pl -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #18 from Srdjan Jankovic <srdjan@catalyst.net.nz> ---
Go on the mainpage. boom, module is missing
Added dependancy. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Dobrica Pavlinusic <dpavlin@rot13.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dpavlin@rot13.org -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=14167 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 --- Comment #19 from Jonathan Druart <jonathan.druart@biblibre.com> --- Srdjan, Did you have a look at the Kyle's try on bug 14167? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |In Discussion --- Comment #20 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- I'm in general preferring the simplicity of Bug 14167 for now. Lets go with that and get this much needed feature into Koha! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13413 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |RESOLVED Resolution|--- |DUPLICATE CC| |tomascohen@gmail.com --- Comment #21 from Tomás Cohen Arazi <tomascohen@gmail.com> --- *** This bug has been marked as a duplicate of bug 14167 *** -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org