diff options
| author | Patrick Williams <patrick@stwcx.xyz> | 2017-07-12 18:37:42 +0300 |
|---|---|---|
| committer | Patrick Williams <patrick@stwcx.xyz> | 2017-07-13 22:03:29 +0300 |
| commit | d69337ec77bef9fc7ffc6810c34ead95a930d03c (patch) | |
| tree | 19848941f12c05f877bf2200bfe063cd2cfe1e85 /meta-phosphor/common/recipes-phosphor/watchdog | |
| parent | 8b60de789d7f553a3480db2590304fbc9a33d1c2 (diff) | |
| download | openbmc-d69337ec77bef9fc7ffc6810c34ead95a930d03c.tar.xz | |
phosphor-watchdog: don't enable watchdog on BMC reboot
There are applications we do not want to reenable after a BMC reboot and
the typical mechanism for that is a ConditionPathExists on a marker file
we create in /run. This was missing for the Host watchdog enable
service, which caused the watchdog to be re-enabled even though the boot
process was complete. Add the ConditionPathExists to prevent it from
becoming enabled.
Resolves openbmc/openbmc#1948.
Change-Id: Ibb64ee40d9b6313bc44205c751d130345be695ed
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Diffstat (limited to 'meta-phosphor/common/recipes-phosphor/watchdog')
| -rw-r--r-- | meta-phosphor/common/recipes-phosphor/watchdog/phosphor-watchdog/obmc-enable-host-watchdog@.service | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/meta-phosphor/common/recipes-phosphor/watchdog/phosphor-watchdog/obmc-enable-host-watchdog@.service b/meta-phosphor/common/recipes-phosphor/watchdog/phosphor-watchdog/obmc-enable-host-watchdog@.service index 1fc88f3324..a921fc3165 100644 --- a/meta-phosphor/common/recipes-phosphor/watchdog/phosphor-watchdog/obmc-enable-host-watchdog@.service +++ b/meta-phosphor/common/recipes-phosphor/watchdog/phosphor-watchdog/obmc-enable-host-watchdog@.service @@ -5,6 +5,7 @@ After=obmc-host-started@%i.target Wants=mapper-wait@-xyz-openbmc_project-watchdog-host%i.service After=mapper-wait@-xyz-openbmc_project-watchdog-host%i.service Conflicts=obmc-host-stop@%i.target +ConditionPathExists=!/run/openbmc/host@%i-on [Service] Type=oneshot |