Alexandros Kosiaris has submitted this change and it was merged.

Change subject: openldap: Use require_package
......................................................................


openldap: Use require_package

Change-Id: If3b2da7a46c4ec2035b482d98b02d8900c2321b7
---
M modules/openldap/manifests/init.pp
1 file changed, 1 insertion(+), 7 deletions(-)

Approvals:
  Alexandros Kosiaris: Verified; Looks good to me, approved



diff --git a/modules/openldap/manifests/init.pp 
b/modules/openldap/manifests/init.pp
index a75e0de..2e6c894 100644
--- a/modules/openldap/manifests/init.pp
+++ b/modules/openldap/manifests/init.pp
@@ -48,13 +48,7 @@
     $ca=undef,
 ) {
 
-    package { [
-        'slapd',
-        'ldap-utils',
-        'python-ldap',
-        ]:
-        ensure => installed,
-    }
+    require_package('slapd', 'ldap-utils', 'python-ldap')
 
     service { 'slapd':
         ensure     => running,

-- 
To view, visit https://gerrit.wikimedia.org/r/249720
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If3b2da7a46c4ec2035b482d98b02d8900c2321b7
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Muehlenhoff <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to