From 546858a1e4d13d179a6af27b474e1396cfdf0c29 Mon Sep 17 00:00:00 2001 From: Michael Krelin Date: Fri, 23 Jul 2004 20:40:46 +0000 Subject: the ability to check/kill/reload any of the processes being monitored added. --- diff --git a/man/dudki.8.in b/man/dudki.8.in index 3011034..05db733 100644 --- a/man/dudki.8.in +++ b/man/dudki.8.in @@ -26,6 +26,7 @@ dudki \- a process monitoring daemon .if \*[longopt] [\fB--ensure\fR] [\fB-t\fR] .if \*[longopt] [\fB--test\fR] +[\fI\fR] .SH DESCRIPTION @@ -45,15 +46,19 @@ Specify the configuration file to use (default is .TP .ie \*[longopt] \fB-k\fR, \fB--kill\fR .el \fB-k\fR -Stop the running instance by sending the \fBSIGTERM\fR signal. +Stop the running instance by sending the \fBSIGTERM\fR signal. If no process +name specified on the command line, dudki kills his own running instance. .TP .ie \*[longopt] \fB-r\fR, \fB--reload\fR .el \fB-r\fR -Reload the running instance by sending the \fBSIGHUP\fR signal. +Reload the running instance by sending the \fBSIGHUP\fR signal. Like with +\fB-k\fR, if no process name specified on the command line, dudki sends +\fBSIGHUP\fR to his own running instance. .TP .ie \*[longopt] \fB-c\fR, \fB--check\fR .el \fB-c\fR -Check if dudki is running. Exit with non-zero status if not. +Check if dudki is running. Exit with non-zero status if not. The same target +rules as in \fB-k\fR and \fB-r\fR apply here. .TP .ie \*[longopt] \fB-e\fR, \fB--ensure\fR .el \fB-e\fR diff --git a/src/dudki.cc b/src/dudki.cc index b4e95a7..e91ad5e 100644 --- a/src/dudki.cc +++ b/src/dudki.cc @@ -62,12 +62,11 @@ int main(int argc,char **argv) { enum { op_default, op_work, - op_hup, - op_term, - op_check, + op_signal, op_ensure, op_test } op = op_default; + int op_signum = 0; while(true) { #define SHORTOPTSTRING "f:hVLrkcet" #ifdef HAVE_GETOPT_LONG @@ -93,7 +92,8 @@ int main(int argc,char **argv) { switch(c) { case 'h': cerr << PHEADER << endl - << PCOPY << endl << endl << + << PCOPY << endl << endl + << " " << argv[0] << " [options] [processes]" << endl << endl << #ifdef HAVE_GETOPT_LONG " -h, --help\n" " --usage display this text\n" @@ -102,9 +102,11 @@ int main(int argc,char **argv) { " -f filename, --config=filename\n" " specify the configuration file to use\n" "\n" - " -k, --kill stop running instance\n" + " -k, --kill stop running instance (send SIGTERM)\n" " -r, --reload reload running instance (send SIGHUP)\n" - " -c, --check check if dudki is running\n" + " -c, --check check if the process is running\n" + " (the above commands operate on dudki itself if no\n" + " process name has been specified)\n" " -e, --ensure ensure that dudki is running\n" " -t, --test test configuration file and exit" #else /* !HAVE_GETOPT_LONG */ @@ -113,9 +115,11 @@ int main(int argc,char **argv) { " -L show license\n" " -f filename specify the configuration file to use\n" "\n" - " -k stop running instance\n" + " -k stop running instance (send SIGTERM)\n" " -r reload running instance (send SIGHUP)\n" - " -c check if dudki is running\n" + " -c check if the process is running\n" + " (the above commands operate on dudki itself if no\n" + " process name has been specified)\n" " -e ensure that dudki is running\n" " -t test configuration file and exit" #endif /* /HAVE_GETOPT_LONG */ @@ -139,21 +143,21 @@ int main(int argc,char **argv) { cerr << "Can't obey two or more orders at once" << endl; exit(1); } - op = op_term; + op = op_signal; op_signum = SIGTERM; break; case 'r': if(op!=op_default) { cerr << "Can't obey two or more orders at once" << endl; exit(1); } - op = op_hup; + op = op_signal; op_signum = SIGHUP; break; case 'c': if(op!=op_default) { cerr << "Can't obey two or more orders at once" << endl; exit(1); } - op = op_check; + op = op_signal; op_signum = 0; break; case 'e': if(op!=op_default) { @@ -187,18 +191,31 @@ int main(int argc,char **argv) { case op_test: cerr << "Configuration OK" << endl; break; - case op_hup: - signal_self(config,SIGHUP); - break; - case op_term: - signal_self(config,SIGTERM); - break; - case op_check: - try{ - signal_self(config,0); - exit(0); + case op_signal: + try { + if(optind>=argc) { + signal_self(config,op_signum); + }else{ + int failures = 0; + for(int narg=optind;nargsecond.signal(op_signum); + }catch(exception& e) { + cerr << "dudki(" << argv[narg] << "): " << e.what() << endl; + failures++; + } + } + if(failures) + throw runtime_error("not all processes have been successfully signaled"); + } + if(!op_signum) + exit(0); }catch(exception& e) { - exit(1); + if(!op_signum) + exit(1); } case op_ensure: try { diff --git a/src/process.cc b/src/process.cc index bfab311..1ffac9f 100644 --- a/src/process.cc +++ b/src/process.cc @@ -15,21 +15,10 @@ using namespace std; #include "configuration.h" void process::check(const string& id,configuration& config) { - bool running = false; - ifstream pids(pidfile.c_str(),ios::in); - if(pids) { - pid_t pid = 0; - pids >> pid; - pids.close(); - if(pid) { - if(!kill(pid,0)) { - running = true; - } - } - } - if(running){ + try { + signal(0); patience = 0; - }else{ + }catch(exception& e) { if(patience>60) { // TODO: configurable patience = 0; }else{ @@ -185,3 +174,16 @@ void process::notify_mailto(const string& email,const string& id,const string& e waitpid(pid,&status,0); // TODO: check the return code } + +void process::signal(int signum) const { + ifstream pids(pidfile.c_str(),ios::in); + if(!pids) + throw runtime_error("no pidfile found"); + pid_t pid = 0; + pids >> pid; + pids.close(); + if(!pid) + throw runtime_error("no pid in pidfile"); + if(kill(pid,signum)) + throw runtime_error("failed to signal process"); +} diff --git a/src/process.h b/src/process.h index b6d7091..27ee049 100644 --- a/src/process.h +++ b/src/process.h @@ -29,6 +29,8 @@ class process { void do_notify(const string& id,const string& event,const string& description,configuration& config); void notify_mailto(const string& email,const string& id,const string& event, const string& description,configuration& config); + + void signal(int signum) const; }; typedef map processes_t; -- cgit v0.9.0.2