JoaoJandre commented on code in PR #10980:
URL: https://github.com/apache/cloudstack/pull/10980#discussion_r2140920666


##########
client/conf/log4j-cloud.xml.in:
##########
@@ -50,9 +50,7 @@ under the License.
       <!-- Append warnings+ to the syslog if it is listening on UDP port -->
       <!-- ============================== -->
 
-      <Syslog name="SYSLOG" host="localhost" facility="LOCAL6">
-         <ThresholdFilter level="WARN" onMatch="ACCEPT" onMismatch="DENY"/>
-         <PatternLayout pattern="%d{DEFAULT} %-5p [%c{1.}] (%t:%x) 
(logid:%X{logcontextid}) %m%ex{filters(${filters})}%n"/>

Review Comment:
   According to the documentation, the Syslog appender accepts a Filter and a 
Layout, same as the other appenders that are defined in this file (see 
https://logging.apache.org/log4j/2.12.x/manual/appenders.html#SyslogAppender).
   In my env I see that the logs are not being appended to syslog, but that is 
because it is not listening for it. If I configure it to listen on port 514, 
and add the `port="514" protocol="UDP"` configuration on the log4j2 xml it 
starts working.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to