Bug 740722 - Add config-check to Pound restart function in init.d
Summary: Add config-check to Pound restart function in init.d
Keywords:
Status: NEW
Alias: None
Product: Fedora EPEL
Classification: Fedora
Component: Pound
Version: el6
Hardware: All
OS: Linux
unspecified
low
Target Milestone: ---
Assignee: Adam Tkac
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-09-23 05:52 UTC by Marcus Smit
Modified: 2016-07-20 09:40 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:


Attachments (Terms of Use)

Description Marcus Smit 2011-09-23 05:52:04 UTC
Suggestion for improvement: restarting the Pound daemon restarts without any check on the configuration. Does the config fail, Pound will not start up. But it did shutdown. My suggestion is to add the config check to the restart function of the init script.

Something like:

        echo -n $"Checking configuration validity... "
        /usr/sbin/pound -cf /etc/pound.cfg
        [ $? -eq 0 ] || exit 1

will do the trick perfectly.

I did not test this, but i'm pretty sure this is applicable to all Pound versions in the EPEL repository.

Also see the nagios package, where this check actually is implemented in the init-script.

Comment 1 Jose Pedro Oliveira 2012-02-08 03:18:22 UTC
Re-assigning this to the Pound package.


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