Add strict validation about boolean parameters

This ensures the parameters used by if-else logic accept only boolean
values because non-boolean can result in unexpected behavior.

Change-Id: I0185a7ae83c778ecc6ae5f466ab8b48acbba42ed

manifests/api.pp

@@ -44,6 +44,10 @@ class placement::api (
   include placement::deps
   include placement::policy
 
+  validate_legacy(Boolean, 'validate_bool', $manage_service)
+  validate_legacy(Boolean, 'validate_bool', $enabled)
+  validate_legacy(Boolean, 'validate_bool', $sync_db)
+
   if $manage_service {
     if $api_service_name == 'httpd' {
       # The following logic is currently required only in Debian, because

manifests/generic_service.pp

@@ -41,6 +41,9 @@ define placement::generic_service(
   include placement::deps
   include placement::params
 
+  validate_legacy(Boolean, 'validate_bool', $manage_service)
+  validate_legacy(Boolean, 'validate_bool', $enabled)
+
   $placement_title = "placement-${name}"
 
   # I need to mark that ths package should be

manifests/init.pp

@@ -36,6 +36,8 @@ class placement(
 
   include placement::deps
 
+  validate_legacy(Boolean, 'validate_bool', $sync_db)
+
   if $sync_db {
     include placement::db::sync
   }