datacentred/datacentred-ldap — pull request #26 by jgrassler (2014-06-19 10:12:42+00:00)
Title: Two fixes for use on RedHat flavoured systems

Hello,

I ran into two issues with datacentred-ldap on CentOS and fixed them:

1) The pidfile/argsfile options in slapd.conf are statically configured to have values suitable for Debian based systems. These values clash with those expected by CentOS' init system. I made them dynamic, so that they are set based on os flavour in c396512.

2) In 5e9aee5 I modified the template for Redhat flavoured systems' sysconfig to match that of your Debian defaults template: I supply the -f option to slapd to explicitely set its config file (in the Debian defaults template there's an extra option for the same thing). Without this change the default /etc/openldap/slapd.d shipped with the openldap RPMs takes precedence over slapd.conf.

Can you please merge these two commits? Thanks!

Cheers,

Johannes

P.S.: I tested my changes on CentOS and they work, but I didn't do any regression testing on Ubuntu. For 5e9aee5 this shouldn't be neccessary, since I only touched the sysconfig.erb template, but you may want to test c396512, just to be on the safe side.