Don't install alert.rules file if there's no content for it

voxpupuli · Dec 5, 2017 · 695cc33 · 695cc33
1 parent 87124c1
commit 695cc33
Showing 1 changed file with 10 additions and 12 deletions.
diff --git a/manifests/alerts.pp b/manifests/alerts.pp
@@ -14,9 +14,8 @@
   String $alertfile_name  = 'alert.rules'
 ) inherits prometheus::params {
 
-  if ( versioncmp($::prometheus::version, '2.0.0') < 0 ){
-
-    if $alerts != [] {
+  if $alerts != [] and $alerts != {} {
+    if ( versioncmp($::prometheus::version, '2.0.0') < 0 ){
 
       file { "${location}/${alertfile_name}":
         ensure  => 'file',
@@ -27,17 +26,16 @@
       }
 
     }
+    else {
 
-  }
-  else {
+      file { "${location}/${alertfile_name}":
+        ensure  => 'file',
+        owner   => $prometheus::user,
+        group   => $prometheus::group,
+        notify  => Class['::prometheus::service_reload'],
+        content => $alerts.to_yaml,
+      }
 
-    file { "${location}/${alertfile_name}":
-      ensure  => 'file',
-      owner   => $prometheus::user,
-      group   => $prometheus::group,
-      notify  => Class['::prometheus::service_reload'],
-      content => $alerts.to_yaml,
     }
-
   }
 }