Ottomata has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/206423

Change subject: Open more ports for impala, include impalad on hadoop worker 
nodes
......................................................................

Open more ports for impala, include impalad on hadoop worker nodes

Bug: T96329
Change-Id: Id72184bac0de070336d00deda0430c146dba2ada
---
M manifests/role/analytics/impala.pp
M manifests/site.pp
2 files changed, 4 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/23/206423/1

diff --git a/manifests/role/analytics/impala.pp 
b/manifests/role/analytics/impala.pp
index 31cb6dc..1a90fc4 100644
--- a/manifests/role/analytics/impala.pp
+++ b/manifests/role/analytics/impala.pp
@@ -21,7 +21,7 @@
 
     ferm::service { 'impalad':
         proto  => 'tcp',
-        port   => '(22000 28000)',
+        port   => '(21000 21050 22000 23000 25000 28000)',
         srange => '$ANALYTICS_NETWORKS',
     }
 }
@@ -35,12 +35,12 @@
 
     ferm::service { 'impala-state-store':
         proto  => 'tcp',
-        port   => '24000',
+        port   => '(24000 25010)',
         srange => '$ANALYTICS_NETWORKS',
     }
     ferm::service { 'impala-catalog':
         proto  => 'tcp',
-        port   => '25020',
+        port   => '(23020 25020 26000)',
         srange => '$ANALYTICS_NETWORKS',
     }
     ferm::service { 'impala-llama':
diff --git a/manifests/site.pp b/manifests/site.pp
index 051c108..2abdbfd 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -122,14 +122,7 @@
     if $::hostname =~ /^analytics101[349]$/ {
         $ganglia_aggregator = true
     }
-    role analytics::hadoop::worker
-
-    # For now, I want to test this out on only one node, to be sure that 
configs
-    # are applied how I expect.  If this works, I will include this class via 
the
-    # role keyword on all workers here.
-    if $hostname == 'analytics1041' {
-        include role::analytics::impala::worker
-    }
+    role analytics::hadoop::worker, analytics::impala::worker
 
     include standard
 }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id72184bac0de070336d00deda0430c146dba2ada
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ottomata <o...@wikimedia.org>

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

Reply via email to