kieker-monitoring / kieker

Kieker's main repository
Apache License 2.0
70 stars 41 forks source link

[KIEKER-422] Java EE example: Include option to use custom Kieker configuration in start.ini #1155

Open rju opened 1 week ago

rju commented 1 week ago

JIRA Issue: KIEKER-422 Java EE example: Include option to use custom Kieker configuration in start.ini Original Reporter: Andre van Hoorn


Checklist:

rju commented 5 days ago

author André van Hoorn -- Wed, 21 Nov 2012 17:07:37 +0100

Option added as comment in changeset:c8e65a33b48c715c3ec1824e4e8c52f88477d702/kieker-git

rju commented 5 days ago

author André van Hoorn -- Wed, 21 Nov 2012 17:09:04 +0100

We should add a copy of Kieker's META-INF directory (including default configurations) to the Jetty dir when packaging the release file.

rju commented 5 days ago

author André van Hoorn -- Thu, 4 Apr 2013 14:46:20 +0200

Implemented in changeset:bace033c8a255f9985ef1b8e6d8e7a94e3be3d69/kieker-git

The example already included a configuration file. However, this was located in the webapps/jpetstore/WEB-INF/classes/META-INF/ directory and, hence, used as Kieker finds it in the classpath. I moved this file to the Jetty's root directory and enabled the (slightly modified options mentioned in comment:1). Also adjusted the check release script.

1. Spring

...
 INFO 2013-04-04 14:26:25,061 kieker.common.logging.LogImplCommonsLogging: Loading configuration from JVM-specified location: 'kieker.monitoring.properties'
 INFO 2013-04-04 14:26:25,116 kieker.common.logging.LogImplCommonsLogging: Current State of kieker.monitoring (1.7-SNAPSHOT) Status: 'enabled'
...

2. AspectJ

...
INFO 2013-04-04 14:35:57,044 kieker.common.logging.LogImplCommonsLogging: Loading configuration from JVM-specified location: 'kieker.monitoring.properties'
 INFO 2013-04-04 14:35:57,074 kieker.common.logging.LogImplCommonsLogging: Current State of kieker.monitoring (1.7-SNAPSHOT) Status: 'enabled'
...

Updated user guide in changeset:4e7b7ddd3767148ec50f874ccf20c4f58a756e12/kieker-git