Merge pull request voxpupuli#49 from sathieu/default_collectors

Use default collectors if "collectors" param is empty
bramblek1 · Sep 24, 2017 · 4288526 · 4288526
2 parents f0c8368 + 7f2bd40
commit 4288526
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/manifests/node_exporter.pp b/manifests/node_exporter.pp
@@ -11,7 +11,7 @@
 #  Directory where binaries are located
 #
 #  [*collectors*]
-#  The set of node node_exporter collectors
+#  The set of node node_exporter collectors. Use default collectors if empty.
 #
 #  [*download_extension*]
 #  Extension for the release binary archive
@@ -119,8 +119,12 @@
     default => undef,
   }
 
-  $str_collectors = join($collectors, ',')
-  $options = "-collectors.enabled=${str_collectors} ${extra_options}"
+  if empty($collectors) {
+    $options = $extra_options
+  } else {
+    $str_collectors = join($collectors, ',')
+    $options = "-collectors.enabled=${str_collectors} ${extra_options}"
+  }
 
   prometheus::daemon { $service_name :
     install_method     => $install_method,