Giuseppe Lavagetto has submitted this change and it was merged. ( 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(-) Approvals: Giuseppe Lavagetto: Verified; Looks good to me, approved Ottomata: Looks good to me, but someone else must approve 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: merged Gerrit-Change-Id: I01ddda470bf38391d687d57ee759fb4fbaf00692 Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Giuseppe Lavagetto <glavage...@wikimedia.org> Gerrit-Reviewer: Giuseppe Lavagetto <glavage...@wikimedia.org> Gerrit-Reviewer: Ottomata <ao...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits