Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 589930 - net-analyzer/icinga2 systemd unit file does not work
Summary: net-analyzer/icinga2 systemd unit file does not work
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Matthew Thode ( prometheanfire )
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-07-28 15:34 UTC by Niklas Haas
Modified: 2017-07-24 02:03 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Niklas Haas 2016-07-28 15:34:32 UTC
This is a packaging error since the bug is introduced with gentoo-specific modifications.

In particular, the Gentoo-supplied /etc/conf.d/icinga2 is missing variables needed for /usr/lib/icinga2/prepare-dirs, and these variables are included in the upstream icinga2.confd file.

I have fixed this issue locally by expanding /etc/conf.d/icinga2 to look like this:

DAEMON=/usr/sbin/icinga2
ICINGA2_CONFIG_FILE=/etc/icinga2/icinga2.conf
ICINGA2_RUN_DIR=/run/icinga2
ICINGA2_STATE_DIR=/var
ICINGA2_PID_FILE=$ICINGA2_RUN_DIR/icinga2.pid
ICINGA2_ERROR_LOG=/var/log/icinga2/error.log
ICINGA2_STARTUP_LOG=/var/log/icinga2/startup.log
ICINGA2_LOG=/var/log/icinga2/icinga2.log
ICINGA2_COMMAND_GROUP=icingacmd

Note: Icinga2 still fails to launch with this fixed conf.d, due to another issue with /usr/lib/icinga2/prepare-dirs, but I suspect that issue is upstream rather than Gentoo-specific. (It fails to create /var/cache/icinga2, so icinga2 crashes on startup)
Comment 1 Niklas Haas 2016-07-28 15:42:47 UTC
Note: Upstream bug for the second issue is at https://dev.icinga.org/issues/12253

I just realized that even if they fix it upstream, the Gentoo package for the existing versions would continue to fail to work, so it might be necessary to patch this in the package as well.
Comment 2 Matthew Thode ( prometheanfire ) archtester Gentoo Infrastructure gentoo-dev Security 2016-07-28 15:51:00 UTC
If a patch is available I can probably just backport it.
Comment 3 Ewoud Kohl van Wijngaarden 2017-07-01 12:52:11 UTC
For the first part I submitted https://github.com/gentoo/gentoo/pull/5020 because I believe this fixes it. For the second part I submitted https://github.com/Icinga/icinga2/pull/5392
Comment 4 Matthew Thode ( prometheanfire ) archtester Gentoo Infrastructure gentoo-dev Security 2017-07-24 02:03:55 UTC
thanks for fixing this, merged it