Ottomata has submitted this change and it was merged.
Change subject: Use ensure_packages for java on hadoop clients
......................................................................
Use ensure_packages for java on hadoop clients
Change-Id: I0b5d91f0647cdf2b370ae6ce4dade8fd1f1777b7
---
M manifests/role/analytics/hadoop.pp
1 file changed, 1 insertion(+), 5 deletions(-)
Approvals:
Ottomata: Looks good to me, approved
jenkins-bot: Verified
diff --git a/manifests/role/analytics/hadoop.pp
b/manifests/role/analytics/hadoop.pp
index f0150bf..a41567d 100644
--- a/manifests/role/analytics/hadoop.pp
+++ b/manifests/role/analytics/hadoop.pp
@@ -290,11 +290,7 @@
#
class role::analytics::hadoop::client inherits role::analytics::hadoop::config
{
# need java before hadoop is installed
- if (!defined(Package['openjdk-7-jdk'])) {
- package { 'openjdk-7-jdk':
- ensure => 'installed',
- }
- }
+ ensure_packages(['openjdk-7-jdk'])
class { 'cdh::hadoop':
cluster_name => $cluster_name,
--
To view, visit https://gerrit.wikimedia.org/r/203093
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0b5d91f0647cdf2b370ae6ce4dade8fd1f1777b7
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
