[MediaWiki-commits] [Gerrit] Enable 'has_spec' on Kartotherian service. - change (operations/puppet)

2016-06-14 Thread Gehel (Code Review) 
Gehel has submitted this change and it was merged.

Change subject: Enable 'has_spec' on Kartotherian service.
..


Enable 'has_spec' on Kartotherian service.

Spec has been written and deployed for Kartotherian. We now need to use it.
To not spam everyone while testing, this is disabled on all servers except
maps2001. This will need to be cleaned once we confirm that all works
correctly.

Bug: T137617
Change-Id: Ibbd69c672b477d7d71326f12fb181c2201d39ab9
---
M hieradata/hosts/maps2001.yaml
M hieradata/role/common/maps/server.yaml
M modules/kartotherian/manifests/init.pp
3 files changed, 7 insertions(+), 0 deletions(-)

Approvals:
  Gehel: Looks good to me, approved
  MaxSem: Looks good to me, but someone else must approve
  jenkins-bot: Verified



diff --git a/hieradata/hosts/maps2001.yaml b/hieradata/hosts/maps2001.yaml
index db7b54b..adafca6 100644
--- a/hieradata/hosts/maps2001.yaml
+++ b/hieradata/hosts/maps2001.yaml
@@ -15,3 +15,5 @@
 tilerator::conf_sources: sources.prod2.yaml
 tilerator::ui::conf_sources: sources.prod2.yaml
 kartotherian::conf_sources: sources.prod2.yaml
+# enable specs only on maps2001 for testing
+kartotherian::enable_spec: true
diff --git a/hieradata/role/common/maps/server.yaml 
b/hieradata/role/common/maps/server.yaml
index 0249efa..47dab51 100644
--- a/hieradata/role/common/maps/server.yaml
+++ b/hieradata/role/common/maps/server.yaml
@@ -12,3 +12,6 @@
   - tilerator-admin
 service::configuration::logstash_host: logstash1001.eqiad.wmnet
 service::configuration::statsd_host: statsd.eqiad.wmnet
+
+# Let's keep spec disabled until it is correctly tested
+kartotherian::enable_spec: false
diff --git a/modules/kartotherian/manifests/init.pp 
b/modules/kartotherian/manifests/init.pp
index c5886c6..b998a04 100644
--- a/modules/kartotherian/manifests/init.pp
+++ b/modules/kartotherian/manifests/init.pp
@@ -8,6 +8,7 @@
 # service
 class kartotherian(
 $conf_sources = 'sources.prod.yaml',
+$enable_spec  = true,
 ) {
 
 $cassandra_kartotherian_user = 'kartotherian'
@@ -19,5 +20,6 @@
 port   => 6533,
 config => template('kartotherian/config.yaml.erb'),
 deployment => 'scap3',
+has_spec   => $enable_spec,
 }
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibbd69c672b477d7d71326f12fb181c2201d39ab9
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Gehel 
Gerrit-Reviewer: Alexandros Kosiaris 
Gerrit-Reviewer: Gehel 
Gerrit-Reviewer: MaxSem 
Gerrit-Reviewer: Yurik 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] Enable 'has_spec' on Kartotherian service. - change (operations/puppet)

2016-06-13 Thread Gehel (Code Review) 
Gehel has uploaded a new change for review.

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

Change subject: Enable 'has_spec' on Kartotherian service.
..

Enable 'has_spec' on Kartotherian service.

Spec has been written and deployed for Kartotherian. We now need to use it.
To not spam everyone while testing, this is disabled on all servers except
maps2001. This will need to be cleaned once we confirm that all works
correctly.

Bug: T137617
Change-Id: Ibbd69c672b477d7d71326f12fb181c2201d39ab9
---
M hieradata/hosts/maps2001.yaml
M hieradata/role/common/maps/server.yaml
M modules/kartotherian/manifests/init.pp
3 files changed, 7 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/28/294028/1

diff --git a/hieradata/hosts/maps2001.yaml b/hieradata/hosts/maps2001.yaml
index db7b54b..adafca6 100644
--- a/hieradata/hosts/maps2001.yaml
+++ b/hieradata/hosts/maps2001.yaml
@@ -15,3 +15,5 @@
 tilerator::conf_sources: sources.prod2.yaml
 tilerator::ui::conf_sources: sources.prod2.yaml
 kartotherian::conf_sources: sources.prod2.yaml
+# enable specs only on maps2001 for testing
+kartotherian::enable_spec: true
diff --git a/hieradata/role/common/maps/server.yaml 
b/hieradata/role/common/maps/server.yaml
index 0249efa..47dab51 100644
--- a/hieradata/role/common/maps/server.yaml
+++ b/hieradata/role/common/maps/server.yaml
@@ -12,3 +12,6 @@
   - tilerator-admin
 service::configuration::logstash_host: logstash1001.eqiad.wmnet
 service::configuration::statsd_host: statsd.eqiad.wmnet
+
+# Let's keep spec disabled until it is correctly tested
+kartotherian::enable_spec: false
diff --git a/modules/kartotherian/manifests/init.pp 
b/modules/kartotherian/manifests/init.pp
index c5886c6..b998a04 100644
--- a/modules/kartotherian/manifests/init.pp
+++ b/modules/kartotherian/manifests/init.pp
@@ -8,6 +8,7 @@
 # service
 class kartotherian(
 $conf_sources = 'sources.prod.yaml',
+$enable_spec  = true,
 ) {
 
 $cassandra_kartotherian_user = 'kartotherian'
@@ -19,5 +20,6 @@
 port   => 6533,
 config => template('kartotherian/config.yaml.erb'),
 deployment => 'scap3',
+has_spec   => $enable_spec,
 }
 }

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

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

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