From ac49d3f894622d85cdc4487e0eeb476d69440994 Mon Sep 17 00:00:00 2001
From: Quentin Manfroi <quentin.manfroi@fr.clara.net>
Date: Tue, 21 Aug 2018 09:44:51 +0200
Subject: [PATCH] MON-290 update php busy worker monitor

---
 middleware/php-fpm/inputs.tf       |  4 ++--
 middleware/php-fpm/monitors-fpm.tf | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/middleware/php-fpm/inputs.tf b/middleware/php-fpm/inputs.tf
index 0650e83..9fbeb84 100644
--- a/middleware/php-fpm/inputs.tf
+++ b/middleware/php-fpm/inputs.tf
@@ -69,12 +69,12 @@ variable "php_fpm_busy_timeframe" {
 
 variable "php_fpm_busy_threshold_warning" {
   description = "php fpm busy warning threshold"
-  default     = 0.8
+  default     = 80
 }
 
 variable "php_fpm_busy_threshold_critical" {
   description = "php fpm busy critical threshold"
-  default     = 0.9
+  default     = 90
 }
 
 variable "php_fpm_connect_silenced" {
diff --git a/middleware/php-fpm/monitors-fpm.tf b/middleware/php-fpm/monitors-fpm.tf
index 0b1e0a1..7ac50c4 100644
--- a/middleware/php-fpm/monitors-fpm.tf
+++ b/middleware/php-fpm/monitors-fpm.tf
@@ -7,10 +7,10 @@ resource "datadog_monitor" "datadog_php_fpm_connect_idle" {
 
   query = <<EOF
     ${var.php_fpm_busy_time_aggregator}(${var.php_fpm_busy_timeframe}): (
-      avg:php_fpm.processes.active${module.filter-tags.query_alert} by {region, host} /
-      ( avg:php_fpm.processes.idle${module.filter-tags.query_alert} by {region, host} +
-       avg:php_fpm.processes.active${module.filter-tags.query_alert} by {region, host} )
-    ) > ${var.php_fpm_busy_threshold_critical}
+      avg:php_fpm.processes.active${module.filter-tags.query_alert} by {region, host, pool} /
+      ( avg:php_fpm.processes.idle${module.filter-tags.query_alert} by {region, host, pool} +
+       avg:php_fpm.processes.active${module.filter-tags.query_alert} by {region, host, pool} )
+    ) * 100 > ${var.php_fpm_busy_threshold_critical}
   EOF
 
   thresholds {
@@ -18,7 +18,7 @@ resource "datadog_monitor" "datadog_php_fpm_connect_idle" {
     critical = "${var.php_fpm_busy_threshold_critical}"
   }
 
-  notify_no_data      = true
+  notify_no_data      = false
   evaluation_delay    = "${var.evaluation_delay}"
   new_host_delay      = "${var.new_host_delay}"
   notify_audit        = false