iredapd.service must after slapd.service

iredmail / iRedMail

Full-featured, open source mail server solution for mainstream Linux/BSD distributions.

https://www.iredmail.org/

GNU General Public License v3.0

1.46k stars 216 forks source link

iredapd.service must after slapd.service #252

Closed bronkoo closed 7 months ago

bronkoo commented 7 months ago

We should extend iredapd.service file to have OpenLDAP up and running: before

After=network.target local-fs.target remote-fs.target slapd.service

Background at forum.

iredmail commented 7 months ago

Fixed. Thanks for the feedback. :) https://github.com/iredmail/iRedAPD/commit/bf33fc4cb5ef73ae4214dfb9857a128589f11246

iredmail commented 7 months ago

New version 5.4.0 is now available for upgrading, with just this small change.

bronkoo commented 7 months ago

Thanks!

Comment: My assumption was to have 'Requires=' in place, to be sure service is up and running, while 'After=' should only give an order, not a check of running slapd.service. But the reality by tests give me the answer from above. May be some farther tests necessary...

iredmail commented 7 months ago

For iRedMail SQL backends, there's no LDAP service at all, so After= is proper.