[jboss-cvs] JBossAS SVN: r86231 - trunk/server/src/etc/conf/default.

[jboss-cvs-commits at lists.jboss.org]

Author: bstansberry at jboss.com
Date: 2009-03-23 17:20:12 -0400 (Mon, 23 Mar 2009)
New Revision: 86231

Modified:
   trunk/server/src/etc/conf/default/jboss-log4j.xml
Log:
[JBAS-6629] Set root logger priority via jboss.server.log.threshold

Modified: trunk/server/src/etc/conf/default/jboss-log4j.xml
===================================================================
--- trunk/server/src/etc/conf/default/jboss-log4j.xml	2009-03-23 21:13:05 UTC (rev 86230)
+++ trunk/server/src/etc/conf/default/jboss-log4j.xml	2009-03-23 21:20:12 UTC (rev 86231)
@@ -25,13 +25,12 @@
       <errorHandler class="org.jboss.logging.util.OnlyOnceErrorHandler"/>
       <param name="File" value="${jboss.server.log.dir}/server.log"/>
       <param name="Append" value="true"/>
-      <!-- 
-         Set the threshold via a system property. Note this is parsed by log4j,
-         so the full JBoss system property format is not supported; e.g.
-         setting a default via ${jboss.server.log.threshold:WARN} will not work.         
-       -->
+      <!-- In AS 5.0.x the server log threshold was set by a system
+           property. In 5.1 and later we are instead using the system
+           property to set the priority on the root logger (see <root/> below)
       <param name="Threshold" value="${jboss.server.log.threshold}"/>
-
+      -->
+      
       <!-- Rollover at midnight each day -->
       <param name="DatePattern" value="'.'yyyy-MM-dd"/>
 
@@ -329,6 +328,12 @@
    <!-- ======================= -->
 
    <root>
+      <!-- 
+         Set the root logger priority via a system property. Note this is parsed by log4j,
+         so the full JBoss system property format is not supported; e.g.
+         setting a default via ${jboss.server.log.threshold:WARN} will not work.         
+       -->
+      <priority value="${jboss.server.log.threshold}"/>
       <appender-ref ref="CONSOLE"/>
       <appender-ref ref="FILE"/>
    </root>