Mark Bergsma has submitted this change and it was merged.

Change subject: Split gmetad hosts and aggregator hosts
......................................................................


Split gmetad hosts and aggregator hosts

Change-Id: I7353649ae155bebee14b8805b8cd4dcf2436a2c3
---
M modules/ganglia_new/manifests/configuration.pp
M modules/ganglia_new/manifests/monitor.pp
M modules/ganglia_new/manifests/monitor/config.pp
M modules/ganglia_new/templates/gmond.conf.erb
4 files changed, 13 insertions(+), 2 deletions(-)

Approvals:
  Mark Bergsma: Verified; Looks good to me, approved



diff --git a/modules/ganglia_new/manifests/configuration.pp 
b/modules/ganglia_new/manifests/configuration.pp
index 8e8c6a8..63f6602 100644
--- a/modules/ganglia_new/manifests/configuration.pp
+++ b/modules/ganglia_new/manifests/configuration.pp
@@ -120,6 +120,11 @@
                'production': {
                        $url = "http://ganglia.wikimedia.org";
                        $gmetad_hosts = [ "208.80.152.15", "208.80.154.150" ]
+                       $aggregator_hosts = {
+                               'pmtpa' => [ "208.80.152.15", "208.80.154.150" 
],
+                               'eqiad' => [ "208.80.152.15", "208.80.154.150" 
],
+                               'esams' => [ "91.198.174.113" ]
+                       }
                        $base_port = 8649
                        $id_prefix = {
                                pmtpa => 0,
@@ -131,6 +136,10 @@
                'labs': {
                        $url = "http://ganglia.wmflabs.org";
                        $gmetad_hosts = [ "10.4.0.79"]
+                       $aggregator_hosts = {
+                               'pmtpa' => [ "10.4.0.79" ],
+                               'eqiad' => [ "10.4.0.79" ],
+                       }
                        $base_port = 8649
                        $id_prefix = {
                                pmtpa => 0,
diff --git a/modules/ganglia_new/manifests/monitor.pp 
b/modules/ganglia_new/manifests/monitor.pp
index 226f7c8..ce0c041 100644
--- a/modules/ganglia_new/manifests/monitor.pp
+++ b/modules/ganglia_new/manifests/monitor.pp
@@ -15,6 +15,7 @@
                production => "${desc} ${::site}",
                labs => $::instanceproject
        }
+       $aggregator_hosts = 
$ganglia_new::configuration::aggregator_hosts[$::site]
        $override_hostname = $::realm ? {
                production => undef,
                labs => $::instancename
@@ -23,6 +24,7 @@
        class { "ganglia_new::monitor::config":
                gmond_port => $gmond_port,
                cname => $cname,
+               aggregator_hosts => $aggregator_hosts,
                override_hostname => $override_hostname
        }
 }
\ No newline at end of file
diff --git a/modules/ganglia_new/manifests/monitor/config.pp 
b/modules/ganglia_new/manifests/monitor/config.pp
index 3fc0895..140e10e 100644
--- a/modules/ganglia_new/manifests/monitor/config.pp
+++ b/modules/ganglia_new/manifests/monitor/config.pp
@@ -1,4 +1,4 @@
-class ganglia_new::monitor::config($gmond_port, $cname, 
$override_hostname=undef) {
+class ganglia_new::monitor::config($gmond_port, $cname, $aggregator_hosts, 
$override_hostname=undef) {
        require ganglia_new::monitor::packages
 
        $aggregator = false
diff --git a/modules/ganglia_new/templates/gmond.conf.erb 
b/modules/ganglia_new/templates/gmond.conf.erb
index 3ff7b5d..e55810f 100644
--- a/modules/ganglia_new/templates/gmond.conf.erb
+++ b/modules/ganglia_new/templates/gmond.conf.erb
@@ -35,7 +35,7 @@
 }
 
 <% if not aggregator -%>
-<% scope.lookupvar("ganglia_new::configuration::gmetad_hosts").each do |host| 
-%>
+<% aggregator_hosts.each do |host| -%>
 udp_send_channel {
   host = <%= host %>
   port = <%= gmond_port %>

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7353649ae155bebee14b8805b8cd4dcf2436a2c3
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Mark Bergsma <m...@wikimedia.org>
Gerrit-Reviewer: Mark Bergsma <m...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to