Ottomata has uploaded a new change for review.
https://gerrit.wikimedia.org/r/307524
Change subject: Make sure zookeeper package is installed on namenodes too
......................................................................
Make sure zookeeper package is installed on namenodes too
Bug: T138263
Change-Id: I092380cc1bc7f03cf21422b0cfb6331b1499d4c0
---
M manifests/hadoop/namenode.pp
M manifests/hadoop/nodemanager.pp
2 files changed, 10 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet/cdh
refs/changes/24/307524/1
diff --git a/manifests/hadoop/namenode.pp b/manifests/hadoop/namenode.pp
index ace2186..8b497eb 100644
--- a/manifests/hadoop/namenode.pp
+++ b/manifests/hadoop/namenode.pp
@@ -17,6 +17,12 @@
}
if ($::cdh::hadoop::ha_enabled and $::cdh::hadoop::zookeeper_hosts) {
+ if !defined(Package['zoookeeper']) {
+ package { 'zookeeper':
+ ensure => 'installed'
+ }
+ }
+
package { 'hadoop-hdfs-zkfc':
ensure => 'installed',
}
diff --git a/manifests/hadoop/nodemanager.pp b/manifests/hadoop/nodemanager.pp
index b6f1b7f..ee6dcf4 100644
--- a/manifests/hadoop/nodemanager.pp
+++ b/manifests/hadoop/nodemanager.pp
@@ -14,8 +14,10 @@
# zookeeper package here explicitly. This avoids
# java.lang.NoClassDefFoundError: org/apache/zookeeper/KeeperException
# errors.
- package { 'zookeeper':
- ensure => 'installed'
+ if !defined(Package['zoookeeper']) {
+ package { 'zookeeper':
+ ensure => 'installed'
+ }
}
# NodeManager (YARN TaskTracker)
--
To view, visit https://gerrit.wikimedia.org/r/307524
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I092380cc1bc7f03cf21422b0cfb6331b1499d4c0
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet/cdh
Gerrit-Branch: master
Gerrit-Owner: Ottomata <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
