Alexandros Kosiaris has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/353118 )
Change subject: parsoid: move 'include standard' to role ...................................................................... parsoid: move 'include standard' to role Change-Id: Ifd355e418d3343f59758a9badab253eaeeaa9557 --- M manifests/site.pp M modules/role/manifests/parsoid.pp 2 files changed, 1 insertion(+), 2 deletions(-) Approvals: Alexandros Kosiaris: Verified; Looks good to me, approved diff --git a/manifests/site.pp b/manifests/site.pp index 252d4cb..089f1f0 100644 --- a/manifests/site.pp +++ b/manifests/site.pp @@ -2712,12 +2712,10 @@ # https://www.mediawiki.org/wiki/Parsoid node /^wtp10(0[1-9]|1[0-9]|2[0-4])\.eqiad\.wmnet$/ { role(parsoid) - include ::standard } node /^wtp20(0[1-9]|1[0-9]|2[0-4])\.codfw\.wmnet$/ { role(parsoid) - include ::standard } # T138650 - tools for the security team diff --git a/modules/role/manifests/parsoid.pp b/modules/role/manifests/parsoid.pp index 18591b2..63af98b 100644 --- a/modules/role/manifests/parsoid.pp +++ b/modules/role/manifests/parsoid.pp @@ -7,6 +7,7 @@ description => "Parsoid ${::realm}" } + include ::standard include ::base::firewall if hiera('has_lvs', true) { -- To view, visit https://gerrit.wikimedia.org/r/353118 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifd355e418d3343f59758a9badab253eaeeaa9557 Gerrit-PatchSet: 2 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Dzahn <[email protected]> Gerrit-Reviewer: Alexandros Kosiaris <[email protected]> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
