[KIEKER-369] bin/log4j.properties in binary and source release archives

rju commented 1 week ago

JIRA Issue: KIEKER-369 bin/log4j.properties in binary and source release archives Original Reporter: Andre van Hoorn

avanhoorn@avanhoorn-thinkpad:0:~/git_work/kieker.releases/1.6 <master> $ unzip -l kieker-1.6_sources.zip | grep "bin/log4j.properties"
avanhoorn@avanhoorn-thinkpad:1:~/git_work/kieker.releases/1.6 <master> $ unzip -l kieker-1.6_binaries.zip | grep "bin/log4j.properties"
avanhoorn@avanhoorn-thinkpad:1:~/git_work/kieker.releases/1.6 <master> $ tar -tzf kieker-1.6_binaries.tar.gz  | grep "bin/log4j.properties"
avanhoorn@avanhoorn-thinkpad:1:~/git_work/kieker.releases/1.6 <master> $ tar -tzf kieker-1.6_sources.tar.gz  | grep "bin/log4j.properties"

For example, when executing bin/traceAnalysis.sh, this leads to the following warning:

avanhoorn@avanhoorn-thinkpad:0:~/svn_work/path/to/kieker-1.6-binaries $ bin/trace-analysis.sh 
log4j:WARN No appenders could be found for logger (kieker.common.logging.LogFactory).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.
usage: kieker.tools.traceAnalysis.TraceAnalysisTool -i <dir1 ... dirN> -o <dir>
       [-p <prefix>] [--plot-Deployment-Sequence-Diagrams]
       [--plot-Assembly-Sequence-Diagrams]
       [--plot-Deployment-Component-Dependency-Graph <node decorations>]
...

Adding the file solves the problem:

avanhoorn@avanhoorn-thinkpad:1:~/svn_work/path/to/kieker-1.6-binaries $ cp ~/git_work/kieker/bin/log4j.properties bin/
avanhoorn@avanhoorn-thinkpad:0:~/svn_work/path/to/kieker-1.6-binaries $ bin/trace-analysis.sh 
usage: kieker.tools.traceAnalysis.TraceAnalysisTool -i <dir1 ... dirN> -o <dir>
       [-p <prefix>] [--plot-Deployment-Sequence-Diagrams]
       [--plot-Assembly-Sequence-Diagrams]
       [--plot-Deployment-Component-Dependency-Graph <node decorations>]
       [--plot-Assembly-Component-Dependency-Graph <node decorations>]
...

This should be included in the release test script

Checklist:

rju commented 3 days ago

author nils-christian -- Tue, 19 Feb 2013 15:21:56 +0100

Or use Kieker parameter to select Java logger.

rju commented 3 days ago

author André van Hoorn -- Tue, 5 Mar 2013 09:56:20 +0100

Missing file added in changeset:cd5f68ebafb40e7b3ad65c11f64d6b0103a35fef/kieker-git and extended the check script:

avhAndrè van Hoorn-ThinkPad-RSS:0:/tmp/kieker-1.7-SNAPSHOT $ bin/trace-analysis.sh 
usage: kieker.tools.traceAnalysis.TraceAnalysisTool -i <dir1 ... dirN> -o <dir>
       [-p <prefix>] [--plot-Deployment-Sequence-Diagrams]
...

In the future, we may want to change the logger to JDK. When starting this activity, reopen.

rju commented 3 days ago

author Jan Waller -- Fri, 15 Mar 2013 16:53:36 +0100

Replying to [avh|comment:3]:
> In the future, we may want to change the logger to JDK. When starting this activity, reopen.

done in 920053c6f45b9a2ab5d92d5f02c4b95fa333a47b

kieker-monitoring / kieker

[KIEKER-369] bin/log4j.properties in binary and source release archives #1154

Checklist: