Ottomata has submitted this change and it was merged.

Change subject: Remove ganglia from jmxtrans kafka broker configs
......................................................................


Remove ganglia from jmxtrans kafka broker configs

We don't use these stats in ganglia anyway, and apparently it is
misconfigured.

Change-Id: I9b3f5489e3c391216fce9438ef313a2f7d794d4a
---
M modules/role/manifests/kafka/analytics/broker.pp
M modules/role/manifests/kafka/main/broker.pp
2 files changed, 0 insertions(+), 2 deletions(-)

Approvals:
  Ottomata: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/role/manifests/kafka/analytics/broker.pp 
b/modules/role/manifests/kafka/analytics/broker.pp
index c9a5f70..1a63a77 100644
--- a/modules/role/manifests/kafka/analytics/broker.pp
+++ b/modules/role/manifests/kafka/analytics/broker.pp
@@ -126,7 +126,6 @@
     }
     class { '::kafka::server::jmxtrans':
         group_prefix => $group_prefix,
-        ganglia      => hiera('ganglia_aggregators', undef),
         statsd       => hiera('statsd', undef),
         jmx_port     => $::role::kafka::analytics::config::jmx_port,
         require      => Class['::kafka::server'],
diff --git a/modules/role/manifests/kafka/main/broker.pp 
b/modules/role/manifests/kafka/main/broker.pp
index 5d4b00b..ce4172b 100644
--- a/modules/role/manifests/kafka/main/broker.pp
+++ b/modules/role/manifests/kafka/main/broker.pp
@@ -78,7 +78,6 @@
     $group_prefix = "kafka.cluster.${cluster_name}."
     class { '::kafka::server::jmxtrans':
         group_prefix => $group_prefix,
-        ganglia      => hiera('ganglia_aggregators', undef),
         statsd       => hiera('statsd', undef),
         jmx_port     => $::role::kafka::analytics::config::jmx_port,
         require      => Class['::kafka::server'],

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9b3f5489e3c391216fce9438ef313a2f7d794d4a
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ottomata <[email protected]>
Gerrit-Reviewer: Ottomata <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to