diff --git a/manifests/params.pp b/manifests/params.pp
index bd63d8cf..8ec6ac6d 100644
--- a/manifests/params.pp
+++ b/manifests/params.pp
@@ -1,54 +1,43 @@
+# == Class: manila::params
+#
+# Parameters for puppet-manila
 #
 class manila::params {
 
   include ::openstacklib::defaults
 
-  $manila_conf = '/etc/manila/manila.conf'
+  $manila_conf          = '/etc/manila/manila.conf'
   $manila_paste_api_ini = '/etc/manila/api-paste.ini'
+  $client_package       = 'python-manilaclient'
+  $db_sync_command      = 'manila-manage db sync'
+  $lio_package_name     = 'targetcli'
 
-  if $::osfamily == 'Debian' {
-    $package_name                = 'manila-common'
-    $client_package              = 'python-manilaclient'
-    $api_package                 = 'manila-api'
-    $api_service                 = 'manila-api'
-    $scheduler_package           = 'manila-scheduler'
-    $scheduler_service           = 'manila-scheduler'
-    $share_package               = 'manila-share'
-    $share_service               = 'manila-share'
-    $db_sync_command             = 'manila-manage db sync'
-    $tgt_package_name            = 'tgt'
-    $tgt_service_name            = 'tgt'
-    $ceph_init_override          = '/etc/init/manila-share.override'
-    $iscsi_helper                = 'tgtadm'
-    $lio_package_name            = 'targetcli'
-    $gluster_client_package_name = 'glusterfs-client'
-    $gluster_package_name        = 'glusterfs-common'
-
-  } elsif($::osfamily == 'RedHat') {
-
-    $package_name                = 'openstack-manila'
-    $client_package              = 'python-manilaclient'
-    $api_package                 = false
-    $api_service                 = 'openstack-manila-api'
-    $scheduler_package           = false
-    $scheduler_service           = 'openstack-manila-scheduler'
-    $share_package               = 'openstack-manila-share'
-    $share_service               = 'openstack-manila-share'
-    $db_sync_command             = 'manila-manage db sync'
-    $tgt_package_name            = 'scsi-target-utils'
-    $tgt_service_name            = 'tgtd'
-    $ceph_init_override          = '/etc/sysconfig/openstack-manila-share'
-    $lio_package_name            = 'targetcli'
-    $gluster_client_package_name = 'glusterfs-fuse'
-    $gluster_package_name        = 'glusterfs'
-
-    if $::operatingsystem == 'RedHat' and (versioncmp($::operatingsystemmajrelease, '7') >= 0) {
-      $iscsi_helper = 'lioadm'
-    } else {
-      $iscsi_helper = 'tgtadm'
+  case $::osfamily {
+    'Debian': {
+      $package_name                = 'manila-common'
+      $api_package                 = 'manila-api'
+      $api_service                 = 'manila-api'
+      $scheduler_package           = 'manila-scheduler'
+      $scheduler_service           = 'manila-scheduler'
+      $share_package               = 'manila-share'
+      $share_service               = 'manila-share'
+      $gluster_client_package_name = 'glusterfs-client'
+      $gluster_package_name        = 'glusterfs-common'
+    }
+    'RedHat': {
+      $package_name                = 'openstack-manila'
+      $api_package                 = false
+      $api_service                 = 'openstack-manila-api'
+      $scheduler_package           = false
+      $scheduler_service           = 'openstack-manila-scheduler'
+      $share_package               = 'openstack-manila-share'
+      $share_service               = 'openstack-manila-share'
+      $gluster_client_package_name = 'glusterfs-fuse'
+      $gluster_package_name        = 'glusterfs'
+    }
+    default: {
+      fail("unsupported osfamily ${::osfamily}, currently Debian and Redhat are the only supported platforms")
     }
 
-  } else {
-    fail("unsupported osfamily ${::osfamily}, currently Debian and Redhat are the only supported platforms")
   }
 }