Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 67955 - nagios does not have reload option in /etc/init.d script
Summary: nagios does not have reload option in /etc/init.d script
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Gentoo Netmon project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-10-17 17:48 UTC by Jason Frank
Modified: 2004-10-20 09:38 UTC (History)
0 users

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


Attachments
Add in reload option to /etc/init.d/nagios (a la xinetd) (nagios-init.d-with-reload.patch,521 bytes, patch)
2004-10-17 17:52 UTC, Jason Frank
Details | Diff
Patch to incorporate config file validation (nagios-init.d-with-reload-try2.patch,1.43 KB, patch)
2004-10-20 09:04 UTC, Jason Frank
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jason Frank 2004-10-17 17:48:53 UTC
nagios is missing the reload option in it's /etc/init.d script.  Patch coming next.
Comment 1 Jason Frank 2004-10-17 17:52:32 UTC
Created attachment 42078 [details, diff]
Add in reload option to /etc/init.d/nagios (a la xinetd)

Add in reload method to /etc/init.d/nagios in the same fashion as xinetd
Comment 2 Eldad Zack (RETIRED) gentoo-dev 2004-10-20 01:04:43 UTC
This is dangerous.
If the configuration is rejected by nagios it will just stop.
You have to check it beforehand and bail if it wrong.
Comment 3 Jason Frank 2004-10-20 09:04:50 UTC
Created attachment 42251 [details, diff]
Patch to incorporate config file validation

I didn't bother to put in config file validation since the restart option
didn't do it either, but I don't disagree with the intent, so I went ahead and
added it.  I wasn't able to find any good examples for what I needed to do, so
some style changes might be necessary.
Comment 4 Eldad Zack (RETIRED) gentoo-dev 2004-10-20 09:38:03 UTC
Hmm. This is not healthy...

I've updated the script to check the config before start/restart and reload.
In portage... 

Thanks!