[issue-680] make statsd exporter mappings optional

voxpupuli · May 5, 2024 · b10b872 · b10b872
1 parent 5dc551e
commit b10b872
Showing 1 changed file with 10 additions and 8 deletions.
diff --git a/manifests/statsd_exporter.pp b/manifests/statsd_exporter.pp
@@ -67,9 +67,9 @@
   String[1] $package_ensure,
   String[1] $package_name,
   String[1] $service_name,
-  Array[Hash] $mappings,
   String[1] $user,
   String[1] $version,
+  Optional[Array[Hash]] $mappings                            = undef,
   String[1] $arch                                            = $prometheus::real_arch,
   Stdlib::Absolutepath $bin_dir                              = $prometheus::bin_dir,
   String[1] $config_mode                                     = $prometheus::config_mode,
@@ -105,13 +105,15 @@
     default => undef,
   }
 
-  file { $mapping_config_path:
-    ensure  => 'file',
-    mode    => $config_mode,
-    owner   => 'root',
-    group   => $group,
-    content => stdlib::to_yaml({ mappings => $mappings }),
-    notify  => $notify_service,
+  if $mappings {
+    file { $mapping_config_path:
+      ensure  => 'file',
+      mode    => $config_mode,
+      owner   => 'root',
+      group   => $group,
+      content => stdlib::to_yaml({ mappings => $mappings }),
+      notify  => $notify_service,
+    }
   }
 
   # Switched to POSIX like flags in version 0.7.0