From 4193a626b290b51ebf411423ac653512dbd1c014 Mon Sep 17 00:00:00 2001
From: Dan Prince <dprince@redhat.com>
Date: Fri, 4 Jan 2013 14:13:16 -0500
Subject: [PATCH] Update ulimit modules to support Red Hat.

Adds a new params module which is used to provide distribution
specific parameters for other modules. The init module was updated
to make use of the new parameters.

Change-Id: Ib81e919a777741d9d58d687672508b8dd53498ff
Reviewed-on: https://review.openstack.org/18997
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
---
 manifests/init.pp   |  5 ++++-
 manifests/params.pp | 17 +++++++++++++++++
 2 files changed, 21 insertions(+), 1 deletion(-)
 create mode 100644 manifests/params.pp

diff --git a/manifests/init.pp b/manifests/init.pp
index edb5f13..80ffc70 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -2,7 +2,9 @@
 #
 class ulimit {
 
-  package { ['libpam-modules', 'libpam-modules-bin']:
+  include ulimit::params
+
+  package { $::ulimit::params::pam_packages:
     ensure => present,
   }
 
@@ -11,4 +13,5 @@ class ulimit {
     owner  => 'root',
     mode   => '0755',
   }
+
 }
diff --git a/manifests/params.pp b/manifests/params.pp
new file mode 100644
index 0000000..b0c5eab
--- /dev/null
+++ b/manifests/params.pp
@@ -0,0 +1,17 @@
+# Class: ulimit::params
+#
+# This class holds parameters that need to be
+# accessed by other classes.
+class ulimit::params {
+  case $::osfamily {
+    'Fedora', 'Redhat': {
+      $pam_packages = ['pam']
+    }
+    'Debian', 'Ubuntu': {
+      $pam_packages = ['libpam-modules', 'libpam-modules-bin']
+    }
+    default: {
+      fail("Unsupported osfamily: ${::osfamily} The 'ulimit' module only supports osfamily Fedora, Redhat, Debian, or Ubuntu.")
+    }
+  }
+}