Gage has submitted this change and it was merged.
Change subject: logstash: Drop full_message field from GELF messages
......................................................................
logstash: Drop full_message field from GELF messages
The nodejs logging library used by Parsoid and OCG duplicates the whole
message in the 'full_message' attribute.
Change-Id: I9d67fb4c8ca0a32cde07690a7164bce799574b59
---
M files/logstash/filter-gelf.conf
1 file changed, 6 insertions(+), 0 deletions(-)
Approvals:
Subramanya Sastry: Looks good to me, but someone else must approve
Gage: Looks good to me, approved
GWicke: Looks good to me, but someone else must approve
jenkins-bot: Verified
diff --git a/files/logstash/filter-gelf.conf b/files/logstash/filter-gelf.conf
index 19e0f71..33be766 100644
--- a/files/logstash/filter-gelf.conf
+++ b/files/logstash/filter-gelf.conf
@@ -69,6 +69,12 @@
action => "replace"
}
}
+
+ # Drop duplicated json content added by nodejs gelf-stream library
+ mutate {
+ remove_field => [ "full_message" ]
+ }
+
prune {
# get rid of a whole bunch of useless _ fields that are duplicated
# We must keep _id, _index, and _type
--
To view, visit https://gerrit.wikimedia.org/r/169727
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9d67fb4c8ca0a32cde07690a7164bce799574b59
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: BryanDavis <[email protected]>
Gerrit-Reviewer: BryanDavis <[email protected]>
Gerrit-Reviewer: Cscott <[email protected]>
Gerrit-Reviewer: GWicke <[email protected]>
Gerrit-Reviewer: Gage <[email protected]>
Gerrit-Reviewer: Subramanya Sastry <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
