Ema has submitted this change and it was merged.
Change subject: RESTBase: enable metrics batching
..
RESTBase: enable metrics batching
RESTBase sends a lot of metrics to graphite. In order to minimise the
amount of requests done (each of which is rather small in payload),
batch consecutive events together.
Change-Id: I9fe06567cfbc2e5ac17de9c6e644a3fd8246d265
---
M modules/restbase/templates/config.labs.yaml.erb
M modules/restbase/templates/config.yaml.erb
2 files changed, 2 insertions(+), 0 deletions(-)
Approvals:
Filippo Giunchedi: Looks good to me, but someone else must approve
Ema: Verified; Looks good to me, approved
GWicke: Looks good to me, but someone else must approve
diff --git a/modules/restbase/templates/config.labs.yaml.erb
b/modules/restbase/templates/config.labs.yaml.erb
index 45730c6..3cae6a1 100644
--- a/modules/restbase/templates/config.labs.yaml.erb
+++ b/modules/restbase/templates/config.labs.yaml.erb
@@ -135,3 +135,4 @@
type: statsd # default, but lets be explicit
host: <%= @statsd_host %>
port: <%= @statsd_port %>
+ batch: true
diff --git a/modules/restbase/templates/config.yaml.erb
b/modules/restbase/templates/config.yaml.erb
index 87aad54..e7b8109 100644
--- a/modules/restbase/templates/config.yaml.erb
+++ b/modules/restbase/templates/config.yaml.erb
@@ -885,3 +885,4 @@
type: statsd # default, but lets be explicit
host: <%= @statsd_host %>
port: <%= @statsd_port %>
+ batch: true
--
To view, visit https://gerrit.wikimedia.org/r/267917
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9fe06567cfbc2e5ac17de9c6e644a3fd8246d265
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Mobrovac
Gerrit-Reviewer: Ema
Gerrit-Reviewer: Filippo Giunchedi
Gerrit-Reviewer: GWicke
Gerrit-Reviewer: Giuseppe Lavagetto
Gerrit-Reviewer: Mobrovac
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits