Petals ESB Container

Remove the hard-coded configuration of logging sub-system

Details

  • Type: Task Task
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 4.1.0
  • Fix Version/s: 4.2.0
  • Component/s: Configuration, Logging
  • Security Level: Public
  • Description:
    Hide

    A lot of stuff about logging configuration is done when starting Petals ESB, see:

    • org.ow2.petals.kernel.server.PetalsServerImpl#initializeLogger
    • org.ow2.petals.kernel.server.PetalsServerImpl#forceInitializeGlobalHandlers
    • org.ow2.petals.kernel.server.PetalsServerImpl#recreateConsoleHandler
    • org.ow2.petals.kernel.server.PetalsServerImpl#recreateConsoleHandler
    • org.ow2.petals.kernel.server.PetalsServerImpl#finalizeLoggingInitialization

    All this seems unneeded. If a dedicated configuration is needed to support MONIT traces, just update documentation and provide a correct pre-configured logging system.
    To prevent error in configuration of the logging system about business traces (level MONIT), we can add a check of the configuration, and errors will be logged as WARNING messages.

    Show
    A lot of stuff about logging configuration is done when starting Petals ESB, see:
    • org.ow2.petals.kernel.server.PetalsServerImpl#initializeLogger
    • org.ow2.petals.kernel.server.PetalsServerImpl#forceInitializeGlobalHandlers
    • org.ow2.petals.kernel.server.PetalsServerImpl#recreateConsoleHandler
    • org.ow2.petals.kernel.server.PetalsServerImpl#recreateConsoleHandler
    • org.ow2.petals.kernel.server.PetalsServerImpl#finalizeLoggingInitialization
    All this seems unneeded. If a dedicated configuration is needed to support MONIT traces, just update documentation and provide a correct pre-configured logging system. To prevent error in configuration of the logging system about business traces (level MONIT), we can add a check of the configuration, and errors will be logged as WARNING messages.
  • Environment:
    -

Issue Links

Activity

Christophe DENEUX made changes - Wed, 11 Jul 2012 - 18:21:53 +0200
Field Original Value New Value
Link This issue blocks PETALSESBCONT-196 [ PETALSESBCONT-196 ]
Christophe DENEUX made changes - Thu, 12 Jul 2012 - 05:45:35 +0200
Priority Blocker [ 1 ]
Description A lot of stuff about logging configuration is done when starting Petals ESB, see:
* {{org.ow2.petals.kernel.server.PetalsServerImpl#initializeLogger}}
* {{org.ow2.petals.kernel.server.PetalsServerImpl#forceInitializeGlobalHandlers}}
* {{org.ow2.petals.kernel.server.PetalsServerImpl#recreateConsoleHandler}}
* {{org.ow2.petals.kernel.server.PetalsServerImpl#recreateConsoleHandler}}
* {{org.ow2.petals.kernel.server.PetalsServerImpl#finalizeLoggingInitialization}}

All this seems unneeded. If a dedicated configuration is needed to support MONIT traces, just update documentation and provide a correct pre-configured logging system.

A lot of stuff about logging configuration is done when starting Petals ESB, see:
* {{org.ow2.petals.kernel.server.PetalsServerImpl#initializeLogger}}
* {{org.ow2.petals.kernel.server.PetalsServerImpl#forceInitializeGlobalHandlers}}
* {{org.ow2.petals.kernel.server.PetalsServerImpl#recreateConsoleHandler}}
* {{org.ow2.petals.kernel.server.PetalsServerImpl#recreateConsoleHandler}}
* {{org.ow2.petals.kernel.server.PetalsServerImpl#finalizeLoggingInitialization}}

All this seems unneeded. If a dedicated configuration is needed to support MONIT traces, just update documentation and provide a correct pre-configured logging system.
To prevent error in configuration of the logging system about business traces (level MONIT), we can add a check of the configuration, and errors will be logged as WARNING messages.

Christophe DENEUX made changes - Mon, 3 Sep 2012 - 12:49:44 +0200
Link This issue depends on PETALSESBCONT-205 [ PETALSESBCONT-205 ]
Christophe DENEUX made changes - Mon, 3 Sep 2012 - 12:50:03 +0200
Status New [ 10000 ] Open [ 10002 ]
Priority Blocker [ 1 ] Major [ 3 ]
Christophe DENEUX made changes - Mon, 3 Sep 2012 - 12:50:20 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Hide
Christophe DENEUX added a comment - Mon, 3 Sep 2012 - 12:50:29 +0200

Fixed in trunk

Show
Christophe DENEUX added a comment - Mon, 3 Sep 2012 - 12:50:29 +0200 Fixed in trunk
Christophe DENEUX made changes - Mon, 3 Sep 2012 - 12:50:29 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Fix Version/s 4.2.0 [ 10356 ]
Resolution Fixed [ 1 ]
Christophe DENEUX made changes - Thu, 7 Mar 2013 - 12:15:10 +0100
Component/s Logging [ 10255 ]
Component/s Administration [ 10052 ]
Christophe DENEUX made changes - Tue, 12 Mar 2013 - 08:17:33 +0100
Component/s Configuration [ 10257 ]
Transition Status Change Time Execution Times Last Executer Last Execution Date
New New Open Open
53d 18h 29m
1
Christophe DENEUX
Mon, 3 Sep 2012 - 12:50:03 +0200
Open Open In Progress In Progress
17s
1
Christophe DENEUX
Mon, 3 Sep 2012 - 12:50:20 +0200
In Progress In Progress Resolved Resolved
9s
1
Christophe DENEUX
Mon, 3 Sep 2012 - 12:50:29 +0200



People

Dates

  • Created:
    Wed, 11 Jul 2012 - 18:20:16 +0200
    Updated:
    Tue, 12 Mar 2013 - 08:17:33 +0100
    Resolved:
    Mon, 3 Sep 2012 - 12:50:29 +0200