Muehlenhoff has submitted this change and it was merged.
Change subject: statistics::cruncher: Move standard and base::firewall includes
into the role
......................................................................
statistics::cruncher: Move standard and base::firewall includes into the role
Change-Id: I43a1e324f405aa1a7106161264f1206c48645024
---
M manifests/role/statistics.pp
M manifests/site.pp
2 files changed, 2 insertions(+), 2 deletions(-)
Approvals:
Muehlenhoff: Verified; Looks good to me, approved
diff --git a/manifests/role/statistics.pp b/manifests/role/statistics.pp
index c1a437a..ae939e7 100644
--- a/manifests/role/statistics.pp
+++ b/manifests/role/statistics.pp
@@ -41,6 +41,8 @@
description => 'Statistics general compute node (non private data)'
}
+ include standard
+ include base::firewall
include role::backup::host
backup::set { 'home' : }
diff --git a/manifests/site.pp b/manifests/site.pp
index 8059bbc..31ca0bb4 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -2382,8 +2382,6 @@
# query results for further processing on this node.
node 'stat1003.eqiad.wmnet' {
role statistics::cruncher
- include standard
- include base::firewall
include passwords::mysql::research
# This file will render at
--
To view, visit https://gerrit.wikimedia.org/r/247223
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I43a1e324f405aa1a7106161264f1206c48645024
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Muehlenhoff <mmuhlenh...@wikimedia.org>
Gerrit-Reviewer: Muehlenhoff <mmuhlenh...@wikimedia.org>
Gerrit-Reviewer: Ottomata <o...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
