362033 – app-admin/monit init.d option to include configtest

Bug 362033 - app-admin/monit init.d option to include configtest

Summary: app-admin/monit init.d option to include configtest

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	New packages (show other bugs)
Hardware:	All Linux

Importance:	Normal enhancement (vote)
Assignee:	Patrick Lauer

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2011-04-05 09:53 UTC by Oleg Gawriloff
Modified:	2011-04-13 15:40 UTC (History)
CC List:	0 users

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Oleg Gawriloff 2011-04-05 09:53:20 UTC

It would be nice that monit.initd include option for configtest. Smth like this:

opts="${opts} reload configtest"

configtest() {
        /usr/bin/monit -t 1>/dev/null 2>&1
        ret=$?
        if [ $ret -ne 0 ]; then
                eerror "${SVCNAME} has detected an error in your setup:"
                /usr/bin/monit -t
        fi
        return $ret
}


Reproducible: Always

Comment 1 Patrick Lauer gentoo-dev

2011-04-13 15:40:44 UTC

+  13 Apr 2011; Patrick Lauer <patrick@gentoo.org> +files/monit.initd-5.0-r1,
+  +monit-5.2.3-r1.ebuild:
+  init script fix for #362033


Excellent idea. Thanks!