Gehel has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/342618 )

Change subject: elasticsearch - enable shipping logs to logstash for 
elasticsearch 5.x
......................................................................

elasticsearch - enable shipping logs to logstash for elasticsearch 5.x

Shipping logs to logstash was disabled pending more testing. Testing has
been done on relforge1001 and seems to be working just fine. The library
used to ship logs is still the same, but the appender used is the one
compatible with log4j2.

Change-Id: I7c7b32b0f651f30918717d11737f5a7a1ff610aa
---
M modules/elasticsearch/templates/log4j2.properties.erb
1 file changed, 9 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/18/342618/1

diff --git a/modules/elasticsearch/templates/log4j2.properties.erb 
b/modules/elasticsearch/templates/log4j2.properties.erb
index 9d0e276..3d47237 100644
--- a/modules/elasticsearch/templates/log4j2.properties.erb
+++ b/modules/elasticsearch/templates/log4j2.properties.erb
@@ -2,7 +2,7 @@
 rootLogger.level = info
 rootLogger.appenderRef.file.ref = file
 <% if @send_logs_to_logstash %>
-#rootLogger.appenderRef.ship_to_logstash.ref = ship_to_logstash
+rootLogger.appenderRef.ship_to_logstash.ref = ship_to_logstash
 <% end %>
 
 # log action execution errors for easier debugging
@@ -59,13 +59,11 @@
 appender.index_indexing_slow_log_file.layout.pattern = 
[%d{ISO8601}][%-5p][%-25c] %m%n
 
 # ship_to_logstash needs to also be added to rootLogger to actually ship logs
-# @TODO This class is for log4j v1, for v2 should we test
-# org.graylog2.log4j2.GelfAppender ?
-#appender.ship_to_logstash.name = ship_to_logstash
-#appender.ship_to_logstash.type = biz.paluch.logging.gelf.log4j.GelfLogAppender
-#appender.ship_to_logstash.host = udp:<%= @graylog_host %>
-#appender.ship_to_logstash.port <%= @graylog_port %>
-#appender.ship_to_logstash.originHost = <%= @hostname %>
-#appender.ship_to_logstash.facility = elasticsearch
-#appender.ship_to_logstash.extractStackTrace = true
-#appender.ship_to_logstash.threshold = DEBUG
+appender.ship_to_logstash.name = ship_to_logstash
+appender.ship_to_logstash.type = Gelf
+appender.ship_to_logstash.host = udp:<%= @graylog_host %>
+appender.ship_to_logstash.port = <%= @graylog_port %>
+appender.ship_to_logstash.originHost = <%= @hostname %>
+appender.ship_to_logstash.facility = elasticsearch
+appender.ship_to_logstash.extractStackTrace = true
+appender.ship_to_logstash.threshold = DEBUG

-- 
To view, visit https://gerrit.wikimedia.org/r/342618
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7c7b32b0f651f30918717d11737f5a7a1ff610aa
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Gehel <guillaume.leder...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to