Muehlenhoff has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/338150 )

Change subject: Drop require_package for python-ldap
......................................................................


Drop require_package for python-ldap

Already declared in modules/ldap/manifests/client/utils.pp

Change-Id: I2cf0501998c1e184fec67d67904e0367a5c7ba3f
---
M modules/role/manifests/openldap/management.pp
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Muehlenhoff: Verified; Looks good to me, approved



diff --git a/modules/role/manifests/openldap/management.pp 
b/modules/role/manifests/openldap/management.pp
index 9bfc6a5..2e404c1 100644
--- a/modules/role/manifests/openldap/management.pp
+++ b/modules/role/manifests/openldap/management.pp
@@ -14,7 +14,7 @@
         password => $ldapconfig['script_user_pass'],
     }
 
-    require_package('python-ldap', 'python-yaml')
+    require_package('python-yaml')
 
     file { '/usr/local/bin/cross-validate-accounts':
         ensure => present,

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2cf0501998c1e184fec67d67904e0367a5c7ba3f
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Muehlenhoff <[email protected]>
Gerrit-Reviewer: Muehlenhoff <[email protected]>

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

Reply via email to