ArielGlenn has submitted this change and it was merged.

Change subject: hadoop install needs java
......................................................................


hadoop install needs java

Change-Id: I52379667dbac77d3c54abca1d214472b97b4ec0f
---
M manifests/role/analytics/hadoop.pp
1 file changed, 3 insertions(+), 1 deletion(-)

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



diff --git a/manifests/role/analytics/hadoop.pp 
b/manifests/role/analytics/hadoop.pp
index df6c4da..13482c3 100644
--- a/manifests/role/analytics/hadoop.pp
+++ b/manifests/role/analytics/hadoop.pp
@@ -23,7 +23,9 @@
 #
 class role::analytics::hadoop::client {
     # need java before hadoop is installed
-    require role::analytics::java
+    if (!defined(Package['openjdk-7-jdk'])) {
+        package { 'openjdk-7-jdk': }
+    }
 
     # include common labs or production hadoop configs
     # based on $::realm

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

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

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

Reply via email to