Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added new global configuration directive shutdown_script. #1

Closed
wants to merge 1 commit into from

Conversation

bfg
Copy link

@bfg bfg commented Jul 16, 2012

This patch adds new global configuration directive "shutdown_script". Using this directive one can, for example, easily shutdown any services started by notitfy_* directive scripts when daemon is shut down.

Example: Run /bin/true on daemon shutdown.
shutdown_script "/bin/true arg1 arg2"

 Example: Run /bin/true on daemon shutdown.
 shutdown_script "/bin/true arg1 arg2"
@acassen
Copy link
Owner

acassen commented Jul 26, 2012

shutdown_script MUST not be called from signal handler. Would be more appropriated to run script at stop_keepalived(void) level so that waitpid (VRRP & CHECK) have completed and child are stopped.

You can even imagine to register multiple shutdown_script for each subsystem to be called in each subsystem: I mean :
shutdown_script_global
shutdown_script_check
shutdown_script_vrrp

Then all subsystems will have the ability to run script in their own "stop_" directive.

@acassen acassen closed this Jul 26, 2012
@kuaikuai kuaikuai mentioned this pull request Jul 23, 2015
acassen pushed a commit that referenced this pull request Oct 28, 2015
acassen added a commit that referenced this pull request Jan 26, 2016
Update configure in line with configure.in - request #1
@Cugar15 Cugar15 mentioned this pull request May 15, 2016
lzmartinico pushed a commit to lzmartinico/keepalived that referenced this pull request Sep 8, 2016
Add DBus functionality to VRRP
pqarmitage pushed a commit that referenced this pull request Nov 26, 2016
@stfast stfast mentioned this pull request Jan 23, 2018
@iliesh iliesh mentioned this pull request Jun 25, 2018
@LinSizhan LinSizhan mentioned this pull request Jul 2, 2018
@boucho-o boucho-o mentioned this pull request Aug 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants