[MediaWiki-commits] [Gerrit] rename misc::parsoid to role::parsoid::production - change (operations/puppet)

2013-11-19 Thread Hashar (Code Review) 
Hashar has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/96270


Change subject: rename misc::parsoid to role::parsoid::production
..

rename misc::parsoid to role::parsoid::production

I am going to need a role class to deploy parsoid on the beta cluster.
The deployment-parsoid2 instance currently uses the same class as the
production one but need some further tweaks.

bug: 57233
Change-Id: Ide6f136c66aebd77f62ec35589c311a8ada9a83f
---
R manifests/role/parsoid.pp
M manifests/site.pp
2 files changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/70/96270/1

diff --git a/manifests/misc/parsoid.pp b/manifests/role/parsoid.pp
similarity index 90%
rename from manifests/misc/parsoid.pp
rename to manifests/role/parsoid.pp
index fa247a1..892900d 100644
--- a/manifests/misc/parsoid.pp
+++ b/manifests/role/parsoid.pp
@@ -2,8 +2,8 @@
 @monitor_group { parsoid_eqiad: description = eqiad parsoid servers }
 @monitor_group { parsoid_pmtpa: description = pmtpa parsoid servers }
 
-class misc::parsoid {
-   system::role { misc::parsoid: description = Parsoid server }
+class role::parsoid::production {
+   system::role { role::parsoid::production: description = Parsoid 
server }
deployment::target { parsoid: }
 
package { [ nodejs, npm, build-essential ]:
diff --git a/manifests/site.pp b/manifests/site.pp
index 95fe209..468e455 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -2897,7 +2897,7 @@
 include standard,
 admins::roots,
 admins::parsoid,
-misc::parsoid
+role::parsoid::production
 
 class { lvs::realserver: realserver_ips = [ 10.2.2.28 ] }
 }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ide6f136c66aebd77f62ec35589c311a8ada9a83f
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar has...@free.fr

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

[MediaWiki-commits] [Gerrit] rename misc::parsoid to role::parsoid::production - change (operations/puppet)

2013-11-19 Thread Faidon Liambotis (Code Review) 
Faidon Liambotis has submitted this change and it was merged.

Change subject: rename misc::parsoid to role::parsoid::production
..


rename misc::parsoid to role::parsoid::production

I am going to need a role class to deploy parsoid on the beta cluster.
The deployment-parsoid2 instance currently uses the same class as the
production one but need some further tweaks.

bug: 57233
Change-Id: Ide6f136c66aebd77f62ec35589c311a8ada9a83f
---
R manifests/role/parsoid.pp
M manifests/site.pp
2 files changed, 3 insertions(+), 3 deletions(-)

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



diff --git a/manifests/misc/parsoid.pp b/manifests/role/parsoid.pp
similarity index 90%
rename from manifests/misc/parsoid.pp
rename to manifests/role/parsoid.pp
index fa247a1..892900d 100644
--- a/manifests/misc/parsoid.pp
+++ b/manifests/role/parsoid.pp
@@ -2,8 +2,8 @@
 @monitor_group { parsoid_eqiad: description = eqiad parsoid servers }
 @monitor_group { parsoid_pmtpa: description = pmtpa parsoid servers }
 
-class misc::parsoid {
-   system::role { misc::parsoid: description = Parsoid server }
+class role::parsoid::production {
+   system::role { role::parsoid::production: description = Parsoid 
server }
deployment::target { parsoid: }
 
package { [ nodejs, npm, build-essential ]:
diff --git a/manifests/site.pp b/manifests/site.pp
index 95fe209..468e455 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -2897,7 +2897,7 @@
 include standard,
 admins::roots,
 admins::parsoid,
-misc::parsoid
+role::parsoid::production
 
 class { lvs::realserver: realserver_ips = [ 10.2.2.28 ] }
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ide6f136c66aebd77f62ec35589c311a8ada9a83f
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar has...@free.fr
Gerrit-Reviewer: Catrope roan.katt...@gmail.com
Gerrit-Reviewer: Faidon Liambotis fai...@wikimedia.org
Gerrit-Reviewer: GWicke gwi...@wikimedia.org
Gerrit-Reviewer: Hashar has...@free.fr
Gerrit-Reviewer: jenkins-bot

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