Filippo Giunchedi has uploaded a new change for review.

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

Change subject: Revert "Add basic alerts on RESTBase error rates and storage 
latencies"
......................................................................

Revert "Add basic alerts on RESTBase error rates and storage latencies"

I prematurely merged this, it broke icinga config reload and it should live in
its own class too.

This reverts commit 5d04bd78d2263d9de33092b8e2c549968001a9c8.

Change-Id: If38a4375e7cafa9468b8c4fef757e59e4f670b24
---
M modules/restbase/manifests/init.pp
1 file changed, 0 insertions(+), 20 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/57/215557/1

diff --git a/modules/restbase/manifests/init.pp 
b/modules/restbase/manifests/init.pp
index 85f6a13..3374828 100644
--- a/modules/restbase/manifests/init.pp
+++ b/modules/restbase/manifests/init.pp
@@ -135,24 +135,4 @@
             '/etc/init.d/restbase'
         ],
     }
-
-    monitoring::graphite_threshold { 'restbase_request_5xx_rate':
-        description     => 'RESTBase requests returning 5xx, in req/s',
-        metric          => 
'restbase.v1_page_html_-title-_-revision--_tid-.GET.5xx.sample_rate',
-        from            => '10min',
-        warning         => '1', # 1 5xx/s
-        critical        => '3', # 5 5xx/s
-        percentage      => '20',
-        contact_group   => 'team-services',
-    }
-
-    monitoring::graphite_threshold { 'restbase_html_storage_hit_latency':
-        description     => 'RESTBase HTML storage load mean latency',
-        metric          => 
'movingMedian(restbase.sys_key-rev-value_-bucket-_-key--_revision--_tid-.GET.2xx.mean,
 15)',
-        from            => '10min',
-        warning         => '25', # 25ms
-        critical        => '50', # 50ms
-        percentage      => '50',
-        contact_group   => 'team-services',
-    }
 }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If38a4375e7cafa9468b8c4fef757e59e4f670b24
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Filippo Giunchedi <fgiunch...@wikimedia.org>

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

Reply via email to