do not override systemd restart config options
There doesn't appear to be any special reason for this service to
override the defaults set by OpenBMC for how many times a service should
restart before giving up.
Recently we saw an issue where this service was failing repeatedly. It
continued to just loop and loop on restarts, generating core dump after
core dump. The whole purpose of the restart limits is to avoid
situations like this.
Tested:
Verified that this change caused srvcfg-manager service to stop after
failing twice
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I420b5de55f368602e17a2b482d29b9d58f16d31a
diff --git a/srvcfg-manager.service b/srvcfg-manager.service
index 391f39c..b7b22df 100644
--- a/srvcfg-manager.service
+++ b/srvcfg-manager.service
@@ -4,8 +4,6 @@
[Service]
Restart=always
ExecStart=/usr/bin/phosphor-srvcfg-manager
-RestartSec=5
-StartLimitInterval=0
SyslogIdentifier=srvcfg-manager
Type=dbus
BusName=xyz.openbmc_project.Control.Service.Manager