diff --git a/manifests/iscsid.pp b/manifests/iscsid.pp
index c0f14d5f..5b31ab29 100644
--- a/manifests/iscsid.pp
+++ b/manifests/iscsid.pp
@@ -17,9 +17,9 @@
 #    Defaults to 'present'
 #
 class openstacklib::iscsid (
-  Boolean $enabled        = true,
-  Boolean $manage_service = true,
-  $package_ensure         = 'present'
+  Boolean $enabled                        = true,
+  Boolean $manage_service                 = true,
+  Stdlib::Ensure::Package $package_ensure = 'present'
 ) {
   include openstacklib::params
 
diff --git a/manifests/openstackclient.pp b/manifests/openstackclient.pp
index 0dc2dfb0..5d84d7b4 100644
--- a/manifests/openstackclient.pp
+++ b/manifests/openstackclient.pp
@@ -13,8 +13,8 @@
 #    Defaults to $openstacklib::params::openstackclient_package_name
 #
 class openstacklib::openstackclient (
-  $package_name   = $openstacklib::params::openstackclient_package_name,
-  $package_ensure = 'present',
+  String[1] $package_name                 = $openstacklib::params::openstackclient_package_name,
+  Stdlib::Ensure::Package $package_ensure = 'present'
 ) inherits openstacklib::params {
   stdlib::ensure_packages($package_name, {
     'ensure' => $package_ensure,