Filippo Giunchedi has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/393784 )
Change subject: role: use scalar() to check statsd udp loss
......................................................................
role: use scalar() to check statsd udp loss
Bug: T181382
Change-Id: Ia37c40d0e7c5e00c34a42d5ae17826e6d6ce8c3f
---
M modules/role/manifests/statsd.pp
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
jenkins-bot: Verified
Filippo Giunchedi: Looks good to me, approved
diff --git a/modules/role/manifests/statsd.pp b/modules/role/manifests/statsd.pp
index 9bbb447..627e6d8 100644
--- a/modules/role/manifests/statsd.pp
+++ b/modules/role/manifests/statsd.pp
@@ -72,7 +72,7 @@
monitoring::check_prometheus { 'statsd_udp_inbound_errors':
description => 'statsd UDP receive errors are elevated',
dashboard_link =>
"https://grafana.wikimedia.org/dashboard/db/graphite-${::site}?orgId=1&refresh=1m&panelId=16&fullscreen";,
- query => "100 *
rate(node_netstat_Udp_InErrors${prometheus_labels}[5m]) /
rate(node_netstat_Udp_InDatagrams${prometheus_labels}[5m])",
+ query => "scalar(100 *
rate(node_netstat_Udp_InErrors${prometheus_labels}[5m]) /
rate(node_netstat_Udp_InDatagrams${prometheus_labels}[5m]))",
warning => 1,
critical => 2,
prometheus_url => "http://prometheus.svc.${::site}.wmnet/ops";,
--
To view, visit https://gerrit.wikimedia.org/r/393784
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia37c40d0e7c5e00c34a42d5ae17826e6d6ce8c3f
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Filippo Giunchedi <[email protected]>
Gerrit-Reviewer: Filippo Giunchedi <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
