[MediaWiki-commits] [Gerrit] operations/puppet[production]: public_dumps: Remove module path and rename to distribution

2017-12-31 Thread Madhuvishy (Code Review)
Madhuvishy has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/401198 )

Change subject: public_dumps: Remove module path and rename to distribution
..


public_dumps: Remove module path and rename to distribution

Removing the module path for public_dumps since it should
really be in the profile path.

Also changing public to distribution, since public is a vague
name, and we could end up with dumps generation and dumps
distribution as 2 distinct roles.

Bug: T171539
Change-Id: If03cf64f3f577f76914d0e0ab2d72a932340ba8f
---
M manifests/site.pp
R modules/profile/manifests/dumps/distribution/server.pp
A modules/role/manifests/dumps/distribution/server.pp
D modules/role/manifests/dumps/public/server.pp
4 files changed, 21 insertions(+), 17 deletions(-)

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



diff --git a/manifests/site.pp b/manifests/site.pp
index df1b8a0..d90fb8a 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -1248,7 +1248,7 @@
 }
 
 node /labstore100[67]\.wikimedia\.org/ {
-role(dumps::public::server)
+role(dumps::distribution::server)
 }
 
 node /labstore200[1-2]\.codfw\.wmnet/ {
diff --git a/modules/public_dumps/manifests/server.pp 
b/modules/profile/manifests/dumps/distribution/server.pp
similarity index 66%
rename from modules/public_dumps/manifests/server.pp
rename to modules/profile/manifests/dumps/distribution/server.pp
index 94e1429..a7d1edc 100644
--- a/modules/public_dumps/manifests/server.pp
+++ b/modules/profile/manifests/dumps/distribution/server.pp
@@ -1,5 +1,9 @@
-# Base class to set up a Public Dumps server
-class public_dumps::server {
+# Profile for Dumps distribution server in the Public VLAN,
+# that serves dumps to Cloud VPS/Stat boxes via NFS,
+# or via web or rsync to mirrors
+
+class profile::dumps::distribution::server {
+class { '::dumpsuser': }
 
 file { '/srv/dumps':
 ensure => 'directory',
diff --git a/modules/role/manifests/dumps/distribution/server.pp 
b/modules/role/manifests/dumps/distribution/server.pp
new file mode 100644
index 000..250130c
--- /dev/null
+++ b/modules/role/manifests/dumps/distribution/server.pp
@@ -0,0 +1,14 @@
+class role::dumps::distribution::server {
+system::role { 'dumps::distribution::server': description => 'labstore 
host in the public VLAN that distributes Dumps to clients via NFS/Web/Rsync' }
+
+include ::standard
+include ::profile::base::firewall
+include ::profile::wmcs::nfs::ferm
+include ::profile::dumps::distribution::server
+include ::profile::dumps::web::rsync_server
+include ::profile::dumps::rsyncer
+include ::profile::dumps::web::dumpstatusfiles_sync
+include ::profile::dumps::web::cleanup
+include ::profile::dumps::web::cleanup_miscdatasets
+
+}
diff --git a/modules/role/manifests/dumps/public/server.pp 
b/modules/role/manifests/dumps/public/server.pp
deleted file mode 100644
index d20ae51..000
--- a/modules/role/manifests/dumps/public/server.pp
+++ /dev/null
@@ -1,14 +0,0 @@
-class role::dumps::public::server {
-system::role { 'dumps::public::server': description => 'labstore host in 
the public VLAN that serves Dumps to clients via NFS/Web/Rsync' }
-
-include ::standard
-include ::profile::base::firewall
-include ::profile::wmcs::nfs::ferm
-include ::profile::dumps::public_server
-include ::profile::dumps::web::rsync_server
-include ::profile::dumps::rsyncer
-include ::profile::dumps::web::dumpstatusfiles_sync
-include ::profile::dumps::web::cleanup
-include ::profile::dumps::web::cleanup_miscdatasets
-
-}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: If03cf64f3f577f76914d0e0ab2d72a932340ba8f
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Madhuvishy 
Gerrit-Reviewer: Giuseppe Lavagetto 
Gerrit-Reviewer: Madhuvishy 
Gerrit-Reviewer: jenkins-bot <>

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


[MediaWiki-commits] [Gerrit] operations/puppet[production]: public_dumps: Remove module path and rename to distribution

2017-12-31 Thread Madhuvishy (Code Review)
Madhuvishy has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/401198 )

Change subject: public_dumps: Remove module path and rename to distribution
..

public_dumps: Remove module path and rename to distribution

Removing the module path for public_dumps since it should
really be in the profile path.

Also changing public to distribution, since public is a vague
name, and we could end up with dumps generation and dumps
distribution as 2 distinct roles.

Bug: T171539
Change-Id: If03cf64f3f577f76914d0e0ab2d72a932340ba8f
---
M manifests/site.pp
R modules/profile/manifests/dumps/distribution/server.pp
A modules/role/manifests/dumps/distribution/server.pp
D modules/role/manifests/dumps/public/server.pp
4 files changed, 21 insertions(+), 17 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/98/401198/1

diff --git a/manifests/site.pp b/manifests/site.pp
index df1b8a0..d90fb8a 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -1248,7 +1248,7 @@
 }
 
 node /labstore100[67]\.wikimedia\.org/ {
-role(dumps::public::server)
+role(dumps::distribution::server)
 }
 
 node /labstore200[1-2]\.codfw\.wmnet/ {
diff --git a/modules/public_dumps/manifests/server.pp 
b/modules/profile/manifests/dumps/distribution/server.pp
similarity index 66%
rename from modules/public_dumps/manifests/server.pp
rename to modules/profile/manifests/dumps/distribution/server.pp
index 94e1429..a7d1edc 100644
--- a/modules/public_dumps/manifests/server.pp
+++ b/modules/profile/manifests/dumps/distribution/server.pp
@@ -1,5 +1,9 @@
-# Base class to set up a Public Dumps server
-class public_dumps::server {
+# Profile for Dumps distribution server in the Public VLAN,
+# that serves dumps to Cloud VPS/Stat boxes via NFS,
+# or via web or rsync to mirrors
+
+class profile::dumps::distribution::server {
+class { '::dumpsuser': }
 
 file { '/srv/dumps':
 ensure => 'directory',
diff --git a/modules/role/manifests/dumps/distribution/server.pp 
b/modules/role/manifests/dumps/distribution/server.pp
new file mode 100644
index 000..250130c
--- /dev/null
+++ b/modules/role/manifests/dumps/distribution/server.pp
@@ -0,0 +1,14 @@
+class role::dumps::distribution::server {
+system::role { 'dumps::distribution::server': description => 'labstore 
host in the public VLAN that distributes Dumps to clients via NFS/Web/Rsync' }
+
+include ::standard
+include ::profile::base::firewall
+include ::profile::wmcs::nfs::ferm
+include ::profile::dumps::distribution::server
+include ::profile::dumps::web::rsync_server
+include ::profile::dumps::rsyncer
+include ::profile::dumps::web::dumpstatusfiles_sync
+include ::profile::dumps::web::cleanup
+include ::profile::dumps::web::cleanup_miscdatasets
+
+}
diff --git a/modules/role/manifests/dumps/public/server.pp 
b/modules/role/manifests/dumps/public/server.pp
deleted file mode 100644
index d20ae51..000
--- a/modules/role/manifests/dumps/public/server.pp
+++ /dev/null
@@ -1,14 +0,0 @@
-class role::dumps::public::server {
-system::role { 'dumps::public::server': description => 'labstore host in 
the public VLAN that serves Dumps to clients via NFS/Web/Rsync' }
-
-include ::standard
-include ::profile::base::firewall
-include ::profile::wmcs::nfs::ferm
-include ::profile::dumps::public_server
-include ::profile::dumps::web::rsync_server
-include ::profile::dumps::rsyncer
-include ::profile::dumps::web::dumpstatusfiles_sync
-include ::profile::dumps::web::cleanup
-include ::profile::dumps::web::cleanup_miscdatasets
-
-}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If03cf64f3f577f76914d0e0ab2d72a932340ba8f
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Madhuvishy 

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