Merge ~ballot/content-cache-charm/+git/content-cache-charm:extend_nrpe_timeout into content-cache-charm:master

[Benjamin Allot]

Increase the timeout of the telegraf metrics check

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

[Junien F (axino)]

+1 - I'd rather use -s but it doesn't really matter

review: Approve

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

Change successfully merged at revision f563a832eb8f7a2f9a70009850e7d06ce9bcf23f

diff --git a/reactive/content_cache.py b/reactive/content_cache.py
index af0376f..758a171 100644
--- a/reactive/content_cache.py
+++ b/reactive/content_cache.py
@@ -496,7 +496,9 @@ def advertise_stats_endpoint():
 @reactive.when_not('nagios-nrpe-telegraf.configured')
 def check_haproxy_alerts():
     nrpe_setup = nrpe.NRPE(hostname=nrpe.get_nagios_hostname(), primary=True)
-    cmd = '/usr/lib/nagios/plugins/check_http -I 127.0.0.1 -p 9103 -u /metrics -r "haproxy_rate"'
+    # Because check_http is really inefficient, the parsing of the metrics is quite slow
+    # hence increasing the timeout to 20 seconds
+    cmd = '/usr/lib/nagios/plugins/check_http -I 127.0.0.1 -p 9103 -u /metrics -r "haproxy_rate" -t 20'
     nrpe_setup.add_check(
         shortname='haproxy_telegraf_metrics',
         description='Verify haproxy metrics are visible via telegraf subordinate',