Faidon Liambotis has submitted this change and it was merged.
Change subject: graphite: fix carbon-relay queue check
......................................................................
graphite: fix carbon-relay queue check
< godog> paravoid: afaict because fullQueueDrops isn't pushed as 0
by carbon-cache when nothing happens, thus no datapoints, not sure
there's a good solution using check_graphite
Use the transformNull() function to transform nulls into 0s.
Change-Id: Icb24dd173a614db0e0282098b8accd483755b5a2
---
M modules/graphite/manifests/monitoring/graphite.pp
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Faidon Liambotis: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/graphite/manifests/monitoring/graphite.pp
b/modules/graphite/manifests/monitoring/graphite.pp
index 0d4da01..9775ef8 100644
--- a/modules/graphite/manifests/monitoring/graphite.pp
+++ b/modules/graphite/manifests/monitoring/graphite.pp
@@ -10,7 +10,7 @@
# is carbon-relay queue full? (i.e. dropping data)
monitoring::graphite_threshold { 'carbon-relay_queue_full':
description => 'carbon-relay queue full',
- metric =>
'sumSeries(carbon.relays.graphite1001-*.destinations.*.fullQueueDrops)',
+ metric =>
'sumSeries(transformNull(carbon.relays.graphite1001-*.destinations.*.fullQueueDrops))',
from => '10minutes',
warning => 200,
critical => 1000,
--
To view, visit https://gerrit.wikimedia.org/r/202677
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icb24dd173a614db0e0282098b8accd483755b5a2
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Faidon Liambotis <[email protected]>
Gerrit-Reviewer: Faidon Liambotis <[email protected]>
Gerrit-Reviewer: Filippo Giunchedi <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
