From d55fa37bed8eb0048abb3e054a60ffdf36b5c546 Mon Sep 17 00:00:00 2001
From: Rafael Romero Carmona <rafael.romero.carmona@fr.clara.net>
Date: Mon, 27 Aug 2018 17:36:18 +0200
Subject: [PATCH] MON-228 Monitors fixed

---
 databases/elasticsearch/monitors-elasticsearch.tf | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/databases/elasticsearch/monitors-elasticsearch.tf b/databases/elasticsearch/monitors-elasticsearch.tf
index aa7af24..6c94622 100644
--- a/databases/elasticsearch/monitors-elasticsearch.tf
+++ b/databases/elasticsearch/monitors-elasticsearch.tf
@@ -182,7 +182,10 @@ resource "datadog_monitor" "node_free_space" {
 
   query = <<EOF
   ${var.node_free_space_time_aggregator}(${var.node_free_space_timeframe}):
-    min:elasticsearch.fs.total.available_in_bytes{${data.template_file.filter.rendered}} by {node_name} / min:elasticsearch.fs.total.total_in_bytes{${data.template_file.filter.rendered}} by {node_name} * 100
+    (min:elasticsearch.fs.total.available_in_bytes{${data.template_file.filter.rendered}} by {node_name}
+    /
+    min:elasticsearch.fs.total.total_in_bytes{${data.template_file.filter.rendered}} by {node_name}
+    ) * 100
   < ${var.node_free_space_threshold_critical}
 EOF
 
@@ -812,7 +815,7 @@ resource "datadog_monitor" "request_cache_evictions_change" {
   // TODO add tags to filter by node type and do not apply this monitor on non-data nodes
   query = <<EOF
   change(${var.request_cache_evictions_change_time_aggregator}(${var.request_cache_evictions_change_timeframe}),${var.request_cache_evictions_change_timeshift}):
-    avg:elasticsearch.indices.request_cache.evictions{${data.template_file.filter.rendered}} by {node_cache}
+    avg:elasticsearch.indices.request_cache.evictions{${data.template_file.filter.rendered}} by {node_name}
   > ${var.request_cache_evictions_change_threshold_critical}
 EOF