[FOLIO-849] Logging formats for backend FOLIO modules Created: 20/Sep/17 Updated: 16/Nov/20 Resolved: 16/Nov/20 |
|
| Status: | Closed |
| Project: | FOLIO |
| Components: | None |
| Affects versions: | None |
| Fix versions: | None |
| Type: | Bug | Priority: | P3 |
| Reporter: | John Malconian | Assignee: | Unassigned |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | devops | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original estimate: | Not Specified | ||
| Issue links: |
|
||||||||||||||||||||||||||||||||
| Sprint: | |||||||||||||||||||||||||||||||||
| Development Team: | Core: Platform | ||||||||||||||||||||||||||||||||
| Description |
|
Logging formats across modules are inconsistent which makes it more difficult to parse logs. Some details of these inconsistencies are highlighted in the comments. Is it possible that we standardize on a common log format? |
| Comments |
| Comment by John Malconian [ 20/Sep/17 ] |
|
Log inconsistency examples: 1. Multiple date formats within modules. From mod-notify (not the only module to exhibit this): 04:03:02 INFO Version HV000001: Hibernate Validator 5.2.4.Final 04:03:06 WARN AbstractKieModule No files found for KieBase defaultKieBase 04:03:07 INFO KieRepositoryImpl KieModule was added: MemoryKieModule[releaseId=org.default:artifact:1.0.0-SNAPSHOT] Sep 16, 2017 4:03:07 AM org.folio.rest.tools.utils.LogUtil 2. Multiple date formats across modules: Sep 16, 2017 4:09:01 AM org.folio.rest.persist.PostgresClient[ [Wed Sep 20 17:59:34 UTC 2017] 04:03:07 3. Log entries span multiple lines. Common across many modules: Sep 20, 2017 5:42:04 AM io.vertx.core.net.impl.ConnectionBase SEVERE: java.lang.IllegalArgumentException: invalid version format: 0 This log entry should be one line. |
| Comment by shale99 [ 27/Mar/18 ] |
|
rmb modules should log as follows (this is as of the 16th of October - commit - 9b29d9cb2873348c87e33282cbb2485efd91e504) %d {dd MMM yyyy HH:mm:ss:SSS}%-5p %C {1}%X {reqId}%m%n this includes mod-notify - if this is not the case, its a bug |
| Comment by Jakub Skoczen [ 16/Nov/20 ] |
|
Resolving as a dupe of
|