-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #14006: Move rudder-slapd to a systemd unit #1743
Fixes #14006: Move rudder-slapd to a systemd unit #1743
Conversation
878904d
to
c1d9076
Compare
Commit modified |
%config(noreplace) /opt/rudder/etc/openldap/slapd.conf | ||
/etc/init.d/rudder-slapd | ||
/usr/lib/systemd/system/rudder-slapd.service | ||
/opt/rudder/etc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you need to add the package-script
c1d9076
to
02a895b
Compare
Commit modified |
sed -i '/^[ \t]*argsfile/d' "${LDAP_CONF}" | ||
|
||
# Migration from pre-5.1, moving settings to slapd.conf | ||
if [ -f /etc/default/rudder-slapd ]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will the file still be there in a postinst ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good question, it seems not, I'll make a copy of the file in preinst to make sure we don't miss it in case of a purge.
02a895b
to
1d07fe2
Compare
Commit modified |
1d07fe2
to
c58cdb2
Compare
Commit modified |
OK, merging this PR |
https://issues.rudder.io/issues/14006