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

Change subject: wdqs - tuning of logback configuration to send logs to logstash
......................................................................

wdqs - tuning of logback configuration to send logs to logstash

* since ${HOSTNAME} is lazy loaded in latest lgoback version, load it
  explicitely
* rename the "program" field to be more explicit that those logs are about
  wdqs

Bug: T172710
Change-Id: I0648d53d73b75c84f3b5ba352caa50237b3b323d
---
M modules/wdqs/manifests/init.pp
M modules/wdqs/manifests/updater.pp
M modules/wdqs/templates/logback.xml.erb
3 files changed, 6 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/06/374806/1

diff --git a/modules/wdqs/manifests/init.pp b/modules/wdqs/manifests/init.pp
index 35c89eb..237b455 100644
--- a/modules/wdqs/manifests/init.pp
+++ b/modules/wdqs/manifests/init.pp
@@ -110,7 +110,7 @@
         mode    => '0644',
     }
 
-    wdqs::logback_config { 'blazegraph':
+    wdqs::logback_config { 'wdqs-blazegraph':
         logstash_host => $logstash_host,
         logstash_port => $logstash_json_port,
     }
diff --git a/modules/wdqs/manifests/updater.pp 
b/modules/wdqs/manifests/updater.pp
index be7b1c8..1382db3 100644
--- a/modules/wdqs/manifests/updater.pp
+++ b/modules/wdqs/manifests/updater.pp
@@ -23,7 +23,7 @@
         before  => Systemd::Unit['wdqs-updater'],
     }
 
-    wdqs::logback_config { 'updater':
+    wdqs::logback_config { 'wdqs-updater':
         pattern       => '%d{HH:mm:ss.SSS} [%thread] %-5level %logger{36} - 
%msg%n',
         logstash_host => $logstash_host,
         logstash_port => $logstash_json_port,
diff --git a/modules/wdqs/templates/logback.xml.erb 
b/modules/wdqs/templates/logback.xml.erb
index 996ee3f..4057019 100644
--- a/modules/wdqs/templates/logback.xml.erb
+++ b/modules/wdqs/templates/logback.xml.erb
@@ -1,5 +1,9 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <configuration>
+
+    <!-- ugly trick to ensure ${HOSTNAME} is evaluated -->
+    <property scope="context" name="hostname" value="${HOSTNAME}" />
+
     <appender name="STDOUT" class="ch.qos.logback.core.ConsoleAppender">
         <encoder class="ch.qos.logback.classic.encoder.PatternLayoutEncoder">
             <pattern><%= @pattern %></pattern>

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0648d53d73b75c84f3b5ba352caa50237b3b323d
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Gehel <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to