Merge "Add python-ldappool as a requirement for LDAP"

redhat-openstack · Nov 4, 2014 · 38518cb · 38518cb
2 parents 2bfe589 + cca5483
commit 38518cb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/manifests/ldap.pp b/manifests/ldap.pp
@@ -84,7 +84,8 @@
   $assignment_driver                   = undef,
 ) {
 
-  package { 'python-ldap':
+  $ldap_packages = ['python-ldap', 'python-ldappool']
+  package { $ldap_packages:
       ensure => present,
   }
 

diff --git a/spec/classes/keystone_ldap_spec.rb b/spec/classes/keystone_ldap_spec.rb
@@ -76,6 +76,7 @@
       }
     end
     it { should contain_package('python-ldap') }
+    it { should contain_package('python-ldappool') }
     it 'should have basic params' do
       should contain_keystone_config('ldap/url').with_value('ldap://foo')
       should contain_keystone_config('ldap/user').with_value('cn=foo,dc=example,dc=com')