[systemd-devel] [PATCH v2 1/3] introduce WatchdogSec and hook up the watchdog with the existing failure logic

Lennart Poettering lennart at poettering.net
Thu Feb 9 11:17:52 PST 2012


On Thu, 09.02.12 10:21, Michael Olbrich (m.olbrich at pengutronix.de) wrote:

> > > +                                be restarted. Defaults to 0s, which
> > > +                                disables this feature.</para></listitem>
> > 
> > This makes me think that it might be a good idea to add a new restart
> > setting for this: Restart=on-watchdog, for people who only want to
> > restart on watchdog failures, but not otherwise? Do you think this could
> > be useful to you? If so, happy to merge a patch for that.
> 
> Hmmm, I think Restart=on-unexpected-failure (= !SERVICE_SUCCESS &&
> !SERVICE_FAILURE_EXIT_CODE) is probable more usefull.

Hmm, not sure I like the name though. An exit code != 0 might still be
unexpected.

Maybe "on-abort-or-timeout" is a better name? (And watchdog would just
be considered a different type of timeout here?)

Lennart

-- 
Lennart Poettering - Red Hat, Inc.


More information about the systemd-devel mailing list