Fix role detection

This PR adds some more test cases for a multi-node 5.5.0 cluster and fixes the role detection. Fixes prometheus-community#109
dominikschulz · Nov 27, 2017 · 0abc5b2 · 0abc5b2
1 parent bdf7fff
commit 0abc5b2
Show file tree

Hide file tree

Showing 2 changed files with 68 additions and 7 deletions.
diff --git a/collector/nodes.go b/collector/nodes.go
@@ -21,8 +21,8 @@ var (
 	defaultNodeLabelValues = func(cluster string, node NodeStatsNodeResponse) []string {
 		// default settings (2.x) and map, which roles to consider
 		roles := map[string]bool{
-			"master": true,
-			"data":   true,
+			"master": false,
+			"data":   false,
 			"ingest": false,
 		}
 		isClientNode := "true"

diff --git a/collector/nodes_test.go b/collector/nodes_test.go