From a0cae2108cbe9d88defce97fec799d1f592b2cc7 Mon Sep 17 00:00:00 2001
From: Stephen Gran <steve@lobefin.net>
Date: Sun, 18 Mar 2012 15:40:01 +0000
Subject: [PATCH] more cleanup

Signed-off-by: Stephen Gran <steve@lobefin.net>
---
 manifests/site.pp                  | 14 +++++++-------
 modules/acpi/manifests/init.pp     |  2 +-
 modules/ferm/manifests/per-host.pp |  2 +-
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/manifests/site.pp b/manifests/site.pp
index b4f44c26..3bf505ce 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -35,7 +35,7 @@ node default {
         finzi,fano,fasch,field:    { include kfreebsd }
     }
 
-    if $::smartarraycontroller {
+    if $::smartarraycontroller == 'true' {
         include debian-proliant
     }
 
@@ -43,11 +43,11 @@ node default {
         include megactl
     }
 
-    if $::mptraid {
+    if $::mptraid == 'true' {
         include raidmpt
     }
 
-    if $::kvmdomain {
+    if $::kvmdomain == 'true' {
         include acpi
     }
 
@@ -71,7 +71,7 @@ node default {
         default: { include nagios::client }
     }
 
-    if $::apache2 {
+    if $::apache2 == 'true' {
          if getfromhash($nodeinfo, 'apache2_security_mirror') {
                 include apache2::security_mirror
          }
@@ -87,7 +87,7 @@ node default {
          include apache2
     }
 
-    if $::rsyncd {
+    if $::rsyncd == 'true' {
         include rsyncd-log
     }
 
@@ -127,11 +127,11 @@ node default {
         }
     }
 
-    if $::brokenhosts {
+    if $::brokenhosts == 'true' {
         include hosts
     }
 
-    if $::portforwarder_user_exists {
+    if $::portforwarder_user_exists == 'true' {
         include portforwarder
     }
 
diff --git a/modules/acpi/manifests/init.pp b/modules/acpi/manifests/init.pp
index af961ba0..ffc779b8 100644
--- a/modules/acpi/manifests/init.pp
+++ b/modules/acpi/manifests/init.pp
@@ -1,5 +1,5 @@
 class acpi {
-    if ! $debarchitecture in ['kfreebsd-amd64', 'kfreebsd-i386'] {
+    if ! ($::debarchitecture in ['kfreebsd-amd64', 'kfreebsd-i386']) {
         package {
             acpid: ensure => installed
         }
diff --git a/modules/ferm/manifests/per-host.pp b/modules/ferm/manifests/per-host.pp
index a5717e15..374da372 100644
--- a/modules/ferm/manifests/per-host.pp
+++ b/modules/ferm/manifests/per-host.pp
@@ -244,7 +244,7 @@ class ferm::per-host {
         }
     }
 
-    if $::rsyncd {
+    if $::rsyncd == 'true' {
         include ferm::rsync
     }
 }
-- 
2.39.2