Giuseppe Lavagetto has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/402841 )

Change subject: site.pp: one role called with role() for stat1005
......................................................................

site.pp: one role called with role() for stat1005

I didn't bother creating a compound role for a machine that looks
definitely a one-off unicorn.

Change-Id: I01ddda470bf38391d687d57ee759fb4fbaf00692
---
M manifests/site.pp
1 file changed, 23 insertions(+), 23 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/41/402841/1

diff --git a/manifests/site.pp b/manifests/site.pp
index 3fafc42..81992cf 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -2031,35 +2031,35 @@
 
 # WIP: stat1002 replacement (T152712)
 node 'stat1005.eqiad.wmnet' {
-    role(statistics::private,
-        # This is a Hadoop client, and should
-        # have any special analytics system users on it
-        # for interacting with HDFS.
-        analytics_cluster::users,
+    role(statistics::private)
+    # This is a Hadoop client, and should
+    # have any special analytics system users on it
+    # for interacting with HDFS.
+    include ::role::analytics_cluster::users
 
-        # Include Hadoop and other analytics cluster
-        # clients so that analysts can access Hadoop
-        # from here.
-        analytics_cluster::client,
+    # Include Hadoop and other analytics cluster
+    # clients so that analysts can access Hadoop
+    # from here.
+    include ::role::analytics_cluster::client
 
-        # Include analytics/refinery deployment target.
-        analytics_cluster::refinery,
+    # Include analytics/refinery deployment target.
+    include ::role::analytics_cluster::refinery
 
-        # Include analytics/refinery checks that send email about
-        # webrequest partitions faultyness.
-        analytics_cluster::refinery::job::data_check,
+    # Include analytics/refinery checks that send email about
+    # webrequest partitions faultyness.
+    include ::role::analytics_cluster::refinery::job::data_check
 
-        # Include analytics/refinery/source guard checks
-        # Disabled due to T166937
-        # analytics_cluster::refinery::job::guard,
+    # Include analytics/refinery/source guard checks
+    # Disabled due to T166937
+    # analytics_cluster::refinery::job::guard,
 
-        # Set up a read only rsync module to allow access
-        # to public data generated by the Analytics Cluster.
-        analytics_cluster::rsyncd,
+    # Set up a read only rsync module to allow access
+    # to public data generated by the Analytics Cluster.
+    include ::role::analytics_cluster::rsyncd
 
-        # Deploy wikimedia/discovery/analytics repository
-        # to this node.
-        elasticsearch::analytics)
+    # Deploy wikimedia/discovery/analytics repository
+    # to this node.
+    include ::role::elasticsearch::analytics
 }
 
 # stat1006 is a general purpose number cruncher for

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I01ddda470bf38391d687d57ee759fb4fbaf00692
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Giuseppe Lavagetto <glavage...@wikimedia.org>

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

Reply via email to