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