diff options
author | William A. Kennington III <wak@google.com> | 2018-10-03 23:36:16 -0700 |
---|---|---|
committer | William A. Kennington III <wak@google.com> | 2018-10-04 00:14:02 -0700 |
commit | 7036c569c8de530165a3ee029cdeaa20fb9e9a41 (patch) | |
tree | bcd48c83f6ac10b9028a89ad2e85853ab227d78e /test/argument_test.hpp | |
parent | d4cbc5aad28f32c90e18d0fdd5ff8d1150fbd663 (diff) | |
download | phosphor-watchdog-7036c569c8de530165a3ee029cdeaa20fb9e9a41.tar.gz phosphor-watchdog-7036c569c8de530165a3ee029cdeaa20fb9e9a41.zip |
watchdog: Handle systemd StartUnit errors
Currently, if systemd returns an error upon trying to StartUnit in the
timeout of the watchdog the entire daemon will crash.
Sep 28 22:59:09 phosphor-watchdog[2462]: Timer Expired
Sep 28 22:59:09 phosphor-watchdog[2462]: watchdog: Timed out
Sep 28 22:59:09 phosphor-watchdog[2462]: terminate called after throwing an instance of 'sdbusplus::exception::SdBusError'
Sep 28 22:59:09 phosphor-watchdog[2462]: what(): sd_bus_call noreply: org.freedesktop.DBus.Error.InvalidArgs: Unit ho
st-watchdog-reset.service is not loaded properly: Invalid argument.
This patch fixes the behavior so we get something more like:
Oct 04 07:12:14 phosphor-watchdog[10897]: watchdog: Timed out
Oct 04 07:12:14 phosphor-watchdog[10897]: watchdog: Failed to start unit
Oct 04 07:12:16 phosphor-watchdog[10897]: watchdog: disabled
Tested:
Ran with a bogus systemd target which used to throw errors which are
now correctly handled without crashing.
Change-Id: I4cf6ffded789e49d2329439165927cc227e2e79e
Signed-off-by: William A. Kennington III <wak@google.com>
Diffstat (limited to 'test/argument_test.hpp')
0 files changed, 0 insertions, 0 deletions