From b8a65368cb39d6d82c4b025b25fdbe868dbbfe89 Mon Sep 17 00:00:00 2001 From: Matthew Barth Date: Fri, 7 Aug 2020 09:35:20 -0500 Subject: [PATCH] fans: Update service start limits for fan watchdog It was found that the fan control services were constantly getting restarted due to not failing within the previous start limits. After experimenting with different combinations of limits, using the default values for StartLimitBurst and StartLimitIntervalSec is sufficient. Tested: Changed limits, powered on, watched service fails in journal until fan watchdog monitor started Resolves: https://github.com/openbmc/phosphor-fan-presence/issues/23 Change-Id: I137807dd8c2e09e3a4019a9ad416b0384824ed65 Signed-off-by: Matthew Barth --- recipes-phosphor/fans/phosphor-fan/fan-watchdog-monitor.conf | 2 -- 1 file changed, 2 deletions(-) diff --git a/recipes-phosphor/fans/phosphor-fan/fan-watchdog-monitor.conf b/recipes-phosphor/fans/phosphor-fan/fan-watchdog-monitor.conf index e72de888..38a37cbd 100644 --- a/recipes-phosphor/fans/phosphor-fan/fan-watchdog-monitor.conf +++ b/recipes-phosphor/fans/phosphor-fan/fan-watchdog-monitor.conf @@ -1,5 +1,3 @@ [Unit] #These overrides allow the fan watchdog to take over when this service dies OnFailure=fan-watchdog-monitor@%n.service -StartLimitIntervalSec=5 -StartLimitBurst=3