Muehlenhoff has submitted this change and it was merged.
Change subject: Rename Hiera variable for aqs hosts
......................................................................
Rename Hiera variable for aqs hosts
As noted in the gerrit review for change 289830 by Andrew Otto.
Change-Id: Iea53df4f4c9ae2f7c08297d66b2d104a53d1bed9
---
M hieradata/role/common/aqs.yaml
M manifests/role/aqs.pp
2 files changed, 2 insertions(+), 2 deletions(-)
Approvals:
Ottomata: Looks good to me, but someone else must approve
Muehlenhoff: Verified; Looks good to me, approved
diff --git a/hieradata/role/common/aqs.yaml b/hieradata/role/common/aqs.yaml
index e9e84ef..17b26c6 100644
--- a/hieradata/role/common/aqs.yaml
+++ b/hieradata/role/common/aqs.yaml
@@ -47,7 +47,7 @@
lvs::realserver::realserver_ips:
- '10.2.2.12' # aqs.svc.eqiad.wmnet
-aqs::hosts:
+aqs_hosts:
- aqs1001.eqiad.wmnet
- aqs1002.eqiad.wmnet
- aqs1003.eqiad.wmnet
diff --git a/manifests/role/aqs.pp b/manifests/role/aqs.pp
index b0e2d82..ce12d1f 100644
--- a/manifests/role/aqs.pp
+++ b/manifests/role/aqs.pp
@@ -56,7 +56,7 @@
# In addition to the IP assigned to the Cassandra multi instances, these
rules
# grant access from the actual AQS hosts
- $aqs_hosts_ferm = join(hiera('aqs::hosts'), ' ')
+ $aqs_hosts_ferm = join(hiera('aqs_hosts'), ' ')
# Cassandra CQL query interface
ferm::service { 'cassandra-analytics-cql':
--
To view, visit https://gerrit.wikimedia.org/r/290246
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iea53df4f4c9ae2f7c08297d66b2d104a53d1bed9
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Muehlenhoff <[email protected]>
Gerrit-Reviewer: Muehlenhoff <[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
