diff options
author | Brad Bishop <bradleyb@fuzziesquirrel.com> | 2020-08-12 14:03:47 +0300 |
---|---|---|
committer | Brad Bishop <bradleyb@fuzziesquirrel.com> | 2020-08-14 02:28:47 +0300 |
commit | 0e72d71a67f8d29df926af2a8c249a521ee62680 (patch) | |
tree | 41b79f72ec15509067c292e957a281e31107585a /meta-phosphor/recipes-core/systemd/phosphor-systemd-policy | |
parent | f14643da2deabab4b79e896acb4440a95aa54889 (diff) | |
download | openbmc-0e72d71a67f8d29df926af2a8c249a521ee62680.tar.xz |
meta-phosphor: systemd-policy: move to systemd
Don't install systemd policy if:
- systemd is not being installed
- a non phosphor distro
Use RRECOMMENDS for easy opt-out in Phosphor based distros.
(From meta-phosphor rev: 2641990831230d85ee7cb5fdc29450ad561a4799)
Change-Id: Ie3154cc6ee8d257ffa92463c49d3755a15379663
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Diffstat (limited to 'meta-phosphor/recipes-core/systemd/phosphor-systemd-policy')
-rw-r--r-- | meta-phosphor/recipes-core/systemd/phosphor-systemd-policy/journald-maxlevel-policy.conf | 6 | ||||
-rw-r--r-- | meta-phosphor/recipes-core/systemd/phosphor-systemd-policy/service-restart-policy.conf | 31 |
2 files changed, 37 insertions, 0 deletions
diff --git a/meta-phosphor/recipes-core/systemd/phosphor-systemd-policy/journald-maxlevel-policy.conf b/meta-phosphor/recipes-core/systemd/phosphor-systemd-policy/journald-maxlevel-policy.conf new file mode 100644 index 000000000..f5f3d0ff1 --- /dev/null +++ b/meta-phosphor/recipes-core/systemd/phosphor-systemd-policy/journald-maxlevel-policy.conf @@ -0,0 +1,6 @@ +# Don't store "debug" messages, only "info" and below. +# Reference: journald.conf(5) + +[Journal] +MaxLevelStore=info +MaxLevelSyslog=info diff --git a/meta-phosphor/recipes-core/systemd/phosphor-systemd-policy/service-restart-policy.conf b/meta-phosphor/recipes-core/systemd/phosphor-systemd-policy/service-restart-policy.conf new file mode 100644 index 000000000..54516c2d4 --- /dev/null +++ b/meta-phosphor/recipes-core/systemd/phosphor-systemd-policy/service-restart-policy.conf @@ -0,0 +1,31 @@ +# This file overrides some defaults for systemd +# +# - Change the RestartSec from 100ms to 1s. +# When a service hits a failure, our new debug collection service kicks +# in. When a core file is involved, it's been found that generating 5 core +# files within ~500ms puts a huge strain on the BMC. Also, if the bmc is +# going to get a fix on a restart of a service, the more time the better +# (think retries on device driver scenarios). +# +# - Change the StartLimitBurst to 2 +# Five just seems excessive for our services in openbmc. In all fail +# scenarios seen so far (other then with phosphor-hwmon), either +# restarting once does the job or restarting all 5 times does not help +# and we just end up hitting the 5 limit anyway. +# +# - Change the StartLimitIntervalSec to 30s +# The BMC CPU performance is already challenged. When a service is +# failing and a core dump is being generated and collected into a dump, +# it's even more challenged. Recent failures have shown situations where +# the service does not fail again until 15-20 seconds after the initial +# failure which means the default of 10s for this results in the service +# being restarted indefinitely. Change this to 30s to only allow a service +# to be restarted StartLimitBurst times within a 30s interval before +# being put in a permanent fail state. +# +# See systemd-system.conf(5) for details on the conf files + +[Manager] +DefaultRestartSec=1s +DefaultStartLimitBurst=2 +DefaultStartLimitIntervalSec=30s |