Dzahn has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/393709 )
Change subject: rename phabricator_server to just phabricator
......................................................................
rename phabricator_server to just phabricator
Similar to gerrit we can also rename "phabricator_server" to just
"phabricator". There were just historic reasons for that.
Change-Id: I1293ef9eb9a779d80fbfb43df127677112d89d06
---
R hieradata/role/common/phabricator.yaml
M manifests/site.pp
M modules/profile/templates/cumin/aliases.yaml.erb
R modules/role/manifests/phabricator.pp
4 files changed, 4 insertions(+), 4 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/09/393709/1
diff --git a/hieradata/role/common/phabricator_server.yaml
b/hieradata/role/common/phabricator.yaml
similarity index 100%
rename from hieradata/role/common/phabricator_server.yaml
rename to hieradata/role/common/phabricator.yaml
diff --git a/manifests/site.pp b/manifests/site.pp
index 2b374d0..8d180a9 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -1142,7 +1142,7 @@
# Phabricator
node /^(phab1001\.eqiad|phab2001\.codfw)\.wmnet$/ {
- role(phabricator_server)
+ role(phabricator)
include ::ganglia
interface::add_ip6_mapped { 'main': }
}
diff --git a/modules/profile/templates/cumin/aliases.yaml.erb
b/modules/profile/templates/cumin/aliases.yaml.erb
index edcef5a..664d3f0 100644
--- a/modules/profile/templates/cumin/aliases.yaml.erb
+++ b/modules/profile/templates/cumin/aliases.yaml.erb
@@ -196,5 +196,5 @@
misc-nonprod: P{O:spare::system or O:test or O:pybaltest or
O:restbase::test_cluster or O:authdns::testns or O:cache::canary}
misc-ops: A:installserver or A:cumin or A:url-downloader or A:icinga or
P{O:logging::kafkatee::webrequest::ops or O:builder or O:syslog::centralserver
or O:pmacct or O:backup::offsite or O:backup} or A:netmon
misc-others: P{O:mw_rc_irc or O:etherpad or O:tor_relay or O:lists}
-misc-releng: P{O:labs::openstack::nodepool or O:phabricator_server} or
A:contint or A:gerrit
+misc-releng: P{O:labs::openstack::nodepool or O:phabricator} or A:contint or
A:gerrit
misc-wmcs: A:labnet or A:labcontrol or A:labservices or A:labweb
diff --git a/modules/role/manifests/phabricator_server.pp
b/modules/role/manifests/phabricator.pp
similarity index 85%
rename from modules/role/manifests/phabricator_server.pp
rename to modules/role/manifests/phabricator.pp
index cbb85df..2db93a0 100644
--- a/modules/role/manifests/phabricator_server.pp
+++ b/modules/role/manifests/phabricator.pp
@@ -1,9 +1,9 @@
# phabricator instance
#
# filtertags: labs-project-deployment-prep labs-project-phabricator
-class role::phabricator_server {
+class role::phabricator {
- system::role { 'phabricator_server':
+ system::role { 'phabricator':
description => 'Phabricator (Main) Server'
}
--
To view, visit https://gerrit.wikimedia.org/r/393709
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I1293ef9eb9a779d80fbfb43df127677112d89d06
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <dz...@wikimedia.org>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
