Muehlenhoff has submitted this change and it was merged.
Change subject: aqs: Include base::firewall in the role
......................................................................
aqs: Include base::firewall in the role
Also include standard inside the role.
Change-Id: Ia9e7688dbacd7f530cb4841e45b8a31adba56be8
---
M manifests/role/aqs.pp
M manifests/site.pp
2 files changed, 2 insertions(+), 3 deletions(-)
Approvals:
Muehlenhoff: Verified; Looks good to me, approved
jenkins-bot: Verified
diff --git a/manifests/role/aqs.pp b/manifests/role/aqs.pp
index 861962c..b48835b 100644
--- a/manifests/role/aqs.pp
+++ b/manifests/role/aqs.pp
@@ -12,6 +12,8 @@
description => 'Analytics Query Service Node',
}
+ include standard
+ include base::firewall
#
# Set up Cassandra for AQS.
diff --git a/manifests/site.pp b/manifests/site.pp
index bd4f3b5..11775d8 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -202,9 +202,6 @@
# Analytics Query Service (RESTBase & Cassandra)
node /aqs100[123]\.eqiad\.wmnet/ {
role aqs
-
- include standard
- include base::firewall
}
--
To view, visit https://gerrit.wikimedia.org/r/246330
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia9e7688dbacd7f530cb4841e45b8a31adba56be8
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Muehlenhoff <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Milimetric <[email protected]>
Gerrit-Reviewer: Muehlenhoff <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
