Dzahn has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/394731 )

Change subject: site/cp: convert "not true"-spare systems to role(test)
......................................................................


site/cp: convert "not true"-spare systems to role(test)

There is role(test) for testing/experiments, so no need
to use role(spare::system) and then comment they are not
real spares :)

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

Approvals:
  jenkins-bot: Verified
  Dzahn: Looks good to me, approved



diff --git a/manifests/site.pp b/manifests/site.pp
index 3e53e0d..c1de7ba 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -203,8 +203,8 @@
 }
 
 node 'cp1046.eqiad.wmnet', 'cp1047.eqiad.wmnet', 'cp1059.eqiad.wmnet', 
'cp1060.eqiad.wmnet' {
-    # ex-cache_maps, not true spares, earmarked for experimentation...
-    role(spare::system)
+    # ex-cache_maps, earmarked for experimentation...
+    role(test)
 }
 
 node /^cp10(4[89]|50|6[234]|7[1-4]|99)\.eqiad\.wmnet$/ {
@@ -228,8 +228,8 @@
 }
 
 node /^cp20(0[39]|15|21)\.codfw\.wmnet$/ {
-    # ex-cache_maps, not true spares, earmarked for experimentation...
-    role(spare::system)
+    # ex-cache_maps, earmarked for experimentation...
+    role(test)
 }
 
 node /^cp20(06|1[28]|25)\.codfw\.wmnet$/ {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib26cc597d04bd84b434f29b569230be8bccfdcf7
Gerrit-PatchSet: 7
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: BBlack <bbl...@wikimedia.org>
Gerrit-Reviewer: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to