Rebased code to use case statement

DRY up a lot of the Puppet code
petems · Nov 12, 2015 · 08112a6 · 08112a6
1 parent 3ff51a3
commit 08112a6
Showing 1 changed file with 9 additions and 10 deletions.
diff --git a/manifests/files.pp b/manifests/files.pp
@@ -53,17 +53,16 @@
   validate_bool($add_mount)
 
   if $ensure == 'present' {
-    if $cmd == 'dd' {
-      exec { "Create swap file ${swapfile}":
-        command => "/bin/dd if=/dev/zero of=${swapfile} bs=1M count=${swapfilesize_mb}",
-        creates => $swapfile,
-        timeout => $timeout,
+    exec { "Create swap file ${swapfile}":
+      creates => $swapfile,
+      timeout => $timeout,
+    }
+    case $cmd {
+      'dd': {
+        Exec["Create swap file ${swapfile}"] { command => "/bin/dd if=/dev/zero of=${swapfile} bs=1M count=${swapfilesize_mb}" }
       }
-    } else {
-      exec { "Create swap file ${swapfile}":
-        command => "/usr/bin/fallocate -l ${swapfilesize_mb}M ${swapfile}",
-        creates => $swapfile,
-        timeout => $timeout,
+      'fallocate': {
+        Exec["Create swap file ${swapfile}"] { command => "/usr/bin/fallocate -l ${swapfilesize_mb}M ${swapfile}" }
       }
     }
     file { $swapfile: