Univention Bugzilla – Full Text Bug Listing |
Summary: | Require slapd in /etc/init.d/nagios-nrpe-server | ||
---|---|---|---|
Product: | UCS | Reporter: | Valentin Heidelberger <heidelberger> |
Component: | Monitoring (Prometheus or Nagios) | Assignee: | UCS maintainers <ucs-maintainers> |
Status: | NEW --- | QA Contact: | UCS maintainers <ucs-maintainers> |
Severity: | normal | ||
Priority: | P5 | CC: | damrose, hahn, requate, steuwer |
Version: | UCS 4.4 | ||
Target Milestone: | --- | ||
Hardware: | Other | ||
OS: | Linux | ||
See Also: | https://forge.univention.org/bugzilla/show_bug.cgi?id=43691 | ||
What kind of report is it?: | Bug Report | What type of bug is this?: | 2: Improvement: Would be a product improvement |
Who will be affected by this bug?: | 1: Will affect a very few installed domains | How will those affected feel about the bug?: | 2: A Pain – users won’t like this once they notice it |
User Pain: | 0.023 | Enterprise Customer affected?: | |
School Customer affected?: | ISV affected?: | ||
Waiting Support: | Flags outvoted (downgraded) after PO Review: | ||
Ticket number: | Bug group (optional): | ||
Max CVSS v3 score: | |||
Attachments: |
Add slapd to Required-Start in nagios-nrpe-server init script
Optionally add slapd to Required-Start in nagios-nrpe-server init script |
Created attachment 10054 [details]
Optionally add slapd to Required-Start in nagios-nrpe-server init script
Another approach, which I personally favour, would be to make the requirement optionally via UCR. Attached is another patch for this. It currently also pastes an empty line below the Required-Start line when I "ucr commit" it. Not sure why that is right now tbh.
@Ingo Would love to get your opinion on this from a product management perspective :) Adding it to the init script in the proposed way would cause the init script to fail on any system where no slapd is installed - that would be memberservers. Comment on attachment 10053 [details]
Add slapd to Required-Start in nagios-nrpe-server init script
You're right of course
Read <https://wiki.debian.org/LSBInitScripts>: Use "Should-Start:" instead of "Required-Start:" as the later would still not work on MemberServers. |
Created attachment 10053 [details] Add slapd to Required-Start in nagios-nrpe-server init script Normally nagios-nrpe-server uses a local user called nagios to start nrpe. A customer has created a "nagios" user in the LDAP because they require it on many other non-UCS systems which are joined to the domain. This resulted in NRPE not starting after the update to 4.3, because NRPE starts before slapd normally. The problem with this specific use-case persist with 4.4. If we changed the requirements of nagios-nrpe-server so that it requires slapd this use case would work again. Attached is a very simple patch that adds slapd to Required-Start in the init script. We'd need to regenerate the systemd service generated by systemd-sysv-generator as well.